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

Gc add on #280

Merged
merged 7 commits into from
Sep 28, 2023
Merged

Gc add on #280

merged 7 commits into from
Sep 28, 2023

Conversation

sluetze
Copy link
Contributor

@sluetze sluetze commented May 15, 2023

Description

added add-on role for garbagecollection

Checklist/ToDo's

  • Added documentation?
  • Added release note entry? ( docs/release-notes.md )
  • Tested?

@rbo
Copy link
Contributor

rbo commented May 15, 2023

/ok-to-test

@skull-cicd
Copy link

skull-cicd bot commented May 15, 2023

💀 There was an error during pre-commit / linter:

[INFO] Initializing environment for https://github.com/ansible-community/ansible-lint.git.
[INFO] Initializing environment for https://github.com/ansible-community/ansible-lint.git:.[community,yamllint].
[INFO] Installing environment for https://github.com/ansible-community/ansible-lint.git.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
Ansible-lint.............................................................Failed
- hook id: ansible-lint
- exit code: 2

[WARNING]: Ansible is being run in a world writable directory
(/workspace/source), ignoring it as an ansible.cfg source. For more information
see
https://docs.ansible.com/ansible/devel/reference_appendices/config.html#cfg-in-
world-writable-dir
[WARNING]: Ansible is being run in a world writable directory
(/workspace/source), ignoring it as an ansible.cfg source. For more information
see
https://docs.ansible.com/ansible/devel/reference_appendices/config.html#cfg-in-
world-writable-dir
WARNING  Loading custom .yamllint config file, this extends our internal yamllint config.
WARNING  Listing 4 violation(s) that are fatal
yaml: no new line character at the end of file (new-line-at-end-of-file)
ansible/add-on-roles/garbagecollection/defaults/main.yml:3

yaml: no new line character at the end of file (new-line-at-end-of-file)
ansible/add-on-roles/garbagecollection/meta/main.yml:20

yaml: no new line character at the end of file (new-line-at-end-of-file)
ansible/add-on-roles/garbagecollection/tasks/post-install.yml:18

yaml: trailing spaces (trailing-spaces)
ansible/roles/openshift-4-cluster/tasks/create.yml:140

You can skip specific rules or tags by adding them to your configuration file:
# .ansible-lint
warn_list:  # or 'skip_list' to silence them completely
  - yaml  # Violations reported by yamllint

Finished with 4 failure(s), 0 warning(s) on 172 files.

@sluetze
Copy link
Contributor Author

sluetze commented May 16, 2023

fixed the lint errors. i obviously wasnt concentrated when building the branch and included some commits which are related to another PR. I close the other PR, since its quite small stuff.

so this also fixes #278

@skull-cicd
Copy link

skull-cicd bot commented May 16, 2023

💀 There was an error during pre-commit / linter:

[INFO] Initializing environment for https://github.com/ansible-community/ansible-lint.git.
[INFO] Initializing environment for https://github.com/ansible-community/ansible-lint.git:.[community,yamllint].
[INFO] Installing environment for https://github.com/ansible-community/ansible-lint.git.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
Ansible-lint.............................................................Failed
- hook id: ansible-lint
- exit code: 2

[WARNING]: Ansible is being run in a world writable directory
(/workspace/source), ignoring it as an ansible.cfg source. For more information
see
https://docs.ansible.com/ansible/devel/reference_appendices/config.html#cfg-in-
world-writable-dir
[WARNING]: Ansible is being run in a world writable directory
(/workspace/source), ignoring it as an ansible.cfg source. For more information
see
https://docs.ansible.com/ansible/devel/reference_appendices/config.html#cfg-in-
world-writable-dir
WARNING  Loading custom .yamllint config file, this extends our internal yamllint config.
WARNING  Listing 2 violation(s) that are fatal
yaml: no new line character at the end of file (new-line-at-end-of-file)
ansible/add-on-roles/garbagecollection/meta/main.yml:21

yaml: trailing spaces (trailing-spaces)
ansible/add-on-roles/garbagecollection/meta/main.yml:21

You can skip specific rules or tags by adding them to your configuration file:
# .ansible-lint
warn_list:  # or 'skip_list' to silence them completely
  - yaml  # Violations reported by yamllint

Finished with 2 failure(s), 0 warning(s) on 172 files.

Copy link
Contributor

@rbo rbo left a comment

Choose a reason for hiding this comment

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

Please remove tags add-ons let's just use post-install and post-install-add-ons thanks.

@sluetze sluetze changed the title WIP: Gc add on Gc add on Jul 10, 2023
@sluetze sluetze requested a review from rbo September 27, 2023 14:25
@rbo
Copy link
Contributor

rbo commented Sep 28, 2023

/retest

@rbo
Copy link
Contributor

rbo commented Sep 28, 2023

LGTM

@rbo rbo merged commit 31a489b into RedHat-EMEA-SSA-Team:devel Sep 28, 2023
2 checks passed
@rbo
Copy link
Contributor

rbo commented Sep 28, 2023

Thank you for your contribution.

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