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

Update distributionUrl maven to 3.9.9 #2247

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jcarranzan
Copy link
Contributor

@jcarranzan jcarranzan commented Dec 12, 2024

Summary

This task is related to the maven upgrade to 3.9.9 and I've used the command to do the changes:
mvn wrapper:wrapper -Dmaven=3.9.9

Please select the relevant options.

  • Bug fix (non-breaking change which fixes an issue)
  • Dependency update
  • Refactoring
  • Backport
  • New scenario (non-breaking change which adds functionality)
  • This change requires a documentation update
  • This change requires execution against OCP (use run tests phrase in comment)

Checklist:

  • Methods and classes used in PR scenarios are meaningful
  • Commits are well encapsulated and follow the best practices

@rsvoboda
Copy link
Member

Latest 3.9.z is 3.9.9, why moving to 3.9.6?

Please run mvn wrapper:wrapper -Dmaven=3.9.9.

There are other changes, e.g. in scripts.

@jcarranzan
Copy link
Contributor Author

Latest 3.9.z is 3.9.9, why moving to 3.9.6?

Please run mvn wrapper:wrapper -Dmaven=3.9.9.

There are other changes, e.g. in scripts.

I noticed that the latest stable version in the 3.9.x series is actually 3.9.9. Initially, I selected 3.9.6 based on previous discussions indicating that 3.9.6 was acceptable.
I’ll proceed with upgrading to 3.9.9 (run mvn wrapper:wrapper -Dmaven=3.9.9) and also update the other MRs I already did related to this in our infrastructure.
Thanks.

@jcarranzan jcarranzan changed the title Update distributionUrl maven to 3.9.6 Update distributionUrl maven to 3.9.9 Dec 13, 2024
Copy link
Member

@michalvavrik michalvavrik left a comment

Choose a reason for hiding this comment

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

There is no reason for tests to be failing, both daily build and my PR that run full CI are green, either you need to rebase this PR on current main or the failures are related. Please have a look.

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.

3 participants