Skip to content

Upgrade Mimir #11274

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Upgrade Mimir #11274

wants to merge 6 commits into from

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Oct 15, 2025 •
edited
Loading

And collect more information, and we seems loose some information in case of IT failures.

Changes:

  • upgrade to Mimir 0.10.0
  • GH action collects IT failsafe outputs as well (as I sus we have something on stdout or stderr)
  • centrally manage Toolbox and Mimir versions
  • make Maven 3 tests use Mimir as well (by doing UW or PW setup for it); so far those did not use Mimir, but went directly to Central instead
  • in maven-executor and maven-cli test disable prefix RRF (for now)

@cstamas
Upgrade Mimir
efbce97 
And collect more information, and we seems loose some information
in case of IT failures.
@cstamas cstamas added this to the 4.1.0 milestone Oct 15, 2025
@cstamas cstamas self-assigned this Oct 15, 2025
@cstamas cstamas marked this pull request as ready for review October 16, 2025 08:54
gnodet
gnodet reviewed Oct 16, 2025
View reviewed changes
...-cli/src/test/java/org/apache/maven/cling/invoker/mvn/resident/ResidentMavenInvokerTest.java Outdated
}

@Disabled("Until we move off fully from File")
@Disabled("Enable when core moves off fully from FS")
Copy link
Contributor

Choose a reason for hiding this comment

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

The FS is ambiguous. We need to enable it when we fully move to the NIO2 api with Path/java.nio.file.Filesystem, rather than File/java.io.FileSystem, right ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, let me reword this

Copy link
Member Author

Choose a reason for hiding this comment

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

    @Disabled("Enable it when fully moved to NIO2 with Path/Filesystem (ie MavenExecutionRequest)")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gnodet gnodet gnodet approved these changes

@slawekjaranowski slawekjaranowski Awaiting requested review from slawekjaranowski

Assignees

@cstamas cstamas

Labels

maintenance

Projects

None yet

Milestone

4.1.0

Development

Successfully merging this pull request may close these issues.

2 participants

@cstamas @gnodet