Skip to content

[ACA-5022] Configure SonarCloud phase2 - #186

Merged
brahmanim merged 7 commits into
redhat-cop:mainfrom
brahmanim:ACA-5022_configure-sonarcloud_phase2
Jun 23, 2026
Merged

[ACA-5022] Configure SonarCloud phase2#186
brahmanim merged 7 commits into
redhat-cop:mainfrom
brahmanim:ACA-5022_configure-sonarcloud_phase2

Conversation

@brahmanim

@brahmanim brahmanim commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

SUMMARY
Resolves ACA-5022

Reported CI Issues
Integration tests #184

@brahmanim
brahmanim force-pushed the ACA-5022_configure-sonarcloud_phase2 branch from 6943266 to 83d2ce2 Compare June 14, 2026 12:47
@brahmanim
brahmanim force-pushed the ACA-5022_configure-sonarcloud_phase2 branch from 83d2ce2 to 6b8d836 Compare June 14, 2026 12:57
@brahmanim
brahmanim force-pushed the ACA-5022_configure-sonarcloud_phase2 branch 9 times, most recently from 915501f to 7537f9a Compare June 15, 2026 15:22
@brahmanim
brahmanim force-pushed the ACA-5022_configure-sonarcloud_phase2 branch 4 times, most recently from 786b5f9 to 4bc71a6 Compare June 16, 2026 07:21
@brahmanim
brahmanim force-pushed the ACA-5022_configure-sonarcloud_phase2 branch from 4bc71a6 to cf066f8 Compare June 16, 2026 07:35
@brahmanim

brahmanim commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

SonarCloud coverage tests for running integration tests are working.
I have checked this workflow, with workflow_run that always runs, and changed to workflow_call, to avoid the security issue of Make sure that no untrusted code is executed from a fork.
There are Multiple integration test failures due to Python\botocore dependencies, which will be handled on a separate PR (see reported CI issue).

@brahmanim
brahmanim marked this pull request as ready for review June 17, 2026 06:13
@brahmanim brahmanim changed the title Aca 5022 configure sonarcloud phase2 [ACA-5022] Configure SonarCloud phase2 Jun 17, 2026
@brahmanim
brahmanim requested a review from beeankha June 22, 2026 10:58
@brahmanim
brahmanim merged commit 39530ae into redhat-cop:main Jun 23, 2026
13 of 14 checks passed
@patchback

patchback Bot commented Jun 23, 2026

Copy link
Copy Markdown

Backport to stable-3: 💔 cherry-picking failed — conflicts found

❌ Failed to cleanly apply 39530ae on top of patchback/backports/stable-3/39530aecf29a4f69da8c07bbd09639652de5b83e/pr-186

Backporting merged PR #186 into main

  1. Ensure you have a local repo clone of your fork. Unless you cloned it
    from the upstream, this would be your origin remote.
  2. Make sure you have an upstream repo added as a remote too. In these
    instructions you'll refer to it by the name upstream. If you don't
    have it, here's how you can add it:
    $ git remote add upstream https://github.com/redhat-cop/cloud.aws_ops.git
  3. Ensure you have the latest copy of upstream and prepare a branch
    that will hold the backported code:
    $ git fetch upstream
    $ git checkout -b patchback/backports/stable-3/39530aecf29a4f69da8c07bbd09639652de5b83e/pr-186 upstream/stable-3
  4. Now, cherry-pick PR [ACA-5022] Configure SonarCloud phase2 #186 contents into that branch:
    $ git cherry-pick -x 39530aecf29a4f69da8c07bbd09639652de5b83e
    If it'll yell at you with something like fatal: Commit 39530aecf29a4f69da8c07bbd09639652de5b83e is a merge but no -m option was given., add -m 1 as follows instead:
    $ git cherry-pick -m1 -x 39530aecf29a4f69da8c07bbd09639652de5b83e
  5. At this point, you'll probably encounter some merge conflicts. You must
    resolve them in to preserve the patch from PR [ACA-5022] Configure SonarCloud phase2 #186 as close to the
    original as possible.
  6. Push this branch to your fork on GitHub:
    $ git push origin patchback/backports/stable-3/39530aecf29a4f69da8c07bbd09639652de5b83e/pr-186
  7. Create a PR, ensure that the CI is green. If it's not — update it so that
    the tests and any other checks pass. This is it!
    Now relax and wait for the maintainers to process your pull request
    when they have some cycles to do reviews. Don't worry — they'll tell you if
    any improvements are necessary when the time comes!

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

@patchback

patchback Bot commented Jun 23, 2026

Copy link
Copy Markdown

Backport to stable-4: 💔 cherry-picking failed — conflicts found

❌ Failed to cleanly apply 39530ae on top of patchback/backports/stable-4/39530aecf29a4f69da8c07bbd09639652de5b83e/pr-186

Backporting merged PR #186 into main

  1. Ensure you have a local repo clone of your fork. Unless you cloned it
    from the upstream, this would be your origin remote.
  2. Make sure you have an upstream repo added as a remote too. In these
    instructions you'll refer to it by the name upstream. If you don't
    have it, here's how you can add it:
    $ git remote add upstream https://github.com/redhat-cop/cloud.aws_ops.git
  3. Ensure you have the latest copy of upstream and prepare a branch
    that will hold the backported code:
    $ git fetch upstream
    $ git checkout -b patchback/backports/stable-4/39530aecf29a4f69da8c07bbd09639652de5b83e/pr-186 upstream/stable-4
  4. Now, cherry-pick PR [ACA-5022] Configure SonarCloud phase2 #186 contents into that branch:
    $ git cherry-pick -x 39530aecf29a4f69da8c07bbd09639652de5b83e
    If it'll yell at you with something like fatal: Commit 39530aecf29a4f69da8c07bbd09639652de5b83e is a merge but no -m option was given., add -m 1 as follows instead:
    $ git cherry-pick -m1 -x 39530aecf29a4f69da8c07bbd09639652de5b83e
  5. At this point, you'll probably encounter some merge conflicts. You must
    resolve them in to preserve the patch from PR [ACA-5022] Configure SonarCloud phase2 #186 as close to the
    original as possible.
  6. Push this branch to your fork on GitHub:
    $ git push origin patchback/backports/stable-4/39530aecf29a4f69da8c07bbd09639652de5b83e/pr-186
  7. Create a PR, ensure that the CI is green. If it's not — update it so that
    the tests and any other checks pass. This is it!
    Now relax and wait for the maintainers to process your pull request
    when they have some cycles to do reviews. Don't worry — they'll tell you if
    any improvements are necessary when the time comes!

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

@sonarqubecloud

Copy link
Copy Markdown

brahmanim added a commit that referenced this pull request Jun 23, 2026
…phase2

[ACA-5022] Configure SonarCloud phase2

(cherry picked from commit 39530ae)
brahmanim added a commit that referenced this pull request Jun 23, 2026
…9530aecf29a4f69da8c07bbd09639652de5b83e/pr-186

[PR #186/39530ae manually backport][stable-4] ACA-5022  Configure SonarCloud phase 2
brahmanim added a commit that referenced this pull request Jun 23, 2026
…phase2

[ACA-5022] Configure SonarCloud phase2

(cherry picked from commit 39530ae)
brahmanim added a commit that referenced this pull request Jun 23, 2026
…9530aecf29a4f69da8c07bbd09639652de5b83e/pr-186

[PR #186/39530ae manually backport][stable-3] ACA-5022 Configure SonarCloud phase 2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants