Closed
Bug 952024
Opened 12 years ago
Closed 11 years ago
Host the working version of mciconf on mozqa.com
Categories
(Mozilla QA Graveyard :: Infrastructure, defect)
Mozilla QA Graveyard
Infrastructure
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cosmin-malutan, Assigned: whimboo)
References
(Blocks 1 open bug, )
Details
Due to fact that the code and staging version would live on github.com/mozilla/mciconf, asdiscused on bug 951578, we would want the live version on mozqa.com
| Assignee | ||
Comment 1•12 years ago
|
||
Brandon, what is the best way to get the code updated on mozqa.com? I assume via a cron job similar to the other external repositories, and an update for puppet?
The code we want to have synced is:
https://github.com/mozilla/mciconf/tree/release
Flags: needinfo?(bburton)
Comment 2•12 years ago
|
||
(In reply to Henrik Skupin (:whimboo) from comment #1)
> Brandon, what is the best way to get the code updated on mozqa.com? I assume
> via a cron job similar to the other external repositories, and an update for
> puppet?
>
> The code we want to have synced is:
> https://github.com/mozilla/mciconf/tree/release
From a quick glance, it looks like this application is static content on the filesystem? is that correct?
Flags: needinfo?(bburton)
| Assignee | ||
Comment 3•12 years ago
|
||
That's correct. There are no scripts running server-side. All is done via JS in the browser.
Something I missed to say is that the target location should be http://www.mozqa.com/mciconf.
Comment 4•12 years ago
|
||
When just pulling down a git checkout, the URL ends up being http://www.mozqa.com/mciconf/app/ , is that fine or should I do some symlink trickery to make it just http://www.mozqa.com/mciconf/ ?
| Assignee | ||
Comment 5•12 years ago
|
||
Cosmin, I assume the app sub folder is not necessary and we can move all the files into the top level folder in the repository. Is that right? If yes, can you please do that?
Flags: needinfo?(cosmin.malutan)
| Reporter | ||
Comment 6•12 years ago
|
||
I moved it.
https://github.com/mozilla/mciconf/pull/7
Flags: needinfo?(cosmin.malutan)
| Assignee | ||
Comment 7•12 years ago
|
||
(In reply to Brandon Burton [:solarce] from comment #4)
> When just pulling down a git checkout, the URL ends up being
> http://www.mozqa.com/mciconf/app/ , is that fine or should I do some symlink
> trickery to make it just http://www.mozqa.com/mciconf/ ?
No, that's fine then. It will work once we push the next version to the release branch. So it looks great. Thanks Brandon!
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Comment 8•12 years ago
|
||
We'll also need to update https://mana.mozilla.org/wiki/display/websites/mozqa.com
| Assignee | ||
Comment 9•12 years ago
|
||
Cosmin, after nearly two weeks I would like to see an update from you here.
Flags: needinfo?(cosmin.malutan)
| Reporter | ||
Comment 10•12 years ago
|
||
Hi, sorry if I missed this, I opened a new pr for this https://github.com/mozilla/mciconf/pull/8. The thing is if I get only this landed then I would have to do a lot of work for the requireJS commit, and I would like to have that landed too since is more profesional to split the application into relevant smaller files instead of a single javascript file.
Again the pr for this is:
https://github.com/mozilla/mciconf/pull/8
If you want to land both the requireJS and removal of app directory commits:
https://github.com/mozilla/mciconf/pull/7
Thanks
Flags: needinfo?(cosmin.malutan)
| Assignee | ||
Comment 11•12 years ago
|
||
We want to have PR #7 without the change for this bug! I already mentioned that over on PR #7 more than a week ago. Please really follow your github email, so that things are not getting massively delayed.
| Reporter | ||
Comment 12•12 years ago
|
||
Henrik, in pr 8, I only removed the app directory.
I will update the pr 7 later(requireJS commit).
| Assignee | ||
Comment 13•11 years ago
|
||
All what we can do here has been done. The mciconf application is now fully functional on mozqa.com.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•