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.

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

[6.12.z] oscap helper function fix #13847

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #13841

Problem Statement

#11544 altered fetch_scap_and_profile_id to use satellite, but the function defines it as the last parameter, while callers use it as the first one, hence AttributeError: 'str' object has no attribute 'cli'

Solution

changing the order of parameters

@Satellite-QE Satellite-QE added 6.12.z Introduced in or relating directly to Satellite 6.12 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Jan 22, 2024
@Gauravtalreja1 Gauravtalreja1 enabled auto-merge (squash) January 23, 2024 10:33
(cherry picked from commit edf63ff)
@Gauravtalreja1 Gauravtalreja1 force-pushed the cherry-pick-6.12.z-edf63fff0bc77af1d83cddc4c58ce5a3b6c972fe branch from f59dce2 to b83a23b Compare January 23, 2024 14:47
@Gauravtalreja1 Gauravtalreja1 enabled auto-merge (squash) January 23, 2024 14:48
@Gauravtalreja1 Gauravtalreja1 merged commit 17b4aa7 into 6.12.z Jan 23, 2024
5 checks passed
@Gauravtalreja1 Gauravtalreja1 deleted the cherry-pick-6.12.z-edf63fff0bc77af1d83cddc4c58ce5a3b6c972fe branch January 23, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.12.z Introduced in or relating directly to Satellite 6.12 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants