Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Manual backport of test: remove unused workflow inputs (#2589) into release/1.1.x #2608

Merged
merged 3 commits into from Jul 21, 2023

Conversation

zalimeni
Copy link
Member

I think this should resolve errors like https://github.com/hashicorp/consul-k8s/actions/runs/5615117399/job/15215290442?pr=2604#step:2:17 due to the missing arg in the target workflow.

Follow-up to #2589.

Checklist:

Changes proposed in this PR:
- Removed unused workflow inputs.

Manual backport of b3769b1.
@zalimeni zalimeni added pr/no-changelog PR does not need a corresponding .changelog entry pr/no-backport signals that a PR will not contain a backport label labels Jul 20, 2023
Copy link
Member

@DanStough DanStough left a comment

Choose a reason for hiding this comment

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

🚀

@zalimeni zalimeni enabled auto-merge (squash) July 20, 2023 20:42
@zalimeni zalimeni force-pushed the zalimeni/fix-backport-pr-test-workflow-1.1.x branch from d3c4b85 to 6965c2b Compare July 21, 2023 13:57
@zalimeni
Copy link
Member Author

zalimeni commented Jul 21, 2023

Force pushed to try a quick cache bust in case that was contributing to golangci-lint timeout issues re: golangci/golangci-lint-action#730. Appears cache-y since it looks specific to my PR, not even just this release branch given https://github.com/hashicorp/consul-k8s-workflows/actions/runs/5602459151/job/15177096250#step:6:50 went through fine.

I'll see if adding a timeout fixes it, and will add some debug logging around memory and CPU so we can diagnose if it happens again.

Update: we already have a 5m timeout on main and the release branches 😮‍💨

@zalimeni
Copy link
Member Author

@zalimeni
Copy link
Member Author

zalimeni commented Jul 21, 2023

Bringing in fix from #2621 to get this to pass (the PRs had a cyclic dependency)

@zalimeni zalimeni merged commit 682ed97 into release/1.1.x Jul 21, 2023
20 of 42 checks passed
@zalimeni zalimeni deleted the zalimeni/fix-backport-pr-test-workflow-1.1.x branch July 21, 2023 17:50
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 pr/no-changelog PR does not need a corresponding .changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants