Skip to content

Add patch position#429

Open
acidjunk wants to merge 2 commits into
SUNET:developfrom
acidjunk:add-patch-position
Open

Add patch position#429
acidjunk wants to merge 2 commits into
SUNET:developfrom
acidjunk:add-patch-position

Conversation

@acidjunk
Copy link
Copy Markdown
Collaborator

@acidjunk acidjunk commented Sep 6, 2025

Copied changes from: https://github.com/SUNET/cnaas-nms/pull/381/files

Solved discussion items

Todo: Not sure about the changes in the Dockerfile (build master vs develop)

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 6, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
10.6% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@indy-independence
Copy link
Copy Markdown
Member

Looks good!
Yes not sure why the BUILDBRANCH would need to be updated, you should be able to override with "--build-arg BUILDBRANCH=master" if needed so just revert that change I think?

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