| *** ralonsoh_ is now known as ralonsoh | 08:08 | |
| slaweq | hi kopecmartin, frickler and other devstack core team members, can You check https://review.opendev.org/c/openstack/devstack/+/908572 when You will have a minute? Thx in advace | 08:33 |
|---|---|---|
| slaweq | this unlocks neutron periodic jobs where sqlalchemy is installed from master branch | 08:33 |
| opendevreview | Martin Kopec proposed openstack/devstack master: Set source and destination params for tests https://review.opendev.org/c/openstack/devstack/+/908331 | 14:03 |
| opendevreview | Martin Kopec proposed openstack/tempest master: Add option to specify source and destination host https://review.opendev.org/c/openstack/tempest/+/891123 | 14:04 |
| opendevreview | Lajos Katona proposed openstack/grenade stable/2023.1: Unmaintained-Only: change stable/yoga to unmaintained https://review.opendev.org/c/openstack/grenade/+/908795 | 14:45 |
| *** jph3 is now known as jph | 15:06 | |
| opendevreview | Lajos Katona proposed openstack/grenade stable/zed: Unmaintained-Only: change stable/yoga to unmaintained https://review.opendev.org/c/openstack/grenade/+/908826 | 15:17 |
| opendevreview | Merged openstack/devstack master: Add support for the pyproject.toml file in setup with constraints https://review.opendev.org/c/openstack/devstack/+/908572 | 16:05 |
| kopecmartin | gmann: seems like finding out all available nodes in devstack isn't as simple as we thought - https://review.opendev.org/c/openstack/devstack/+/908331/comment/507c0980_c2e90ae9/ | 20:41 |
| gmann | kopecmartin: but we do the same in tempest tests right? | 20:42 |
| gmann | kopecmartin: maybe list service will be easy, https://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L688 | 20:44 |
| kopecmartin | maybe the second node isn't deployed at the time of running 'openstack hypervisor list' | 20:44 |
| gmann | getting it from service list or hypervisor list is same | 20:44 |
| gmann | but does not tempest install/configure happen after devstack setup all the nova nodes | 20:44 |
| kopecmartin | that I don't know | 20:44 |
| gmann | it is from extra scripts so it is running after all enabled services are setup https://github.com/openstack/devstack/blob/master/extras.d/80-tempest.sh | 20:45 |
| kopecmartin | maybe a wrong openstack command? instead of 'openstack hypervisor list' we could run 'openstack cluster node list'? | 20:50 |
| kopecmartin | it's been a while i read the documentation | 20:50 |
| * kopecmartin now searching random things on the internet | 20:51 | |
| gmann | kopecmartin: tempest is installed at the end here even after the service_check on each enabled services, example https://zuul.opendev.org/t/openstack/build/db9affc6308a4311968037dd7442653b/log/controller/logs/devstacklog.txt#19824 | 20:55 |
| kopecmartin | or 'openstack host list' instead of 'openstack hypervisor list' | 20:55 |
| gmann | kopecmartin: use service list and then filter the smae way this test does https://github.com/openstack/tempest/blob/master/tempest/api/compute/base.py#L688 | 20:55 |
| gmann | host list is deprecated API from nova | 20:55 |
| gmann | service list is right one | 20:56 |
| kopecmartin | but that link is for tempest , i don't understand how that helps me in devstack | 20:56 |
| kopecmartin | plus one is python and one bash | 20:56 |
| gmann | I mean its just a example of which API can give us host info | 20:57 |
| kopecmartin | we should train an AI for these things, i really don't want to see this patch again .. | 20:58 |
| gmann | bcz without running those new tests it is not good to merge as we found one typo there when try to run them and there might be other error exist so it is hard to merge the test which is not running in upstream gate or at least we have proof of 3rd party/downstream CI result | 20:58 |
| kopecmartin | what if we hardcoded 'compute1' and 'controller' for this job specifically? | 21:05 |
| kopecmartin | it could work if tempest client would understand 'controller' instead of 'np0036730222' | 21:06 |
| * kopecmartin checking the output of LOOP [emit-job-header : Print node information] | 21:07 | |
| gmann | I do not think that will work, as nova expect host name in server boot API not the service name | 21:07 |
| gmann | https://docs.openstack.org/api-ref/compute/#create-server | 21:08 |
| kopecmartin | hmm, then maybe an ansible role/task ran after devstack but before tempest | 21:08 |
| kopecmartin | ansible has all the info in the inventory - it would be almost a copy paste of this https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/emit-job-header/tasks/main.yaml | 21:08 |
| kopecmartin | well "copy paste" .. the same but different :D | 21:09 |
| kopecmartin | but finding the host name is one thing, editing tempest.conf another .. but i recall i did something similar a while back, maybe i can use that if i find it | 21:10 |
| opendevreview | Martin Kopec proposed openstack/tempest master: Add set-src-dest-host role https://review.opendev.org/c/openstack/tempest/+/908866 | 21:43 |
| opendevreview | Martin Kopec proposed openstack/tempest master: Add scenario test: test instances with cinder volumes https://review.opendev.org/c/openstack/tempest/+/898732 | 22:47 |
| opendevreview | Martin Kopec proposed openstack/tempest master: Add set-src-dest-host role https://review.opendev.org/c/openstack/tempest/+/908866 | 22:50 |
| opendevreview | Martin Kopec proposed openstack/tempest master: Add set-src-dest-host role https://review.opendev.org/c/openstack/tempest/+/908866 | 23:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!