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

Test loader namespaceing #56513

Merged
merged 15 commits into from
Sep 30, 2020
Merged

Test loader namespaceing #56513

merged 15 commits into from
Sep 30, 2020

Conversation

dwoz
Copy link
Contributor

@dwoz dwoz commented Apr 3, 2020

What does this PR do?

What issues does this PR fix or reference?

Previous Behavior

Remove this section if not relevant

New Behavior

Remove this section if not relevant

Tests written?

[NOTICE] Bug fixes or features added to Salt require tests.
Please review the test documentation for details on how to implement tests into Salt's test suite.

Yes/No

Commits signed with GPG?

Yes/No

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

@dwoz dwoz requested a review from a team as a code owner April 3, 2020 02:02
@ghost ghost requested a review from cmcmarrow April 3, 2020 02:02
@Ch3LL Ch3LL removed the request for review from a team April 15, 2020 14:13
@dwoz dwoz added has-failing-test Needs-Testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases labels May 13, 2020
@dwoz dwoz force-pushed the loader_namespace branch from 0979d1a to cf8c617 Compare June 14, 2020 02:39
@sagetherage sagetherage added the Magnesium Mg release after Na prior to Al label Jun 23, 2020
@garethgreenaway
Copy link
Contributor

re-run full all

2 similar comments
@garethgreenaway
Copy link
Contributor

re-run full all

@garethgreenaway
Copy link
Contributor

re-run full all

@dwoz dwoz force-pushed the loader_namespace branch from 8ee3439 to d2655ed Compare July 27, 2020 22:54
@dwoz
Copy link
Contributor Author

dwoz commented Jul 27, 2020

re-run full all

@sagetherage sagetherage assigned Ch3LL and unassigned dwoz Aug 6, 2020
@Ch3LL Ch3LL force-pushed the loader_namespace branch from d2655ed to ab39a52 Compare August 12, 2020 16:57
@Ch3LL
Copy link
Contributor

Ch3LL commented Aug 12, 2020

re-run full all

@dwoz dwoz removed the Needs-Testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases label Aug 13, 2020
@garethgreenaway
Copy link
Contributor

re-run full all

@Ch3LL
Copy link
Contributor

Ch3LL commented Sep 22, 2020

re-run full all

@garethgreenaway
Copy link
Contributor

re-run full all

@garethgreenaway
Copy link
Contributor

re-run full all

@garethgreenaway
Copy link
Contributor

re-run full all

…lding the eventpublisher_process from 2 seconds to 8 seconds.
@garethgreenaway
Copy link
Contributor

re-run full all

@Ch3LL
Copy link
Contributor

Ch3LL commented Sep 28, 2020

re-run full all

@Ch3LL
Copy link
Contributor

Ch3LL commented Sep 28, 2020

re-run full all

@Ch3LL Ch3LL changed the title [WIP] Test loader namespaceing Test loader namespaceing Sep 30, 2020
@dwoz dwoz merged commit 16af697 into saltstack:master Sep 30, 2020
@dwoz
Copy link
Contributor Author

dwoz commented Oct 20, 2020

Addresses #58369

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Magnesium Mg release after Na prior to Al
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants