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

Schedule salt smoke tests on micro-os #20447

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Schedule salt smoke tests on micro-os #20447

wants to merge 1 commit into from
+10 −2

Conversation

rfan1
Copy link
Contributor

@rfan1 rfan1 commented Oct 22, 2024 •
edited
Loading

@rfan1 rfan1 added the WIP Work in progress label Oct 22, 2024
@github-actions GitHub Actions
Copy link

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files.

@rfan1 rfan1 added the notready label Oct 22, 2024
@rfan1 rfan1 force-pushed the salt_micro branch 3 times, most recently from 53c4982 to 3011743 Compare October 22, 2024 08:57
@rfan1 rfan1 changed the title Schedule salt smoke tests on slem/micro-os Schedule salt smoke tests on micro-os Oct 22, 2024
@rfan1 rfan1 added qe-core Tag used to filter PR's created by QE-Core's members or are assigned to them and removed notready WIP Work in progress labels Oct 22, 2024
@mloviska
Copy link
Contributor

Wait, hasn't it failed on 5.5 as the salt is pre-installed on older versions?

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

@foursixnine foursixnine foursixnine approved these changes

@dzedro dzedro dzedro approved these changes

@jlausuch jlausuch Awaiting requested review from jlausuch jlausuch is a code owner

Assignees
No one assigned
Labels
qe-core Tag used to filter PR's created by QE-Core's members or are assigned to them
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rfan1 @mloviska @foursixnine @dzedro