| *** dviroel is now known as dviroel|pto | 00:40 | |
| opendevreview | Lewis Denny proposed openstack/tripleo-docs master: Update node delete host name list command https://review.opendev.org/c/openstack/tripleo-docs/+/856218 | 00:56 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: RabbitMQ: Fix missing bind mount for .key file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856219 | 01:04 |
| opendevreview | Lewis Denny proposed openstack/tripleo-ansible master: Migrate tripleo_ovn_metadata host prep tasks to tripleo-ansible https://review.opendev.org/c/openstack/tripleo-ansible/+/856220 | 01:08 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Cinder/Manila: Remove unused logic for docker_volumes/envionments https://review.opendev.org/c/openstack/puppet-tripleo/+/854941 | 01:20 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Ironic: Fix incorrect type for integer values https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855158 | 01:21 |
| opendevreview | Merged openstack/puppet-tripleo master: remove unicode from code https://review.opendev.org/c/openstack/puppet-tripleo/+/854483 | 01:21 |
| *** ysandeep|out is now known as ysandeep | 01:25 | |
| opendevreview | Lewis Denny proposed openstack/tripleo-ansible master: renaming ovn standalone role hostprep.yml to install.yml https://review.opendev.org/c/openstack/tripleo-ansible/+/856222 | 01:25 |
| opendevreview | Lewis Denny proposed openstack/tripleo-heat-templates master: Use tripleo_ovn standalone role for host_prep_tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/842044 | 01:28 |
| opendevreview | Merged openstack/tripleo-common master: Fix condition for CHROOT, missing quotes https://review.opendev.org/c/openstack/tripleo-common/+/856154 | 01:59 |
| opendevreview | Lewis Denny proposed openstack/tripleo-ansible master: renaming ovn standalone role hostprep.yml to install.yml https://review.opendev.org/c/openstack/tripleo-ansible/+/856222 | 02:32 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Nova: Set unprivileged_userfaultfd only when post-copy is enabled https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856025 | 03:09 |
| opendevreview | Rabi Mishra proposed openstack/python-tripleoclient master: Use host network for db_sync https://review.opendev.org/c/openstack/python-tripleoclient/+/856226 | 03:47 |
| opendevreview | Steve Baker proposed openstack/tripleo-common master: Populate /var/www/cgi-bin/ironic/ with wsgi app https://review.opendev.org/c/openstack/tripleo-common/+/854459 | 04:02 |
| opendevreview | Steve Baker proposed openstack/tripleo-common master: Do not install syslinux-tftpboot https://review.opendev.org/c/openstack/tripleo-common/+/854059 | 04:02 |
| *** ysandeep is now known as ysandeep|afk | 04:04 | |
| opendevreview | Merged openstack/tripleo-heat-templates master: Remove argument to ssh-keygen for key size https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854602 | 04:38 |
| opendevreview | Merged openstack/puppet-tripleo master: mysql_bundle: Remove redundant override of mysqld options https://review.opendev.org/c/openstack/puppet-tripleo/+/855104 | 04:38 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: RabbitMQ: Fix missing bind mount for .key file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856219 | 04:50 |
| lecris[m] | Quick question, how do I trigger a recheck? | 04:53 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Redis: Migrate environment/volumes definition https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855113 | 04:53 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: MariaDB: Migrate environment/volumes definition https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855101 | 04:55 |
| *** ysandeep|afk is now known as ysandeep | 04:58 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: MySQL: Use common volumes in containers https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856228 | 05:05 |
| opendevreview | Rabi Mishra proposed openstack/python-tripleoclient master: Use host network for db_sync https://review.opendev.org/c/openstack/python-tripleoclient/+/856226 | 05:21 |
| ykarel | lecris[m], can post comment "recheck" on the patch | 05:35 |
| ykarel | or you mean jobs not triggered even after you commented? | 05:35 |
| lecris[m] | It might be the latter | 05:36 |
| lecris[m] | So firstly there is no new patch, because the build error is unrelated to the commit, it randomly timed out | 05:36 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Cinder/Manila: Remove unused logic for docker_volumes/envionments https://review.opendev.org/c/openstack/puppet-tripleo/+/854941 | 05:36 |
| ykarel | lecris[m], patch link | 05:37 |
| lecris[m] | So, do I simply reply to any comment with `recheck`? I don't think it submitted a new job (https://review.opendev.org/c/openstack/tripleo-ansible/+/856062) | 05:37 |
| ykarel | lecris[m], jobs are running https://zuul.openstack.org/status#856062 | 05:38 |
| lecris[m] | Oh, ok. Is it possible to submit only to rdo? | 05:39 |
| ykarel | lecris[m], for that can comment: check-rdo | 05:39 |
| ykarel | with recheck all upstream and rdo get's triggered | 05:39 |
| lecris[m] | Thanks for the tip | 05:39 |
| lecris[m] | Does it matter if it has a quote reply? | 05:40 |
| ykarel | yes it needs to be without quotes | 05:40 |
| lecris[m] | clarification quotes as in quoting the message of the CI, not the "" | 05:41 |
| ykarel | it should match the regex https://github.com/openstack/project-config/blob/master/zuul.d/pipelines.yaml#L24 | 05:41 |
| lecris[m] | Oh boy, regex black-magic. Should be able to test that though | 05:42 |
| ykarel | okk i doubt it will work with that as will not match the regex ^ | 05:42 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ci master: puppet: Test pacemaker services by multinode ipa job https://review.opendev.org/c/openstack/tripleo-ci/+/856229 | 05:42 |
| lecris[m] | tripleo-ci-centos-9-scenario001-ceph-nightly might not be picking up on errors in `openstack overcloud ceph deploy`? There is an issue currently that the container is one version ahead of the `cephadm` causing it to fail. | 05:51 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New tripleo_ironic role https://review.opendev.org/c/openstack/tripleo-ansible/+/855873 | 06:10 |
| opendevreview | Luca Miccini proposed openstack/puppet-pacemaker master: Expose run_arping to control IP collisions detection https://review.opendev.org/c/openstack/puppet-pacemaker/+/856233 | 06:32 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-ansible master: Add unified_observability_data_only condition to node-exporter component https://review.opendev.org/c/openstack/tripleo-ansible/+/854176 | 06:40 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Add observability_data_only bool https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854178 | 06:44 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-ansible master: Add observability_data_only condition to node-exporter component https://review.opendev.org/c/openstack/tripleo-ansible/+/854176 | 06:44 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Add observability_data_only bool https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854178 | 06:45 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Add observability_data_only bool https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854178 | 06:46 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Add observability_data_only bool https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854178 | 06:46 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: sshd: Strictly validate value for PasswordAuthentication https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856234 | 06:54 |
| tkajinam | I'm afraid I broke multinode-ipa job in master and we need https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856219 to fix it. | 06:59 |
| * tkajinam is still waiting for CI | 06:59 | |
| dciabrin | tkajinam, preemptive +2 as it lgtm | 07:06 |
| tkajinam | dciabrin, thanks ! | 07:07 |
| lecris[m] | Question about rabbitmq. Is the address supposed to be `controller-0.internalapi.domain.com` isntead of `overcloud.internalapi.domain.com`? | 07:15 |
| tkajinam | rabbitmq does not use vip so the local hostname should be correct | 07:22 |
| lecris[m] | Kay, then I still have no idea why rabbitmq is failing on me with tls-e | 07:24 |
| opendevreview | amolkahat proposed openstack/tripleo-quickstart master: Mixed OS conditional dlrn_hash_tag for component line https://review.opendev.org/c/openstack/tripleo-quickstart/+/853860 | 07:28 |
| lecris[m] | Also, tkajinam the leftover hiera of rabbitmq that I encountered are in the container itself ` tripleomastercentos9 / openstack-rabbitmq`? | 07:28 |
| lecris[m] | s/`?/`/ | 07:29 |
| tkajinam | I didn't get what you mean | 07:32 |
| tkajinam | hieradata is not part of containers | 07:32 |
| *** jpena|off is now known as jpena | 07:34 | |
| lecris[m] | tkajinam:... (full message at https://matrix.org/_matrix/media/r0/download/matrix.org/StadSNhGwhHgYmIAvAbAQyvk) | 07:36 |
| ramishra_ | tkajinam: do we've a LP for https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_073/847635/1/gate/tripleo-ci-centos-9-standalone-on-multinode-ipa/073087b/logs/undercloud/var/log/containers/rabbitmq/startup_log ? | 07:36 |
| * ramishra_ noticed the patch to fix it | 07:36 | |
| lecris[m] | ramishra_: Isn't it this https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856219? | 07:37 |
| ramishra_ | lecris[m]: yes, I was looking for bug report | 07:38 |
| tkajinam | ramishra_, no I've not yet created one | 07:39 |
| chkumar|ruck | tkajinam: thank you for fixing ipa failure quickly Do we have a bug opened for that? | 07:44 |
| *** ysandeep is now known as ysandeep|lunch | 07:45 | |
| chkumar|ruck | creating one | 07:50 |
| tkajinam | chkumar|ruck, sorry not yet. submitted that patch before reporting that bug. | 07:54 |
| chkumar|ruck | tkajinam: https://bugs.launchpad.net/tripleo/+bug/1988949 | 07:59 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart-extras master: Adding temporary workaround for libvirt-failures https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/856244 | 08:36 |
| tkajinam | chkumar|ruck, ipa job passed with https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856219 so I've pushed it to gate. I need to leave soon but that one needs some attention just in case it fails at gate for an intermittent issue | 08:51 |
| tkajinam | I've pushed a patch which would save us from a similar problem https://review.opendev.org/c/openstack/tripleo-ci/+/856229 | 08:51 |
| tkajinam | I need to leave for a while but will check status later | 08:52 |
| chkumar|ruck | tkajinam: ++ thank you :-) | 08:57 |
| *** ysandeep|lunch is now known as ysandeep | 09:10 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Switch default firewall engine https://review.opendev.org/c/openstack/tripleo-heat-templates/+/852808 | 09:21 |
| *** jgilaber_ is now known as jgilaber | 09:45 | |
| *** ysandeep is now known as ysandeep|afk | 10:17 | |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart-extras master: Adding temporary workaround for libvirt-failures https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/856244 | 10:32 |
| *** rlandy|out is now known as rlandy | 10:34 | |
| chkumar|ruck | bogdando: I will take a look at molecule failure today https://review.opendev.org/c/openstack/tripleo-ansible/+/842152 | 10:40 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Do not change ownership recursive for Swift https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855541 | 10:56 |
| opendevreview | amolkahat proposed openstack/tripleo-quickstart master: Mixed OS conditional dlrn_hash_tag for component line https://review.opendev.org/c/openstack/tripleo-quickstart/+/853860 | 11:15 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment https://review.opendev.org/c/openstack/tripleo-ansible/+/852386 | 11:19 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Remove newline https://review.opendev.org/c/openstack/tripleo-ansible/+/856265 | 11:22 |
| *** ysandeep|afk is now known as ysandeep | 11:30 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Remove newline https://review.opendev.org/c/openstack/tripleo-ansible/+/856265 | 11:31 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Make tripleo_nftables compatible with puppet-firewall https://review.opendev.org/c/openstack/tripleo-ansible/+/856266 | 11:45 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Victoria tear down : multinode and undercloud jobs https://review.opendev.org/c/openstack/tripleo-ci/+/853931 | 11:46 |
| opendevreview | Archana Singh proposed openstack/tripleo-upgrade stable/queens: Ensure fencing is correctly handled during update https://review.opendev.org/c/openstack/tripleo-upgrade/+/856126 | 11:53 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart-extras master: Adding temporary workaround for libvirt-failures https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/856244 | 12:10 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Fix wrong separator(,) used to list multiple dependent services https://review.opendev.org/c/openstack/tripleo-ansible/+/849007 | 12:43 |
| opendevreview | Merged openstack/tripleo-heat-templates master: RabbitMQ: Fix missing bind mount for .key file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856219 | 12:43 |
| opendevreview | Merged openstack/tripleo-validations master: Refactoring and improvement of check_ironic_boot_config tests https://review.opendev.org/c/openstack/tripleo-validations/+/781125 | 12:43 |
| opendevreview | Merged openstack/tripleo-common master: Fix missing healthcheck script for manila-api/share https://review.opendev.org/c/openstack/tripleo-common/+/849379 | 12:43 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Manila: Fix missing healthcheck for api and share https://review.opendev.org/c/openstack/tripleo-heat-templates/+/849380 | 12:43 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Remove leftover of the tripleo::firewall class https://review.opendev.org/c/openstack/tripleo-heat-templates/+/845867 | 12:44 |
| opendevreview | Merged openstack/tripleo-puppet-elements master: Remove unused puppet-ipaclient https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/842009 | 12:44 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Make tripleo_nftables compatible with puppet-firewall https://review.opendev.org/c/openstack/tripleo-ansible/+/856266 | 13:07 |
| beagles | slagle, if you get a sec - can I get a review of https://review.opendev.org/c/openstack/tripleo-ansible/+/853181 and https://review.opendev.org/c/openstack/tripleo-heat-templates/+/853499 ? | 13:37 |
| ysandeep | ade_lee, fyi.. links for zuul to fips jobs | 13:38 |
| ysandeep | check: https://zuul.openstack.org/builds?job_name=tripleo-ci-centos-9-standalone-fips&skip=0 | 13:38 |
| ysandeep | periodic: https://zuul.openstack.org/buildset/71e50c59a12842978a2e285c475f0291 | 13:38 |
| ysandeep | periodic line have some other jobs apart from fips jobs, you can search 'fips' for fips enabled jobs | 13:39 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart-extras master: Adding temporary workaround for libvirt-failures https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/856244 | 13:40 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible stable/train: Fix wrong separator(,) used to list multiple dependent services https://review.opendev.org/c/openstack/tripleo-ansible/+/856282 | 13:42 |
| slagle | beagles: +w | 13:43 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Ensure package is installed before downgrading https://review.opendev.org/c/openstack/tripleo-ansible/+/856283 | 13:45 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Ensure package is installed before downgrading https://review.opendev.org/c/openstack/tripleo-ansible/+/856283 | 13:46 |
| Tengu | slagle: heya! would you have a moment to talk about the tripleo_httpd* roles and the comments you had? | 13:46 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Remove newline https://review.opendev.org/c/openstack/tripleo-ansible/+/856265 | 13:48 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Make tripleo_nftables compatible with puppet-firewall https://review.opendev.org/c/openstack/tripleo-ansible/+/856266 | 13:48 |
| slagle | Tengu: sure! | 13:51 |
| Tengu | slagle: https://meet.google.com/jhe-fhqv-htf | 13:51 |
| beagles | slagle++ thanks! | 13:52 |
| slagle | Tengu: oh, no. thought you meant here | 13:52 |
| Tengu | oh. | 13:52 |
| Tengu | slagle: fine as well :) | 13:52 |
| slagle | did you respond on the patch? /me looks | 13:52 |
| Tengu | slagle: yep, I did | 13:52 |
| Tengu | slagle: mostly OK, but the structure/usage of the tripleo_httpd_vhost role isn't the same as the other "standalone" roles you created. | 13:53 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/train: Do not change ownership recursive for Swift https://review.opendev.org/c/openstack/tripleo-heat-templates/+/855542 | 13:53 |
| Tengu | slagle: so it may not match the layout you may want, though that layout is probably not suited for that kind of module imho | 13:53 |
| slagle | Tengu: ok, the layout is fine then | 13:54 |
| Tengu | slagle: cool :) | 13:54 |
| Tengu | so I may just push the change removing the files/.gitkeep and that's about it | 13:55 |
| ade_lee | ysandeep, thanks! | 13:55 |
| slagle | Tengu: well I still don't get the roles vars thing | 13:55 |
| slagle | Tengu: putting them there, does not equate to "don't modify" | 13:55 |
| Tengu | slagle: where would you put them then? | 13:56 |
| slagle | why not defaults/main.yml? | 13:57 |
| Tengu | slagle: https://opendev.org/openstack/tripleo-ansible/src/branch/master/_skeleton_role_/vars/main.yml.j2#L18-L20 | 13:57 |
| Tengu | is it misleading? | 13:57 |
| Tengu | slagle: also: https://opendev.org/openstack/tripleo-ansible/src/branch/master/_skeleton_role_/defaults/main.yml.j2#L18 | 13:58 |
| Tengu | sooo... at least according to those embedded comments, it's correct | 13:59 |
| slagle | ok, fair enough. that's not an ansible thing afaik. but if we invented that in tripleo, ok | 14:00 |
| Tengu | heh.. | 14:01 |
| slagle | precedence is the difference, which causes massive confusion when you wonder why your vars aren't taking effect | 14:01 |
| slagle | play vars would be ignored, but not block vars. who remembers that? :) | 14:02 |
| opendevreview | chandan kumar proposed openstack/tripleo-ansible master: Ensure package is installed before downgrading https://review.opendev.org/c/openstack/tripleo-ansible/+/856283 | 14:02 |
| Tengu | so in there, the _default aren't intended to be modified, as there are other, with similar name, intended to extend the _default | 14:02 |
| Tengu | it's a "smart" way to ensure the bare minimal defaults are always set | 14:02 |
| slagle | yes, smart in quotes. e.g., not smart :) | 14:03 |
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix BLS entries with bad machine-id https://review.opendev.org/c/openstack/tripleo-ansible/+/855984 | 14:03 |
| Tengu | ;) | 14:03 |
| chkumar|ruck | bogdando: slagle once zuul passes please merge this https://review.opendev.org/c/openstack/tripleo-ansible/+/856283/ | 14:03 |
| Tengu | it's "not the worst". and if really we do want to override them, well... we still can | 14:04 |
| Tengu | slagle: and about the "parameters.yaml" for the verification? are we good as-is, or... ? | 14:05 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Remove newline https://review.opendev.org/c/openstack/tripleo-ansible/+/856265 | 14:10 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Allow to remove an nft rule based on its comment https://review.opendev.org/c/openstack/tripleo-ansible/+/852386 | 14:10 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New roles for apache/httpd management https://review.opendev.org/c/openstack/tripleo-ansible/+/853481 | 14:11 |
| Tengu | ah. dang. I should have rebased it as well... | 14:11 |
| Tengu | or... nope. podman provisioner, so we're good. | 14:12 |
| slagle | Tengu: yea, it's fine as-is if those task files can't be used on their own | 14:12 |
| Tengu | slagle: it's not intended at least. Now, if really someone wants to hurt themself... ;) | 14:12 |
| Tengu | slagle: and for the tripleo_ironic - as answered, we can set a default value, though it will be empty, because there's no actual default... | 14:13 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Switch default firewall engine https://review.opendev.org/c/openstack/tripleo-heat-templates/+/852808 | 14:16 |
| *** ysandeep is now known as ysandeep|out | 14:16 | |
| slagle | Tengu: i think that's what we usually do for most other roles, an empty default. | 14:16 |
| slagle | that way you can see what all needs to be set in one place | 14:16 |
| Tengu | slagle: ok, I can of course do that. makes sense then - but maybe I should ensure they are actually set to something non-null, like I did in the vhost role. | 14:17 |
| slagle | well either way, should check whether it's set or defined | 14:17 |
| Tengu | if not set right now, it would just fail with "unknown parameter" | 14:18 |
| Tengu | :) | 14:18 |
| Tengu | gimme a minute or 2. | 14:18 |
| bogdando | slagle, bshephar: hi. Do you think the multi-rhel work in progress in master branch, could become a requirement for standalone roles to function in stable/Wallaby?.. | 14:23 |
| bogdando | the context is: we are asked to backport standalone roles for external (EDPM) scope in W | 14:23 |
| slagle | bogdando: almost certain multi-rhel is also going back to W at this point | 14:24 |
| bogdando | just wondering if the roles merged in master, then can it be just backported, or other deps? | 14:24 |
| bogdando | right, but would it become a dependency | 14:24 |
| bogdando | on which we should wait before backporting standalone DPM | 14:24 |
| opendevreview | yatin proposed openstack/tripleo-heat-templates master: Switch default ML2 plugin in Undercloud to OVN https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850158 | 14:24 |
| bogdando | I can think at least of data structures changes to represent images data for roles, or may be not, it would keep getting a string value | 14:25 |
| slagle | bogdando: i don't know about "would it become" naturally. if it's there, or we plan it to be there, we can depend on it if we need it | 14:25 |
| slagle | i wouldn't block or wait on multi-rhel for anything. | 14:25 |
| bogdando | thanks, that was my assumption also | 14:26 |
| slagle | once the implementation is decided, we can always make changes on already merged standalone roles, of which there are just a few | 14:26 |
| bogdando | yea | 14:26 |
| bogdando | ok then I need to fix molecule failing and hopefully we can merge libvirt and compute roles into master | 14:26 |
| odyssey4me | slagle Tengu there is some guidance about vars vs defaults in https://redhat-cop.github.io/automation-good-practices/#_vars_vs_defaults | 14:27 |
| bogdando | btw, I cannot get why it fails, the same scenario passes on my env | 14:27 |
| slagle | bogdando: so how did you get your molecule tests to run locally? and what is supposed to be in /opt/yum.repos.d on the host? | 14:27 |
| slagle | i can't figure out what creates that | 14:27 |
| bogdando | I reproduced, with zuul, the env from https://review.opendev.org/c/openstack/tripleo-ansible/+/842152 | 14:28 |
| bogdando | and developing molecules there | 14:28 |
| bogdando | and if I run pytest, it works | 14:28 |
| Tengu | odyssey4me: so it kind of match my understanding, apparently :). And the embedded comments. | 14:28 |
| bogdando | but fails in upstream job :/ | 14:28 |
| bogdando | see the msg in comments | 14:28 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New tripleo_ironic role https://review.opendev.org/c/openstack/tripleo-ansible/+/855873 | 14:29 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New tripleo_ironic role https://review.opendev.org/c/openstack/tripleo-ansible/+/855873 | 14:29 |
| slagle | bogdando: i was just trying to use molecule and not test_molecule.py | 14:29 |
| bogdando | whatever it should be there in /opt, it should be the same on my reproduced env and upstream zuul nodes | 14:29 |
| Tengu | hm. missed the rebase. | 14:29 |
| slagle | bogdando: it seems there is too much zuul embedded logic | 14:30 |
| slagle | expecting stuff under /opt, etc | 14:30 |
| bogdando | molecule way works also, just trying to follow closely that that job runs | 14:30 |
| bogdando | (pytest) | 14:30 |
| slagle | we ought to be able to use molecule from a checkout | 14:30 |
| slagle | it copied the repo config from my fedora host, put it under /opt, mounted it into the ubi9 container, where it promptly tried to use fedora 9 repos. which are a bit outdated | 14:31 |
| bogdando | '/etc/libvirt/virtqemud.conf: No such file or directory' doesn't look zuul/CI specifc... | 14:32 |
| bogdando | also the log shows configuration was applied to that file, and the other ones | 14:32 |
| bogdando | but yes, those things surely can be improved to allow local runs | 14:34 |
| bogdando | not only on zuul workers deployed like the job does | 14:34 |
| bogdando | s/zuul/nodepool/* | 14:36 |
| bogdando | slagle: do you need repro commands? | 14:36 |
| slagle | i'm just going to try and get it to work with locally with molecule | 14:36 |
| bogdando | makes sense | 14:36 |
| bogdando | omitting preparations, if any, it should be like cd tripleo-ansible/roles/tripleo_nova_libvirt/ | 14:37 |
| chkumar|ruck | bogdando: hello | 14:38 |
| chkumar|ruck | bogdando: https://58d288df402434e7c3fd-04f41080b257670dc6eeadaffc790e1a.ssl.cf5.rackcdn.com/842152/41/check/tripleo-ansible-centos-stream-molecule-tripleo_nova_libvirt/41bccfe/job-output.txt | 14:38 |
| bogdando | and molecule --base-config ../../../.config/molecule/config_podman.yml create and so on | 14:38 |
| chkumar|ruck | you were talking about this error Error: error creating build container: copying system image from manifest list: Source image rejected: None of the signatures were accepted, reasons: open /etc/pki/rpm-gpg/RPM-GPG-KEY-redhat-beta: no such file or directory; open /etc/pki/rpm-gpg/RPM-GPG-KEY-redhat-beta: | 14:38 |
| bogdando | or just config, if delegated | 14:38 |
| chkumar|ruck | bogdando: https://review.opendev.org/c/openstack/tripleo-ansible/+/856283 will fix it | 14:39 |
| bogdando | another example: molecule --base-config ../../../.config/molecule/config_podman.yml converge -s default -- --tags install --skip-tags configure,run | 14:39 |
| bogdando | but currently, it fails 'configure' scenario, with delegated driver (config.yml) | 14:39 |
| bogdando | chkumar|ruck: not sure that's related, but I'm aware of that blocker | 14:39 |
| bogdando | the failure I'm talking about is 2022-09-02 14:09:42.399608 | centos-9-stream | stderr: 'grep: /etc/libvirt/virtqemud.conf: No such file or directory' | 14:40 |
| chkumar|ruck | checking previous logs | 14:41 |
| bogdando | chkumar|ruck: https://f78eba48db27f93d7a35-b4998686e736196b5d74af7a908c113b.ssl.cf5.rackcdn.com/842152/41/check/tripleo-ansible-centos-stream-molecule-tripleo_nova_libvirt/c8041ff/job-output.txt | 14:42 |
| chkumar|ruck | [tripleo_nova_libvirt : Copy initial config files from nova_libvirt_config container got skipped | 14:46 |
| chkumar|ruck | 2022-09-02 14:09:42.398600 | centos-9-stream | TASK [tripleo_nova_libvirt : Copy initial config files from nova_libvirt_config container] *** | 14:46 |
| bogdando | chkumar|ruck: indeed! it is not skipped on my env tho | 14:50 |
| bogdando | interesting | 14:51 |
| chkumar|ruck | https://codesearch.opendev.org/?q=verifier%3A&i=nope&literal=nope&files=molecule.yml&excludeFiles=&repos= | 14:51 |
| chkumar|ruck | bogdando: can we use verifier to test infra for podman | 14:51 |
| chkumar|ruck | just for consistency | 14:51 |
| chkumar|ruck | by default it is going to use ansible | 14:51 |
| bogdando | I like the idea, though not sure what it is | 14:52 |
| chkumar|ruck | bogdando: you can run the job on rdo side also with testproject | 14:52 |
| chkumar|ruck | and hold the node and debug it there | 14:52 |
| chkumar|ruck | bogdando: if still comes, let me know, I will take a look at this tomorrow morning | 14:54 |
| bogdando | actually, if I ran molecule --base-config ../../../.config/molecule/config.yml converge -s configure -- --skip-tags run, it only executes converge once, while that log above looks like executed it twice | 14:55 |
| bogdando | weird, I configured no idempotency stage for it | 14:55 |
| bogdando | https://review.opendev.org/c/openstack/tripleo-ansible/+/842152/41/tripleo_ansible/roles/tripleo_nova_libvirt/molecule/configure/molecule.yml#25 | 14:55 |
| bogdando | Tengu, chkumar|ruck: ah, that's molecule --base-config ../../../.config/molecule/config.yml test -s configure -- --skip-tags run executes converge twice, and 2nd time it gets skipped on my env as well, BUT, it finishes w/o errors :) | 15:05 |
| bogdando | chkumar|ruck, tkajinam, slagle: nvm, I got that reproduced on my env | 15:13 |
| bogdando | thanks for looking | 15:13 |
| bogdando | all I needed to do is removing /etc/libvirt from host before running the above molecule test command | 15:14 |
| bogdando | as the copy task gets skipped for the 2nd converge run, verifier fails to fetch config files to check it | 15:14 |
| opendevreview | Ronelle Landy proposed openstack/openstack-tempest-skiplist master: Remove test_minimum_basic_instance_hard_reboot_after_vol_snap_deletion https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/850155 | 15:47 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: [DNM] testing hard_reboot tests https://review.opendev.org/c/openstack/tripleo-ci/+/850156 | 15:49 |
| hjensas | ramishra_: you around? Can we use anchors/aliases in heat environment files? | 15:53 |
| hjensas | ramishra_: We have to document a workaround, and it involves duplicating firewall rules on multiple roles. For example: https://paste.opendev.org/show/boJBqiufhPusPi2ZfGdz/ | 15:54 |
| hjensas | ramishra_: I figured if I could add the ExtraFirewallRules entries once add and anchor, and then reference it for subsequent roles. | 15:55 |
| hjensas | bshephar: ^ do you know? | 15:56 |
| *** marios is now known as marios|out | 16:00 | |
| slagle | chkumar|ruck: hi, not sure if you know the history here, but is there any specific reason to use ubi9 as the base image in our molecule tests? | 16:06 |
| slagle | seems like quay.io/centos/centos:stream9 might be a better choice | 16:06 |
| slagle | the zuul jobs copy the stream9 repos, gpg keys, and dnf vars into the ubi9 image, so that it looks like stream9 | 16:07 |
| slagle | i'm thinking, why not just use it instead? | 16:07 |
| lecris[m] | I give up, rdo-zuul does not want to collaborate with me even though I'm sure the errors are unrelated to this. | 16:17 |
| lecris[m] | https://review.opendev.org/c/openstack/tripleo-ansible/+/856062 | 16:17 |
| lecris[m] | Can it be merged without rdo? | 16:18 |
| *** jpena is now known as jpena|off | 16:34 | |
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix BLS entries with bad machine-id https://review.opendev.org/c/openstack/tripleo-ansible/+/855984 | 16:40 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Enable designate-dashboard https://review.opendev.org/c/openstack/tripleo-heat-templates/+/853499 | 16:52 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Nova: Squash NovaLibvirtCommon resource https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856301 | 16:57 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Nova: Squash NovaLibvirtCommon resource https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856301 | 17:01 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove reference to unused hieradata keys https://review.opendev.org/c/openstack/puppet-tripleo/+/831779 | 17:28 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Pacemaker: Replace hiera by lookup (2) https://review.opendev.org/c/openstack/puppet-tripleo/+/843161 | 17:30 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Redis: Remove unused parameters https://review.opendev.org/c/openstack/puppet-tripleo/+/855847 | 17:30 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Fix ignored enable_aodh_expirer hieradata key https://review.opendev.org/c/openstack/puppet-tripleo/+/849244 | 17:32 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Fix ignored EnableAodhExpirer https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856310 | 17:37 |
| *** dviroel is now known as dviroel|out | 17:58 | |
| opendevreview | Merged openstack/openstack-tempest-skiplist master: Remove test_minimum_basic_instance_hard_reboot_after_vol_snap_deletion https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/850155 | 18:29 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Add support for log offloading over TCP https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850148 | 18:50 |
| fultonj | slagle: would you mind reviewing https://review.opendev.org/c/openstack/python-tripleoclient/+/851236 ? | 19:01 |
| fultonj | as announced on https://lists.openstack.org/pipermail/openstack-discuss/2022-August/030092.html | 19:02 |
| * fultonj wanted to make sure PTL was ok with it | 19:02 | |
| opendevreview | John Fulton proposed openstack/tripleo-docs stable/wallaby: Document steps to migrate away from Derived Parameters https://review.opendev.org/c/openstack/tripleo-docs/+/856203 | 19:04 |
| slagle | fultonj: sure | 19:04 |
| fultonj | thanks | 19:04 |
| *** dasm is now known as dasm|off | 21:11 | |
| opendevreview | Merged openstack/tripleo-heat-templates master: [ML2 OVN] Adding DisableOvnDhcpForBaremetalPorts parameter https://review.opendev.org/c/openstack/tripleo-heat-templates/+/850594 | 21:20 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: Run molecule tests locally without zuul https://review.opendev.org/c/openstack/tripleo-ansible/+/856339 | 21:36 |
| *** rlandy is now known as rlandy|out | 22:03 | |
| opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Remove old network_data examples https://review.opendev.org/c/openstack/tripleo-heat-templates/+/856343 | 22:44 |
| bshephar | hjensas: I'm not too sure about environment files. I guess you probably can since it's a yaml construct not related to Heat specifically. And we just parse the files with the python yaml library. So it should work | 22:48 |
| bshephar | hjensas: Why do those params need to be role specific like that though? Why not just set the rules globally? | 22:49 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!