Skip to content

Conversation

@sachaudh
Copy link
Contributor

@sachaudh sachaudh commented Jan 8, 2026

Description

Updates the UI permission check for the Base Images route from Image to ImageAdministration to align with the backend.

Additionally, restricts modification actions (add, edit, delete) to users with write access for ImageAdministration. Users with read-only access will see a read-only view of the base images table.

User-facing documentation

  • CHANGELOG.md is updated OR update is not needed

Testing and quality

  • the change is production ready: the functionality is gated by ROX_BASE_IMAGE_DETECTION feature flag
  • CI results are inspected

Automated testing

  • modified existing tests

How I validated my change

  • Unit tests pass
  • Checked the UI with/without permissions

Screenshots

No Read/Write Access

Screenshot 2026-01-09 at 7 18 03 AM

Read Access Only

Screenshot 2026-01-09 at 7 17 03 AM

Read/Write Access

Screenshot 2026-01-09 at 7 17 12 AM

@openshift-ci
Copy link

openshift-ci bot commented Jan 8, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@rhacs-bot
Copy link
Contributor

rhacs-bot commented Jan 8, 2026

Images are ready for the commit at 64b712a.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.10.x-750-g64b712a077.

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.93%. Comparing base (0041570) to head (64b712a).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #18415      +/-   ##
==========================================
- Coverage   48.93%   48.93%   -0.01%     
==========================================
  Files        2631     2631              
  Lines      197964   197964              
==========================================
- Hits        96872    96868       -4     
- Misses      93703    93708       +5     
+ Partials     7389     7388       -1     
Flag Coverage Δ
go-unit-tests 48.93% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sachaudh sachaudh marked this pull request as ready for review January 9, 2026 14:54
@sachaudh sachaudh requested a review from a team as a code owner January 9, 2026 14:54
@sachaudh sachaudh requested review from a team and removed request for a team January 9, 2026 15:01
@sachaudh sachaudh force-pushed the ROX-32557-update-base-images-permission branch from 7b11122 to 623aa05 Compare January 9, 2026 17:17
@sachaudh
Copy link
Contributor Author

sachaudh commented Jan 9, 2026

Tests should be resolved once #18422 goes in. The reason is explained in this comment: #18422 (comment)

@sachaudh sachaudh force-pushed the ROX-32557-update-base-images-permission branch from 623aa05 to 64b712a Compare January 12, 2026 15:11
@sachaudh sachaudh merged commit b85ebba into master Jan 12, 2026
90 checks passed
@sachaudh sachaudh deleted the ROX-32557-update-base-images-permission branch January 12, 2026 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants