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

[TECH DEBT] Add a label to run a full test run #64539

Closed
MKLeb opened this issue Jun 23, 2023 · 1 comment · Fixed by #64547
Closed

[TECH DEBT] Add a label to run a full test run #64539

MKLeb opened this issue Jun 23, 2023 · 1 comment · Fixed by #64547
Assignees
@s0undt3ch @MKLeb
Labels
tech-debt Tests

Comments

@MKLeb
Copy link
Contributor

MKLeb commented Jun 23, 2023

Description of the tech debt to be addressed, include links and screenshots

We need a label that allows a full test run to be run on PRs, only assignable by dev-core members. This will allow us to trigger full runs on PRs that may have a broad scope, or perhaps the reach of which is unclear. This way, we can catch stray failures before we merge and see them in nightly builds.
@MKLeb MKLeb self-assigned this Jun 23, 2023
@s0undt3ch s0undt3ch self-assigned this Jun 27, 2023
s0undt3ch added a commit to s0undt3ch/salt that referenced this issue Jun 27, 2023
@s0undt3ch
Choose a full test run when a PR has the test:full label
02d7ce4 
Fixes saltstack#64539

Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
@s0undt3ch s0undt3ch linked a pull request Jun 27, 2023 that will close this issue
[3006.x] Switch to using tools and re-use the event payload available instead of querying again #64547
Merged
s0undt3ch added a commit to s0undt3ch/salt that referenced this issue Jun 27, 2023
@s0undt3ch
Choose a full test run when a PR has the test:full label
315f0aa 
Fixes saltstack#64539

Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
s0undt3ch added a commit to s0undt3ch/salt that referenced this issue Jun 27, 2023
@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 added a commit to s0undt3ch/salt that referenced this issue Jun 27, 2023
@s0undt3ch
Choose a full test run when a PR has the test:full label
5067110 
Fixes saltstack#64539

Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
s0undt3ch added a commit that referenced this issue Jun 27, 2023
@s0undt3ch
Choose a full test run when a PR has the test:full label
3fe2f44 
Fixes #64539

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

This has been implemented in the 3006.x branch and will be merged forwarded into master soon.

s0undt3ch added a commit to s0undt3ch/salt that referenced this issue Jun 27, 2023
@s0undt3ch
Choose a full test run when a PR has the test:full label
72de783 
Fixes saltstack#64539

Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
s0undt3ch added a commit to s0undt3ch/salt that referenced this issue Jun 28, 2023
@s0undt3ch
Choose a full test run when a PR has the test:full label
d27ce77 
Fixes saltstack#64539

Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@s0undt3ch s0undt3ch

@MKLeb MKLeb

Labels
tech-debt Tests
Projects
None yet
Milestone
No milestone
3 participants
@s0undt3ch @OrangeDog @MKLeb