| *** ykarel|away is now known as ykarel | 04:14 | |
| ykarel | tkajinam, puppet-nova train tag created wrongly | 06:57 |
|---|---|---|
| ykarel | https://github.com/openstack/releases/blob/master/deliverables/train/puppet-nova.yaml#L33-L40 | 06:58 |
| ykarel | both 15.7.0 and 15.8.0 pointing same commit | 06:58 |
| ykarel | noticed via https://review.rdoproject.org/r/c/puppet/puppet-nova-distgit/+/34167 | 06:58 |
| ykarel | looking if other failures are also same | 06:59 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova stable/train: Bump metadata to create the real final train release https://review.opendev.org/c/openstack/puppet-nova/+/796766 | 06:59 |
| tkajinam | ^^ ykarel | 06:59 |
| tkajinam | sorry about that... | 06:59 |
| ykarel | tkajinam, also issue with puppet-murano | 07:01 |
| ykarel | metadata is 15.4.1 https://github.com/openstack/puppet-murano/commit/7a3a8d7e58dfa64b490ce71791262f4403545693 | 07:01 |
| ykarel | while release is 15.5.0 | 07:01 |
| ykarel | https://review.opendev.org/c/openstack/releases/+/792820/6/deliverables/train/puppet-murano.yaml | 07:01 |
| ykarel | rest all good | 07:01 |
| tkajinam | ykarel, thanks for double checking... | 07:03 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-murano stable/train: Fix inconsistent version in metadata.json https://review.opendev.org/c/openstack/puppet-murano/+/796768 | 07:03 |
| tkajinam | ykarel, it's always tough to create the correct release... I though I checked all thoroughly but there are still mistakes left... | 07:04 |
| * tkajinam sighs | 07:05 | |
| tkajinam | ykarel, https://review.opendev.org/c/openstack/releases/+/796770 | 07:06 |
| tkajinam | ykarel, hope this fixes the problems ... | 07:06 |
| ykarel | tkajinam, yeap it could happen, aren't u using some script to fetch latest metadata and commit to prepare releases? | 07:07 |
| ykarel | that might handle some misses | 07:07 |
| tkajinam | I'm not aware of any existing tooling | 07:07 |
| tkajinam | and it's time to develop it :-) | 07:07 |
| * tkajinam sighs | 07:07 | |
| tkajinam | actually I thought of that after making a mistake | 07:08 |
| tkajinam | but left that every time when a release was made | 07:08 |
| ykarel | tkajinam, me too not aware if there is some tooling | 07:12 |
| tkajinam | ykarel, I'll create that one when I nominate to Puppet PTL :-P | 07:12 |
| ykarel | but too fetch hashes i use git ls-remote https://opendev.org/openstack/puppet-murano refs/heads/stable/train | awk '{print $1}' | 07:12 |
| tkajinam | which means never at this moment | 07:12 |
| tkajinam | ykarel, yeah | 07:12 |
| tkajinam | tricky part is that we should rebase that release commit before merging | 07:12 |
| tkajinam | or, yes | 07:13 |
| tkajinam | you are correct | 07:13 |
| tkajinam | I was thinking to create a centralized config file | 07:13 |
| tkajinam | and generate metadata.conf as well as release patch automatically | 07:13 |
| ykarel | tkajinam, rebase is not that necessary, if using merged commit via git ls-remote | 07:14 |
| ykarel | but yes rebase is safer | 07:14 |
| tkajinam | ykarel, yeah I see that command picks up the latest | 07:14 |
| tkajinam | commit | 07:14 |
| tkajinam | which would mean the commit itself some case, merge commit in the other cases | 07:14 |
| ykarel | yes | 07:14 |
| tkajinam | makes sense | 07:20 |
| *** rpittau|afk is now known as rpittau | 07:22 | |
| *** jpena|off is now known as jpena | 07:31 | |
| tkajinam | ykarel, will you update the release patch once correct versions are released ? | 07:32 |
| tkajinam | the release patch in rdo I mean | 07:32 |
| ykarel | tkajinam, the bot will create new reviews | 07:33 |
| tkajinam | ook | 07:33 |
| ykarel | so we can abandon the old oens | 07:33 |
| tkajinam | so it's automated :-) | 07:33 |
| ykarel | yes :) we just need to trigger merge | 07:33 |
| tkajinam | ack. that's good to know | 07:34 |
| ykarel | if that's green | 07:34 |
| opendevreview | Daniel Alvarez proposed openstack/puppet-neutron master: [ovn metadata] Add report agent option https://review.opendev.org/c/openstack/puppet-neutron/+/796511 | 07:49 |
| opendevreview | Daniel Alvarez proposed openstack/puppet-neutron master: [ovn metadata] Add report agent option https://review.opendev.org/c/openstack/puppet-neutron/+/796511 | 08:18 |
| *** ykarel is now known as ykarel|lunch | 08:38 | |
| opendevreview | Daniel Alvarez proposed openstack/puppet-neutron master: [ovn metadata] Add report agent option https://review.opendev.org/c/openstack/puppet-neutron/+/796511 | 09:15 |
| opendevreview | Ebbex proposed openstack/puppet-cinder master: Allow for '-' in volume type property values https://review.opendev.org/c/openstack/puppet-cinder/+/796409 | 09:29 |
| *** ykarel|lunch is now known as ykarel | 09:45 | |
| *** jpena is now known as jpena|lunch | 11:34 | |
| *** amoralej is now known as amoralej|lunch | 12:00 | |
| opendevreview | Merged openstack/puppet-oslo master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-oslo/+/792569 | 12:07 |
| opendevreview | Merged openstack/puppet-oslo master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-oslo/+/796373 | 12:07 |
| *** jpena|lunch is now known as jpena | 12:32 | |
| *** amoralej|lunch is now known as amoralej | 13:02 | |
| opendevreview | Merged openstack/puppet-murano stable/train: Fix inconsistent version in metadata.json https://review.opendev.org/c/openstack/puppet-murano/+/796768 | 14:08 |
| opendevreview | Merged openstack/puppet-nova master: Add parameter to set the libvirt max_number of guests with encrypted memory https://review.opendev.org/c/openstack/puppet-nova/+/795951 | 14:26 |
| opendevreview | Merged openstack/puppet-nova stable/train: Bump metadata to create the real final train release https://review.opendev.org/c/openstack/puppet-nova/+/796766 | 14:26 |
| *** ykarel is now known as ykarel|away | 15:10 | |
| opendevreview | Martin Schuppert proposed openstack/puppet-nova stable/wallaby: Add parameter to set the libvirt max_number of guests with encrypted memory https://review.opendev.org/c/openstack/puppet-nova/+/796880 | 15:27 |
| *** ysandeep is now known as ysandeep|out | 15:33 | |
| *** rpittau is now known as rpittau|afk | 16:09 | |
| *** amoralej is now known as amoralej|off | 16:35 | |
| *** jpena is now known as jpena|off | 16:36 | |
| *** mschuppert3 is now known as mschuppert | 17:33 | |
| opendevreview | Merged openstack/puppet-oslo stable/victoria: Use tripleo puppet job instaed of tripleo job https://review.opendev.org/c/openstack/puppet-oslo/+/796558 | 22:35 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!