Skip to content

[TEST] Fix distro tests for linux packages#569

Merged
adnapibar merged 3 commits into
opensearch-project:mainfrom
adnapibar:packaging-tests
May 16, 2021
Merged

[TEST] Fix distro tests for linux packages#569
adnapibar merged 3 commits into
opensearch-project:mainfrom
adnapibar:packaging-tests

Conversation

@adnapibar
Copy link
Copy Markdown
Contributor

@adnapibar adnapibar commented Apr 16, 2021

Description

This PR includes changes that fix the failing OpenSearch distribution tests for packages (linux-archive, linux-archive-aarch64, deb, rpm, docker) on supported linux distros.

Note: The oss tag will be removed in a separate issue.

Issues Resolved

Relates #538

Check List

  • All tests pass
  • 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.

Signed-off-by: Rabi Panda adnapibar@gmail.com

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

✅   Gradle Wrapper Validation success a28fa4f36d91fe30dda09697176734fdd420c3c0

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

✅   DCO Check Passed a28fa4f36d91fe30dda09697176734fdd420c3c0

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

✅   Gradle Precommit success a28fa4f36d91fe30dda09697176734fdd420c3c0

@adnapibar
Copy link
Copy Markdown
Contributor Author

adnapibar commented Apr 16, 2021

start gradle check

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

❌   Gradle Check failure a28fa4f36d91fe30dda09697176734fdd420c3c0
Log 97

Reports 97

@adnapibar adnapibar mentioned this pull request Apr 16, 2021
Closed
2 tasks
@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

✅   Gradle Wrapper Validation success 74b3c8e800749adeca972e6fa2c107a1ea5ea66a

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

✅   DCO Check Passed 74b3c8e800749adeca972e6fa2c107a1ea5ea66a

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

✅   Gradle Precommit success 74b3c8e800749adeca972e6fa2c107a1ea5ea66a

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

✅   Gradle Wrapper Validation success 140179f3c5ac33b85fc4a85938aacf628bf20c8c

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

✅   DCO Check Passed 140179f3c5ac33b85fc4a85938aacf628bf20c8c

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

✅   Gradle Precommit success 140179f3c5ac33b85fc4a85938aacf628bf20c8c

@adnapibar
Copy link
Copy Markdown
Contributor Author

adnapibar commented Apr 16, 2021

start gradle check

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 16, 2021

✅   Gradle Check success 140179f3c5ac33b85fc4a85938aacf628bf20c8c
Log 101

Reports 101

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 19, 2021

✅   DCO Check Passed acfe3b6704905547f177ce8014ad12700d8f0df0

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 19, 2021

✅   Gradle Wrapper Validation success acfe3b6704905547f177ce8014ad12700d8f0df0

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 19, 2021

✅   Gradle Precommit success acfe3b6704905547f177ce8014ad12700d8f0df0

@adnapibar adnapibar requested a review from nknize April 20, 2021 00:52
@adnapibar
[TEST] Fix distro tests for linux packages
c5d9953 
This commit includes changes that fix the failing OpenSearch distribution tests for packages (linux-archive, linux-archive-aarch64, debian, rpm, docker) on supported linux distros.

Signed-off-by: Rabi Panda <adnapibar@gmail.com>
@adnapibar
Fix license string for rpm distros.
21920db 
Signed-off-by: Rabi Panda <adnapibar@gmail.com>
@adnapibar
Rebase with main and remove "-oss" from the changes.
ad500ab 
Signed-off-by: Rabi Panda <adnapibar@gmail.com>
@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 20, 2021

✅   DCO Check Passed ad500ab

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 20, 2021

✅   Gradle Wrapper Validation success ad500ab

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 20, 2021

✅   Gradle Precommit success ad500ab

@adnapibar
Copy link
Copy Markdown
Contributor Author

adnapibar commented Apr 20, 2021

start gradle check

@odfe-release-bot
Copy link
Copy Markdown

odfe-release-bot commented Apr 20, 2021

✅   Gradle Check success ad500ab
Log 118

Reports 118

@adnapibar adnapibar added >test-failure Test failure from CI, local build, etc. v1.0.0 Version 1.0.0 v1.0.0-alpha1 Version 1.0.0 alpha 1 v2.0.0 Version 2.0.0 labels Apr 22, 2021
harold-wang
harold-wang reviewed Apr 26, 2021
View reviewed changes
Comment thread distribution/packages/build.gradle
// this is package indepdendent configuration
ospackage {
maintainer 'OpenSearch Team'
maintainer 'OpenSearch Team <opensearch@amazon.com>'
Copy link
Copy Markdown
Contributor

@harold-wang harold-wang Apr 26, 2021

Choose a reason for hiding this comment

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

Do we want to change it to opensearch.org domain instead of amazon domain?

Copy link
Copy Markdown
Contributor Author

@adnapibar adnapibar Apr 26, 2021

Choose a reason for hiding this comment

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

This is the email being used currently as there is no functioning email with the opensearch.org domain yet.

@adnapibar adnapibar removed the v1.0.0-alpha1 Version 1.0.0 alpha 1 label May 7, 2021
@adnapibar adnapibar mentioned this pull request May 11, 2021
Closed
@adnapibar adnapibar requested a review from harold-wang May 12, 2021 14:26
setiah
setiah approved these changes May 15, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@setiah setiah left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM!

@setiah
Copy link
Copy Markdown
Contributor

setiah commented May 15, 2021

start gradle check

@opensearch-ci-bot
Copy link
Copy Markdown
Collaborator

opensearch-ci-bot commented May 15, 2021

❌   Gradle Check failure ad500ab
Log 182

Reports 182

@adnapibar
Copy link
Copy Markdown
Contributor Author

adnapibar commented May 15, 2021

start gradle check

@opensearch-ci-bot
Copy link
Copy Markdown
Collaborator

opensearch-ci-bot commented May 15, 2021

✅   Gradle Check success ad500ab
Log 184

Reports 184

@adnapibar adnapibar merged commit c2e816e into opensearch-project:main May 16, 2021
@adnapibar adnapibar added the pending backport Identifies an issue or PR that still needs to be backported label May 19, 2021
@adnapibar adnapibar mentioned this pull request May 20, 2021
Merged
dblock pushed a commit that referenced this pull request May 20, 2021
@adnapibar
[TEST] Fix failing distro tests for linux packages (#569) (#733)
3290932 
Changes to fix the failing OpenSearch distribution tests for packages (linux-archive, linux-archive-aarch64, debian, rpm, docker) on supported linux distros. 

Signed-off-by: Rabi Panda <adnapibar@gmail.com>
@adnapibar adnapibar removed the pending backport Identifies an issue or PR that still needs to be backported label May 21, 2021
@adnapibar adnapibar mentioned this pull request May 21, 2021
Closed
@adnapibar adnapibar deleted the packaging-tests branch November 15, 2022 07:47
@adnapibar adnapibar restored the packaging-tests branch November 15, 2022 07:47
@adnapibar adnapibar deleted the packaging-tests branch November 15, 2022 07:47
ritty27 pushed a commit to ritty27/OpenSearch that referenced this pull request May 12, 2024
@reta
Exclude Apache HttpClient5 transitive dependencies on Apache HttpCore5 (
ccbe10a 
opensearch-project#569)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nknize nknize Awaiting requested review from nknize

@harold-wang harold-wang Awaiting requested review from harold-wang

1 more reviewer

@setiah setiah setiah approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

>test-failure Test failure from CI, local build, etc. v1.0.0 Version 1.0.0 v2.0.0 Version 2.0.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@adnapibar @odfe-release-bot @setiah @opensearch-ci-bot @harold-wang