Install correct version of networking-mlnx for openstack release

[jovial]

Updated neutron_mlnx_agent_pip_packages_override to use the stackhpc branch based on openstack_release.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: stackhpc/stackhpc-kayobe-config/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: ff14a096-3682-4ba8-8693-8d358e9ac8d7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

📝 Summary

Update neutron_mlnx_agent_pip_packages_override to install the StackHPC networking-mlnx branch that matches the configured OpenStack release.

Walkthrough

The Kolla build configuration now installs networking-mlnx from the release-specific StackHPC branch instead of the upstream master branch.

Changes

MLNX package source selection

Layer / File(s)	Summary
Update MLNX package override etc/kayobe/kolla.yml	The networking-mlnx package reference now uses the stackhpc/{{ openstack_release }} branch.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: 🟡 Moderate · up to 468d4

The change can select a nonexistent networking-mlnx branch for the default OpenStack release, which may stop the MLNX image build. Update the release-to-branch mapping before merging.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

---

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.}

[coderabbitai]

Actionable comments posted: 1

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 56e17421-f7cd-422d-aa05-6a4d6228e5ca

📥 Commits

Reviewing files that changed from the base of the PR and between 7fa8c88 and 468d407.

📒 Files selected for processing (1)

• etc/kayobe/kolla.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Review details

⏰ Context from checks skipped due to timeout. (3)

• GitHub Check: stackhpc/check-review
• GitHub Check: Build Kayobe Image / Build kayobe image
• GitHub Check: stackhpc/check

[jovial]

We likely need to restore the installation of networking-mlnx into neutron and nova containers since I believe they are not installed by default.

[jovial]

Will need to rebase on #2573 and rebuild.

[jovial]

Potentially this is the default. Its probably the neutron and nova plugins we need.

[jovial]

Trying to rebuild images again.

[mnasiadka]

Closing and reopening because there's a pep8 job stuck on zuul side.

[jovial]

Trying to rebuild here: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/35616612798