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

Fixes #37433 - Set indentation when calling subscription_manager_setup #10200

Closed
wants to merge 1 commit into from

Conversation

stejskalleos
Copy link
Contributor

Incorrect indentation in the snippet is causing yaml error in the cloud_init_default.erb

Follow up to #10153
Detailed explanation #10153 (comment)

@stejskalleos
Copy link
Contributor Author

@ShimShtein I don't know why Katello is failing, but it's not because of my changes.

ShimShtein
ShimShtein previously approved these changes Jun 10, 2024
Copy link
Member

@ShimShtein ShimShtein left a comment

Choose a reason for hiding this comment

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

LGTM, @ekohl any concerns?

Incorrect indentation in the snippet is causing yaml
error in the cloud_init_default.erb
@ekohl
Copy link
Member

ekohl commented Jul 29, 2024

@stejskalleos I think the test failures are related

@stejskalleos
Copy link
Contributor Author

PR is not needed anymore. #10153 has been updated and merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants