Add CI, pre-commit hooks #269
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fix support for SSPv2
Bumps [ansible](https://github.com/ansible-community/ansible-build-data) from 2.10.7 to 8.5.0. - [Changelog](https://github.com/ansible-community/ansible-build-data/blob/main/docs/release-process.md) - [Commits](ansible-community/ansible-build-data@2.10.7...8.5.0) --- updated-dependencies: - dependency-name: ansible dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Bruce Becker <[email protected]>
Signed-off-by: Bruce Becker <[email protected]>
WIP rciam#268 Signed-off-by: Bruce Becker <[email protected]>
Signed-off-by: Bruce Becker <[email protected]>
Signed-off-by: Bruce Becker <[email protected]>
Ci/validate
Bump ansible from 2.10.7 to 8.5.0
Signed-off-by: Bruce Becker <[email protected]>
Signed-off-by: Bruce Becker <[email protected]>
Fix/apiserver
brucellino
added a commit
to brucellino/rciam-deploy
that referenced
this pull request
Mar 21, 2024
WIP rciam#269 Signed-off-by: Bruce Becker <[email protected]>
|
@NicolasLiampotis from my side, ready to merge. |
brucellino
changed the title
style(attrauth): fix lint issues in playbook style(apache): fix ansible-lint issues in defaults and handlers style(apache): fix lint in OS-dependent configuration tasks style(apache): fix linting issues in ssl config tasks style(apache): apply ansible-lint's opinionated fix to role style(apache): align name of handler to respect ansible-lint rules style(apache): apply lint fixes to main tasks style(comange-registry): apply linting rules to defaults style(comange-registry): apply linting rules to configure tasks style(comanage): declare no changes in ad-hoc commands This should be updated when we know how to identify a change in order to ensure idempotency style(comanage): split long install command over several lines style(comanage): apply linting fixes to install tasks style(comanage): explicitly ignore idempotency in cache clear task style(comanage): explicitly ignore idempotency in static files tasks style(php): fix linting errors in role defaults style(php): fix linting errors in role defaults style(php): fix linting errors in role taks and vars style(shibboleth-sp): fix linting errors in role defaults style(shibboleth-sp): fix linting errors in role main tasks style(shibboleth-sp): fix all linting errors in role WIP rciam#270 WIP rciam#267 style(common): apply linting rules to role defaults style(common): apply linting rules to role defaults style(common): fix linting errors in main role tasks style(common): apply linting fixes to role tasks and vars style(ssp): make role defaults pass ansible lint chore(deps): freeze current requirements for dependabot style(ssp): apply style rules to ssp handlers style(ssp): reduce linting warnings in role composer tasks style(ssp): apply linting rules to configure-common tasks style(ssp): ensure configure-saml and shib tasks pass lint style(ssp): ensure authservers playbook passes Ansible lint WIP rciam#270 style(authzkeys): conform to Ansible Lint style style(cache): conform to ansible style guide style(cert2saml): conform to ansible style for ssp module role style(comanage): conform comanage playbook and roles to Ansible style guide style(d4science): conform playbook to Ansible lint style(postgres): conform db playbook and postgres role to ansible lint style(egi-igtf): conform EGI IGTF role and playbook to Ansible Lint refactor(egi-igtf): move task to handler style(deep2saml): conform playbook to Ansible Lint style(federation-registry): conform to ansible lint Some task modules have been changed from postgresql_query to postgresql_script style(fedreg): conform to ansible lint style(firewall): conform to ansible lint style(google2saml): conform to ansible lint chore: ignore editor cruft style(keycloak): conform to Ansible Lint style(keycloak): conform to Ansible Lint style(metrics): conform metrics servers to Ansible Lint style(monservers): conform monservers to Ansible Lint style(oauth2saml): conform to Ansible Lint style(ldap): conform to Ansible Lint style(orcid): conform to Ansible Lint style(rciam-keycloak,nginx,rsyslog,webproxy): conform to Ansible Lint style(oidc): conform to Ansible Lint style(registry-ui): conform to Ansible Lit chore: add detect-secrets pre-commit hook style(saml,oidc): conform to Ansible Lint style(secrets): conform to Ansible Lint style(site): conform to Ansible Lint style(ssp): conform to Ansible Lint fix(apache): fix module name style(utils): conform to Ansible Lint style: conform to Ansible Lint --------- Signed-off-by: Bruce Becker <[email protected]> Signed-off-by: Bruce Becker <[email protected]>
|
@NicolasLiampotis This now passes Ansible Lint |
|
bump @NicolasLiampotis |
Before merging this, I suggest closing #272 and #274 to ensure we have a stable release in master. Merging now could potentially break several roles in the devel branch. |
Bumps [dnspython](https://github.com/rthalley/dnspython) from 2.1.0 to 2.6.1. - [Release notes](https://github.com/rthalley/dnspython/releases) - [Changelog](https://github.com/rthalley/dnspython/blob/main/doc/whatsnew.rst) - [Commits](rthalley/dnspython@v2.1.0...v2.6.1) --- updated-dependencies: - dependency-name: dnspython dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [jinja2](https://github.com/pallets/jinja) from 3.1.3 to 3.1.4. - [Release notes](https://github.com/pallets/jinja/releases) - [Changelog](https://github.com/pallets/jinja/blob/main/CHANGES.rst) - [Commits](pallets/jinja@3.1.3...3.1.4) --- updated-dependencies: - dependency-name: jinja2 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [cryptography](https://github.com/pyca/cryptography) from 42.0.5 to 43.0.1. - [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst) - [Commits](pyca/cryptography@42.0.5...43.0.1) --- updated-dependencies: - dependency-name: cryptography dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a few bits and pieces to help maintaining the project:
Closed #268
Closed #267
Closed #270