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

lower freq for build for 1.3.1 opensearch-dashboards #1941

Merged

Conversation

abhinavGupta16
Copy link
Contributor

Signed-off-by: Abhinav Gupta [email protected]

Description

Lower build freq for release versions

Issues Resolved

Post release activity - #1885

Check List

  • Commits are signed per the DCO using --signoff

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.

@abhinavGupta16 abhinavGupta16 requested a review from a team as a code owner April 8, 2022 00:46
@abhinavGupta16 abhinavGupta16 self-assigned this Apr 8, 2022
@codecov-commenter
Copy link

codecov-commenter commented Apr 8, 2022

Codecov Report

Merging #1941 (adb73e5) into main (91d0a57) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #1941   +/-   ##
=========================================
  Coverage     94.60%   94.60%           
  Complexity       20       20           
=========================================
  Files           178      178           
  Lines          3633     3633           
  Branches         27       27           
=========================================
  Hits           3437     3437           
  Misses          192      192           
  Partials          4        4           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 91d0a57...adb73e5. Read the comment docs.

Copy link
Member

@peterzhuamazon peterzhuamazon left a comment

Choose a reason for hiding this comment

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

No need to rebuild just remove it.
Same for 1.2.x

@abhinavGupta16
Copy link
Contributor Author

abhinavGupta16 commented Apr 8, 2022

No need to rebuild just remove it. Same for 1.2.x

I have removed it. Although, how do we decide which ones to keep and which ones to remove?

@peterzhuamazon peterzhuamazon merged commit 4d76087 into opensearch-project:main Apr 12, 2022
@peterzhuamazon peterzhuamazon deleted the abhng-lower-freq-1.3.1 branch April 12, 2022 21:33
prudhvigodithi pushed a commit to prudhvigodithi/opensearch-build that referenced this pull request Apr 13, 2022
…arch-project#1947)

Signed-off-by: Sagar Upadhyaya <[email protected]>

Adding code block for maven plugin zips

Signed-off-by: pgodithi <[email protected]>

Adding code block for maven plugin zips

Signed-off-by: pgodithi <[email protected]>

zip maven publish

Signed-off-by: pgodithi <[email protected]>

added zipsmaven dir check

Signed-off-by: pgodithi <[email protected]>

Adding better gradle project dir

Signed-off-by: pgodithi <[email protected]>

test maven snapshots

Signed-off-by: pgodithi <[email protected]>

test maven snapshots

Signed-off-by: pgodithi <[email protected]>

test maven snapshots

Signed-off-by: pgodithi <[email protected]>

test maven snapshots

Signed-off-by: pgodithi <[email protected]>

test maven snapshots

Signed-off-by: pgodithi <[email protected]>

test maven snapshots

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

Removing build script for PA (opensearch-project#1952)

Signed-off-by: Sagar Upadhyaya <[email protected]>

update release issue with steps to update ansible and helm (opensearch-project#1942)

* update release issue with steps to update ansible and helm

Signed-off-by: Abhinav Gupta <[email protected]>

* updated test as sample PR

Signed-off-by: Abhinav Gupta <[email protected]>

* adding periods

Signed-off-by: Abhinav Gupta <[email protected]>

Fix the path for maven sign and staging job (opensearch-project#1954)

* Fix the path for staging maven

Signed-off-by: Zelin Hao <[email protected]>

* Add the distribution folder tar to the artifact path and update test

Signed-off-by: Zelin Hao <[email protected]>

[OSD][2.0.0] use rc1 qualifier (opensearch-project#1949)

Signed-off-by: Kawika Avilla <[email protected]>

install createrepo in docker image for RPM yum repo support (opensearch-project#1955)

* install createrepo in docker image for RPM yum repo support

Signed-off-by: Tianle Huang <[email protected]>

* fix

Signed-off-by: Tianle Huang <[email protected]>

* update opensearch to use new image

Signed-off-by: Tianle Huang <[email protected]>

Add systemd docker for rpm service test (opensearch-project#1958)

* Add systemd based image for yum install testing and more

Signed-off-by: Peter Zhu <[email protected]>

* More packages to add to the images

Signed-off-by: Peter Zhu <[email protected]>

lower freq for build for 1.3.1 opensearch-dashboards (opensearch-project#1941)

* lower freq for build for 1.3.1 opensearch-dashboards

Signed-off-by: Abhinav Gupta <[email protected]>

* removed build for 1.2.x and 1.3.1

Signed-off-by: Abhinav Gupta <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

Added notifications-core and notifications to 2.0 and add integTest manifest (opensearch-project#1957)

* Added notifications-core and notifications to 2.0.

Signed-off-by: dblock <[email protected]>

* Remove scripts from opensearch-build.

Signed-off-by: dblock <[email protected]>

* Remove dashboards notifications scripts to use the default one

Signed-off-by: Peter Zhu <[email protected]>

* Remove CI checks for lack of gradlew.

Signed-off-by: dblock <[email protected]>

* Fix the second subfolder location after core

Signed-off-by: Peter Zhu <[email protected]>

* Add notifications related build scripts to build repo

Signed-off-by: Peter Zhu <[email protected]>

* Tweak build scripts to correctly move the zips

Signed-off-by: Peter Zhu <[email protected]>

Co-authored-by: Peter Zhu <[email protected]>

Make sure notifications is built correctly while passing ci checks (opensearch-project#1959)

Signed-off-by: Peter Zhu <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>

test snapshot build

Signed-off-by: pgodithi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants