Skip to content

fix: bump ai-navigator-app to 0.5.2 (release-2.17)#4795

Open
lukeogg wants to merge 1 commit intorelease-2.17from
logg/release-2.17/ai-navigator-0.5.2-fix
Open

fix: bump ai-navigator-app to 0.5.2 (release-2.17)#4795
lukeogg wants to merge 1 commit intorelease-2.17from
logg/release-2.17/ai-navigator-0.5.2-fix

Conversation

@lukeogg
Copy link
Copy Markdown
Contributor

@lukeogg lukeogg commented Apr 18, 2026

Summary

Backport of the AI Navigator license-extraction fix for non-Nutanix infrastructure to NKP 2.17.x.

  • Bumps applications/ai-navigator-app/ from 0.5.2/0.5.3/
  • Chart ref.tag bumped 0.5.10.5.2 (the 0.5.2 directory on release-2.17 was introduced by the refactor: use ${releaseName}-${appVersion} prefix in resources #4247 templating refactor without a corresponding chart release; this PR is the first time a real 0.5.2 chart is actually pulled)
  • Chart directory advanced to 0.5.3 so Flux/kommander-cli see a new version dir and reconcile.

Refs

  • Backport tracking: NCN-113186
  • Original bug: NCN-113010 — 401 Unauthorized on Azure when using NKP/Nutanix license
  • Upstream fix: mesosphere/ai-navigator-app#118
  • Chart release: mesosphere/ai-navigator-app v0.5.2
  • Published chart: oci://ghcr.io/mesosphere/charts/ai-navigator-app:0.5.2

Test plan

  • CI (validate-licenses, flux validation) passes
  • Deploy NKP 2.17.2-dev build on Azure with NKP/Nutanix Ultimate license applied; AI Navigator chat request returns 200 (no longer 401)
  • Nutanix-infra regression check

Backport fix for license extraction on non-Nutanix infrastructure.

Chart tag: 0.5.2 (directory bumped to 0.5.3 to force reconciliation, since
the 0.5.2 directory on release-2.17 was reserved by the #4247 refactor
without a corresponding chart release).

Refs:
- NCN-113186 (backport tracking)
- NCN-113010 (original bug: 401 Unauthorized on Azure with NKP license)
- mesosphere/ai-navigator-app#118 (upstream fix)
- mesosphere/ai-navigator-app v0.5.2

Made-with: Cursor
@github-actions github-actions Bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Apr 18, 2026
@mesosphere-ci mesosphere-ci added applications/ai-navigator-app ok-to-test Signals mergebot that CI checks are ready to be kicked off update-licenses signals mergebot to update licenses.d2iq.yaml and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Apr 18, 2026
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 24594523301

Warning

No base build found for commit 3ff5cfa on release-2.17.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 33.945%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 327
Covered Lines: 111
Line Coverage: 33.94%
Coverage Strength: 1.39 hits per line

💛 - Coveralls

Copy link
Copy Markdown

@ragnar-howler ragnar-howler left a comment

Choose a reason for hiding this comment

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

🤖 Automated PR Review Complete

This PR has been scanned for:

  • ✅ Common security issues
  • ✅ Code style consistency
  • ✅ Test coverage indicators
  • ✅ Documentation completeness

Stats: +2 -2 in 6 files

Please address any automated checks above and request human review when ready.

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

Labels

applications/ai-navigator-app ok-to-test Signals mergebot that CI checks are ready to be kicked off ready-for-review update-licenses signals mergebot to update licenses.d2iq.yaml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants