Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix test AK create with CV #13642

Merged
merged 1 commit into from
Jan 8, 2024
Merged

Fix test AK create with CV #13642

merged 1 commit into from
Jan 8, 2024

Conversation

vsedmik
Copy link
Contributor

@vsedmik vsedmik commented Jan 8, 2024

Problem Statement

This parametrized test has been failing in 6.15 and 6.14 for some time. The error is

E   Warning: Option --environment is deprecated. Use --lifecycle-environment instead
E   Could not create the activation key:
E     Validation failed: Content view 'wMjLDMSyxhndvsDSHyvRFeLOBOZEhOXzLLPwsLcrCJvvAhR' is not in environment 'Library'

When creating an AK via UI we provide LCE first and then CV, which needs to be published/promoted to that LCE. Using that validation for CLI is right.

Solution

  1. Publish the CV so that it IS in Library LCE.
  2. Replace deprecated environment with newer lifecycle-environment.

@vsedmik vsedmik added CLI Issues and PRs involving the CLI Easy Fix :) Easiest Fix to review and quick merge request. CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 labels Jan 8, 2024
@vsedmik vsedmik self-assigned this Jan 8, 2024
@vsedmik vsedmik requested a review from a team as a code owner January 8, 2024 09:40
@vsedmik
Copy link
Contributor Author

vsedmik commented Jan 8, 2024

trigger: test-robottelo
pytest: tests/foreman/cli/test_activationkey.py -k test_positive_create_with_cv

@Gauravtalreja1 Gauravtalreja1 merged commit ffadf54 into SatelliteQE:master Jan 8, 2024
github-actions bot pushed a commit that referenced this pull request Jan 8, 2024
@vsedmik @web-flow
Fix test AK create with CV (#13642)
e7ca64e 
(cherry picked from commit ffadf54)
github-actions bot pushed a commit that referenced this pull request Jan 8, 2024
@vsedmik @web-flow
Fix test AK create with CV (#13642)
83fd8e3 
(cherry picked from commit ffadf54)
This was referenced Jan 8, 2024
Merged
Merged
shweta83 pushed a commit to shweta83/robottelo that referenced this pull request Apr 10, 2024
@vsedmik @shweta83
Fix test AK create with CV (SatelliteQE#13642)
fdf1b87
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@vsedmik vsedmik

Labels
6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches CLI Issues and PRs involving the CLI Easy Fix :) Easiest Fix to review and quick merge request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vsedmik @Gauravtalreja1