✨ Bring in maven index from release

[jmle]

https://issues.redhat.com/browse/MTA-6231

Summary by CodeRabbit

• New Features

  • The container image now includes pre-packaged Maven index metadata, making it available to the application at runtime.

• Chores

  • Updated the Docker build process to download and incorporate Maven index data via an additional build stage.
  • Established a dedicated working directory in the image for the bundled index data.
  • Preserved existing build stages while integrating the new data into the final image.

[coderabbitai]

Walkthrough

Adds a new Docker multi-stage build step (index-download) that installs wget/zip, downloads and unpacks Maven index data, and copies two metadata files into the final image at /usr/local/etc/maven-index.txt and /usr/local/etc/maven-index.idx. Existing stages remain; final image now contains Maven index metadata.

Changes

Cohort / File(s)	Summary
Docker build pipeline Dockerfile	Introduces index-download stage (installs wget, zip), sets WORKDIR /maven-index-data, downloads/unzips Maven index archive, and copies central.archive-metadata.txt and central.archive-metadata.idx into final image as /usr/local/etc/maven-index.txt and /usr/local/etc/maven-index.idx.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Dev as Docker Build
  participant IDX as index-download Stage
  participant REM as Remote Maven Index
  participant FIN as Final Image

  Dev->>IDX: Build index-download stage
  IDX->>REM: wget Maven index ZIP
  REM-->>IDX: ZIP bytes
  IDX->>IDX: unzip to /maven-index-data

  Dev->>FIN: Build final stage
  FIN->>IDX: COPY central.archive-metadata.txt
  FIN->>IDX: COPY central.archive-metadata.idx
  note over FIN: Files placed at\n/usr/local/etc/maven-index.txt\n/usr/local/etc/maven-index.idx

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

• pranavgaikwad
• eemcmullan
• aufi

Poem

I hopped through layers, stage by stage,
Fetching Maven’s secrets like a sage.
Two little files, snug where they sit,
Tucked in the image—perfect fit.
Zip, unzip, whiskers twitch—hooray!
Build bunnies ship faster today. 🐇📦

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title clearly summarizes the main change of including the Maven index in the release image by highlighting "Bring in maven index," which aligns with the Dockerfile updates.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f0b6d70 and 37c343d.

📒 Files selected for processing (1)

• Dockerfile (3 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Build tackle2-addon-analyzer

[coderabbitai]

⚠️ Potential issue | 🔴 Critical

Install unzip before using it.

The new stage invokes unzip, but UBI’s zip package does not ship the unzip binary, so this layer fails during the build. Install unzip (or both zip unzip) before running the command.

-RUN microdnf install -y wget zip && microdnf clean all && rm -rf /var/cache/dnf
+RUN microdnf install -y wget unzip && microdnf clean all && rm -rf /var/cache/dnf

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	RUN microdnf install -y wget zip && microdnf clean all && rm -rf /var/cache/dnf
	WORKDIR /maven-index-data
	#TODO: get latest release when we get to update them periodically
	RUN wget --quiet https://github.com/konveyor/maven-search-index/releases/download/v0.0.1/maven-index-data-v0.0.1.zip -O maven-index-data.zip && unzip maven-index-data.zip && rm maven-index-data.zip
	RUN microdnf install -y wget unzip && microdnf clean all && rm -rf /var/cache/dnf
	WORKDIR /maven-index-data
	#TODO: get latest release when we get to update them periodically
	RUN wget --quiet https://github.com/konveyor/maven-search-index/releases/download/v0.0.1/maven-index-data-v0.0.1.zip -O maven-index-data.zip && unzip maven-index-data.zip && rm maven-index-data.zip

🤖 Prompt for AI Agents

In Dockerfile around lines 29 to 32, the RUN uses the unzip binary but only
installs the zip package (which on UBI doesn't provide unzip); update the RUN
that currently does "microdnf install -y wget zip ..." to include unzip (e.g.,
microdnf install -y wget zip unzip ...) so the unzip command is available, keep
the subsequent microdnf clean and rm -rf /var/cache/dnf as-is to avoid leaving
package cache.

[shawn-hurley]

How do we keep this up to date are there going to be version bumps?

[jmle]

We will have version bumps yeah, but for the moment we are going to stick to the current version. We can grab the latest version and hardcoded here or pass it through an argument in the future.