| sames | hello! Quick question. I'm striking out trying to make nova_flavor properties idempotent. I'm guessing that I have some bad syntax but can't seem to figure it out. Puppet always returns: | 13:41 |
|---|---|---|
| sames | Notice: /Stage[main]/Profiles::Ostack_controller/Nova_flavor[m1.pico]/properties: properties changed { } to { 'aggregate_instance_extra_specs:common' => 'true' } (corrective) | 13:41 |
| sames | latest resource: nova_flavor { 'm1.pico': ensure => present, id => '024dd0df-c1b5-4da9-b6f9-4c0c910f5941', ram => '64', disk => '1', vcpus => '1', properties => { 'aggregate_instance_extra_specs:common' => 'true' }, } | 13:42 |
| sames | NOTE: This is using puppet modules for 2023.1. | 13:43 |
| sames | openstack flavor show does show properties as populated (properties="{'aggregate_instance_extra_specs:common': 'true'}") so why it's changing from {} is unclear | 13:48 |
| tkajinam | sames, I think I found the wrong logic. I guess something was updated in openstackclient but our module wasn't updated timely | 14:58 |
| sames | ah, sweet. So not my fault :). Will this be fixed for the 2024.1 release? I can manually make changes locally to test... | 15:09 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_flavor: Fix broken idempotency with properties https://review.opendev.org/c/openstack/puppet-nova/+/914813 | 15:18 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_flavor: Convert and validate numeric values https://review.opendev.org/c/openstack/puppet-nova/+/914815 | 15:28 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_flavor: Fix broken idempotency with properties https://review.opendev.org/c/openstack/puppet-nova/+/914813 | 15:39 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_flavor: Fix broken idempotency with properties https://review.opendev.org/c/openstack/puppet-nova/+/914813 | 15:40 |
| sames | For what it's worth... fix 914813 fixes the issue I'm seeing. thank you. | 16:36 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_flavor: Fix broken idempotency with properties https://review.opendev.org/c/openstack/puppet-nova/+/914813 | 17:01 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_flavor: Fix broken idempotency with properties https://review.opendev.org/c/openstack/puppet-nova/+/914813 | 17:09 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_flavor: Fix broken idempotency with properties https://review.opendev.org/c/openstack/puppet-nova/+/914813 | 17:35 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: nova_flavor: Fix broken idempotency with properties https://review.opendev.org/c/openstack/puppet-nova/+/914813 | 17:35 |
| opendevreview | Merged openstack/puppet-magnum master: Refactor resource dependencies https://review.opendev.org/c/openstack/puppet-magnum/+/914737 | 22:36 |
| opendevreview | Merged openstack/puppet-manila master: Require enabled_share_backends https://review.opendev.org/c/openstack/puppet-manila/+/914576 | 22:43 |
| opendevreview | Merged openstack/puppet-octavia stable/2023.1: Use different resource name for persistence db setup https://review.opendev.org/c/openstack/puppet-octavia/+/914592 | 22:43 |
| opendevreview | Merged openstack/puppet-watcher master: cache: Support new redis options https://review.opendev.org/c/openstack/puppet-watcher/+/914574 | 22:43 |
| opendevreview | Merged openstack/puppet-barbican master: [p11_crypto_plugin] login should be secret https://review.opendev.org/c/openstack/puppet-barbican/+/914746 | 22:58 |
| opendevreview | Merged openstack/puppet-keystone master: Use native interface to inject vhost configuration https://review.opendev.org/c/openstack/puppet-keystone/+/914050 | 23:04 |
| opendevreview | Merged openstack/puppet-keystone stable/2023.1: identity provider: Drop reference to removed parameters https://review.opendev.org/c/openstack/puppet-keystone/+/914640 | 23:04 |
| opendevreview | Merged openstack/puppet-cinder master: Require enabled_backends https://review.opendev.org/c/openstack/puppet-cinder/+/914575 | 23:09 |
| opendevreview | Merged openstack/puppet-cinder master: rbd: Purge rbd_cluster_name by default https://review.opendev.org/c/openstack/puppet-cinder/+/914577 | 23:09 |
| opendevreview | Merged openstack/puppet-cinder master: rbd: Simplify handling of rbd_cluster_name https://review.opendev.org/c/openstack/puppet-cinder/+/914578 | 23:09 |
| opendevreview | Merged openstack/puppet-cinder master: rbd: More strictly validate rbd_ceph_conf https://review.opendev.org/c/openstack/puppet-cinder/+/914659 | 23:19 |
| opendevreview | Merged openstack/puppet-octavia stable/2023.2: Use different resource name for persistence db setup https://review.opendev.org/c/openstack/puppet-octavia/+/907922 | 23:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!