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 #19874: rudder package does not print the plugin content at install time #3854

Conversation

Fdall
Copy link
Contributor

@Fdall Fdall commented Aug 27, 2021

@@ -515,6 +515,12 @@ def extract_archive_from_rpkg(rpkgPath, dst, archive):
tar = run(
['tar', 'xvJ', '--no-same-owner', '-C', dst], input=ar[1], check=True, capture_output=True
)
files_trace = '\n {dst}'.format(dst=dst).join(tar[1].decode('utf-8').splitlines())
Copy link
Member

Choose a reason for hiding this comment

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

do
archive_contnet = tar[1].decode('utf-8')

and use it here and as return value

@Fdall
Copy link
Contributor Author

Fdall commented Aug 27, 2021

PR updated with a new commit

@Fdall
Copy link
Contributor Author

Fdall commented Aug 31, 2021

PR rebased

@Fdall Fdall force-pushed the bug_19874/rudder_package_does_not_print_the_plugin_content_at_install_time branch from 4674af7 to b7ae0cc Compare August 31, 2021 13:21
@Fdall
Copy link
Contributor Author

Fdall commented Aug 31, 2021

PR updated with a new commit

@Normation-Quality-Assistant
Copy link
Contributor

OK, squash merging this PR

@Normation-Quality-Assistant Normation-Quality-Assistant force-pushed the bug_19874/rudder_package_does_not_print_the_plugin_content_at_install_time branch from 01f754f to 328a76b Compare August 31, 2021 14:19
@Normation-Quality-Assistant Normation-Quality-Assistant merged commit 328a76b into Normation:branches/rudder/6.2 Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants