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

Sort the return list from the fileserver.envs runner #39209

Merged
merged 1 commit into from Feb 7, 2017

Conversation

Projects
None yet
3 participants
@terminalmage
Copy link
Contributor

commented Feb 7, 2017

This makes it easier to find envs in the list when there are many of them (e.g. when gitfs is used)

@cachedout cachedout merged commit dd44045 into saltstack:2016.3 Feb 7, 2017

3 of 6 checks passed

default Pull Requests » Salt PR - Main Build #1419
Details
jenkins/PR/salt-pr-linode-ubuntu14-n Pull Requests » Salt Linode Ubuntu14.04 #8252 — FAILURE
Details
jenkins/PR/salt-pr-rs-cent7-n Pull Requests » Salt PR - RS CentOS 7 #8482 — FAILURE
Details
jenkins/PR/salt-pr-clone Pull Requests » Salt PR - Clone #10964 — SUCCESS
Details
jenkins/PR/salt-pr-docs-n Pull Requests » salt-pr-docs-n #3809 — SUCCESS
Details
jenkins/PR/salt-pr-lint-n Pull Requests » Salt PR - Code Lint #8335 — SUCCESS
Details
@terminalmage

This comment has been minimized.

Copy link
Contributor Author

commented Feb 7, 2017

@rallytime Keep an eye on this as it is merged forward, it is likely to conflict since runner returns have changed since 2016.3 and we no longer check for an outputter in the runner function itself. The merge conflict resolution should be fairly straightforward.

@rallytime

This comment has been minimized.

Copy link
Contributor

commented Feb 7, 2017

@terminalmage Ok sounds good. Thank you for the heads-up.

@terminalmage terminalmage deleted the terminalmage:sorted-envs branch Apr 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.