-
Notifications
You must be signed in to change notification settings - Fork 505
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
BIGTOP-4035: Fix ranger deb packaging and puppet script #1204
Conversation
@JiaLiangC Thank you for the update, and sorry that my previous comment was insufficient. For Rocky 8, we still have to keep the package name as "python36", otherwise we get the following error. To put it simply, we have to use "python36" only for Rocky 8 and "python3" for other distros including CentOS 7.
Other updates are supposed to work as expected, so I'll merge it once the issue above is fixed. |
@sekikn |
Hi @JiaLiangC |
|
Thank you for the update and advice @JiaLiangC @guyuqi. As Yuqi said, you can use the images published on DockerHub for running build (use bigtop/slaves for this purpose, in which both puppet and toolchain are installed) and smoke test (use bigtop/puppet, in which only puppet is installed).
But if we rebuild it based on the latest Rocky image, facter recognizes it as "Rocky".
So the following code
should be as follows. Would you fix it?
Regarding the trunk images, they should be periodically built and pushed into DockerHub so that developers can use the latest version. I'll configure the CI for doing so later. |
@sekikn already fixed |
+1, thank you so much for your contribution @JiaLiangC! I ensured the latest PR works with CentOS 7, Rocky 8 and Ubuntu 22.04. Only with Debian 11, running smoke test failed with the following error. But the current PR works with the most of the distros we support, so let's merge it first and then address the problem as another issue. I'll file it later.
|
* BIGTOP-4035: Fix ranger deb packaging and puppet script * fix rocky8 python package name * add rocky for latest Rocky images --------- Co-authored-by: jialiang <jialiangcaimd@gmail.com> (cherry picked from commit 63706a1) Signed-off-by: Nazarii Melnyk <nmelnyk@rework-space.com>
Description of PR
Fix ranger deb packaging and puppet script
How was this patch tested?
manual test
For code changes: