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

install_vm.py: add possibility to install GUI system #7004

Merged
merged 3 commits into from May 13, 2021

Conversation

matusmarhefka
Copy link
Member

This PR introduces --install-gui option into the install_vm.py script so it is easy to test also installations with GUI.

Additional changes:

I have tested both installations, with and without GUI, for the following:

  • RHEL7
  • RHEL8
  • Fedora 34

@pep8speaks
Copy link

Hello @matusmarhefka! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 118:100: E501 line too long (109 > 99 characters)
Line 166:22: E225 missing whitespace around operator
Line 168:26: E225 missing whitespace around operator
Line 188:100: E501 line too long (387 > 99 characters)

@openscap-ci
Copy link
Collaborator

Changes identified:
Others:
 Changes in Python files.

Show details

Others:
 Python abstract syntax tree change found in tests/install_vm.py.

Recommended tests to execute:
 (cd build && cmake ../ && ctest -j4)

@jan-cerny
Copy link
Collaborator

/retest

@jan-cerny
Copy link
Collaborator

Works amazing, I was able to install a F34 VM with GUI.

@jan-cerny jan-cerny merged commit e92a835 into ComplianceAsCode:master May 13, 2021
@jan-cerny jan-cerny self-assigned this May 13, 2021
@jan-cerny jan-cerny added this to the 0.1.57 milestone May 13, 2021
@matusmarhefka matusmarhefka deleted the install_vm_gui branch May 13, 2021 13:57
vojtapolasek pushed a commit that referenced this pull request May 24, 2021
install_vm.py: add possibility to install GUI system

(cherry picked from commit e92a835)
@vojtapolasek vojtapolasek added the backported-into-stabilization PRs which were cherry-picked during stabilization process. label May 24, 2021
@vojtapolasek vojtapolasek modified the milestones: 0.1.57, 0.1.56 May 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported-into-stabilization PRs which were cherry-picked during stabilization process.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants