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

Temporary comment out the macos build section snapshots #2442

Conversation

peterzhuamazon
Copy link
Member

Signed-off-by: Peter Zhu zhujiaxi@amazon.com

Description

Temporary comment out the macos build section snapshots

Issues Resolved

opensearch-project/alerting#516

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.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon added the bug Something isn't working label Aug 9, 2022
@peterzhuamazon peterzhuamazon self-assigned this Aug 9, 2022
@peterzhuamazon peterzhuamazon requested a review from a team as a code owner August 9, 2022 01:09
@codecov-commenter
Copy link

codecov-commenter commented Aug 9, 2022

Codecov Report

Merging #2442 (b3ca0b3) into main (9a60062) will increase coverage by 0.20%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #2442      +/-   ##
============================================
+ Coverage     94.63%   94.83%   +0.20%     
============================================
  Files           211      157      -54     
  Lines          4322     4144     -178     
  Branches         29       19      -10     
============================================
- Hits           4090     3930     -160     
+ Misses          226      214      -12     
+ Partials          6        0       -6     
Impacted Files Coverage Δ
src/system/temporary_directory.py 85.71% <0.00%> (-11.43%) ⬇️
src/system/os.py 93.75% <0.00%> (-6.25%) ⬇️
src/jenkins/BundleManifest.groovy
tests/jenkins/jobs/BuildYumRepo_Jenkinsfile
...enkins/jobs/BuildDockerImage_Qualifier_Jenkinsfile
tests/jenkins/jobs/SignArtifacts_Jenkinsfile
...bs/PrintArtifactDownloadUrlsForStaging_Jenkinsfile
...PromoteArtifacts_OpenSearch_Dashboards_Jenkinsfile
...nkins/jobs/RpmOpenSearchDistValidation_Jenkinsfile
...ests/jenkins/jobs/AssembleManifest_tar_Jenkinsfile
... and 46 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@prudhvigodithi
Copy link
Collaborator

@peterzhuamazon we already have the way include the platform, can we please use this

    platforms:
      - linux

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon
Copy link
Member Author

@peterzhuamazon we already have the way include the platform, can we please use this

    platforms:
      - linux

I was thinking it is also building plugins as a bundle but apparently only pushing core since it is just a snapshot.
Updated now.

Thanks.

@peterzhuamazon peterzhuamazon mentioned this pull request Aug 9, 2022
48 tasks
@peterzhuamazon peterzhuamazon merged commit 5c9a05d into opensearch-project:main Aug 9, 2022
@peterzhuamazon peterzhuamazon deleted the opensearch-2.2.0-removemacos branch August 9, 2022 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants