Skip to content

Conversation

@sandeep318kumar
Copy link
Contributor

What changes were proposed in this pull request?

upgrading commons-collections to 4.4

(Please fill in changes proposed in this fix)

How was this patch tested?

tested on installing a local cluster.
all services and ambari server was start with no error.

(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)

Please review Ambari Contributing Guide before opening a pull request.

@JiaLiangC JiaLiangC force-pushed the upgrade/jdk-spring-dependencies branch from c675284 to 39b6434 Compare November 8, 2024 06:14
@JiaLiangC
Copy link
Contributor

@sandeep318kumar Since the previous /jdk-spring-dependencies was not correct, I updated it and submitted a PR for verification. All tests have passed. Your PR's CI/CD failed. You can take a look at the PR I submitted based on upgrade/jdk-spring-dependencies. The CI/CD has passed: #3873"

Let me know if you need further adjustments!

@sandeep318kumar
Copy link
Contributor Author

Few test cases have failed in CI. Working on fixing them.

@sandeep318kumar
Copy link
Contributor Author

sandeep318kumar commented Dec 16, 2024

Http status 400- bad request is coming while opening the CI run.

image

@JiaLiangC @virajjasani @zRains any idea? how to view CI runs now?

@JiaLiangC
Copy link
Contributor

@sandeep318kumar The Apache foundation team updated the CI environment, which caused incorrect redirect links here. I'll provide you with a link, please check if you can open it. https://ci-hadoop.apache.org/job/Ambari/job/Ambari-PreCommit-GitHub-PR/view/change-requests/job/PR-3871/
image

@JiaLiangC
Copy link
Contributor

@sandeep318kumar
Copy link
Contributor Author

Thanks @JiaLiangC I am able to open this.

@sandeep318kumar sandeep318kumar force-pushed the commons4.4 branch 2 times, most recently from 01aecc8 to 092b1e9 Compare December 16, 2024 19:45
…esourceProviderTest& ExecutionSchedulerTest& AmbariProxiedUserDetailsServiceTest
@sandeep318kumar sandeep318kumar force-pushed the commons4.4 branch 2 times, most recently from 938308d to 92b5823 Compare January 2, 2025 07:00
@JiaLiangC JiaLiangC deleted the branch apache:upgrade/jdk-spring-dependencies January 22, 2025 02:08
@JiaLiangC JiaLiangC closed this Jan 22, 2025
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.

2 participants