| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_aggregate: Fix match for array hosts https://review.opendev.org/c/openstack/puppet-nova/+/931754 | 01:03 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/puppet-glance master: Use common function to parse python dict https://review.opendev.org/c/openstack/puppet-glance/+/931744 | 01:46 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-glance master: Use common function to parse python dict https://review.opendev.org/c/openstack/puppet-glance/+/931744 | 01:48 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_aggregate: Fix match for array hosts https://review.opendev.org/c/openstack/puppet-nova/+/931754 | 01:52 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_aggregate: Fix match for array hosts https://review.opendev.org/c/openstack/puppet-nova/+/931754 | 01:53 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova stable/2024.2: Drop unicode prefix from command output https://review.opendev.org/c/openstack/puppet-nova/+/931855 | 01:56 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Use only Hash value for properties https://review.opendev.org/c/openstack/puppet-nova/+/931720 | 01:56 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_aggregate: Fix match for array hosts https://review.opendev.org/c/openstack/puppet-nova/+/931754 | 01:57 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Use only Hash value for properties https://review.opendev.org/c/openstack/puppet-nova/+/931720 | 01:58 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tempest stable/2024.2: Support customizing catalog_type options https://review.opendev.org/c/openstack/puppet-tempest/+/931735 | 02:05 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tempest stable/2024.1: Support customizing catalog_type options https://review.opendev.org/c/openstack/puppet-tempest/+/931758 | 02:07 |
| opendevreview | Merged openstack/puppet-tempest master: Move descriptions of *_catalog_type options https://review.opendev.org/c/openstack/puppet-tempest/+/931760 | 02:18 |
| opendevreview | Merged openstack/puppet-nova stable/2024.2: Drop unicode prefix from command output https://review.opendev.org/c/openstack/puppet-nova/+/931855 | 02:40 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova stable/2024.1: Drop unicode prefix from command output https://review.opendev.org/c/openstack/puppet-nova/+/931859 | 02:46 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Fix lack of parse logic after resource creation https://review.opendev.org/c/openstack/puppet-nova/+/931862 | 03:33 |
| opendevreview | Merged openstack/puppet-nova stable/2024.1: Drop unicode prefix from command output https://review.opendev.org/c/openstack/puppet-nova/+/931859 | 03:39 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Fix lack of parse logic after resource creation https://review.opendev.org/c/openstack/puppet-nova/+/931862 | 04:07 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Use only Hash value for properties https://review.opendev.org/c/openstack/puppet-nova/+/931720 | 04:18 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Fix lack of parse logic after resource creation https://review.opendev.org/c/openstack/puppet-nova/+/931862 | 04:22 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Fix lack of parse logic after resource creation https://review.opendev.org/c/openstack/puppet-nova/+/931862 | 04:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-glance master: Use common function to parse python dict https://review.opendev.org/c/openstack/puppet-glance/+/931744 | 05:12 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Use common function to parse python dict https://review.opendev.org/c/openstack/puppet-cinder/+/931893 | 09:49 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: cinder_qos: Fix parsing of associations field https://review.opendev.org/c/openstack/puppet-cinder/+/931894 | 09:59 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: cinder_qos: Fix parsing of associations field https://review.opendev.org/c/openstack/puppet-cinder/+/931894 | 11:47 |
| zigo | tkajinam: Hi there! | 11:51 |
| zigo | I'm trying to upgrade to latest puppet-openstack, and now I'm getting this: | 11:51 |
| zigo | Oslo::Messaging::Notifications[keystone_config]: parameter 'transport_url' expects an Oslo::TransportURL = Variant[Oslo::TransportURL::Rabbit = Pattern[/^(rabbit|kombu):\/\/[^\/].*/], Oslo::TransportURL::Fake = Pattern[/^(fake):\/\/$/], Openstacklib::ServiceDefault = Enum['<SERVICE DEFAULT>']] value, got String | 11:51 |
| zigo | What am I doing wrong? Should I change my call to tranport_url() ? | 11:51 |
| zigo | How does it work? | 11:52 |
| tkajinam | zigo, what is the transport protocol you sue ? | 11:53 |
| zigo | Is this because I'm pushing 3 servers? | 11:53 |
| tkajinam | you *use* | 11:54 |
| zigo | I'm doing: rabbit://login:pass@server-1:5671,login:pass@server-2:5671,login:pass@server-3:5671 | 11:54 |
| zigo | Maybe the regex only accept *one* server? | 11:54 |
| zigo | (which is what I'm supposed to do, but I'm still using the old way...) | 11:55 |
| tkajinam | no it should check only prefix | 11:55 |
| zigo | Oh, maybe because I'm setting it to empty string?!? | 11:55 |
| * zigo double checks | 11:56 | |
| zigo | Yeah, that must be it. | 11:56 |
| zigo | I'm setting driver=noop, and transport_url to empty. | 11:56 |
| zigo | That must be it. | 11:56 |
| tkajinam | so the validation works as expected | 11:57 |
| tkajinam | you should omit transport_url if you don't need it | 11:57 |
| tkajinam | drop the parameter or use undef (which is internally dropped by puppet) | 11:57 |
| zigo | Or use service_default instead? | 11:57 |
| zigo | Ok, will try undef. | 11:57 |
| tkajinam | service_default fact may work, too | 11:57 |
| tkajinam | I mean os_service_default fact | 11:58 |
| zigo | Yeah. | 11:58 |
| zigo | Thanks. | 11:58 |
| zigo | Trying undef... | 11:59 |
| zigo | WORKS ! :) | 11:59 |
| zigo | Great... :P | 11:59 |
| zigo | So one last problem, and I'll be good to go. | 12:00 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: cinder_qos: Autorequire associated volume type https://review.opendev.org/c/openstack/puppet-cinder/+/931907 | 12:24 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Test cinder_qos in acceptance tests https://review.opendev.org/c/openstack/puppet-cinder/+/931908 | 12:24 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-manila master: Expose configure_service https://review.opendev.org/c/openstack/puppet-manila/+/931910 | 12:39 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Deploy block-storage service type https://review.opendev.org/c/openstack/puppet-cinder/+/931915 | 13:17 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Test cinder_qos in acceptance tests https://review.opendev.org/c/openstack/puppet-cinder/+/931908 | 13:18 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Test cinder_qos in acceptance tests https://review.opendev.org/c/openstack/puppet-cinder/+/931908 | 13:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-manila master: Use common function to parse python dict https://review.opendev.org/c/openstack/puppet-manila/+/931950 | 14:01 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Deploy block-storage service type https://review.opendev.org/c/openstack/puppet-cinder/+/931915 | 14:35 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Test cinder_qos in acceptance tests https://review.opendev.org/c/openstack/puppet-cinder/+/931908 | 14:43 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Deploy block-storage service type https://review.opendev.org/c/openstack/puppet-cinder/+/931915 | 14:59 |
| opendevreview | Merged openstack/puppet-manila master: Expose configure_service https://review.opendev.org/c/openstack/puppet-manila/+/931910 | 15:18 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Deploy block-storage service type https://review.opendev.org/c/openstack/puppet-cinder/+/931915 | 15:19 |
| opendevreview | Merged openstack/puppet-cinder master: Use common function to parse python dict https://review.opendev.org/c/openstack/puppet-cinder/+/931893 | 15:24 |
| opendevreview | Merged openstack/puppet-glance master: Use common function to parse python dict https://review.opendev.org/c/openstack/puppet-glance/+/931744 | 15:27 |
| opendevreview | Merged openstack/puppet-nova master: nova_aggregate: Fix match for array hosts https://review.opendev.org/c/openstack/puppet-nova/+/931754 | 15:31 |
| opendevreview | Merged openstack/puppet-nova master: Fix lack of parse logic after resource creation https://review.opendev.org/c/openstack/puppet-nova/+/931862 | 15:35 |
| opendevreview | Merged openstack/puppet-tempest stable/2024.2: Support customizing catalog_type options https://review.opendev.org/c/openstack/puppet-tempest/+/931735 | 15:41 |
| opendevreview | Merged openstack/puppet-tempest stable/2024.1: Support customizing catalog_type options https://review.opendev.org/c/openstack/puppet-tempest/+/931758 | 15:41 |
| opendevreview | Merged openstack/puppet-cinder master: cinder_qos: Fix parsing of associations field https://review.opendev.org/c/openstack/puppet-cinder/+/931894 | 15:42 |
| opendevreview | Merged openstack/puppet-manila master: Use common function to parse python dict https://review.opendev.org/c/openstack/puppet-manila/+/931950 | 15:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!