Skip to content
This repository was archived by the owner on Nov 9, 2017. It is now read-only.

Conversation

jezdez
Copy link
Contributor

@jezdez jezdez commented Jul 15, 2014

This is needed work around the change in chief that adds a timespamp to the log URL: mozilla/chief@41af16d

This is needed work around the change in chief that adds a timespamp to the log URL: mozilla/chief@41af16d
@willkg
Copy link
Contributor

willkg commented Jul 15, 2014

One of the problems we have is that our sites have very different versions of chief. The version that kitsune has uses /logs/. It's essentially a directory being served by apache with logs of all the deployments. Fjord has a more recent version of chief which I think uses /history/ (I need to check, but monsters are in my kitchen demanding I come down "RIGHT NOW!").

This PR will break kitsune's use of chief-james, but might fix it for fjord and I bet you're suggesting it because it fixes it for mdn.

Anyhow, so the problem is that we've all got different versions of chief that have different ways of doing things. Maybe we should move the logs url to settings?

@mythmon
Copy link
Owner

mythmon commented Jul 15, 2014

+1 to making this a setting.

@jezdez: I have a branch that is partially done that also does that nasty thing we talked about in which I add an html parser and go figure out the correct link.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants