| amotoki | gmann: https://review.opendev.org/c/openstack/neutron/+/801478 which fixes tempest-full/slow failure just landed in neutron. I also sent a mail to the -discuss list. | 01:29 |
|---|---|---|
| opendevreview | Pavan Kesava Rao proposed openstack/tempest master: Add test to verify FQDN hostname sanitization https://review.opendev.org/c/openstack/tempest/+/795699 | 02:42 |
| gmann | amotoki: thanks | 02:45 |
| *** rpittau|afk is now known as rpittau | 07:49 | |
| *** bhagyashris_ is now known as bhagyashris | 08:05 | |
| *** bhagyashris_ is now known as bhagyashris | 09:35 | |
| *** elodilles is now known as elodilles_afk | 12:48 | |
| gryf | hello. | 12:52 |
| gryf | I have a job, which is inherited (actually, there is a quite a chain of inheritance, although not so complicated), where on base parent there are defined (in job.vars.devsta_services) devstack services | 12:54 |
| gryf | although in inherited job, I cannot see any of them on generated local.conf on zuul side. | 12:55 |
| gryf | there are also used host-vars on the child job, and I'm wondering, if that could be a reson for | 12:56 |
| gryf | overwriting, instead of merging dicts with vars. | 12:56 |
| gryf | because it seems like host-vars.devstack_services overwrites vars.devstack_services from base job. | 12:57 |
| gryf | or, maybe, I misunderstand the concept. | 12:58 |
| sean-k-mooney | host-vars has prescidence over vars i belive | 13:07 |
| sean-k-mooney | in ansible | 13:07 |
| sean-k-mooney | so you can use vars to set default and override per host | 13:08 |
| sean-k-mooney | https://docs.ansible.com/ansible/latest/user_guide/playbooks_variables.html#understanding-variable-precedence | 13:09 |
| sean-k-mooney | so host_vars superseeds group_vars | 13:10 |
| gryf | that was my thinking, although it looks like all the services from vars.devstack_services has been replaced by those in host-vars.devstack_services | 13:10 |
| sean-k-mooney | where is the job definiton | 13:11 |
| gryf | sean-k-mooney, I'm currently doodling with it here: https://review.opendev.org/c/openstack/kuryr-kubernetes/+/801618 | 13:11 |
| sean-k-mooney | its perfectly fine to do service overides like this https://github.com/openstack/os-vif/blob/master/.zuul.yaml#L49 | 13:11 |
| gryf | multinode job (only the first for now) is at my interest. | 13:12 |
| sean-k-mooney | why not just use group vars | 13:14 |
| gryf | right, but I'd like to change some of the default ones (which are defined in job.vars.devstack_services) with other in child jobs (through host-vars.devstack_services), so what I'd expect is to get combined vars from parent and child | 13:14 |
| sean-k-mooney | or vars | 13:14 |
| sean-k-mooney | what you are doing does look like it should work | 13:15 |
| sean-k-mooney | but i have only use vars and group vars with the devstack jobs in the past | 13:15 |
| gryf | well, I'd like to change only controller. and as for compute1 I'd like to have a different set of services. | 13:15 |
| sean-k-mooney | right so normally you modify the default use for the conrotl with vars | 13:15 |
| gryf | so I've jumped in the wild idea to select it through the host-vars ;) | 13:16 |
| sean-k-mooney | then use group-vars: subnode | 13:16 |
| sean-k-mooney | to override that for the subnodes | 13:16 |
| sean-k-mooney | the files you are modifying here in zuul are also not actully ansible | 13:16 |
| sean-k-mooney | they are zuul job definions which are not entirely the same thing | 13:16 |
| gryf | I understand. | 13:17 |
| gryf | ok, I'll try to resolve this using group-vars. | 13:17 |
| gryf | thanks, sean-k-mooney. | 13:17 |
| sean-k-mooney | https://zuul-ci.org/docs/zuul/reference/job_def.html#attr-job.vars | 13:18 |
| sean-k-mooney | so vars is merged with the prevsion defintions before being passed to ansible | 13:18 |
| gryf | right | 13:19 |
| gryf | and that is working flawlessly. | 13:19 |
| sean-k-mooney | but that is not mentioned for https://zuul-ci.org/docs/zuul/reference/job_def.html#attr-job.host-vars | 13:19 |
| sean-k-mooney | or https://zuul-ci.org/docs/zuul/reference/job_def.html#attr-job.group-vars | 13:19 |
| sean-k-mooney | so i dont think there is inheriticae in those cases | 13:19 |
| gryf | hm. | 13:20 |
| sean-k-mooney | gryf: it is proably worth askign on #zuul | 13:20 |
| gryf | ok. I'll give it a try. | 13:20 |
| gryf | thanks again. | 13:20 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Add vgpu resize tests https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/790304 | 13:55 |
| *** elodilles_afk is now known as elodilles | 14:34 | |
| opendevreview | Ananya proposed opendev/elastic-recheck master: Run elastic-recheck container https://review.opendev.org/c/opendev/elastic-recheck/+/729623 | 14:43 |
| opendevreview | Ananya proposed opendev/elastic-recheck master: Run elastic-recheck container https://review.opendev.org/c/opendev/elastic-recheck/+/729623 | 15:02 |
| opendevreview | Pavan Kesava Rao proposed openstack/tempest master: Add test to verify FQDN hostname sanitization https://review.opendev.org/c/openstack/tempest/+/795699 | 15:21 |
| opendevreview | Ananya proposed opendev/elastic-recheck master: Run elastic-recheck container https://review.opendev.org/c/opendev/elastic-recheck/+/729623 | 15:52 |
| *** pojadhav is now known as pojadhav|afk | 15:56 | |
| *** rpittau is now known as rpittau|afk | 16:02 | |
| opendevreview | Pavan Kesava Rao proposed openstack/devstack master: Enable tempest tests for hostname sanitization https://review.opendev.org/c/openstack/devstack/+/801366 | 16:17 |
| opendevreview | Ghanshyam proposed openstack/tempest master: Add openstacksdk-functional-devstack in integrated gates https://review.opendev.org/c/openstack/tempest/+/801920 | 18:43 |
| opendevreview | Pavan Kesava Rao proposed openstack/tempest master: Add test to verify FQDN hostname sanitization https://review.opendev.org/c/openstack/tempest/+/795699 | 19:06 |
| opendevreview | Merged openstack/tempest master: Improve dynamic cred log to include scope https://review.opendev.org/c/openstack/tempest/+/801132 | 19:29 |
| opendevreview | Merged openstack/tempest master: Add keystone unified limits client https://review.opendev.org/c/openstack/tempest/+/788345 | 19:33 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!