Skip to content
This repository was archived by the owner on Sep 17, 2024. It is now read-only.

ci: provision e2-small workers#2212

Merged
mdelapenya merged 4 commits intoelastic:mainfrom
v1v:feature/small-workers
Mar 9, 2022
Merged

ci: provision e2-small workers#2212
mdelapenya merged 4 commits intoelastic:mainfrom
v1v:feature/small-workers

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Mar 8, 2022 •
edited
Loading

What does this PR do?

I wanna explore if a smaller CI workers for the existing parallel stages work as expected since the heavy-lift is done in the remote workers instead.

From n1-highmem-8 to e2-small

image

image

image

Why is it important?

Cheaper builds.

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Mar 8, 2022

This pull request does not have a backport label. Could you fix it @v1v? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 7./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label Mar 8, 2022
v1v
v1v commented Mar 8, 2022
View reviewed changes
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Mar 8, 2022 •
edited
Loading

💔 Tests Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview [preview](https://ci-stats.elastic.co/app/apm/services/beats-ci/transactions/view?rangeFrom=2022-03-08T15:49:52.077Z&rangeTo=2022-03-08T16:09:52.077Z&transactionName=BUILD e2e-tests/e2e-testing-mbp/PR-{number}&transactionType=job&latencyAggregationType=avg&traceId=489157549ee0ba7cc46ac7eca94ae126&transactionId=e98cfae312e95734)

Expand to view the summary

Build stats

  • Start Time: 2022-03-08T15:59:52.077+0000

  • Duration: 51 min 44 sec

Test stats 🧪

Test Results
Failed 1
Passed 275
Skipped 0
Total 276

Test errors 1

Expand to view the tests failures

Initializing / End-To-End Tests / kubernetes-autodiscover_debian_amd64_elastic-agent / Logs collection from running pod – elastic-agent standalone
    Expand to view the error details

     Step "elastic-agent" collects events with "kubernetes.pod.name:a-pod": context deadline exceeded

  • no stacktrace

Steps errors 2

Expand to view the steps failures

Shell Script
  • Took 18 min 34 sec . View more details here
  • Description: ssh -tt -o TCPKeepAlive=yes -o ServerAliveInterval=60 -o ServerAliveCountMax=10 -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i /var/lib/jenkins/workspace/PR-2212-5-6f18e0a7-fdf5-430b-9a84-b4569c0a7fe5/e2essh admin@3.20.223.100 -- 'sudo bash /home/admin/e2e-testing/.ci/scripts/functional-test.sh "elastic-agent && ~@nightly && ~@skip:amd64" '
Archive the artifacts
  • Took 0 min 0 sec . View more details here
  • Description: [2022-03-08T16:47:40.196Z] Archiving artifacts script returned exit code 2

🐛 Flaky test report

❕ There are test failures but not known flaky tests.

Expand to view the summary

Genuine test errors 1

💔 There are test failures but not known flaky tests, most likely a genuine test failure.

  • Name: Initializing / End-To-End Tests / kubernetes-autodiscover_debian_amd64_elastic-agent / Logs collection from running pod – elastic-agent standalone

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

v1v
v1v commented Mar 8, 2022
View reviewed changes
v1v
v1v commented Mar 8, 2022
View reviewed changes
@v1v v1v added backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify backport-v8.1.0 Automated backport with mergify labels Mar 8, 2022
@v1v
Copy link
Copy Markdown
Member Author

v1v commented Mar 8, 2022

/test

@v1v v1v marked this pull request as ready for review March 8, 2022 15:59
@v1v v1v requested a review from a team March 8, 2022 15:59
@mergify mergify bot removed the backport-skip Skip notification from the automated backport with mergify label Mar 8, 2022
@v1v v1v changed the title ci: provision flyweight workers ci: provision e2-small workers Mar 8, 2022
@v1v v1v self-assigned this Mar 8, 2022
mdelapenya
mdelapenya approved these changes Mar 8, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@mdelapenya mdelapenya left a comment

Choose a reason for hiding this comment

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

This is great! I had it in my mind but instead reducing the size of the VMs (in a follow-up).

💯

@mdelapenya mdelapenya merged commit e7e2d1b into elastic:main Mar 9, 2022
mergify bot pushed a commit that referenced this pull request Mar 9, 2022
@v1v
ci: provision e2-small workers (#2212)
53c22f7 
* ci: provision flyweight workers

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

(cherry picked from commit e7e2d1b)
@mergify mergify bot mentioned this pull request Mar 9, 2022
Merged
mergify bot pushed a commit that referenced this pull request Mar 9, 2022
@v1v
ci: provision e2-small workers (#2212)
6ea828b 
* ci: provision flyweight workers

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

(cherry picked from commit e7e2d1b)
@mergify mergify bot mentioned this pull request Mar 9, 2022
Merged
mergify bot pushed a commit that referenced this pull request Mar 9, 2022
@v1v
ci: provision e2-small workers (#2212)
1a4e832 
* ci: provision flyweight workers

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

(cherry picked from commit e7e2d1b)
@mergify mergify bot mentioned this pull request Mar 9, 2022
Merged
mdelapenya pushed a commit that referenced this pull request Mar 9, 2022
@mergify @v1v
ci: provision e2-small workers (#2212) (#2215)
118a0f8 
* ci: provision flyweight workers

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

(cherry picked from commit e7e2d1b)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
mdelapenya pushed a commit that referenced this pull request Mar 9, 2022
@mergify @v1v
ci: provision e2-small workers (#2212) (#2216)
87cce2e 
* ci: provision flyweight workers

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

(cherry picked from commit e7e2d1b)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
mdelapenya pushed a commit that referenced this pull request Mar 9, 2022
@mergify @v1v
ci: provision e2-small workers (#2212) (#2214)
2625a43 
* ci: provision flyweight workers

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

* Update .ci/Jenkinsfile

(cherry picked from commit e7e2d1b)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
mdelapenya added a commit that referenced this pull request Mar 17, 2022
@mdelapenya
Merge branch 'main' into EricDavisX-patch-x3
47c3951 
* main: (268 commits)
  bump stack version 8.2.0-ff67d7b8 (#2242)
  ci: periodic builds for the last two minor versions (#2241)
  bump stack version 8.2.0-9bac538c (#2240)
  fix: add end-of-file line in templates (#2237)
  chore: add Christos to SSH users
  bump stack version 8.2.0-fee3b8d2 (#2234)
  Separate ES workload from Agent spec and make it req for the scenarios (#2203)
  bump stack version 8.2.0-d02c907a (#2228)
  bump stack version 8.2.0-63265ec9 (#2225)
  fix: wrong link (#2220)
  chore: define provider for each test suite (#2213)
  ci: provision e2-small workers (#2212)
  bump stack version 8.2.0-a12f7069 (#2209)
  chore: support for Oracle Linux 8 (#2208)
  bump stack version 8.2.0-190af159 (#2206)
  bump stack version 8.2.0-fdde08ec (#2200)
  bump stack version 8.2.0-43df679f (#2193)
  chore: use trace level in logs (#2130)
  bump stack version 8.2.0-bdf2ad74 (#2188)
  feat: support downloading project artifacts for the new bucket layout (#2172)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@mdelapenya mdelapenya mdelapenya approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@v1v v1v

Labels

backport-v7.17.0 Automated backport with mergify backport-v8.0.0 Automated backport with mergify backport-v8.1.0 Automated backport with mergify

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@v1v @elasticmachine @mdelapenya