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.

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

Replace refs with tags for 2.7.0 manifests and remove cron #3460

Merged
merged 6 commits into from
May 3, 2023

Conversation

zelinh
Copy link
Member

@zelinh zelinh commented May 2, 2023

Description

Replace refs with tags for 2.7.0
Remove cron build job for 2.6.1
Remove cron perf-test job for 2.5.0

Issues Resolved

Part of #3230

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@zelinh zelinh changed the title Replace refs with tags for 2.7.0 Replace refs with tags for 2.7.0 manifests May 2, 2023
Signed-off-by: Zelin Hao <[email protected]>
@zelinh zelinh changed the title Replace refs with tags for 2.7.0 manifests Replace refs with tags for 2.7.0 manifests and remove cron for 2.6.1 May 2, 2023
@zelinh zelinh mentioned this pull request May 2, 2023
42 tasks
Signed-off-by: Zelin Hao <[email protected]>
@zelinh zelinh changed the title Replace refs with tags for 2.7.0 manifests and remove cron for 2.6.1 Replace refs with tags for 2.7.0 manifests and remove cron May 2, 2023
@codecov-commenter
Copy link

codecov-commenter commented May 2, 2023

Codecov Report

Merging #3460 (3fada68) into main (f13a33b) will increase coverage by 0.09%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #3460      +/-   ##
==========================================
+ Coverage   91.72%   91.81%   +0.09%     
==========================================
  Files         172      181       +9     
  Lines        4991     5268     +277     
==========================================
+ Hits         4578     4837     +259     
- Misses        413      431      +18     

see 9 files with indirect coverage changes

Copy link
Member

@gaiksaya gaiksaya left a comment

Choose a reason for hiding this comment

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

@bbarani Can we remove 1.3.9 performance test runs as well?

H 21 * * * %BUNDLE_MANIFEST_URL=https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/2.5.0/latest/linux/arm64/tar/dist/opensearch/manifest.yml;TEST_WORKLOAD=http_logs;TEST_ITERATIONS=250;WARMUP_ITERATIONS=0
H 21 * * 5 %BUNDLE_MANIFEST_URL=https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/2.5.0/latest/linux/x64/tar/dist/opensearch/manifest.yml;TEST_WORKLOAD=http_logs;TEST_ITERATIONS=3;WARMUP_ITERATIONS=2
H 21 * * 5 %BUNDLE_MANIFEST_URL=https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/2.5.0/latest/linux/x64/tar/dist/opensearch/manifest.yml;TEST_WORKLOAD=http_logs;TEST_ITERATIONS=250;WARMUP_ITERATIONS=0

H 19 * * * %BUNDLE_MANIFEST_URL=https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/2.6.0/latest/linux/arm64/tar/dist/opensearch/manifest.yml;TEST_ITERATIONS=3;WARMUP_ITERATIONS=2
Copy link
Member

Choose a reason for hiding this comment

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

Please remove 2.6.0 as well

Copy link
Member Author

Choose a reason for hiding this comment

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

We would need 1.3.9 perf for upcoming 1.3.10 release right? @bbarani

Copy link
Member

Choose a reason for hiding this comment

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

We have numbers from previous runs so should be good. Waiting on reply from @bbarani Feel free to remove 2.6.0 till then,

Copy link
Member

Choose a reason for hiding this comment

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

Can we add 2.8.0 to the list? @zelinh

Copy link
Member

Choose a reason for hiding this comment

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

Lets remove 1.3.9 and add 1.3.10.

Copy link
Member Author

Choose a reason for hiding this comment

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

We would have a separate PR to add 1.3.10 and remove 1.3.9 once the release cycle for 1.3.10 starts.

zelinh added 2 commits May 2, 2023 17:29
Signed-off-by: Zelin Hao <[email protected]>
Signed-off-by: Zelin Hao <[email protected]>
@zelinh
Copy link
Member Author

zelinh commented May 3, 2023

The CI failure is not related to this PR. Merging this for now.

@zelinh zelinh merged commit d017316 into opensearch-project:main May 3, 2023
@zelinh zelinh deleted the replace-270-tags branch May 3, 2023 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants