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
CLOUDSTACK-10341: Reduce systemvmtemplate size, install nftables #2506
Conversation
|
while improving, I wonder do we already have Any thoughts? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Thanks @resmo, we indeed install packages with the --no-install-recommends flag https://github.com/apache/cloudstack/blob/master/tools/appliance/systemvmtemplate/scripts/install_systemvm_packages.sh#L46 |
3c0b93b
to
9343e22
Compare
This reduces systemvmtemplate size by 600MB and installs nftables, updates iptables. Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
…te use UNKNOWN Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
9343e22
to
a7d33a7
Compare
|
@blueorangutan package |
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1833 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Manual tests have passed, let me kick a final run. |
|
@rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests |
|
Trillian test result (tid-2424)
|
|
Trillian test result (tid-2423)
|
|
Test LGTM, failures are due to environment issues or not related to this PR. Merging this based on results and two review LGTMs. |
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
|
Trillian test result (tid-2428)
|
Description
This reduces systemvmtemplate size by 600MB and installs nftables,
updates iptables.
Types of changes
Screenshots (if appropriate):
How Has This Been Tested?
Manually built using packer and tested locally by running smoketests.
Checklist: