| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Make password in octavia::neutron class required https://review.opendev.org/c/openstack/puppet-octavia/+/941596 | 00:11 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Drop explicit install of ruby-augeas https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941613 | 00:21 |
| tkajinam | tobias-urdin, ugh yes. I planned to make it required when octaiva removed neutronclient but that was missed | 00:22 |
| opendevreview | Merged openstack/puppet-openstack-integration master: Use octavia::neutron class https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941595 | 02:05 |
| opendevreview | Merged openstack/puppet-openstack-integration master: Unpin bundler in litmus https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941561 | 02:05 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: DNM: Check litmus https://review.opendev.org/c/openstack/puppet-nova/+/941562 | 02:11 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ovn master: DNM: Check litmus https://review.opendev.org/c/openstack/puppet-ovn/+/941617 | 02:28 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceph master: Bump upper version of puppetlabs-apt https://review.opendev.org/c/openstack/puppet-ceph/+/941618 | 02:37 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack_spec_helper master: Install ruby-augeas https://review.opendev.org/c/openstack/puppet-openstack_spec_helper/+/941621 | 03:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ovn master: DNM: Check litmus https://review.opendev.org/c/openstack/puppet-ovn/+/941617 | 03:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Drop explicit install of ruby-augeas https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941613 | 06:25 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack_spec_helper master: Install ruby-augeas https://review.opendev.org/c/openstack/puppet-openstack_spec_helper/+/941621 | 06:25 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ovn master: DNM: Check litmus https://review.opendev.org/c/openstack/puppet-ovn/+/941617 | 06:25 |
| tobias-urdin | tkajinam: wonder if we should backport a fix that sets everything in octavia::neutron to os_service_default so that old service_auth is used by default until (with above) octavia::neutron becomes required | 07:17 |
| tobias-urdin | or we can just leave it and perhaps update the release note | 07:19 |
| tkajinam | tobias-urdin, no octavia removed usage of service_auth for neutron interaction in 2023.2 so these options are really mandatory | 07:53 |
| tkajinam | so I think the right approach is to backport your change to make users aware that they should really configure these options | 07:54 |
| tkajinam | oh, wait. I misunderstood the release when it was removed. Let me double check | 07:54 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Make password in octavia::neutron class required https://review.opendev.org/c/openstack/puppet-octavia/+/941596 | 09:08 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Make password in octavia::neutron class required https://review.opendev.org/c/openstack/puppet-octavia/+/941596 | 09:09 |
| tkajinam | tobias-urdin, ok I now understand the problem and agree with you. | 09:12 |
| tkajinam | I though it may not read the whole neutron section if it lacks required option but in fact it attempts to merge these sections, which doesn't make quote good sense to me honestly... | 09:13 |
| tkajinam | tobias-urdin, alternatively we can purge the other options completely if password is not given. Though I feel like just backporting your change to require password may make more sense, as we may want to encourage people to use the new options | 09:14 |
| tobias-urdin | tkajinam: either way works for me :) | 10:44 |
| tobias-urdin | yea the merge part is what messed things up a bit | 10:44 |
| tkajinam | if the section is incomplete then there is no reasonable reason to use anything from it, imho | 10:47 |
| tkajinam | tobias-urdin, let's backport your change. | 10:47 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Drop explicit install of ruby-augeas https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941613 | 10:54 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration stable/2024.2: Unpin bundler in litmus https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941630 | 10:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Drop explicit install of ruby-augeas https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941613 | 12:35 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Drop explicit install of ruby-augeas https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941613 | 12:35 |
| *** tkajinam is now known as Guest9129 | 13:39 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-aodh master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-aodh/+/941639 | 13:42 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-barbican master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-barbican/+/941640 | 13:43 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceilometer master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-ceilometer/+/941641 | 13:43 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceph master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-ceph/+/941642 | 13:44 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-cinder/+/941643 | 13:44 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cloudkitty master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-cloudkitty/+/941644 | 13:45 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-designate master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-designate/+/941645 | 13:45 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-glance master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-glance/+/941646 | 13:46 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-gnocchi master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-gnocchi/+/941647 | 13:46 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-heat master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-heat/+/941648 | 13:46 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-horizon master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-horizon/+/941649 | 13:47 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ironic master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-ironic/+/941650 | 13:47 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-keystone/+/941651 | 13:47 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-magnum master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-magnum/+/941652 | 13:48 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-barbican master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-barbican/+/941640 | 13:49 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-manila master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-manila/+/941654 | 13:49 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-mistral master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-mistral/+/941655 | 13:50 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-neutron/+/941656 | 13:50 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-nova/+/941657 | 13:51 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-octavia/+/941658 | 13:51 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack_extras master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-openstack_extras/+/941659 | 13:52 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-openstacklib/+/941660 | 13:52 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-oslo master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-oslo/+/941661 | 13:52 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ovn master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-ovn/+/941662 | 13:53 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-placement master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-placement/+/941663 | 13:53 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-swift master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-swift/+/941664 | 13:53 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tempest master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-tempest/+/941665 | 13:54 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-trove master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-trove/+/941666 | 13:54 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-vitrage master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-vitrage/+/941667 | 13:54 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-watcher master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-watcher/+/941668 | 13:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-zaqar master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-zaqar/+/941669 | 13:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cloudkitty master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-cloudkitty/+/941644 | 13:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Make puppet 8 litmus job voting https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941675 | 15:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-watcher master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-watcher/+/941668 | 15:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Make puppet 8 litmus job voting https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941675 | 15:24 |
| opendevreview | Merged openstack/puppet-aodh master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-aodh/+/941639 | 15:35 |
| opendevreview | Merged openstack/puppet-ceilometer master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-ceilometer/+/941641 | 15:36 |
| opendevreview | Merged openstack/puppet-magnum master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-magnum/+/941652 | 15:38 |
| opendevreview | Merged openstack/puppet-barbican master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-barbican/+/941640 | 15:40 |
| opendevreview | Merged openstack/puppet-ironic master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-ironic/+/941650 | 15:40 |
| opendevreview | Merged openstack/puppet-heat master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-heat/+/941648 | 15:40 |
| opendevreview | Merged openstack/puppet-ceph master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-ceph/+/941642 | 15:41 |
| opendevreview | Merged openstack/puppet-glance master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-glance/+/941646 | 15:41 |
| opendevreview | Merged openstack/puppet-horizon master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-horizon/+/941649 | 15:41 |
| opendevreview | Merged openstack/puppet-nova master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-nova/+/941657 | 15:41 |
| opendevreview | Merged openstack/puppet-manila master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-manila/+/941654 | 15:41 |
| opendevreview | Merged openstack/puppet-keystone master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-keystone/+/941651 | 15:41 |
| opendevreview | Merged openstack/puppet-designate master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-designate/+/941645 | 15:41 |
| opendevreview | Merged openstack/puppet-cloudkitty master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-cloudkitty/+/941644 | 15:42 |
| opendevreview | Merged openstack/puppet-gnocchi master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-gnocchi/+/941647 | 15:42 |
| opendevreview | Merged openstack/puppet-openstack_extras master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-openstack_extras/+/941659 | 15:42 |
| opendevreview | Merged openstack/puppet-cinder master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-cinder/+/941643 | 15:42 |
| opendevreview | Merged openstack/puppet-openstacklib master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-openstacklib/+/941660 | 15:43 |
| opendevreview | Merged openstack/puppet-swift master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-swift/+/941664 | 15:43 |
| opendevreview | Merged openstack/puppet-oslo master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-oslo/+/941661 | 15:43 |
| opendevreview | Merged openstack/puppet-ovn master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-ovn/+/941662 | 15:43 |
| opendevreview | Merged openstack/puppet-zaqar master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-zaqar/+/941669 | 15:44 |
| opendevreview | Merged openstack/puppet-vitrage master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-vitrage/+/941667 | 15:44 |
| opendevreview | Merged openstack/puppet-mistral master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-mistral/+/941655 | 15:44 |
| opendevreview | Merged openstack/puppet-neutron master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-neutron/+/941656 | 15:44 |
| opendevreview | Merged openstack/puppet-trove master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-trove/+/941666 | 15:45 |
| opendevreview | Merged openstack/puppet-placement master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-placement/+/941663 | 15:45 |
| opendevreview | Merged openstack/puppet-tempest master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-tempest/+/941665 | 15:46 |
| opendevreview | Merged openstack/puppet-watcher master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-watcher/+/941668 | 15:46 |
| opendevreview | Merged openstack/puppet-ceph master: Bump upper version of puppetlabs-apt https://review.opendev.org/c/openstack/puppet-ceph/+/941618 | 16:09 |
| opendevreview | Merged openstack/puppet-octavia master: Make password in octavia::neutron class required https://review.opendev.org/c/openstack/puppet-octavia/+/941596 | 17:11 |
| opendevreview | Merged openstack/puppet-octavia master: Mark puppet 8 supported https://review.opendev.org/c/openstack/puppet-octavia/+/941658 | 17:11 |
| opendevreview | Merged openstack/puppet-openstack-integration master: Drop explicit install of ruby-augeas https://review.opendev.org/c/openstack/puppet-openstack-integration/+/941613 | 17:38 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!