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

fileclient: decode file list from master #29947

Merged
merged 2 commits into from Jan 19, 2016

Conversation

Projects
None yet
4 participants
@jfindlay
Contributor

jfindlay commented Dec 22, 2015

Fixes #29918.

@jfindlay jfindlay added the Core label Dec 22, 2015

@cachedout

This comment has been minimized.

Show comment
Hide comment
@cachedout

cachedout Dec 29, 2015

Contributor

@jfindlay This doesn't pass the test suite.

@terminalmage Could you also have a look at this, please? Probably @s0undt3ch too.

Contributor

cachedout commented Dec 29, 2015

@jfindlay This doesn't pass the test suite.

@terminalmage Could you also have a look at this, please? Probably @s0undt3ch too.

@s0undt3ch

This comment has been minimized.

Show comment
Hide comment
@s0undt3ch

s0undt3ch Dec 31, 2015

Member

One VM failed to deploy and the 3 failures on the other VM seem unrelated, let's re-trigger it.

Member

s0undt3ch commented Dec 31, 2015

One VM failed to deploy and the 3 failures on the other VM seem unrelated, let's re-trigger it.

@cachedout

This comment has been minimized.

Show comment
Hide comment
@cachedout

cachedout Jan 4, 2016

Contributor

Go Go Jenkins!

Contributor

cachedout commented Jan 4, 2016

Go Go Jenkins!

@basepi

This comment has been minimized.

Show comment
Hide comment
@basepi

basepi Jan 6, 2016

Collaborator

I think we'll probably hold off on this until we can get the test suite running properly again.

Collaborator

basepi commented Jan 6, 2016

I think we'll probably hold off on this until we can get the test suite running properly again.

@cachedout

This comment has been minimized.

Show comment
Hide comment
@cachedout

cachedout Jan 13, 2016

Contributor

@jfindlay Can you please rebase this today?

Contributor

cachedout commented Jan 13, 2016

@jfindlay Can you please rebase this today?

cachedout added a commit that referenced this pull request Jan 19, 2016

Merge pull request #29947 from jfindlay/remote_decode
fileclient: decode file list from master

@cachedout cachedout merged commit 3c12b45 into saltstack:2015.8 Jan 19, 2016

4 of 5 checks passed

default Merged build finished.
Details
jenkins/salt-pr-clone Salt PR - Clone Repository #12796 — SUCCESS
Details
jenkins/salt-pr-lint-n Salt PR - Code Lint #12484 — SUCCESS
Details
jenkins/salt-pr-rs-cent7-n Salt PR - RS CentOS 7 #11379 — SUCCESS
Details
jenkins/salt-pr-rs-ubuntu14.04-n Salt PR - RS Ubuntu 14 #8866 — SUCCESS
Details

@jfindlay jfindlay deleted the jfindlay:remote_decode branch Jan 20, 2016

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