Skip to content

Add linting Git workflow and local linting env#11

Merged
GavinHeff merged 16 commits into
mainfrom
addLinting
Jun 23, 2025
Merged

Add linting Git workflow and local linting env#11
GavinHeff merged 16 commits into
mainfrom
addLinting

Conversation

@GavinHeff
Copy link
Copy Markdown
Contributor

Added super-lint.env so linting can be performed locally, along with
ansible-lint, yamllint and Git workflow for linting.

@GavinHeff GavinHeff force-pushed the addLinting branch 2 times, most recently from a959349 to acae190 Compare May 20, 2025 15:30
@JohnGarbutt
Copy link
Copy Markdown

@GavinHeff have you tested this out manually on an Azimuth env? It looks like we are missing a CI job for this to test it out.

Comment thread .ansible-lint.yml Outdated
Comment thread ui-meta/sample-appliance.yml
Comment thread ui-meta/sample-appliance.yml Outdated
m-bull
m-bull previously requested changes Jun 17, 2025
Comment thread sample-appliance.yml Outdated
Comment thread sample-appliance.yml Outdated
Comment thread sample-appliance.yml Outdated
Comment thread sample-appliance.yml Outdated
Comment thread sample-appliance.yml Outdated
GavinHeff and others added 4 commits June 17, 2025 16:37
Co-authored-by: Matt Anson <matta@stackhpc.com>
Co-authored-by: Matt Anson <matta@stackhpc.com>
Co-authored-by: Matt Anson <matta@stackhpc.com>
Co-authored-by: Matt Anson <matta@stackhpc.com>
openstack_compute_floatingip_associate_v2 is deprecated,
updated to openstack_network_floatingip_associate_v2 and
changed instance_id to port_id. In roles/cluster_infra/
templates/resources.tf.j2
@GavinHeff GavinHeff force-pushed the addLinting branch 4 times, most recently from b677a23 to cf24801 Compare June 19, 2025 14:12
Using ansible.builtin.service in place of service and
adding handlers for roles/load_balancer/tasks/main.yml
@m-bull
Copy link
Copy Markdown
Contributor

m-bull commented Jun 23, 2025

Deployment tested by hand and working

@sd109 sd109 self-requested a review June 23, 2025 14:06
Copy link
Copy Markdown
Contributor

@sd109 sd109 left a comment

Choose a reason for hiding this comment

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

LGTM

@m-bull m-bull dismissed their stale review June 23, 2025 14:08

stale

@GavinHeff GavinHeff merged commit dfc169a into main Jun 23, 2025
9 checks passed
@GavinHeff GavinHeff deleted the addLinting branch June 23, 2025 14:20
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.

5 participants