Skip to content

Fix Amazon Linux 2 Python 3 support and other clean up #1401

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

Merged
merged 28 commits into from
Jan 6, 2020

Conversation

dmurphy18
Copy link
Contributor

What does this PR do?

Provides bootstrap support for Amazon Linux 2 Python 3 support, and other minor cleanup
Work-in-progress to utilize testing

What issues does this PR fix or reference?

#1391

Previous Behavior

Installed Python 2 Amazon Linux 2 salt-minion, when Python 3 Amazon Linux 2 salt-minion was desired

New Behavior

Installs Python 3 Amazon Linux 2 salt-minion via bootstrap with '-x python3' parameter.

@dmurphy18 dmurphy18 requested review from s0undt3ch and Ch3LL December 11, 2019 20:52
@bradrydzewski
Copy link

Just a heads up that you will want to remove the node selector section when using Drone cloud, otherwise the pipelines will sit in a pending state until they match a runner with this label. Ideally Drone would error when we know a certain configuration will never match a runner. To address this I created harness/harness#2894.

@dmurphy18 dmurphy18 changed the title Fix Amazon Linux 2 Python 3 support and other clean up WIP Fix Amazon Linux 2 Python 3 support and other clean up Dec 17, 2019
@vutny
Copy link
Contributor

vutny commented Dec 22, 2019

Looks good, the CI test seems to be passed.

Copy link
Contributor

@s0undt3ch s0undt3ch left a comment

Choose a reason for hiding this comment

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

Please remove the version bump.

@dmurphy18 dmurphy18 requested a review from s0undt3ch January 2, 2020 19:16
Copy link
Contributor

@vutny vutny left a comment

Choose a reason for hiding this comment

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

LGTM

@Ch3LL Ch3LL merged commit 39f4557 into saltstack:develop Jan 6, 2020
@dmurphy18 dmurphy18 deleted the fix_amzn2 branch July 15, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants