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

UYUNI MASTER - REFENV - rename module names and remove configurations to use cucumber testsuite default values #1474

Merged

Conversation

maximenoel8
Copy link
Contributor

@maximenoel8 maximenoel8 commented Nov 26, 2024

What does this PR do?

  • Modules components changes:

    • rename module names using _ to separate words and make sure module names are compatible with cucumber testsuite module.
    • remove name parameter to use default cucumber testsuite names. This will automatically use the new hostnames
  • Remove venv-salt references:

    • Eliminates all venv-salt references since it is now enabled by default in the updated cucumber testsuite module, simplifying configuration and reducing redundancy.

Related to: https://github.com/SUSE/spacewalk/issues/25062
Specific sub-task linked into this PR thread history
Depends on: https://gitlab.suse.de/galaxy/infrastructure/-/merge_requests/988
Depends on uyuni-project/sumaform#1662 but can be merge before. Using this changes in a temporary branch.

@maximenoel8 maximenoel8 requested a review from a team November 26, 2024 00:54
@maximenoel8 maximenoel8 self-assigned this Nov 26, 2024
@maximenoel8 maximenoel8 changed the title Rename uyuni master NUE refenv modules UYUNI MASTER - REFENV - rename module names and remove configurations to use cucumber testsuite default values Nov 26, 2024
@maximenoel8 maximenoel8 merged commit 71f7583 into SUSE:master Nov 26, 2024
1 check passed
@maximenoel8 maximenoel8 deleted the rename_modules_uyun_ref_master_NUE branch November 26, 2024 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants