| opendevreview | Merged openstack/puppet-watcher master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-watcher/+/794133 | 00:01 |
|---|---|---|
| opendevreview | Merged openstack/puppet-zaqar master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-zaqar/+/794134 | 00:01 |
| opendevreview | Merged openstack/puppet-cloudkitty master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-cloudkitty/+/794097 | 00:01 |
| opendevreview | Merged openstack/puppet-aodh master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-aodh/+/793967 | 00:05 |
| opendevreview | Merged openstack/puppet-nova master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-nova/+/794113 | 00:05 |
| opendevreview | Merged openstack/puppet-placement master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-placement/+/794120 | 00:09 |
| opendevreview | Merged openstack/puppet-heat master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-heat/+/794103 | 00:11 |
| opendevreview | Merged openstack/puppet-rally master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-rally/+/794122 | 00:14 |
| opendevreview | Merged openstack/puppet-trove master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-trove/+/794128 | 00:14 |
| opendevreview | Merged openstack/puppet-senlin master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-senlin/+/794124 | 00:18 |
| opendevreview | Merged openstack/puppet-qdr master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-qdr/+/794121 | 00:23 |
| opendevreview | Merged openstack/puppet-openstacklib master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-openstacklib/+/794116 | 00:33 |
| opendevreview | Merged openstack/puppet-murano master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-murano/+/794111 | 00:34 |
| opendevreview | Merged openstack/puppet-ceph master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-ceph/+/794095 | 00:36 |
| opendevreview | Merged openstack/puppet-panko master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-panko/+/794119 | 00:42 |
| opendevreview | Merged openstack/puppet-neutron master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-neutron/+/794112 | 00:42 |
| opendevreview | Merged openstack/puppet-manila master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-manila/+/794108 | 00:45 |
| opendevreview | Merged openstack/puppet-ovn master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-ovn/+/794118 | 00:47 |
| opendevreview | Merged openstack/puppet-glance master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-glance/+/794101 | 00:47 |
| opendevreview | Merged openstack/puppet-octavia master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-octavia/+/794114 | 00:48 |
| opendevreview | Merged openstack/puppet-mistral master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-mistral/+/794109 | 00:51 |
| *** ysandeep has joined #puppet-openstack | 00:52 | |
| opendevreview | Merged openstack/puppet-openstack_extras master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-openstack_extras/+/794115 | 00:55 |
| opendevreview | Merged openstack/puppet-swift master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-swift/+/794125 | 01:00 |
| opendevreview | Merged openstack/puppet-keystone master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-keystone/+/794106 | 01:40 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-monasca master: DNM: Test downgraded puppet-python https://review.opendev.org/c/openstack/puppet-monasca/+/794362 | 01:51 |
| opendevreview | Merged openstack/puppet-ec2api master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-ec2api/+/794099 | 02:02 |
| opendevreview | Merged openstack/puppet-panko master: Stop acceptance tests https://review.opendev.org/c/openstack/puppet-panko/+/791916 | 02:10 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-monasca master: DNM: Test downgraded puppet-python https://review.opendev.org/c/openstack/puppet-monasca/+/794362 | 02:14 |
| opendevreview | Merged openstack/puppet-vitrage master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-vitrage/+/794129 | 02:14 |
| opendevreview | Merged openstack/puppet-aodh master: Fix leftover related to $pyvers variable https://review.opendev.org/c/openstack/puppet-aodh/+/793510 | 02:27 |
| opendevreview | Merged openstack/puppet-aodh master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-aodh/+/793130 | 02:27 |
| *** ysandeep is now known as ysandeep|afk | 03:23 | |
| *** ysandeep|afk has quit IRC | 03:54 | |
| *** ykarel|away has joined #puppet-openstack | 03:56 | |
| opendevreview | Merged openstack/puppet-aodh master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-aodh/+/792001 | 04:10 |
| *** ykarel|away is now known as ykarel | 04:14 | |
| *** ysandeep|afk has joined #puppet-openstack | 04:30 | |
| *** ysandeep|afk is now known as ysandeep|ruck | 04:35 | |
| opendevreview | Merged openstack/puppet-openstack-integration master: Downgrade puppet-python to 5.0.0 https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794170 | 04:35 |
| opendevreview | Merged openstack/puppet-openstack-integration master: Glance: Enable db purge job https://review.opendev.org/c/openstack/puppet-openstack-integration/+/792743 | 04:35 |
| *** ysandeep__ has joined #puppet-openstack | 04:38 | |
| *** ysandeep|ruck has quit IRC | 04:44 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tempest master: Deprecate support for Panko https://review.opendev.org/c/openstack/puppet-tempest/+/794151 | 04:52 |
| opendevreview | Merged openstack/puppet-monasca master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-monasca/+/794110 | 05:06 |
| *** ysandeep__ is now known as ysandeep_afk | 05:26 | |
| *** ysandeep_afk is now known as ysandeep|ruck | 05:51 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-monasca stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-monasca/+/794374 | 06:10 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-freezer stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-freezer/+/794375 | 06:10 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-vswitch stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-vswitch/+/794376 | 06:10 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceilometer stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-ceilometer/+/794377 | 06:11 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tacker stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-tacker/+/794378 | 06:11 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-cinder/+/794379 | 06:11 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-sahara stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-sahara/+/794380 | 06:12 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-barbican stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-barbican/+/794381 | 06:12 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tempest stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-tempest/+/794382 | 06:12 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-gnocchi stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-gnocchi/+/794383 | 06:12 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ironic stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-ironic/+/794384 | 06:12 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-horizon stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-horizon/+/794385 | 06:13 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-designate stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-designate/+/794386 | 06:14 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-magnum stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-magnum/+/794387 | 06:14 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-watcher stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-watcher/+/794388 | 06:14 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-zaqar stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-zaqar/+/794389 | 06:14 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cloudkitty stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-cloudkitty/+/794390 | 06:14 |
| opendevreview | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794399 | 06:15 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-zaqar stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-zaqar/+/794389 | 06:15 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cloudkitty stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-cloudkitty/+/794390 | 06:15 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-aodh stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-aodh/+/794391 | 06:15 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-nova/+/794392 | 06:16 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-placement stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-placement/+/794413 | 06:16 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-heat stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-heat/+/794414 | 06:17 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-rally stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-rally/+/794415 | 06:17 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-trove stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-trove/+/794416 | 06:17 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-qdr stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-qdr/+/794417 | 06:18 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-senlin stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-senlin/+/794418 | 06:18 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-openstacklib/+/794419 | 06:18 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-murano stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-murano/+/794420 | 06:19 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-panko stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-panko/+/794421 | 06:20 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-neutron/+/794422 | 06:20 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-manila stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-manila/+/794423 | 06:20 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ovn stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-ovn/+/794424 | 06:21 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-glance stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-glance/+/794425 | 06:21 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-octavia/+/794426 | 06:22 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-mistral stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-mistral/+/794427 | 06:22 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack_extras stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-openstack_extras/+/794428 | 06:22 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-swift stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-swift/+/794429 | 06:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-keystone/+/794430 | 06:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-keystone/+/794430 | 06:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ec2api stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-ec2api/+/794431 | 06:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-vitrage stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-vitrage/+/794432 | 06:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-keystone/+/794430 | 06:24 |
| *** amoralej|off has joined #puppet-openstack | 06:28 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/wallaby: Skip content provider job for irrelevant files https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794433 | 06:29 |
| *** amoralej|off is now known as amoralej | 06:30 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/victoria: Remove all TripleO jobs in CentOS7 https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794434 | 06:37 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/victoria: Remove all TripleO jobs in CentOS7 https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794434 | 06:43 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/victoria: Use low-memory-usage.yaml in tripleo jobs https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794435 | 06:44 |
| opendevreview | Merged openstack/puppet-freezer stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-freezer/+/794375 | 06:49 |
| opendevreview | Merged openstack/puppet-monasca stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-monasca/+/794374 | 06:50 |
| opendevreview | Merged openstack/puppet-vswitch stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-vswitch/+/794376 | 06:59 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/victoria: Victoria and before: Update TripleO job definitions https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794434 | 07:02 |
| opendevreview | Merged openstack/puppet-qdr stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-qdr/+/794417 | 07:02 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/victoria: Victoria and before: Update TripleO job definitions https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794410 | 07:03 |
| opendevreview | Merged openstack/puppet-senlin stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-senlin/+/794418 | 07:03 |
| opendevreview | Merged openstack/puppet-rally stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-rally/+/794415 | 07:05 |
| opendevreview | Merged openstack/puppet-tacker stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-tacker/+/794378 | 07:06 |
| opendevreview | Merged openstack/puppet-ovn stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-ovn/+/794424 | 07:09 |
| opendevreview | Merged openstack/puppet-aodh stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-aodh/+/794391 | 07:11 |
| opendevreview | Merged openstack/puppet-sahara stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-sahara/+/794380 | 07:11 |
| opendevreview | Merged openstack/puppet-murano stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-murano/+/794420 | 07:12 |
| opendevreview | Merged openstack/puppet-placement stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-placement/+/794413 | 07:12 |
| opendevreview | Merged openstack/puppet-manila stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-manila/+/794423 | 07:13 |
| opendevreview | Merged openstack/puppet-magnum stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-magnum/+/794387 | 07:13 |
| opendevreview | Merged openstack/puppet-ceilometer stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-ceilometer/+/794377 | 07:13 |
| opendevreview | Merged openstack/puppet-cinder stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-cinder/+/794379 | 07:14 |
| opendevreview | Merged openstack/puppet-openstacklib stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-openstacklib/+/794419 | 07:15 |
| opendevreview | Merged openstack/puppet-trove stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-trove/+/794416 | 07:15 |
| opendevreview | Merged openstack/puppet-horizon stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-horizon/+/794385 | 07:16 |
| opendevreview | Merged openstack/puppet-cloudkitty stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-cloudkitty/+/794390 | 07:17 |
| opendevreview | Merged openstack/puppet-barbican stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-barbican/+/794381 | 07:18 |
| opendevreview | Merged openstack/puppet-vitrage stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-vitrage/+/794432 | 07:18 |
| opendevreview | Merged openstack/puppet-openstack_extras stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-openstack_extras/+/794428 | 07:19 |
| *** jpena|off is now known as jpena | 07:20 | |
| *** tosky has joined #puppet-openstack | 07:20 | |
| opendevreview | Merged openstack/puppet-gnocchi stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-gnocchi/+/794383 | 07:22 |
| opendevreview | Merged openstack/puppet-glance stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-glance/+/794425 | 07:23 |
| opendevreview | Merged openstack/puppet-designate stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-designate/+/794386 | 07:23 |
| opendevreview | Merged openstack/puppet-neutron stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-neutron/+/794422 | 07:23 |
| opendevreview | Merged openstack/puppet-mistral stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-mistral/+/794427 | 07:24 |
| opendevreview | Merged openstack/puppet-panko stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-panko/+/794421 | 07:27 |
| opendevreview | Merged openstack/puppet-zaqar stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-zaqar/+/794389 | 07:29 |
| opendevreview | Merged openstack/puppet-tempest stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-tempest/+/794382 | 07:31 |
| opendevreview | Merged openstack/puppet-watcher stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-watcher/+/794388 | 07:34 |
| *** rpittau|afk is now known as rpittau | 07:34 | |
| opendevreview | Merged openstack/puppet-nova stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-nova/+/794392 | 07:35 |
| opendevreview | Merged openstack/puppet-octavia stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-octavia/+/794426 | 07:38 |
| opendevreview | Merged openstack/puppet-swift stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-swift/+/794429 | 07:40 |
| opendevreview | Merged openstack/puppet-keystone stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-keystone/+/794430 | 07:55 |
| *** derekh has joined #puppet-openstack | 08:11 | |
| opendevreview | Matthias Runge proposed openstack/puppet-panko master: Retire puppet-panko. https://review.opendev.org/c/openstack/puppet-panko/+/791890 | 08:27 |
| *** ysandeep|ruck is now known as ysandeep|lunch | 08:32 | |
| *** ykarel is now known as ykarel|lunch | 08:48 | |
| opendevreview | Merged openstack/puppet-ec2api stable/wallaby: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-ec2api/+/794431 | 09:15 |
| *** ysandeep|lunch is now known as ysandeep|ruck | 09:41 | |
| *** ykarel|lunch is now known as ykarel | 09:49 | |
| *** jpena is now known as jpena|lunch | 11:33 | |
| opendevreview | Alfredo Moralejo proposed openstack/puppet-nova master: Creates initiatorname.iscsi in compute nodes if it doesn't exist https://review.opendev.org/c/openstack/puppet-nova/+/794574 | 11:44 |
| *** ysandeep|ruck is now known as ysandeep|debug_ssn | 12:01 | |
| *** amoralej is now known as amoralej|lunch | 12:12 | |
| *** jpena|lunch is now known as jpena | 12:23 | |
| *** amoralej|lunch is now known as amoralej | 13:07 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-horizon master: Add support for independent manila-dashboard configurations https://review.opendev.org/c/openstack/puppet-horizon/+/790350 | 14:03 |
| *** ysandeep|debug_ssn is now known as ysandeep | 14:05 | |
| *** ysandeep is now known as ysandeep|ruck | 14:05 | |
| *** ade_lee has quit IRC | 14:12 | |
| *** ade_lee has joined #puppet-openstack | 14:13 | |
| *** ade_lee has quit IRC | 14:21 | |
| *** ade_lee has joined #puppet-openstack | 14:21 | |
| amoralej | tkajinam, wrt https://review.opendev.org/c/openstack/puppet-nova/+/794574 | 14:27 |
| tkajinam | amoralej, yes | 14:27 |
| amoralej | from iscsi-initiator side, are we managing out of puppet-nova? | 14:28 |
| tkajinam | in current integration job we don't manage anything about iscsid | 14:28 |
| tkajinam | but that is because the default works | 14:28 |
| tkajinam | I mean the default settings applied by package installation works | 14:29 |
| amoralej | shouln't puppet nova take care of configuring it if needed to work? | 14:29 |
| amoralej | we can apply in p-o-i for CI as you said, but it may fail out of it | 14:29 |
| amoralej | i'd say it'd be good that nova::compute sets what is needed to have a functional compute node? | 14:30 |
| tkajinam | we need the same in puppet-cinder | 14:30 |
| tkajinam | and puppet-glance in case glance uses cinder backend and cinder uses iscsi storage as its backend | 14:31 |
| tkajinam | I think adding a new class to maintain iscsi to openstacklib would make sense | 14:32 |
| tkajinam | considering that iscsi is not specific requirement by nova | 14:32 |
| tkajinam | I didn't come up with puppet-openstacklib when I wrote that comment | 14:33 |
| tkajinam | amoralej, does it make sense ? | 14:33 |
| amoralej | in puppet-cinder there is code to manage the tgtd part | 14:33 |
| amoralej | https://github.com/openstack/puppet-cinder/blob/master/manifests/backend/iscsi.pp | 14:33 |
| amoralej | for backend | 14:33 |
| tkajinam | no that is not for client | 14:33 |
| tkajinam | that is for server side | 14:34 |
| amoralej | client is needed in cinder server part?= | 14:34 |
| tkajinam | that iscsi backend is the driver which use lvm and provide access to lvs via iscsi | 14:34 |
| tkajinam | and tgt is used to set up iscsi targets | 14:34 |
| tkajinam | we still need to configure iscsi clients | 14:34 |
| tkajinam | like initiatornames.iscsi | 14:35 |
| tkajinam | which is not maintained by puppet-cinder, either | 14:35 |
| amoralej | ah, i though in server we only needed the server part | 14:35 |
| amoralej | not the client one | 14:35 |
| amoralej | tbh | 14:35 |
| tkajinam | we need that | 14:35 |
| tkajinam | cinder need to connect to that volume by iscsi | 14:35 |
| tkajinam | to read/write data from volumes | 14:35 |
| amoralej | ok | 14:35 |
| amoralej | let me think | 14:35 |
| amoralej | in fact, this is just a simple change, i'm thinking in fixing at packaging | 14:36 |
| amoralej | at once for all | 14:36 |
| amoralej | in os-brick side | 14:36 |
| tkajinam | that's also an option | 14:37 |
| *** ykarel is now known as ykarel|away | 14:37 | |
| tkajinam | but I think adding ability to maintain the base service which is used in several components would be a good idea | 14:37 |
| tkajinam | instead of keeping that "unmanaged" | 14:37 |
| tkajinam | (though if there is any good existing modules, then we would be able to use it instead... | 14:38 |
| tkajinam | like https://github.com/tedivm/puppet-iscsi ? | 14:40 |
| amoralej | https://forge.puppet.com/tags/iscsi | 14:40 |
| amoralej | nothing promising | 14:40 |
| amoralej | i think it'd be good to maintain the service, if we need to :) | 14:40 |
| amoralej | otherwise we may overengineer it | 14:40 |
| tkajinam | tobias-urdin, do you have any recommendation about the existing module to maintain iscsid ? | 14:41 |
| amoralej | also, it seems that tripleo is not doing much config for iscsi | 14:42 |
| amoralej | https://github.com/openstack/tripleo-common/blob/547b94d08ee1c2c11d7e4f16e8bc99a5023df2a9/container-images/tcib/base/os/iscsid/iscsid.yaml | 14:42 |
| amoralej | https://github.com/openstack/tripleo-common/blob/547b94d08ee1c2c11d7e4f16e8bc99a5023df2a9/container-images/kolla/iscsid/extend_start.sh | 14:43 |
| tkajinam | yeah | 14:43 |
| tkajinam | one option would be select one existing module which looks most promising | 14:44 |
| tkajinam | and create a composition layer to define required items like dependencies | 14:45 |
| amoralej | the one updated 2 years ago seems to be debian/ubuntu only | 14:45 |
| tkajinam | I think the top one updated 6 years ago is most "popular" | 14:46 |
| tkajinam | looking at total downloads | 14:46 |
| tkajinam | which is the github repo I posted | 14:46 |
| amoralej | 6 years old... | 14:48 |
| amoralej | dunno | 14:48 |
| tkajinam | :-P | 14:48 |
| amoralej | i'm afraid of making it more complicated that it requires tbh | 14:48 |
| tkajinam | I'll go for adding a tiny implementation unless we here any recommendation | 14:49 |
| tkajinam | yeah | 14:49 |
| amoralej | so, you vote for doing a iscsi initiator class in some of the existing openstack modules and just include it from nova/cinder/glance? | 14:52 |
| tkajinam | yes | 14:55 |
| tkajinam | we'd need to give some more thoughts about inclusion part, though | 14:55 |
| tkajinam | (whether we make the class included automatically or not | 14:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib master: WIP: Add support for iscsid configuration https://review.opendev.org/c/openstack/puppet-openstacklib/+/794605 | 15:02 |
| tkajinam | ^^^ created a draft | 15:02 |
| amoralej | you are fast :) | 15:04 |
| amoralej | tkajinam++ | 15:04 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib master: WIP: Add support for iscsid configuration https://review.opendev.org/c/openstack/puppet-openstacklib/+/794605 | 15:05 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib master: WIP: Add support for iscsid configuration https://review.opendev.org/c/openstack/puppet-openstacklib/+/794605 | 15:12 |
| tkajinam | amoralej, because I picked the core logic from your patch :-) | 15:13 |
| amoralej | i just left a comment btw | 15:16 |
| amoralej | if you prefer to use systemd iscsi-init unit instead of that command | 15:16 |
| tkajinam | amoralej, ah ok | 15:19 |
| amoralej | in fact, it's even trickier :) | 15:19 |
| amoralej | tkajinam, we need iscsid running? | 15:19 |
| amoralej | or it's just convenient? | 15:20 |
| tkajinam | we need iscsid for any iscsi operations | 15:20 |
| amoralej | but we never enabled... i'm not sure now, tbh | 15:20 |
| amoralej | yes it is | 15:22 |
| amoralej | so iscsid is enabled/started by default in centos8? | 15:22 |
| tkajinam | let me check | 15:23 |
| *** ykarel|away has quit IRC | 15:24 | |
| amoralej | no, i just tested, so what's starting it?... | 15:25 |
| tkajinam | I guess iscsiadm command brings it up automatically | 15:27 |
| amoralej | so, to make it more cumbersome, if iscsid is started with systemd, it first run to iscsi-init | 15:28 |
| amoralej | to create the file | 15:28 |
| amoralej | but in my test iscsid was not started | 15:28 |
| amoralej | with p-o-i | 15:28 |
| amoralej | i though that os-brick may manage iscsi attachments with iscsiadmin without iscsid | 15:29 |
| amoralej | but i'm probably wrong | 15:30 |
| tkajinam | give me a sec | 15:30 |
| tkajinam | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_6b0/792743/2/check/puppet-openstack-integration-6-scenario002-tempest-centos-8-stream/6b09e3e/logs/systemctl.txt | 15:31 |
| tkajinam | in senario002 which used cinder iscsi backend, iscsid is started | 15:31 |
| tkajinam | while in scenario001 which uses ceph backend(not iscsi), iscsid is not started | 15:31 |
| tkajinam | https://0735506f8023839b161e-58818f8d6a88f31ae0841c158a7f2252.ssl.cf2.rackcdn.com/792743/2/check/puppet-openstack-integration-6-scenario001-tempest-centos-8-stream/bebc480/logs/systemctl.txt | 15:31 |
| tkajinam | so it is likely that iscsiadm command brings up iscsid.service | 15:32 |
| tkajinam | in both case iscsid.socket is enabled. IIRC isciadm talk to this socket and that triggers start of iscsid.service | 15:32 |
| amoralej | yep | 15:32 |
| amoralej | ahhhh | 15:32 |
| amoralej | makes sense | 15:33 |
| amoralej | yep, tkajinam i just tested it's as you said, when i execute iscsiadm it starts iscsid, and in fact it createds the initiatorname by running iscsi-init service | 15:47 |
| amoralej | the problem is that os-brick checks the existence of the initiatorname.scsi file before running any iscsiadm command | 15:48 |
| tkajinam | amoralej, I feel like we should fix os-brick | 15:49 |
| tkajinam | if generation of initiatorname.scsi is taken care automatically | 15:49 |
| amoralej | things are never as simple as it looks like :) | 15:49 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib master: Add support for iscsid configuration https://review.opendev.org/c/openstack/puppet-openstacklib/+/794605 | 15:50 |
| amoralej | cinderclient.exceptions.ClientException: Unable to update attachment.(Invalid input received: Connector doesn't have required information: initiator | 15:54 |
| amoralej | that's the actual error, as there is no initiatorname.iscsi, it can't create the attachment for the volume | 15:55 |
| amoralej | so iscsiadm is not even tried | 15:55 |
| amoralej | https://logserver.rdoproject.org/78/33878/3/check/periodic-xena-puppet-scenario001-centos-stream-9/9cda28c/logs/weirdo-project/logs/nova/nova-compute.txt.gz | 15:55 |
| amoralej | if you want to check logs | 15:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: WIP: Enable iscsid configuration https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794635 | 15:56 |
| amoralej | https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794635 | 15:58 |
| tkajinam | amoralej, hmmmmmmm | 15:58 |
| tkajinam | so it tries to cage /etc/iscsi/initiatorname.iscsi | 15:58 |
| tkajinam | to get initiator name | 15:58 |
| tkajinam | :-P | 15:58 |
| amoralej | yes | 15:59 |
| tkajinam | now I understand that sad circular dependencies :-) | 16:00 |
| amoralej | yep, exactly :) | 16:01 |
| tkajinam | I'll take some time tomorrow (actually it's already "today") about that problem | 16:03 |
| tkajinam | so far let's see how the patches to openstacklib and p-o-i work | 16:03 |
| amoralej | ack, thanks for your help tkajinam | 16:06 |
| *** rpittau is now known as rpittau|afk | 16:12 | |
| *** amoralej is now known as amoralej|off | 16:23 | |
| *** jpena is now known as jpena|off | 16:27 | |
| *** ysandeep|ruck is now known as ysandeep|away | 16:27 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-horizon master: Add support for independent manila-dashboard configurations https://review.opendev.org/c/openstack/puppet-horizon/+/790350 | 16:27 |
| *** derekh has quit IRC | 17:06 | |
| tobias-urdin | tkajinam: sorry been afk and going offline now - but reading iscsi gave me shivers, never looked back after going ceph :D | 17:12 |
| * tobias-urdin out | 17:14 | |
| *** ysandeep|away has quit IRC | 17:31 | |
| *** amoralej|off has quit IRC | 18:21 | |
| *** amoralej|off has joined #puppet-openstack | 18:30 | |
| *** amoralej|off has quit IRC | 19:37 | |
| *** tosky has quit IRC | 23:00 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!