Skip to content

CI: Ignore beta Sonar rule text:s8565#6921

Merged
gmuloc merged 1 commit intoaristanetworks:develfrom
gmuloc:ci-ignore-sonar-rule-s8565
May 5, 2026
Merged

CI: Ignore beta Sonar rule text:s8565#6921
gmuloc merged 1 commit intoaristanetworks:develfrom
gmuloc:ci-ignore-sonar-rule-s8565

Conversation

@gmuloc
Copy link
Copy Markdown
Contributor

@gmuloc gmuloc commented May 5, 2026

Change Summary

Right now merge queue is failing because of a new rule introduced May 4th 2026 that requires us to add a lock file to the git repo to pin dependencies. We don't want to do this now as pyavd is a library (this is one of the reason suggested to omit the rule: https://sonarcloud.io/organizations/aristanetworks-1/rules?open=text%3AS8565&rule_key=text%3AS8565)

Related Issue(s)

CI failing in merge queueu

Component(s) name

ci

Proposed changes

Add voodoo syntax to sonar-cloud.properties to ignore the rule

How to test

need to merge \(╯°□°)╯

@gmuloc gmuloc requested review from a team as code owners May 5, 2026 10:32
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

Review docs on Read the Docs

To test this pull request:

# Create virtual environment for this testing below the current directory
python -m venv test-avd-pr-6921
# Activate the virtual environment
source test-avd-pr-6921/bin/activate
# Install all requirements including PyAVD
pip install "pyavd[ansible] @ git+https://github.com/gmuloc/avd.git@ci-ignore-sonar-rule-s8565#subdirectory=python-avd" --force
# Point Ansible collections path to the Python virtual environment
export ANSIBLE_COLLECTIONS_PATH=$VIRTUAL_ENV/ansible_collections
# Install Ansible collection
python -m ansible.cli.galaxy collection install git+https://github.com/gmuloc/avd.git#/ansible_collections/arista/avd/,ci-ignore-sonar-rule-s8565 --force
cd test-avd-pr-6921
# Run your playbook using `python -m ansible.cli.playbook path/to/playbook.yml ...`

You can also test this PR using AVD playground:

  • Rebase your branch to makes sure it is up-to-date and has latest lab topologies for example inventories
  • Authenticate on labs.arista.com. Use your email! Not login.
  • Start the playground
  • In the lab selector UI pick "I want to use a specific AVD fork"
  • Enter following parameters:
GitHub org: gmuloc
Repository name: avd
Branch: ci-ignore-sonar-rule-s8565
  • Select an example inventory to test the PR
  • Once the AVD Playground setup will be finished, type make start and test anything once the lab is up

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 5, 2026

@gmuloc gmuloc enabled auto-merge May 5, 2026 12:07
@gmuloc gmuloc disabled auto-merge May 5, 2026 12:07
@gmuloc gmuloc merged commit e685e90 into aristanetworks:devel May 5, 2026
83 checks passed
@github-actions github-actions Bot added the rn: CI label May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants