opendevreview | Merged openstack/devstack stable/yoga: Cap max microversions for stable/yoga https://review.opendev.org/c/openstack/devstack/+/835226 | 00:15 |
---|---|---|
opendevreview | Ghanshyam proposed openstack/grenade master: Remove sourcing of userrc_early https://review.opendev.org/c/openstack/grenade/+/835531 | 03:54 |
*** soniya29 is now known as soniya29|rover | 04:12 | |
opendevreview | Ghanshyam proposed openstack/grenade master: Update the master grenade setting for yoga to master upgrade https://review.opendev.org/c/openstack/grenade/+/835232 | 04:36 |
*** abhishekk is now known as akekane|home | 04:54 | |
*** akekane|home is now known as abhishekk | 04:54 | |
*** soniya29|rover is now known as soniya29|rover|afk | 05:13 | |
*** soniya29|rover|afk is now known as soniya29|rover | 05:31 | |
opendevreview | Dr. Jens Harbott proposed openstack/devstack master: Move openEuler job to experimental pipeline https://review.opendev.org/c/openstack/devstack/+/835352 | 07:18 |
opendevreview | Dr. Jens Harbott proposed openstack/devstack master: Move openEuler job to experimental pipeline https://review.opendev.org/c/openstack/devstack/+/835352 | 07:18 |
*** jbadiapa|off is now known as jbadiapa | 07:20 | |
*** jpena|off is now known as jpena | 07:38 | |
opendevreview | Martin Kopec proposed openstack/devstack stable/yoga: Cap stable/yoga network, swift, volume api_extensions for tempest https://review.opendev.org/c/openstack/devstack/+/835227 | 07:57 |
*** soniya29|rover is now known as soniya29|rover|afk | 08:42 | |
*** soniya29|rover|afk is now known as soniya29|rover | 09:23 | |
opendevreview | Merged openstack/devstack-plugin-nfs master: remove unicode from code https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/769649 | 09:55 |
*** soniya29|rover is now known as soniya29|rover|call | 12:45 | |
*** soniya29|rover|call is now known as soniya29|rover|afk | 13:21 | |
*** soniya29|rover|afk is now known as soniya29|rover | 13:38 | |
opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 13:59 |
opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: [DNM][CI] Add CEPHADM_DEPLOY flag to py3 tests https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/834223 | 14:00 |
opendevreview | Merged openstack/devstack master: Move openEuler job to experimental pipeline https://review.opendev.org/c/openstack/devstack/+/835352 | 14:03 |
*** pojadhav is now known as pojadhav|dinner | 14:27 | |
kopecmartin | #startmeeting qa | 15:00 |
opendevmeet | Meeting started Tue Mar 29 15:00:15 2022 UTC and is due to finish in 60 minutes. The chair is kopecmartin. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'qa' | 15:00 |
kopecmartin | #link https://wiki.openstack.org/wiki/Meetings/QATeamMeeting#Weekly_QA_Team_meeting | 15:00 |
kopecmartin | agenda ^^ | 15:00 |
gmann | o/ | 15:00 |
kopecmartin | o/ | 15:01 |
frickler | \o | 15:01 |
kopecmartin | o/ | 15:02 |
kopecmartin | let's start | 15:02 |
kopecmartin | #topic Announcement and Action Item (Optional) | 15:02 |
kopecmartin | we should be done with releases | 15:02 |
kopecmartin | couple of patches are hanging there though | 15:02 |
gmann | few grenade setup are pending, which is in gate | 15:02 |
gmann | but we can discuss that later | 15:02 |
gmann | yeah | 15:03 |
kopecmartin | ok | 15:03 |
kopecmartin | no other updates from my side | 15:03 |
kopecmartin | #topic Yoga Priority Items progress | 15:03 |
kopecmartin | any updates on this front? | 15:04 |
kopecmartin | i'm going through and am checking the status of the efforts | 15:04 |
gmann | nothing much from me. just reviewed the rbac patch on devstack side. | 15:05 |
kopecmartin | there is a long list of patches for the manager cleanup i'm failing to get to | 15:05 |
gmann | yeah it is all in plugins sides, I did not see revision after my first review | 15:05 |
gmann | but keeping eyes on those. | 15:05 |
gmann | may be it will be good to use the single topic for those, replace-tempest-sm | 15:06 |
kopecmartin | the patches should be under one topic | 15:06 |
kopecmartin | #link https://review.opendev.org/q/topic:tempest-scenario-manager-cleanup | 15:06 |
gmann | cool | 15:07 |
gmann | I think there are many new one, I will review those | 15:07 |
kopecmartin | cool thanks | 15:07 |
kopecmartin | #topic OpenStack Events Updates and Planning | 15:08 |
kopecmartin | PTG is gonna be next week | 15:08 |
kopecmartin | which reminds, we're cancelling the office hour for the next week, right? | 15:08 |
gmann | yeah | 15:08 |
kopecmartin | ack, i'll send an email | 15:08 |
kopecmartin | #link https://etherpad.opendev.org/p/qa-zed-ptg | 15:09 |
kopecmartin | topics ^^ | 15:09 |
kopecmartin | i'm about to add 2 more | 15:09 |
kopecmartin | regarding the current status and next plans for s-rbac and fips | 15:09 |
kopecmartin | i'll finalize the order of the topics by the end of the week and will send another email to the ML announcing it | 15:10 |
gmann | kopecmartin: we are also going to discuss both in TC PTG and on FIPs to be a goal or not | 15:10 |
gmann | for rbac, nova is ready with new rbac and that i will target for zed cycle to migrate tempest to new policy | 15:10 |
kopecmartin | good, right during the monday's session? | 15:10 |
gmann | kopecmartin: no, in thursday or friday | 15:11 |
kopecmartin | okey | 15:11 |
gmann | but we can discuss/plan the QA part as per the current progress on rbac #link http://lists.openstack.org/pipermail/openstack-discuss/2022-March/027837.html | 15:11 |
gmann | which is nova and keystone for now | 15:12 |
gmann | neutron in progress so that also a candidate for QA work | 15:12 |
kopecmartin | sounds good | 15:13 |
kopecmartin | #topic Gate Status Checks | 15:13 |
kopecmartin | #link https://review.opendev.org/q/label:Review-Priority%253D%252B2+status:open+(project:openstack/tempest+OR+project:openstack/patrole+OR+project:openstack/devstack+OR+project:openstack/grenade+OR+project:openstack/hacking) | 15:13 |
kopecmartin | no high priority patches | 15:14 |
frickler | I noticed hacking doesn't have RP | 15:14 |
slaweq | Hi. If you will be talking about neutron RBAC QA I will be interested to join 🙂 | 15:14 |
frickler | otherwise I'd tagged my py310 fix | 15:14 |
gmann | slaweq: great | 15:14 |
frickler | #link https://review.opendev.org/c/openstack/hacking/+/835413 | 15:14 |
gmann | frickler: I think there are very less traffic there so we can consider all review there as quick to do | 15:15 |
frickler | yeah, makes sense, but kopecmartin can drop it from the url above then as it will never match | 15:15 |
gmann | +2, lgtm | 15:15 |
gmann | ah, right. good point | 15:16 |
kopecmartin | slaweq: good, thanks, i'll keep that in mind | 15:16 |
kopecmartin | frickler: hacking patch merged | 15:16 |
frickler | thx | 15:16 |
kopecmartin | sure, i'll drop it | 15:16 |
kopecmartin | #topic Periodic jobs Status Checks | 15:17 |
kopecmartin | periodic stable | 15:17 |
kopecmartin | #link https://zuul.openstack.org/builds?job_name=tempest-full-xena&job_name=tempest-full-wallaby-py3&job_name=tempest-full-victoria-py3&job_name=tempest-full-ussuri-py3&pipeline=periodic-stable | 15:17 |
kopecmartin | periodic master | 15:17 |
kopecmartin | #link https://zuul.openstack.org/builds?project=openstack%2Ftempest&project=openstack%2Fdevstack&pipeline=periodic | 15:17 |
kopecmartin | seems all good | 15:18 |
frickler | pretty green | 15:18 |
kopecmartin | oh, that reminds me, is it time to add tempest yoga jobs? | 15:18 |
frickler | remember for next week to add periodic-weekly | 15:18 |
gmann | we did right no? | 15:19 |
frickler | ah, no, we didn't move fedora there | 15:19 |
frickler | then disregard | 15:19 |
gmann | #link https://review.opendev.org/c/openstack/tempest/+/835229/1/zuul.d/project.yaml | 15:20 |
gmann | kopecmartin: you mean in url? | 15:20 |
kopecmartin | gmann: oh, thanks, i forgot i pushed that review .. yeah, in that case i just need to update the url in the ageda | 15:20 |
kopecmartin | agenda | 15:20 |
gmann | kopecmartin: :) | 15:20 |
kopecmartin | frickler: periodic-weekly? | 15:20 |
gmann | we do not have peridioc-weekly jobs I think | 15:21 |
frickler | I thought we wanted to move the fedora jobs there, but then we didn't | 15:21 |
gmann | ah, yeah. | 15:21 |
frickler | or was that normal periodic? | 15:21 |
*** pojadhav|dinner is now known as pojadhav | 15:21 | |
frickler | anyway, not happening for now | 15:21 |
gmann | it was normal periodic but we reverted that | 15:21 |
kopecmartin | ah, i see, okey then | 15:22 |
kopecmartin | #topic Distros check | 15:23 |
kopecmartin | our new topic :) | 15:23 |
kopecmartin | centos 8/9 | 15:23 |
kopecmartin | #link https://zuul.openstack.org/builds?job_name=tempest-full-centos-9-stream&job_name=tempest-full-py3-centos-8-stream&job_name=devstack-platform-centos-8-stream&job_name=devstack-platform-centos-9-stream&skip=0 | 15:23 |
kopecmartin | fedora | 15:23 |
kopecmartin | #link https://zuul.openstack.org/builds?job_name=devstack-platform-fedora-latest&skip=0 | 15:23 |
kopecmartin | openEuler | 15:23 |
kopecmartin | #link https://zuul.openstack.org/builds?job_name=devstack-platform-openEuler-20.03-SP2+&skip=0 | 15:23 |
kopecmartin | and debian | 15:23 |
kopecmartin | #link https://zuul.openstack.org/builds?job_name=devstack-platform-debian-bullseye&skip=0 | 15:23 |
gmann | we can drop centos8 stream now? in zed TC defined testing runtime is centos9 stream only | 15:23 |
kopecmartin | gmann: ack, will do | 15:24 |
gmann | but can we do? i mean many jobs are still centos8-stream so we cannot drop it from devstackat least at this stage ? | 15:24 |
kopecmartin | centos 9 looks quite green \o/ | 15:24 |
gmann | frickler: usually we do keep support for one more cycle than they are not defined/targetted for testing? | 15:25 |
kopecmartin | oh, i meant we can delete that from the link in the agenda, for the rest i really need to check , yeah, it may be used on many places | 15:25 |
gmann | ah I mean testing and devstack support for centos8-stream not just list to monitor | 15:25 |
kopecmartin | yup, i get it now | 15:26 |
frickler | I think we would get pushback from various projects when dropping c8s already | 15:26 |
frickler | and as long as we need to keep support, we should also keep testing | 15:27 |
kopecmartin | need to do a research for that, we can discuss on ptg | 15:27 |
kopecmartin | frickler: correct | 15:27 |
gmann | yeah, let's wait. may be we can do in AA | 15:27 |
gmann | and see how projects are moving to centos9-stream in zed | 15:27 |
gmann | kopecmartin: +1 on check in PTG | 15:27 |
frickler | it sure would be good to incentivise (sp?) projects to migrate | 15:28 |
frickler | but even adding a 10 foot deprecation warning would mostly go unnoticed I fear | 15:28 |
kopecmartin | :D likely | 15:29 |
gmann | :) yeah. that does not help actually | 15:29 |
gmann | frickler: may be drop, break and revert. that is good warning signal :) | 15:30 |
opendevreview | Merged openstack/hacking master: Fix test errors with python3.10 https://review.opendev.org/c/openstack/hacking/+/835413 | 15:30 |
frickler | gmann: ack, I was thinking the same but hesitated to propose it ;) | 15:30 |
gmann | and as it is just start of cycle, we can try | 15:30 |
kopecmartin | :D | 15:30 |
gmann | kopecmartin: what you say? drop, send ML, if we get pushback then revert saying move your jobs soon | 15:31 |
kopecmartin | in case we will want to drop that job, yeah, it sounds like an effective way how to do it | 15:32 |
kopecmartin | if we drop | 15:32 |
kopecmartin | let's see how it goes on ptg | 15:32 |
gmann | yeah distro drop+testing goes together | 15:32 |
kopecmartin | moving on | 15:33 |
opendevreview | Brian Haley proposed openstack/devstack master: Add support for IPv6 tunnel endpoints https://review.opendev.org/c/openstack/devstack/+/710519 | 15:33 |
kopecmartin | Changes with Review-Priority == +1 | 15:33 |
kopecmartin | #link https://review.opendev.org/q/label:Review-Priority%253D%252B1+status:open+(project:openstack/tempest+OR+project:openstack/patrole+OR+project:openstack/devstack+OR+project:openstack/grenade+OR+project:openstack/hacking) | 15:34 |
opendevreview | Brian Haley proposed openstack/tempest master: Add an IPv6 tunnel endpoint job definition https://review.opendev.org/c/openstack/tempest/+/828431 | 15:34 |
kopecmartin | #link https://review.opendev.org/c/openstack/devstack/+/835227 | 15:34 |
kopecmartin | #link https://review.opendev.org/c/openstack/devstack/+/835228/2 | 15:34 |
frickler | someone proposed "fixing" setup.py for devstack | 15:34 |
haleyb | oops, sorry for interrupting | 15:34 |
frickler | so I though why not drop it instead? | 15:34 |
frickler | haleyb: np, submitting patches shouldn't stop just because we are meeting | 15:35 |
kopecmartin | frickler: sounds good to me, there isn't any info which wouldn't be somewhere else in the doc too, such as ML or the home page | 15:36 |
gmann | I think it make sense to drop. it create more confusion than thinking of in doc building | 15:36 |
kopecmartin | so there isn't any reason why to keep the files then i guess | 15:36 |
gmann | yeah | 15:36 |
frickler | 835228 needs a rebase I think | 15:37 |
gmann | +2 no setup one | 15:37 |
opendevreview | Martin Kopec proposed openstack/devstack stable/yoga: Stop installing Tempest at system wide for stable branch https://review.opendev.org/c/openstack/devstack/+/835228 | 15:38 |
gmann | yeah, I think extensions are good now. it can be rebased or just rebase on master | 15:38 |
kopecmartin | #topic Open Discussion | 15:39 |
kopecmartin | anything for the open discussion? | 15:39 |
frickler | https://review.opendev.org/c/openstack/devstack/+/832873 may be interesting for you to look at | 15:40 |
frickler | after some discussion, I'd go the simple way of simply adding everything we need to PATH hardcoded, because it doesn't cost much | 15:41 |
haleyb | since i wound-up here could i ask a question about the two IPv6 patches i just rebased? one devstack, one tempest | 15:43 |
haleyb | i hadn't added to agenda | 15:43 |
kopecmartin | frickler: it makes sense, i can't think of reasons why not to do so | 15:44 |
kopecmartin | haleyb: sure, can you send the links? | 15:44 |
*** soniya29|rover is now known as soniya29|rover|dinner | 15:44 | |
kopecmartin | let's have a ook | 15:44 |
kopecmartin | look | 15:44 |
haleyb | https://review.opendev.org/c/openstack/devstack/+/710519 and https://review.opendev.org/c/openstack/tempest/+/828431 | 15:45 |
haleyb | my question was mostly about the ipv6 jobs - i added a new one "ipv6-all" so as to not break the 'ipv6-only' users | 15:45 |
haleyb | but i didn't know if having yet another one was good, or should it be folded into the other | 15:46 |
haleyb | the devstack change has been on my list for a while, and i think gets things to almost 100% ipv6 | 15:46 |
haleyb | i've so far only tested with a neutron patch depending on them | 15:47 |
frickler | not answering your question, but how about making a job that also only uses v6 for tenant networks? | 15:48 |
frickler | I just happened to get that to work in kolla https://review.opendev.org/c/openstack/kolla-ansible/+/712768 | 15:48 |
haleyb | frickler: that would be the next logical step, since those would support IPv6 endpoints, the only thing left would be the multi-node job definitions, which i haven't had time to get working yet | 15:49 |
frickler | but I think we should keep those two jobs at least for a while. | 15:49 |
frickler | we could deprecate and later remove the ipv6-only job | 15:50 |
haleyb | frickler: yes, that would be an option once the new one is working everywhere. i haven't actually looked to see what repos inherit from those jobs to then put test patches out | 15:52 |
kopecmartin | good, so it sounds like it's a good idea to have it as a separate job for now | 15:52 |
kopecmartin | i'll do the review a bit later | 15:53 |
gmann | I think most of the project have the IPv6-only jobs | 15:53 |
gmann | can we just modify the original job itself with TUNNEL_IP_VERSION check ? | 15:53 |
haleyb | yes, searching in codesearch for devstack-tempest-ipv6 has a lot | 15:53 |
gmann | yeah, it was a community wide goal and I added in most of the projects | 15:54 |
gmann | and projects can enable it in their ipv6-only jobs if they wat | 15:54 |
gmann | or we can do enable it by default in ipv6-only and if anyone is breaking then disable/fix there ? | 15:54 |
frickler | +1 for the latter, seems reasonable | 15:55 |
haleyb | gmann: i guess that was my question, and i didn't want to break anything... | 15:55 |
kopecmartin | gmann: yup, that's a good idea (the last one), +1 | 15:55 |
frickler | breaking things but providing a fix along with it seems fine | 15:56 |
kopecmartin | well, sometimes breaking is good | 15:56 |
kopecmartin | frickler: yeah | 15:56 |
kopecmartin | anything else? | 15:57 |
frickler | thinking about it, likely few projects would even care how tunneling is set up | 15:57 |
gmann | yeah | 15:57 |
frickler | maybe octavia | 15:57 |
haleyb | gmann: so just so i understand correctly, modify the devstack change to enable the tunnel version as well, then the tempest change is almost not needed | 15:57 |
gmann | in start of cycle we can try these things | 15:57 |
gmann | haleyb: yeah, and it get enable in every *ipv6-only* jobs on projects side automatically. | 15:58 |
gmann | and after that we can send it on ML if anyone is surprised with that if broken/ | 15:58 |
haleyb | gmann: they could be. as i said the only thing i didn't get working yet is multi-node w/ipv6, but there aren't jobs for that yet anyways | 15:59 |
kopecmartin | let's skip bug triage today | 15:59 |
gmann | haleyb: yeah there are very few I think | 15:59 |
haleyb | thanks for the feedback everyone | 15:59 |
kopecmartin | gmann: when you have a moment, can you check these 2 (relation chain): https://review.opendev.org/c/openstack/tempest/+/835271 | 15:59 |
kopecmartin | haleyb: yw | 15:59 |
gmann | haleyb: but you can disable in multinode job until you make it working | 16:00 |
gmann | kopecmartin: ack, will check after nova meeting | 16:00 |
kopecmartin | thanks | 16:00 |
kopecmartin | that's it, time for the office hour is up | 16:00 |
kopecmartin | see you everyone on PTG | 16:00 |
frickler | thx all | 16:00 |
kopecmartin | #endmeeting | 16:01 |
opendevmeet | Meeting ended Tue Mar 29 16:01:05 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/qa/2022/qa.2022-03-29-15.00.html | 16:01 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/qa/2022/qa.2022-03-29-15.00.txt | 16:01 |
opendevmeet | Log: https://meetings.opendev.org/meetings/qa/2022/qa.2022-03-29-15.00.log.html | 16:01 |
gmann | o/. thanks everyone | 16:01 |
gmann | frickler: dansmith kopecmartin : grenade setup for stable/yoga-> master is working now https://review.opendev.org/c/openstack/grenade/+/835232/3 and userrc_early fix https://review.opendev.org/c/openstack/grenade/+/835531/1 | 16:26 |
*** soniya29|rover|dinner is now known as soniya29|rover | 16:31 | |
*** jpena is now known as jpena|off | 16:46 | |
frickler | gmann: one thing I forgot to mention was the tag_attach fix I made. IIUC the original idea with the SSHABLE fix was that only volume detach would be affected? because this now is for volume attachment | 17:43 |
frickler | and it might also be good to check for other similar patterns still remaining | 17:43 |
frickler | also might be worth mentioning to nova team to try and find a better solution, since I think this may also happen for real world scenarios | 17:45 |
gmann | frickler: ok, we can also monitor based on failure happening in which operation. | 17:45 |
gmann | yeah | 17:45 |
opendevreview | Merged openstack/devstack master: Drop setup.py and setup.cfg https://review.opendev.org/c/openstack/devstack/+/835442 | 17:50 |
frickler | gmann: actually, looking at the failures again and at the code, the issue seems to have been not volume attachment failing, but volume attachment succeeding too fast | 17:54 |
frickler | there is no waiting anywhere in this block, so the verify ssh call is made before the instance has finished booting https://opendev.org/openstack/tempest/src/branch/master/tempest/api/compute/servers/test_device_tagging.py#L410-L425 | 17:56 |
frickler | https://edbf05b13b5adc234293-64e8601ca0a2527bb84fce93b634fc17.ssl.cf1.rackcdn.com/835434/1/check/devstack-platform-debian-bullseye/e092cca/job-output.txt | 17:57 |
frickler | in the console log for the instance, vdb is attached after 18s, while resizing / runs until 44s | 17:58 |
gmann | frickler: but this test does wait for ssh-able (after your patch) so it is not the server ready issue? | 18:01 |
gmann | ohk so that failure is before that merged. got it | 18:02 |
gmann | frickler: I will check such pattern if other test also does ssh command execute, that is good candidate to wait for SSH-able. | 18:03 |
frickler | ack | 18:10 |
opendevreview | Brian Haley proposed openstack/devstack master: Add support for IPv6 tunnel endpoints https://review.opendev.org/c/openstack/devstack/+/710519 | 18:11 |
opendevreview | Frank Ritchie proposed openstack/patrole master: Update variable names for min/max microversion https://review.opendev.org/c/openstack/patrole/+/835743 | 18:25 |
opendevreview | Frank Ritchie proposed openstack/patrole master: Update variable names for min/max microversion https://review.opendev.org/c/openstack/patrole/+/835743 | 18:26 |
sean-k-mooney | o/ | 19:22 |
sean-k-mooney | has anyone ever tried to resue the devstack ansible roles or playbooks form ansible outside of zuul | 19:22 |
sean-k-mooney | ok never mind they are not resuabel as none of the devstack roles are actully packaged as glaxy roles so i can list the devstack repo as a reqruiemetn and hten use the roles form it | 19:31 |
*** lpiwowar9 is now known as lpiwowar | 19:31 | |
clarkb | note that isn't required to use roles in ansible. YOu just need them in your role lookup path | 19:32 |
sean-k-mooney | ya im just trying to figure out how to automaically add them to the path for molecue to find them | 19:32 |
sean-k-mooney | im currently trying to write some playbooks/roles to deploy devstack automatically on vms provision using molecue and vagrant-libvirt | 19:33 |
sean-k-mooney | i was trying to figure out if i shoud just templeation the devstack local.conf and invoke it stack.sh form the shel module or if i could reuse the ansible roles in devstack or zull-jobs | 19:34 |
clarkb | Re galaxy, the requirement to have everything on github meant we never bothered to look into it and it hasn't relaly been a problem. I think glaxy doesn't require that anymore but we made due without so meh | 19:34 |
sean-k-mooney | well you dont actully have to have it in github it can use a tar or any git repo | 19:35 |
clarkb | The role in zuul-jobs is intended to be very basic. Deploy a default devstack basically without any other configuration. | 19:35 |
sean-k-mooney | i was using opendev.org in my requirements | 19:35 |
clarkb | sean-k-mooney: yes but that is realtively new. And if you've spent years working around that the motiviation to change now is low | 19:35 |
sean-k-mooney | clarkb: actully i did not plan ot use the zuul job devstack role | 19:36 |
clarkb | that said if someone wanted to figure it out it should be doable. Its just extremely low on my priority list since we already do thing without galaxy in our world due to the historicaly requirement | 19:36 |
sean-k-mooney | i was thinking more of the other common roles | 19:36 |
sean-k-mooney | ack im just rying to figure out how i can reuse the roles we have form my own playbooks | 19:37 |
sean-k-mooney | clarkb: so you woudl suggest just cloning devstack locally and adding it to the roles path? | 19:37 |
clarkb | sean-k-mooney: I think that is roughly what we do for our ansible stuff. Things get cloned/checked out and ansible is told where to find them | 19:37 |
sean-k-mooney | i have a bootstrap repo command so i guess i can do that there | 19:42 |
sean-k-mooney | https://github.com/SeanMooney/ansible_role_devstack/blob/master/bootstrap-repo.sh | 19:42 |
sean-k-mooney | clarkb: looks like ooo ansibel do somethign similar | 19:47 |
sean-k-mooney | https://opendev.org/openstack/tripleo-ansible/src/branch/master/.config/molecule/config.yml#L33= | 19:47 |
sean-k-mooney | if i add zuul-jobs and devstack as git submodules i could them add there roles to the roles path i guess | 19:48 |
sean-k-mooney | clarkb: quick queition where is the "zuul" variable defiend | 20:10 |
sean-k-mooney | for example https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/validate-host/tasks/main.yaml#L3 | 20:10 |
sean-k-mooney | i assume its defined in the inventory | 20:10 |
sean-k-mooney | generated by zuul | 20:10 |
sean-k-mooney | https://zuul-ci.org/docs/zuul/latest/job-content.html#zuul-variables so ya i assume those are all define in the inveotry or passed when zuul is invoking ansible | 20:14 |
sean-k-mooney | clarkb: ok this seames to work. at least for roles that dont depend on zuul paramaters | 20:35 |
sean-k-mooney | https://github.com/SeanMooney/ansible_role_devstack/commit/118e60a988a6d440c9dbc162f375c5fa5e0d7881 | 20:35 |
sean-k-mooney | im not sure how practical it is to actully do this so i might revert that and go back to plan b which was just generate a local.conf and invoke devstack with the shell modules | 20:36 |
clarkb | I think you'd need to set zuul parameters in your inventory or otherwise | 20:36 |
clarkb | thats all that zuul does with them (it puts them in the inventory) | 20:37 |
sean-k-mooney | yes either set the vars or in the inventory | 20:37 |
sean-k-mooney | ensure-pip does not use any zuul vars so it jsut works when reused | 20:38 |
sean-k-mooney | i would gess most of the ensure-* ones woudl be resuable | 20:41 |
*** lpiwowar8 is now known as lpiwowar | 21:23 | |
opendevreview | Merged openstack/tempest master: Use wrap_socket from ssl.SSLContext instead of ssl https://review.opendev.org/c/openstack/tempest/+/835271 | 21:55 |
opendevreview | Merged openstack/grenade master: Remove sourcing of userrc_early https://review.opendev.org/c/openstack/grenade/+/835531 | 21:55 |
opendevreview | Merged openstack/grenade master: Update the master grenade setting for yoga to master upgrade https://review.opendev.org/c/openstack/grenade/+/835232 | 21:55 |
opendevreview | Frank Ritchie proposed openstack/patrole master: [WIP] Update variable names for min/max microversion https://review.opendev.org/c/openstack/patrole/+/835743 | 22:26 |
opendevreview | Frank Ritchie proposed openstack/patrole master: [WIP] Update variable names for min/max microversion https://review.opendev.org/c/openstack/patrole/+/835743 | 22:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!