Skip to content

Conversation

@tgravescs
Copy link
Contributor

Pass along the acl settings when we launch a container so that they can be applied to viewing the logs on a running NodeManager.

@JoshRosen
Copy link
Contributor

Jenkins, retest this please.

@tgravescs
Copy link
Contributor Author

hmm, odd is jenkins not working.

@tgravescs
Copy link
Contributor Author

Jenkins, test this please

@SparkQA
Copy link

SparkQA commented Aug 29, 2014

QA tests have started for PR 2185 at commit e81a0e0.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 29, 2014

QA tests have finished for PR 2185 at commit e81a0e0.

  • This patch fails unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@tgravescs
Copy link
Contributor Author

Interesting failure, must have something to do with default user running. I'll investigate.

@SparkQA
Copy link

SparkQA commented Aug 29, 2014

QA tests have started for PR 2185 at commit 7df51a1.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 29, 2014

QA tests have finished for PR 2185 at commit 7df51a1.

  • This patch passes unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@tgravescs
Copy link
Contributor Author

Jenkins, test this please

@SparkQA
Copy link

SparkQA commented Sep 4, 2014

QA tests have started for PR 2185 at commit 6f94b5a.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 4, 2014

QA tests have finished for PR 2185 at commit 6f94b5a.

  • This patch passes unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@asfgit asfgit closed this in 51b53a7 Sep 5, 2014
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.

3 participants