| *** ykarel_ is now known as ykarel | 06:23 | |
| *** bhagyashris_ is now known as bhagyashris|ruck | 07:56 | |
| jm1 | frickler, kopecmartin: Please review https://review.opendev.org/c/openstack/devstack/+/828280 :) | 08:38 |
|---|---|---|
| *** jpena|off is now known as jpena | 09:42 | |
| opendevreview | Merged openstack/devstack master: Fix installation with OVN backend and compilation https://review.opendev.org/c/openstack/devstack/+/828877 | 10:47 |
| opendevreview | Elod Illes proposed openstack/devstack stable/wallaby: nova: Enable apic removal workaround for bug #1939108 when using QEMU https://review.opendev.org/c/openstack/devstack/+/805632 | 13:38 |
| dansmith | gmann: so I need to depends-on this from a nova patch to get a test run right? https://review.opendev.org/c/openstack/tempest/+/830670 | 16:26 |
| gmann | dansmith: yeah, you can do that. just to check but it will be running. | 16:27 |
| dansmith | yeah, okay | 16:27 |
| gmann | dansmith: or you can add in nova for irrelevant files . | 16:27 |
| dansmith | not sure I understand that last bit | 16:28 |
| gmann | dansmith: I mean we need to add irrelevant-files in nova zuul config so that it does not run for doc etc changes, like we did for other jobs | 16:29 |
| gmann | that is one change we need to do for this new job | 16:29 |
| dansmith | ah | 16:29 |
| gmann | we cannot do that for template so need to do per job | 16:30 |
| dansmith | ack | 16:31 |
| gmann | tc-members: need 1 or 2 more votes on this - https://review.opendev.org/c/openstack/governance/+/830915 https://review.opendev.org/c/openstack/governance/+/830910/3 | 16:47 |
| gmann | dansmith: ^^ as you are here | 16:47 |
| dansmith | done | 16:51 |
| *** jpena is now known as jpena|off | 17:31 | |
| opendevreview | Michael Johnson proposed openstack/devstack master: Fix tls-proxy on newer versions of openssl https://review.opendev.org/c/openstack/devstack/+/831245 | 18:45 |
| *** artom__ is now known as artom | 19:02 | |
| artom | jparker__, so, https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/829886/2 for you to look at, I think I've finally understood the resources cleanup situation | 19:02 |
| * jparker__ looking | 19:03 | |
| jparker__ | artom: this all LGTM, let me run it against the SR-IOV bed I have downstream real quick | 19:10 |
| jparker__ | artom: I do need to readd the admin client to the server creation for those sr-iov tests | 19:10 |
| jparker__ | artom: the ports are created via admin client, w/o it the default guest creation can't find the ports | 19:11 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Readd admin client for sriov guest creation https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/831275 | 19:28 |
| jparker__ | artom: no rush but can you take a look at this when you get a chance https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/831275 | 19:35 |
| artom | jparker__, hrrm, do the ports *have* to be created with the admin client? | 19:36 |
| artom | jparker__, I think the correct thing to do here would be to stop creating the ports with the admin client, no? | 19:37 |
| jparker__ | artom: let me try that, whenever I first created that module I had some issues with the ports not being admin | 19:38 |
| artom | Or does specifying the binding type preclude us from doing that... | 19:38 |
| artom | I mean, that ^^ would be a bug, because I would expect non-admins to be able to consume SRIOV ports | 19:38 |
| artom | Or we're doing something wrong | 19:38 |
| jparker__ | artom: I honestly can't remember, I know something led me to switch to admin back then | 19:38 |
| jparker__ | artom: I've got the bed ready right now so let me try | 19:39 |
| artom | jparker__, I think that's worth looking into, no? | 19:39 |
| jparker__ | artom: oh yes I agree that better approach is no admin | 19:39 |
| jparker__ | artom: although I use admin to force a server schedule | 19:39 |
| jparker__ | artom: so i dont know if that would be a problem with finding ports | 19:39 |
| artom | jparker__, so, if you're forcing the scheduling to be on the same host as another server, we can use the same_host scheduler hint | 19:40 |
| artom | In fact, we probably should, as it's better to not be admin if we can help it | 19:41 |
| jparker__ | artom: ack let me update the commit to use the hint/non-admin approach | 19:42 |
| opendevreview | Merged openstack/whitebox-tempest-plugin master: Cleanup after every test instead of after the whole test class https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/829886 | 20:39 |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Deploy with cephadm https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484 | 21:37 |
| opendevreview | Merged openstack/whitebox-tempest-plugin master: Add barbican to our devstack job https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/779043 | 23:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!