update dockerfiles to use Fedora
ClosedPublic

Authored by garretraziel on Feb 2 2016, 10:33 AM.

Details

Summary

Update dockerfiles to newest openQA and base them on Fedora image.

Test Plan

Go through README and try to start openQA through Docker.

Diff Detail

Repository
rOPENQA fedora_openqa
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
garretraziel retitled this revision from to update dockerfiles to use Fedora.Feb 2 2016, 10:33 AM
garretraziel updated this object.
garretraziel edited the test plan for this revision. (Show Details)
garretraziel added reviewers: adamwill, jskladan.

I haven't tested this, but one comment - if we're gonna use fake auth, it'd be worth stressing *really hard* not to make this remotely connectable at all, maybe bump up the strength of that warning in the docs a bit?

I haven't tested this, but one comment - if we're gonna use fake auth, it'd be worth stressing *really hard* not to make this remotely connectable at all, maybe bump up the strength of that warning in the docs a bit?

I agree. I thought that we'll use Docker only for development anyway, so using Fake auth would be much easier.

garretraziel updated this revision to Diff 1892.Feb 3 2016, 8:41 AM
  • change method when setting OpenID on Docker, warn about deployment with Docker
jskladan accepted this revision.Feb 3 2016, 1:15 PM

Or we could just make it OpenID by default, and extend the script that allows changing the OpenID provider, to also be able to change the authmethod to Fake. How about that?

Otherwise, LGTM

This revision is now accepted and ready to land.Feb 3 2016, 1:15 PM
This revision was automatically updated to reflect the committed changes.