feat(ENG-11221): modify alpine upstream rsa fields - #284
Conversation
There was a problem hiding this comment.
Pull request overview
Adds CLI table output support for Alpine upstream RSA signature verification fields, aligning cloudsmith upstream alpine ls with newly exposed API configuration properties.
Changes:
- Appends Alpine-only RSA fields to upstream table rows (
rsa_key_inline,rsa_key_url,rsa_verification,rsa_verification_status). - Adds corresponding Alpine-only table headers for the new RSA fields.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
poconnor-cloud
changed the title
|
It's worth moving this one back to draft, as without the update to the cloudsmith-api bindings, these changes won't exist yet. Once the API changes are merged, then a new set of bindings can be generated within |
- Simplify the alpine RSA field null-coalescing in print_upstreams() to the single `u.get(...) or ""` idiom already used elsewhere in build_row(), and truncate rsa_key_url like the other unbounded upstream_url/rsa_key_inline columns for consistent table widths. - Add the missing @pytest.mark.integration marker to test_alpine_upstream_ls_pretty_rsa_columns, which calls the live Cloudsmith API but was previously running in neither the unit nor the integration CI job. - Give rsa_key_url a value distinct from upstream_url so the assertion can actually catch a dropped RSA Key URL column, replace the tautological "RSA Verification"/"RSA Verification Status" substring asserts with a single occurrence count, and assert on rendered rsa_verification/rsa_verification_status values to cover the two remaining untested RSA columns. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Closing this out for now; revisiting later alongside the frontend work. No one is currently using the explicitly provided RSA keys for Alpine and Wolfi, and there's still outstanding UI work needed to surface this properly. Since both are unresolved, it makes more sense to tackle them together than to ship one half now. |
Description
Add CLI support for the new RSA signature verification fields on Alpine upstreams. The web-server API now exposes rsa_key_inline, rsa_key_url, rsa_verification, and rsa_verification_status on Alpine upstream configs. This change surfaces those fields in the
cloudsmith upstream alpine lstable output.Create and update commands require no changes since they pass JSON config directly to the API.
Type of Change
Additional Notes
Requires a cloudsmith-api SDK bump when the new SDK version ships with the updated AlpineUpstream model; until then, the fields will gracefully show empty strings via