Skip to content

Backport of Skip acceptance tests if PR changes are only in .release directory into release/1.9.x - #5567

Open
hc-github-team-consul-core wants to merge 1 commit into
release/1.9.xfrom
backport/shashank/update-skip-files-list/logically-humane-giraffe
Open

Backport of Skip acceptance tests if PR changes are only in .release directory into release/1.9.x#5567
hc-github-team-consul-core wants to merge 1 commit into
release/1.9.xfrom
backport/shashank/update-skip-files-list/logically-humane-giraffe

Conversation

@hc-github-team-consul-core

Copy link
Copy Markdown
Collaborator

Backport

This PR is auto-generated from #5530 to be assessed for backporting due to the inclusion of the label backport/1.9.x.

🚨

Warning automatic cherry-pick of commits failed. If the first commit failed,
you will see a blank no-op commit below. If at least one commit succeeded, you
will see the cherry-picked commits up to, not including, the commit where
the merge conflict occurred.

The person who merged in the original PR is:
@shashankNandigama
This person should resolve the merge-conflict(s) by either:

  • Manually completing the cherry picks into this branch
  • Creating a new branch and manually cherry-picking all commits being backported

merge conflict error: POST https://api.github.com/repos/hashicorp/consul-k8s/merges: 409 Merge conflict []

The below text is copied from the body of the original PR.


This PR is intended to skip the acceptance and unit tests workflows when there is a PR with only changes in /release/* files.
Acceptance tests are not impacted by the changes in .release directory, but we can't add this condition in reusable-conditional-skip.yml becuase that is used in build.yml as well. Adding this dir in reusable-skip workflow skips build.yml for changes in security-scan.hcl which is required for prepare workflow. So created a separate task in pr.yml to skip only pr tests and code coverage task for PRs with changes in .release directory.


Overview of commits

@hashicorp-cla-app

hashicorp-cla-app Bot commented Aug 7, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@shashankNandigama
shashankNandigama marked this pull request as ready for review August 9, 2026 03:34
@shashankNandigama
shashankNandigama requested a review from a team as a code owner August 9, 2026 03:34
@shashankNandigama
shashankNandigama enabled auto-merge (squash) August 10, 2026 05:55
@shashankNandigama shashankNandigama added the pr/no-backport signals that a PR will not contain a backport label label Aug 10, 2026
…5530)

* add release dir to conditional skip exclusion list

* skip acceptance tests only for changes in .release dir

* skip coverage.yml on /release/* file changes

Signed-off-by: Manisha Kumari <manisha.kumari@hashicorp.com>

* skip coverage workflow

Signed-off-by: Manisha Kumari <manisha.kumari@hashicorp.com>

* reverted permissions block

Signed-off-by: Manisha Kumari <manisha.kumari@hashicorp.com>

---------

Signed-off-by: Manisha Kumari <manisha.kumari@hashicorp.com>
Co-authored-by: Manisha Kumari <manisha.kumari@hashicorp.com>
@shashankNandigama
shashankNandigama force-pushed the backport/shashank/update-skip-files-list/logically-humane-giraffe branch from e81916a to 20d5102 Compare August 10, 2026 13:21
auto-merge was automatically disabled August 10, 2026 13:24

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/no-backport signals that a PR will not contain a backport label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants