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

Refs #37130 - Set the mode on the generated bootstrap RPM #477

Merged
merged 1 commit into from
Mar 15, 2024

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Mar 1, 2024

@evgeni I should have tried harder to test it the first time.

@ehelms
Copy link
Member Author

ehelms commented Mar 1, 2024

We cannot use an implicit file resource here because the file that is generated depends on build time results of the rpmbuild process.

@ehelms
Copy link
Member Author

ehelms commented Mar 7, 2024

Tests are passing now that the puppet fix is available.

@evgeni
Copy link
Member

evgeni commented Mar 12, 2024

Should we do the same for the SRPM for consistency? It's not needed during deployment, but we do publish it in pub and if it's not readable it's useless.

@ehelms ehelms force-pushed the refs-37130-1 branch 2 times, most recently from e15b94c to 5197ba7 Compare March 13, 2024 13:18
Copy link
Contributor

@wbclark wbclark left a comment

Choose a reason for hiding this comment

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

In for a dime, in for a dollar...

lib/puppet/provider/bootstrap_rpm/bootstrap_rpm.rb Outdated Show resolved Hide resolved
lib/puppet/provider/bootstrap_rpm/bootstrap_rpm.rb Outdated Show resolved Hide resolved
lib/puppet/provider/bootstrap_rpm/bootstrap_rpm.rb Outdated Show resolved Hide resolved
Copy link
Member

@evgeni evgeni left a comment

Choose a reason for hiding this comment

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

Nice!

Copy link
Contributor

@wbclark wbclark left a comment

Choose a reason for hiding this comment

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

LGTM

@ehelms ehelms merged commit c2e868e into theforeman:master Mar 15, 2024
9 checks passed
@ekohl ekohl added the Bug label May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants