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.

Sign up for GitHub

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

Jump to bottom

[3006.x] Switch to using tools and re-use the event payload available instead of querying again #64547

Merged
merged 5 commits into from
Jun 27, 2023
Merged

[3006.x] Switch to using tools and re-use the event payload available instead of querying again #64547

merged 5 commits into from
Jun 27, 2023

Conversation

s0undt3ch
Copy link
Collaborator

What does this PR do?

See title

@s0undt3ch s0undt3ch added this to the Sulfur v3006.2 milestone Jun 26, 2023
@s0undt3ch s0undt3ch requested a review from a team as a code owner June 26, 2023 17:28
@s0undt3ch s0undt3ch requested review from Ch3LL and removed request for a team June 26, 2023 17:28
@salt-project-bot-prod-environment salt-project-bot-prod-environment bot changed the title Switch to using tools and re-use the event payload available instead of querying again [3006.x] Switch to using tools and re-use the event payload available instead of querying again Jun 26, 2023
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 18:29 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 18:31 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 18:40 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 18:46 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 18:51 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 19:00 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 20:11 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 20:11 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 20:11 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 20:11 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 20:11 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 26, 2023 20:11 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch added the test:core Enable Core Tests label Jun 26, 2023
@s0undt3ch s0undt3ch added the test:slow Enable Slow Tests label Jun 27, 2023
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 09:05 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 09:05 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 09:05 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 09:06 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 09:22 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 09:22 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 10:16 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 10:16 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 10:16 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch linked an issue Jun 27, 2023 that may be closed by this pull request
[TECH DEBT] Add a label to run a full test run #64539
Closed
@s0undt3ch
Don't override the on jinja block on the CI workflow template
dd1d2c0 
Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
@s0undt3ch
Switch to using tools and re-use the event payload available instea…
83ef736 
…d of querying again

Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
@s0undt3ch
Copy link
Collaborator Author

/CC @cmcmarrow

@s0undt3ch
Choose a full test run when a PR has the test:full label
4335199 
Fixes saltstack#64539

Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
@s0undt3ch
Concentrate test selection to a single place
de50ebf 
Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 12:14 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 12:14 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 12:15 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 12:32 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 12:32 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 12:37 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch removed test:core Enable Core Tests test:slow Enable Slow Tests labels Jun 27, 2023
@s0undt3ch
Enable code coverage on pull-requests by setting the test:coverage …
528242c 
…label

Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
@s0undt3ch s0undt3ch requested a review from cmcmarrow June 27, 2023 15:53
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 16:53 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 16:53 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 16:53 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 16:54 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 17:12 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci June 27, 2023 17:13 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch merged commit 2d824c9 into saltstack:3006.x Jun 27, 2023
@s0undt3ch s0undt3ch deleted the hotfix/tools-labels branch June 27, 2023 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@twangboy twangboy twangboy approved these changes

@cmcmarrow cmcmarrow cmcmarrow approved these changes

@Ch3LL Ch3LL Awaiting requested review from Ch3LL Ch3LL was automatically assigned from saltstack/team-core

Assignees
No one assigned
Labels
Sulfur v3006.2
Projects
None yet
Milestone
Sulfur v3006.2
Development

Successfully merging this pull request may close these issues.

[TECH DEBT] Add a label to run a full test run
3 participants
@s0undt3ch @twangboy @cmcmarrow