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

Enable unlimited Salt job logging #547

Merged
merged 1 commit into from Dec 23, 2016

Conversation

@aneeshusa
Copy link
Member

aneeshusa commented Dec 7, 2016

Add some docs in tree as well about this feature,
to avoid forgetting to update the wiki when the PR lands.

This is a potential (partial?) fix for #437, with two caveats:

  • I'm not sure of a way to "tag" a highstate deployment with the revision/PR being deployed; we'll still need to keep track of that manually
  • The keep_jobs config option is a number of hours (the default is 24), so we can set it a concrete number instead of 0 for unlimited if you'd like.

This change is Reviewable

@aneeshusa
Copy link
Member Author

aneeshusa commented Dec 23, 2016

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Dec 23, 2016

Same as other, r=me after rebase so that it has the non-broken intermittent tracker revision :-)

Thanks!

@bors-servo
Copy link
Contributor

bors-servo commented Dec 23, 2016

The latest upstream changes (presumably #546) made this pull request unmergeable. Please resolve the merge conflicts.

Add some docs in tree as well about this feature,
to avoid forgetting to update the wiki when the PR lands.
@aneeshusa aneeshusa force-pushed the aneeshusa:enable-unlimited-job-logging branch from ff56d2a to b7c88c7 Dec 23, 2016
@aneeshusa
Copy link
Member Author

aneeshusa commented Dec 23, 2016

@bors-servo r=larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Dec 23, 2016

📌 Commit b7c88c7 has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Dec 23, 2016

Testing commit b7c88c7 with merge 719458c...

bors-servo added a commit that referenced this pull request Dec 23, 2016
…rgstrom

Enable unlimited Salt job logging

Add some docs in tree as well about this feature,
to avoid forgetting to update the wiki when the PR lands.

This is a potential (partial?) fix for #437, with two caveats:
- I'm not sure of a way to "tag" a highstate deployment with the revision/PR being deployed; we'll still need to keep track of that manually
- The `keep_jobs` config option is a number of hours (the default is 24), so we can set it a concrete number instead of 0 for unlimited if you'd like.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/547)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Dec 23, 2016

☀️ Test successful - status-travis

@bors-servo bors-servo merged commit b7c88c7 into servo:master Dec 23, 2016
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.