Skip to content

Fixes #39330 - Update bats test for removed CV/LCE activation key params#1947

Merged
zjhuntin merged 1 commit into
theforeman:masterfrom
jeremylenz:fix-activation-key-cv-lce-params
May 29, 2026
Merged

Fixes #39330 - Update bats test for removed CV/LCE activation key params#1947
zjhuntin merged 1 commit into
theforeman:masterfrom
jeremylenz:fix-activation-key-cv-lce-params

Conversation

@jeremylenz
Copy link
Copy Markdown
Contributor

@jeremylenz jeremylenz commented May 29, 2026

Summary

Test plan

  • Run the fb-katello-content bats pipeline and confirm the "create activation key" test passes

🤖 Generated with Claude Code

Replace deprecated --content-view and --lifecycle-environment flags with
the new --content-view-environments="LCE/CV" format in hammer activation-key create.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jeremylenz
Copy link
Copy Markdown
Contributor Author

@jturel thanks for the ack :) If you can merge, feel free (I don't seem to have access.)

@jturel
Copy link
Copy Markdown
Contributor

jturel commented May 29, 2026

Sadly I can't :(

@chris1984
Copy link
Copy Markdown
Member

@ekohl can this get a merge?

Copy link
Copy Markdown
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

I'm looking at https://projects.theforeman.org/issues/39330 and it only refers to a Jira issue when it was deprecated. Not on my work machine so I can't view it.

So my question is whether will fail on still supported Katello versions. If so, this really deserves a version check.

@jeremylenz
Copy link
Copy Markdown
Contributor Author

The new content_view_environments params have been available for quite some time now, and I'm pretty sure they would work on all supported versions. Is that enough to not require a version check?

@zjhuntin zjhuntin merged commit 7378c8f into theforeman:master May 29, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants