Skip to content

[1.2] Bump reports plugins to 1.2.1#253

Merged
joshuali925 merged 4 commits intoopensearch-project:1.2from
joshuali925:bump-1.2.1
Dec 11, 2021
Merged

[1.2] Bump reports plugins to 1.2.1#253
joshuali925 merged 4 commits intoopensearch-project:1.2from
joshuali925:bump-1.2.1

Conversation

@joshuali925
Copy link
Copy Markdown
Member

@joshuali925 joshuali925 commented Dec 10, 2021

Signed-off-by: Joshua Li joshuali925@gmail.com

Description

Bump reports plugins to 1.2.1 for opensearch 1.2.1 release

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • 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.

@joshuali925
Bump reports plugins to 1.2.1
9922500 
Signed-off-by: Joshua Li <joshuali925@gmail.com>
zhongnansu
zhongnansu previously approved these changes Dec 10, 2021
View reviewed changes
ps48
ps48 previously approved these changes Dec 10, 2021
View reviewed changes
@joshuali925
Revert version bump for reporting frontend
ca49ef4 
Signed-off-by: Joshua Li <joshuali925@gmail.com>
@joshuali925 joshuali925 dismissed stale reviews from ps48 and zhongnansu via ca49ef4 December 10, 2021 23:29
This was referenced Dec 11, 2021
Closed
Merged
@peternied
Copy link
Copy Markdown
Member

peternied commented Dec 11, 2021

@joshuali925 E: Failed to fetch http://security.ubuntu.com/ubuntu/pool/main/n/nss/libnss3-dev_3.49.1-1ubuntu1.5_amd64.deb 404 Not Found [IP: 52.250.76.244 80] Can you look into this?

@joshuali925
Copy link
Copy Markdown
Member Author

joshuali925 commented Dec 11, 2021

@joshuali925 E: Failed to fetch http://security.ubuntu.com/ubuntu/pool/main/n/nss/libnss3-dev_3.49.1-1ubuntu1.5_amd64.deb 404 Not Found [IP: 52.250.76.244 80] Can you look into this?

This is frontend CI failing, which was fixed in #245 . I did not pick it over because we are only bumping/releasing backend plugin, the frontend CI can be ignored

@joshuali925 joshuali925 changed the title Bump reports plugins to 1.2.1 [1.2] Bump reports plugins to 1.2.1 Dec 11, 2021
@zhongnansu
Copy link
Copy Markdown
Member

zhongnansu commented Dec 11, 2021

@joshuali925 CI failed. The integ test needs the latest 1.2.1.0 job-scheduler artifacts to be in test/resources

@joshuali925
Copy link
Copy Markdown
Member Author

joshuali925 commented Dec 11, 2021

@joshuali925 CI failed. The integ test needs the latest 1.2.1.0 job-scheduler artifacts to be in test/resources

building now

@joshuali925
Add job scheduler 1.2.1.0 artifact
ff2282b 
Signed-off-by: Joshua Li <joshuali925@gmail.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 11, 2021 •
edited
Loading

Codecov Report

Merging #253 (9ab1da7) into 1.2 (290523a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##                1.2     #253   +/-   ##
=========================================
  Coverage     64.54%   64.54%           
  Complexity      294      294           
=========================================
  Files            97       97           
  Lines          4045     4045           
  Branches        620      620           
=========================================
  Hits           2611     2611           
  Misses         1293     1293           
  Partials        141      141
Flag Coverage Δ
dashboards-reports 74.86% <ø> (ø)
reports-scheduler 54.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 290523a...9ab1da7. Read the comment docs.

zhongnansu
zhongnansu previously approved these changes Dec 11, 2021
View reviewed changes
@joshuali925
Fix frontend CI
9ab1da7 
Signed-off-by: Joshua Li <joshuali925@gmail.com>
@joshuali925 joshuali925 merged commit 3ae565f into opensearch-project:1.2 Dec 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ps48 ps48 ps48 approved these changes

+1 more reviewer

@zhongnansu zhongnansu zhongnansu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@joshuali925 @peternied @zhongnansu @codecov-commenter @ps48