Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci: workflow on PR, merges and nightlies #23

Merged
merged 2 commits into from
Feb 9, 2022
Merged

ci: workflow on PR, merges and nightlies #23

merged 2 commits into from
Feb 9, 2022

Conversation

v1v
Copy link
Member

@v1v v1v commented Feb 8, 2022 •
edited
Loading

What

Workflow for the main pipeline

On PR

  • Unit tests always
  • Be able to trigger any other test suite through github comments:
    • run e2e tests or github label ci:end-to-end
    • run integration tests or github label ci:integration
    • run package or github label ci:package

After PR merge

  • Same as above again
  • Run e2e
  • Run packaging tests

Daily

Support the active development branches for main, 7.minor, 8.minor and 8.next

  • Run full e2e test suite
  • Run integration tests

Issue

Closes #20

Follow up

  • ITs and e2e are not yet implemented, so that's up to the team to define how to call them

@v1v v1v requested review from narph and a team February 9, 2022 12:42
@v1v v1v self-assigned this Feb 9, 2022
narph
narph approved these changes Feb 9, 2022
View reviewed changes
Copy link
Contributor

@narph narph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@v1v v1v merged commit 5ccf1b1 into elastic:main Feb 9, 2022
narph referenced this pull request in narph/elastic-agent-temp Feb 21, 2022
@v1v @narph
ci: workflow on PR, merges and nightlies (#23)
adbf464
narph pushed a commit that referenced this pull request Mar 3, 2022
@v1v @narph
ci: workflow on PR, merges and nightlies (#23)
3ad0b69
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@narph narph narph approved these changes

Assignees

@v1v v1v

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CI setup
2 participants
@v1v @narph