| dmsimard | jrosser: o/ could you remind me what was that branch about ara downloading sqlite databases ? noonedeadpunk and I chatted about it recently :p | 02:16 |
|---|---|---|
| *** chandankumar is now known as chkumar|rover | 05:07 | |
| noonedeadpunk | dmsimard: well, I looked at the proposed code and I was thinking it was a bit different when I talked to you :D | 08:55 |
| noonedeadpunk | As for some reason, I thought that it was providing path to the database during runtime and not during wsgi startup | 08:56 |
| noonedeadpunk | but it was this commit https://github.com/jrosser/ara/commit/f9af69eaef4ea1228f4fc641e36b9d8df5adbaaa | 08:57 |
| noonedeadpunk | let me spend some time on it first - and I will return back to you :) | 08:58 |
| noonedeadpunk | As I can recall providing DB to Django at least during runtime with POST or smth... But it was years ago, so can't really recall details | 09:03 |
| dmsimard | ah, yes, I half remember now | 11:12 |
| dmsimard | if we're going to do something that resembles what we talked about, it should be generic and not tightly coupled to zuul | 11:13 |
| dmsimard | I don't know what this would look like in practice but it seems you have an idea so just let me know :p | 11:15 |
| noonedeadpunk | yes, totally, I don't wanna do smth zuul specific | 11:21 |
| noonedeadpunk | What I had in mind is give ARA path/url as GET? request, so it could load it and render content. | 11:25 |
| dmsimard | yes, I still think there are security implications (download and run this arbitrary file from this arbitrary location) so it should definitely be hidden under some toggle that defaults to off | 11:28 |
| dmsimard | but we can try and find out if we can come up with something that makes sense | 11:28 |
| noonedeadpunk | yeah, totally | 11:30 |
| dmsimard | it would probably be a specific backend like remote_dswlit | 11:47 |
| dmsimard | err, mobile typing | 11:47 |
| dmsimard | remote_sqlite instead of distributed_sqlite | 11:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova master: Install openvswith repo for Nova https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/873368 | 13:25 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_nova master: Install openvswitch repo for RDO scenario https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/873368 | 13:25 |
| jrosser | noonedeadpunk: part of the thing with ARA retrieving the sqlite was having to convert the build uuid (or even change no. + revision) into the sqlite swift URL, so whilst it can be as simple as GET?sqlite=<some_url> there needs to be some other zuul specific code | 17:14 |
| jrosser | *zuul specific code somewhere | 17:15 |
| noonedeadpunk | jrosser: I think with artifacts that's not a big deal? | 17:19 |
| jrosser | oh possibly not - idk about that | 17:20 |
| noonedeadpunk | Or well, at least zuul is aware about it's relative URL of where some artifact is stored on swift. built docs are good example kind of | 17:20 |
| noonedeadpunk | But dunno, maybe some issue will raise if I decide to prefix ARA server URL in the beginning... | 17:21 |
| noonedeadpunk | but yes, instead of simply storing it in logs, I was thinking to make an artifact out of it | 17:23 |
| jrosser | it's like we also want to lodge a URL as an artifact too | 17:23 |
| jrosser | to just click and it take you to ARA with the right ?<blah> on the end | 17:23 |
| noonedeadpunk | yep | 17:24 |
| noonedeadpunk | well.. maybe it will be harder then I imagine... | 17:25 |
| noonedeadpunk | will see.... | 17:25 |
| noonedeadpunk | I think you can provide any arbitrary data as artifact url. | 17:25 |
| noonedeadpunk | it's just a matter of how to get a relative path to uploaded file. As upload artifact and push artifact URL to zuul - are 2 different things | 17:26 |
| noonedeadpunk | So you can upload stuff and as return get URL, and then prefix it and push as artifact | 17:27 |
| noonedeadpunk | At least it works like that in my head :D | 17:27 |
| noonedeadpunk | we did smth like that on our local zuul, but it's way older that what we have in opendev | 17:28 |
| opendevreview | Merged openstack/openstack-ansible master: Install curl by defining binary that is provided https://review.opendev.org/c/openstack/openstack-ansible/+/872896 | 23:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!