*** armax has quit IRC | 00:03 | |
*** tetsuro has joined #openstack-release | 00:21 | |
*** diablo_rojo has joined #openstack-release | 00:37 | |
*** mriedem has quit IRC | 01:00 | |
*** ekcs has quit IRC | 01:10 | |
*** armax has joined #openstack-release | 01:36 | |
*** diablo_rojo has quit IRC | 01:38 | |
*** armax has quit IRC | 01:42 | |
*** hongbin has joined #openstack-release | 01:49 | |
*** armax has joined #openstack-release | 02:05 | |
*** ekcs has joined #openstack-release | 02:24 | |
*** ekcs has quit IRC | 02:25 | |
*** ekcs has joined #openstack-release | 02:26 | |
*** hongbin has quit IRC | 02:36 | |
*** ykarel has joined #openstack-release | 02:39 | |
*** ricolin has joined #openstack-release | 02:47 | |
*** ykarel_ has joined #openstack-release | 02:48 | |
*** ykarel has quit IRC | 02:51 | |
*** dave-mccowan has quit IRC | 02:53 | |
*** ekcs has quit IRC | 03:09 | |
*** ekcs has joined #openstack-release | 03:21 | |
*** ekcs has quit IRC | 03:22 | |
*** udesale has joined #openstack-release | 04:07 | |
*** ykarel_ has quit IRC | 04:56 | |
*** ekcs has joined #openstack-release | 05:07 | |
*** ekcs has quit IRC | 05:07 | |
*** ykarel_ has joined #openstack-release | 05:16 | |
*** jtomasek has joined #openstack-release | 05:37 | |
*** zbr|ruck has quit IRC | 05:43 | |
*** zbr has joined #openstack-release | 05:44 | |
*** radeks has joined #openstack-release | 06:06 | |
*** e0ne has joined #openstack-release | 06:16 | |
*** pcaruana has joined #openstack-release | 06:42 | |
*** rpittau|afk is now known as rpittau | 06:47 | |
*** slaweq has joined #openstack-release | 06:55 | |
*** e0ne has quit IRC | 06:56 | |
*** amoralej|off is now known as amoralej | 07:21 | |
*** ykarel_ is now known as ykarel | 07:26 | |
*** hberaud|gone is now known as hberaud | 07:31 | |
*** witek has joined #openstack-release | 07:37 | |
*** e0ne has joined #openstack-release | 07:39 | |
*** jtomasek_ has joined #openstack-release | 07:40 | |
*** jtomasek has quit IRC | 07:43 | |
*** ykarel is now known as ykarel|lunch | 07:56 | |
*** rpittau is now known as rpittau|bbl | 08:01 | |
ttx | processing the approved autoreleases... | 08:17 |
---|---|---|
ttx | evrardjp: could you re-review https://review.opendev.org/#/c/683860/ ? | 08:18 |
*** cdent has joined #openstack-release | 08:20 | |
openstackgerrit | Merged openstack/releases master: Auto release request for barbican-tempest-plugin https://review.opendev.org/683999 | 08:24 |
openstackgerrit | Merged openstack/releases master: Auto release request for trove-tempest-plugin https://review.opendev.org/684022 | 08:24 |
*** e0ne has quit IRC | 08:28 | |
openstackgerrit | Merged openstack/releases master: Auto release request for qinling https://review.opendev.org/684091 | 08:28 |
openstackgerrit | Merged openstack/releases master: Auto release request for qinling-dashboard https://review.opendev.org/684089 | 08:28 |
openstackgerrit | Merged openstack/releases master: Auto release request for trove-dashboard https://review.opendev.org/684110 | 08:28 |
openstackgerrit | Merged openstack/releases master: Auto release request for trove https://review.opendev.org/684111 | 08:34 |
openstackgerrit | Merged openstack/releases master: Auto release request for searchlight https://review.opendev.org/684103 | 08:34 |
openstackgerrit | Merged openstack/releases master: Auto release request for solum https://review.opendev.org/684106 | 08:34 |
*** priteau has joined #openstack-release | 08:34 | |
openstackgerrit | Merged openstack/releases master: Auto release request for solum-dashboard https://review.opendev.org/684105 | 08:38 |
openstackgerrit | Merged openstack/releases master: Auto release request for aodh https://review.opendev.org/684044 | 08:38 |
openstackgerrit | Merged openstack/releases master: Auto release request for telemetry-tempest-plugin https://review.opendev.org/684020 | 08:38 |
openstackgerrit | Merged openstack/releases master: Auto release request for searchlight-ui https://review.opendev.org/684102 | 08:38 |
openstackgerrit | Merged openstack/releases master: Release zun 4.0.0 rc1 https://review.opendev.org/681958 | 08:38 |
openstackgerrit | Merged openstack/releases master: Auto release request for watcher-dashboard https://review.opendev.org/684112 | 08:38 |
openstackgerrit | Merged openstack/releases master: Auto release request for watcher https://review.opendev.org/684113 | 08:38 |
openstackgerrit | Merged openstack/releases master: Auto release request for networking-bgpvpn https://review.opendev.org/684075 | 08:38 |
openstackgerrit | Merged openstack/releases master: Auto release request for magnum-tempest-plugin https://review.opendev.org/684012 | 08:38 |
openstackgerrit | Merged openstack/releases master: Release ironic 13.0.0 and create stable/train https://review.opendev.org/684303 | 08:38 |
openstackgerrit | Merged openstack/releases master: Auto release request for networking-sfc https://review.opendev.org/684080 | 08:38 |
*** zbr is now known as zbr|ruck | 08:40 | |
openstackgerrit | Merged openstack/releases master: Auto release request for networking-midonet https://review.opendev.org/684076 | 08:40 |
openstackgerrit | Merged openstack/releases master: Auto release request for networking-bagpipe https://review.opendev.org/684074 | 08:40 |
openstackgerrit | Merged openstack/releases master: Auto release request for neutron-dynamic-routing https://review.opendev.org/684081 | 08:40 |
openstackgerrit | Merged openstack/releases master: Auto release request for neutron-vpnaas https://review.opendev.org/684083 | 08:40 |
openstackgerrit | Merged openstack/releases master: Auto release request for networking-odl https://review.opendev.org/684077 | 08:40 |
openstackgerrit | Merged openstack/releases master: Auto release request for monasca-tempest-plugin https://review.opendev.org/684015 | 08:41 |
openstackgerrit | Merged openstack/releases master: Auto release request for compute-hyperv https://review.opendev.org/684052 | 08:50 |
*** ykarel|lunch is now known as ykarel | 08:53 | |
openstackgerrit | Merged openstack/releases master: Auto release request for oswin-tempest-plugin https://review.opendev.org/684017 | 08:55 |
openstackgerrit | Thierry Carrez proposed openstack/releases master: Auto release request for ironic-tempest-plugin https://review.opendev.org/684009 | 08:55 |
openstackgerrit | Merged openstack/releases master: Release ironic-prometheus-exporter 1.0.0 and create stable/train https://review.opendev.org/684313 | 08:55 |
openstackgerrit | Merged openstack/releases master: Release ironic-python-agent 5.0.0 and create stable/train https://review.opendev.org/684306 | 08:55 |
openstackgerrit | Merged openstack/releases master: Auto release request for neutron-fwaas https://review.opendev.org/684082 | 08:55 |
openstackgerrit | Merged openstack/releases master: Auto release request for monasca-agent https://review.opendev.org/684030 | 08:55 |
openstackgerrit | Merged openstack/releases master: Auto release request for monasca-ceilometer https://review.opendev.org/684032 | 08:56 |
openstackgerrit | Merged openstack/releases master: Auto release request for monasca-events-api https://review.opendev.org/684033 | 08:56 |
openstackgerrit | Merged openstack/releases master: Auto release request for monasca-log-api https://review.opendev.org/684034 | 08:56 |
openstackgerrit | Merged openstack/releases master: Auto release request for monasca-transform https://review.opendev.org/684037 | 08:56 |
openstackgerrit | Merged openstack/releases master: Auto release request for cloudkitty-dashboard https://review.opendev.org/684051 | 08:56 |
openstackgerrit | Merged openstack/releases master: Auto release request for cloudkitty-tempest-plugin https://review.opendev.org/684002 | 08:56 |
openstackgerrit | Merged openstack/releases master: Auto release request for zaqar-ui https://review.opendev.org/684114 | 08:56 |
evrardjp | ttx: hey, I will start my review day indeed :) | 08:57 |
evrardjp | ttx: merging the patch you asked for, because of ML email. | 08:58 |
openstackgerrit | Merged openstack/releases master: Auto release request for zaqar https://review.opendev.org/684115 | 08:59 |
openstackgerrit | Merged openstack/releases master: Auto release request for zaqar-tempest-plugin https://review.opendev.org/684024 | 08:59 |
openstackgerrit | Thierry Carrez proposed openstack/releases master: Auto release request for cyborg-tempest-plugin https://review.opendev.org/684004 | 09:00 |
evrardjp | ttx: mmm I have missed your patch for neutron lbaas removal when proposing the series, so maybe there are things you shouldn't merge | 09:01 |
ttx | the lbaas proposals failed validation anyway | 09:01 |
ttx | I think I abandoned them yesterday | 09:01 |
ttx | yeah I did | 09:02 |
ttx | https://review.opendev.org/#/q/status:abandoned+project:openstack/releases+branch:master+topic:rc1-autorelease | 09:02 |
ttx | The others I abandoned are those that duplicated an existing PTL request | 09:03 |
ttx | evrardjp: I recommend you focus on the ones with one +2, those can merge now if you confirm the version on them makes sense | 09:04 |
ttx | Those where the version is a no-brainer I approved directly if they had PTL+1 | 09:06 |
evrardjp | you're awesome ! | 09:07 |
evrardjp | ok makes sense | 09:07 |
evrardjp | it is weird to vote on my own patches in releases but ... ok :) | 09:07 |
ttx | evrardjp: I consider those patches were proposed by a bot :) We just need to doublecheck the version makes sense with the list-changes output | 09:09 |
openstackgerrit | Merged openstack/releases master: Release python-cloudkittyclient 3.1.0 https://review.opendev.org/683860 | 09:10 |
evrardjp | ttx: yeah that was my thinking | 09:11 |
ttx | hmm we really need to merge https://review.opendev.org/#/c/683985/ | 09:11 |
evrardjp | ttx: I realised that the rc template is incorrect | 09:11 |
evrardjp | hahah | 09:11 |
evrardjp | great minds think alike | 09:11 |
ttx | I'll abuse powers and merge it now | 09:11 |
evrardjp | ttx: wait | 09:11 |
evrardjp | the title is inconsistent | 09:11 |
ttx | should have done it earlier | 09:11 |
*** dtantsur has joined #openstack-release | 09:12 | |
ttx | too late | 09:12 |
ttx | it's not completely wrong | 09:12 |
evrardjp | it's not in fact | 09:12 |
evrardjp | it's just that rc's are not in release announce | 09:12 |
evrardjp | thanks for merging it then ttx | 09:13 |
ttx | should really have thought to do it BEFORE I unleashed the Kraken | 09:13 |
evrardjp | ^ this is just the reason I want random names as release names | 09:14 |
evrardjp | For K. | 09:14 |
ttx | "things you can release" | 09:14 |
ttx | Movie quotes including the name in a release context | 09:14 |
ttx | http://www.quodb.com/search/release | 09:15 |
evrardjp | OH GOD | 09:15 |
ttx | "Release valve" (Star Trek) | 09:16 |
evrardjp | Wouldn't that be a problem with trademarks? | 09:16 |
evrardjp | Anyway, nothing can beat kraken | 09:17 |
ttx | well, you don;t have to SAY star trek | 09:17 |
ttx | "Release the world engine." (Superman) | 09:17 |
evrardjp | ttx: I meant valve would be a problem | 09:18 |
evrardjp | Because... Valve | 09:18 |
ttx | hah yes | 09:18 |
ttx | How about.. Virus. "A virus, a selective virus. Release it near the aliens." (District 9) | 09:18 |
evrardjp | Works for Virus and Aliens I guess | 09:18 |
ttx | man, this keeps on giving | 09:18 |
evrardjp | hahah | 09:19 |
ttx | "The Quaffle is released and the game begins!" | 09:19 |
ttx | "Release the swine." (Pirates of the Caribbean: On Stranger Tides) | 09:19 |
evrardjp | I wonder if there is something for every letter | 09:21 |
ttx | Oh, there is | 09:21 |
evrardjp | Z would be hard | 09:21 |
ttx | "Sign this release. Make an X if you can't write." (Carrie) | 09:21 |
evrardjp | "Slow release. Bring you down gradually." | 09:22 |
evrardjp | hostage for h | 09:22 |
evrardjp | Wodewick (life of brian) | 09:23 |
evrardjp | omg | 09:23 |
evrardjp | that keeps giving | 09:23 |
ttx | "Zoomonkey" | 09:23 |
evrardjp | hahahahaha | 09:23 |
ttx | "like releasing a zoo monkey into the wild?" (Review: Road Rage; Orgy (2014)) | 09:23 |
ttx | You are free to suggest any word that appears together with release in that database | 09:24 |
ttx | Highness. "Highness, when your father releases me, I intend to return home." (Gladiator) | 09:24 |
openstackgerrit | Merged openstack/releases master: Auto release request for blazar-dashboard https://review.opendev.org/684046 | 09:24 |
evrardjp | that's definitely funnier than cities. | 09:25 |
ttx | Squeeze - "Squeeze. Release." (Leon) | 09:25 |
ttx | Wild - "and release it into the wild." (Monsters Inc) | 09:26 |
openstackgerrit | Merged openstack/releases master: Release monasca-common 2.16.1 for Train https://review.opendev.org/683986 | 09:26 |
openstackgerrit | Merged openstack/releases master: Fixing RC template https://review.opendev.org/683985 | 09:26 |
ttx | I'll propose it, I think | 09:26 |
openstackgerrit | Mark Goddard proposed openstack/releases master: Release python-ironicclient 3.1.0 for Train https://review.opendev.org/684660 | 09:26 |
evrardjp | haha. Propose it for A-Z! | 09:27 |
evrardjp | Kraken! Kraken! | 09:27 |
ttx | I want to play now! Could propose it post-Vancouver (because that one just matches the damn city, so it's a closure for what began in Austin | 09:27 |
evrardjp | not sure what you meant there. But ok | 09:29 |
evrardjp | Oh I see | 09:29 |
ttx | Oh, man. Wodewick. 'Yeah! Release "Wodewick"! Release "Wodewick"!' (Monty Python, Life of Brian) | 09:29 |
ttx | That is so perfect | 09:29 |
evrardjp | Yeah saw it, commented it above :) | 09:29 |
evrardjp | yeah | 09:30 |
evrardjp | that's seriously the second best | 09:30 |
ttx | It's even better I'd say | 09:30 |
evrardjp | I am not sure I want to wait for another round till W to see that happen | 09:30 |
ttx | I won;t wait 13 years for it | 09:30 |
evrardjp | HAHHAHAHAH | 09:30 |
ttx | Alright let's do this | 09:30 |
evrardjp | :D | 09:31 |
ttx | I'll form a proposal | 09:31 |
*** e0ne has joined #openstack-release | 09:31 | |
evrardjp | ttx: I approved a rc :/ | 09:32 |
ttx | which one? | 09:32 |
evrardjp | oh wait, the template already merged | 09:32 |
ttx | yeah, should be good | 09:32 |
evrardjp | murano dashboard | 09:32 |
evrardjp | let's see if that is ok | 09:32 |
evrardjp | else we might need to rebase I guess | 09:32 |
evrardjp | (not sure we need though) | 09:33 |
evrardjp | we would* need | 09:33 |
*** electrofelix has joined #openstack-release | 09:35 | |
openstackgerrit | Merged openstack/releases master: Release muranodashboard for train rc1 https://review.opendev.org/684186 | 09:43 |
*** hberaud is now known as hberaud|school-r | 09:56 | |
openstackgerrit | Mark Goddard proposed openstack/releases master: Release python-ironicclient 3.1.0 for Train https://review.opendev.org/684660 | 09:57 |
*** rpittau|bbl is now known as rpittau | 09:57 | |
*** hberaud|school-r is now known as hberaud | 10:01 | |
openstackgerrit | Merged openstack/releases master: Release murano-tempest-plugin for train https://review.opendev.org/684190 | 10:22 |
openstackgerrit | Merged openstack/releases master: Release murano-agent for train rc1 https://review.opendev.org/684188 | 10:22 |
openstackgerrit | Merged openstack/releases master: Release murano for train rc1 https://review.opendev.org/684185 | 10:22 |
openstackgerrit | Mark Goddard proposed openstack/releases master: Release python-ironicclient 3.1.0 for Train https://review.opendev.org/684660 | 10:22 |
*** pcaruana has quit IRC | 10:23 | |
openstackgerrit | Merged openstack/releases master: Release senlin-dashboard https://review.opendev.org/683695 | 10:26 |
*** hberaud has quit IRC | 10:28 | |
*** hberaud has joined #openstack-release | 10:29 | |
evrardjp | ttx: not sure about the urgency in here: https://review.opendev.org/#/c/684036/2 doesn't Witold have a week? | 10:31 |
evrardjp | still, I see no reason to not branch either :) | 10:31 |
*** pkovar has joined #openstack-release | 10:34 | |
ttx | hmm let me check what we said | 10:34 |
*** pcaruana has joined #openstack-release | 10:36 | |
ttx | yeah until next week to branch | 10:36 |
ttx | for intermediary things | 10:36 |
evrardjp | witek: What's the reason for your comment "Postponing cutting stable branch"? | 10:39 |
openstackgerrit | zhurong proposed openstack/releases master: Auto release request for ceilometer https://review.opendev.org/684049 | 10:40 |
ttx | "Hummingbird, you are released." (White House Down) -- damn, too many letters | 10:40 |
openstackgerrit | zhurong proposed openstack/releases master: Auto release request for panko https://review.opendev.org/684087 | 10:40 |
*** cdent has quit IRC | 10:41 | |
evrardjp | ttx: wow. Do you mean this as a message for rax/swift? | 10:41 |
evrardjp | For some it might be considered offensive :p | 10:42 |
ttx | Offsensive? How about... "Oswald is released." (JFK) | 10:43 |
ttx | oooh... "Release the Sardaukar." (Dune) | 10:44 |
evrardjp | nice! | 10:44 |
ttx | Still looking for good X and Y | 10:45 |
openstackgerrit | Merged openstack/releases master: Auto release request for watcher-tempest-plugin https://review.opendev.org/684023 | 10:51 |
evrardjp | ttx: for Y we can still have "You" "I release you" from LOTR | 10:52 |
openstackgerrit | Merged openstack/releases master: Auto release request for solum-tempest-plugin https://review.opendev.org/684019 | 10:54 |
evrardjp | I have trouble watching some of the logs :/ | 10:54 |
*** witek has quit IRC | 10:57 | |
*** udesale has quit IRC | 11:15 | |
ttx | hah | 11:27 |
openstackgerrit | Merged openstack/releases master: Auto release request for cinder-tempest-plugin https://review.opendev.org/684001 | 11:28 |
*** cdent has joined #openstack-release | 11:37 | |
openstackgerrit | Thierry Carrez proposed openstack/releases master: Sync ironic-prometheus-exporter releases https://review.opendev.org/684721 | 11:40 |
ttx | posted an analysis of the ironic-prometheus-exproter job fail | 11:41 |
*** witek has joined #openstack-release | 11:41 | |
ttx | dtantsur: TheJulia ^ | 11:42 |
dtantsur | ugh | 11:44 |
ttx | not sure that will pass tests. If not I'll keep 1.0.0 in there | 11:45 |
dtantsur | ttx: we'll figure it out, thanks. I wonder if we can have a validation in the CI | 11:45 |
ttx | probably. check if already on PyPI... | 11:46 |
ttx | dtantsur: maybe you can +1 https://review.opendev.org/#/c/684721/ | 11:46 |
dtantsur | ttx: do you think we can remove the created 1.0.0 tag? will be super confusing | 11:47 |
ttx | dtantsur: that does not remove the tag... Just omits it in releases.o.o listing | 11:47 |
dtantsur | ttx: I'd prefer we remove the tag and re-add it at the released location (I wonder what was it) | 11:48 |
dtantsur | anyway, +1 to the 1.1.0 release | 11:48 |
witek | evrardjp: we still have one important change in review which I wanted to merge before branching, but I think it can wait; I'll update the release change | 11:48 |
ttx | well the issue is that 1.0.0 is before 0.1.0 | 11:48 |
ttx | (on PyPI) | 11:48 |
ttx | and we can't put that one back into the box | 11:48 |
ttx | (cat is out) | 11:49 |
dtantsur | UGH | 11:49 |
ttx | dtantsur: the best way is probably to have a good synced version in both areas, so 1.1.0 | 11:49 |
dtantsur | ttx: yep, to be clear, I'd prefer to do both 1.1.0 release and removal of 1.0.0 tag | 11:50 |
ttx | whoever checked PyPI when the repo was added missed that 1.0.0 | 11:50 |
*** iurygregory has joined #openstack-release | 11:51 | |
dtantsur | ttx: it was iurygregory | 11:51 |
iurygregory | dtantsur, I'm here o/ | 11:51 |
iurygregory | ttx, sorry for the trouble >.< | 11:51 |
ttx | yeah, I'll see if we can do that. One issue is that the branch exists from that tag... so I'd like to see if that patch passes | 11:51 |
dtantsur | iurygregory: so, we have 0.1.0 which is a sane upstream tag. and we have 1.0.0 which was released from something ancient | 11:51 |
dtantsur | and now we have 1.0.0 tag from HEAD and 1.0.0 release from.. whatever? | 11:52 |
openstackgerrit | Witold Bedyk proposed openstack/releases master: Auto release request for monasca-persister https://review.opendev.org/684036 | 11:52 |
dtantsur | ttx: will it help if iurygregory removes the 1.0.0 release from PyPI? and we remove the 1.0.0 tag? | 11:52 |
ttx | we can't really remove that one from PyPI that would confuse consumers even more. We have a short window to remove the 1.0.0 tag on our side, hoping people are not cloning that as we speak | 11:52 |
ttx | fungi: thoughts? | 11:53 |
iurygregory | I would say we don't have consumers.. | 11:53 |
ttx | Once something is released, you don;t erase it | 11:53 |
iurygregory | gotcha | 11:54 |
ttx | you just fix it going forward | 11:54 |
ttx | we complain enough when others do that to us | 11:54 |
dtantsur | yeah, it may break people who might have done "ironic-prometheus-exporter===1.0.0" in their constraints | 11:54 |
ttx | arguably the tag on our side did not trigger a release | 11:54 |
ttx | so we could technically remove it | 11:54 |
dtantsur | it's probably reasonably safe, yes | 11:55 |
ttx | but since the version history is now broken on pypi the best is to just do a 1.1.0 | 11:55 |
*** weshay has joined #openstack-release | 11:55 | |
dtantsur | yep, 1.1.0 is a must now. the only thing I wonder is whether we've had any breaking changes since that ancient 1.0.0 | 11:55 |
dtantsur | iurygregory: ^^^? | 11:55 |
dtantsur | mgoddard: JFYI the scrollback ^^^ | 11:56 |
iurygregory | dtantsur, I would say that the 1.0.0 in pypi is the first running version for ipmi only and with only a few metrics support | 11:57 |
dtantsur | iurygregory: but have we broken anything since? | 11:57 |
iurygregory | we broke rdo when we move the repo to opendev | 11:58 |
dtantsur | ttx: it passed. I wonder if we need to change branch location to the git hash of 1.0.0 | 11:58 |
iurygregory | so we fixed with https://review.opendev.org/#/c/669885/ | 11:58 |
dtantsur | ah, the branch hasn't been created yet. then we're fine. | 11:59 |
ttx | it's been created | 11:59 |
ttx | there are even changes proposed to it | 11:59 |
ttx | so removing it will cause more problems than it solves | 11:59 |
iurygregory | changes proposed to stable/train you mean? | 12:00 |
iurygregory | oh OpenStack Release Bot | 12:00 |
dtantsur | ttx: ah, right. then probably we need a git hash as 'location' | 12:01 |
ttx | it will be fine with the 1.1.0 tag as location | 12:01 |
ttx | it's the same ref in the end | 12:01 |
dtantsur | oh. right. I'm not very smart today :D | 12:02 |
dtantsur | let's go ahead with this one, I guess | 12:02 |
ttx | yeah.. evrardjp could you +2a it ? | 12:03 |
ttx | then I'll see with fungi if it's worth trying to remove the 1.0.0 tag from our repo | 12:03 |
evrardjp | fine for me | 12:08 |
dtantsur | on a related note, dhellmann, could you approve https://review.opendev.org/#/c/684376/ please? | 12:08 |
dtantsur | (not sure why I called it related, probably just too many things at the same time) | 12:09 |
evrardjp | ttx: done | 12:10 |
ttx | evrardjp: thx | 12:10 |
dtantsur | ttx, evrardjp, could you also check https://review.opendev.org/#/c/684660/ ASAP please? we've got some serious issues with 3.0.0.. | 12:10 |
evrardjp | dtantsur: ok reviewing rn | 12:11 |
ttx | yeah, was waiting on prometheanfire to +1 it | 12:12 |
fungi | ttx: anything which has already fetched new refs from that repo (for example, the zuul mergers and executors) won't know to delete their local copies of that tag. tag deletion doesn't propagate on fetch/update | 12:12 |
ttx | yeah, that was what I feared | 12:13 |
ttx | something will have cloned that and has to live with it anyzay | 12:13 |
ttx | +w+ | 12:13 |
evrardjp | ttx: I was reading his email, I thought it was an approval | 12:13 |
dtantsur | ttx: ah. I'm not sure it's practical to have an FFE discussion for it, though, given that it breaks tripleo and a few other projects.. | 12:13 |
evrardjp | but re-reading it.. makes it less obvious | 12:13 |
fungi | in which case at a minimum, things like release notes jobs will continue to include it unless we go around manually purging that tag from all the infrastructure | 12:13 |
ttx | dtantsur: I'll give him an hour and +2a it if he's not around | 12:14 |
evrardjp | is this the first time this happens? | 12:14 |
evrardjp | fungi: ^ | 12:14 |
evrardjp | to know if we should tackle the problem by writing a convoluted script do clean up tags | 12:14 |
evrardjp | to* | 12:14 |
ttx | dtantsur: i agree that the benefits outweigh the drawbacks and the RFE should be granted. Just want to give prometheanfire a chance to express that before I push it | 12:15 |
*** witek has quit IRC | 12:16 | |
dtantsur | sorry for this screw-up :( ironicclient code is quite a mess, it's very hard to refactor it without breaking | 12:16 |
ttx | evrardjp: it would only clean the clones we control | 12:16 |
evrardjp | that's the reason I voted positively. Making it impossible to use by consumers like bifrost ... is a big deal (or maybe I misunderstood) | 12:16 |
ttx | better to own that you have a bad tag in your history | 12:16 |
dtantsur | evrardjp: yep, bifrost is broken with 3.0.0 | 12:17 |
dtantsur | so is tripleo | 12:17 |
dtantsur | or anything that used mistral with ironic actions | 12:17 |
evrardjp | :( | 12:17 |
evrardjp | I will keep my vote then | 12:17 |
evrardjp | I hope you've added functional tests for this now :) | 12:17 |
dtantsur | mistral itself is partly to blame for its problems, but still | 12:17 |
ttx | I like to blame mistral | 12:18 |
evrardjp | haha | 12:18 |
dtantsur | evrardjp: we're migrating ironic-inspector from ironicclient to openstacksdk, we just didn't manage to finish it in train | 12:18 |
evrardjp | I am not here to blame.. just trying to help :) | 12:18 |
fungi | evrardjp: it's not at all the first time an incorrect tag number was pushed to an openstack repository, but it happens far less now that we have the release team reviewing tag proposals rather than all the ptls individually pushing tags to gerrit like in the old days | 12:18 |
dtantsur | which is going to be interesting, because any breakage in openstacksdk will break A LOT of people :) | 12:18 |
evrardjp | though the occasional blame is fun :p | 12:18 |
dtantsur | hehe | 12:18 |
ttx | https://imgflip.com/i/3bi8vj | 12:19 |
evrardjp | fungi: If I read your message differently, maybe we should do some pre-validation | 12:19 |
evrardjp | some more* | 12:19 |
evrardjp | ttx: hahahah | 12:19 |
evrardjp | dtantsur: you should totally click the "like" and "share" button on ttx link | 12:20 |
*** witek has joined #openstack-release | 12:20 | |
dtantsur | :D | 12:20 |
openstackgerrit | Merged openstack/releases master: Sync ironic-prometheus-exporter releases https://review.opendev.org/684721 | 12:24 |
* dtantsur fingers crossed | 12:24 | |
ttx | Now let's see if it runs ok :) | 12:24 |
*** dave-mccowan has joined #openstack-release | 12:28 | |
openstackgerrit | Kota Tsuyuzaki proposed openstack/releases master: Release storlets for train https://review.opendev.org/684733 | 12:34 |
dtantsur | ++ | 12:34 |
*** cdent has quit IRC | 12:36 | |
*** dave-mccowan has quit IRC | 12:42 | |
*** dave-mccowan has joined #openstack-release | 12:43 | |
*** amoralej is now known as amoralej|lunch | 12:44 | |
*** mriedem has joined #openstack-release | 12:47 | |
dtantsur | ttx: seems to have worked! | 12:48 |
TheJulia | Ttx, dtantsur: o/ from training. I guess I’m personally okay if we can’t get the latest ironic-Prometheus-exporter code on the co-ordinated release. Consumers of it will want latest, not a cycle release much as how ironic operates with releases on master branch during the cycle | 12:51 |
dtantsur | TheJulia: don't worry, we've already re-released it | 12:51 |
TheJulia | Woot, okay. Now back to zombie state | 12:51 |
dtantsur | braaaaaiinzzz | 12:52 |
dtantsur | TheJulia: the only thing you need to be aware of (no actions required) is that ironicclient 3.0.0 is so borken, we're excluding it from requirements and re-releasing stable/train :( | 12:53 |
TheJulia | Woot. :( | 12:53 |
TheJulia | It happens sadly | 12:53 |
dtantsur | yeah | 12:53 |
dtantsur | I had to dive into SessionClient and it's horrible | 12:53 |
dtantsur | considering rewriting it completely in Ussuri | 12:54 |
TheJulia | Please just put a giant warning on it :) | 12:54 |
TheJulia | “Go use openstacksdk, k thx bai” | 12:54 |
TheJulia | :) | 12:54 |
dtantsur | ++ | 12:54 |
* TheJulia engages maximum zombie state with minimal sleep last night | 12:55 | |
dtantsur | I'm pondering replacing the internals with openstacksdk's adapters | 12:55 |
TheJulia | That would be 4.0.0 though, fwiw | 12:55 |
dtantsur | ah, right, go get some sleep (and let's stop spamming the channel of people who're presumably trying to work) | 12:55 |
dtantsur | heh, depends on how much I break it in the process | 12:55 |
TheJulia | Training starts in 5 anyway | 12:56 |
*** witek has quit IRC | 13:08 | |
*** udesale has joined #openstack-release | 13:22 | |
*** witek has joined #openstack-release | 13:22 | |
*** amoralej|lunch is now known as amoralej | 13:23 | |
*** dave-mccowan has quit IRC | 13:26 | |
fungi | heads up, twine 2.0 was released today and is dropping support for python 2.7 and 3.5. as long as we're running it with python3 on bionic that's probably fine but keep an eye out for pypi upload failures in the next 24 hours: https://groups.google.com/forum/#!topic/pypa-dev/a2emyNrGrA4 | 13:34 |
mtreinish | fungi: thanks for that heads up, I didn't notice that | 13:44 |
* mtreinish goes to check automation for projects outside of openstack | 13:45 | |
mtreinish | I'm surprised they dropped 3.5 though, it's still a supported version | 13:45 |
*** ykarel is now known as ykarel|afk | 13:51 | |
*** slaweq has quit IRC | 13:54 | |
*** tosky has joined #openstack-release | 13:55 | |
*** ykarel|afk has quit IRC | 13:58 | |
prometheanfire | dtantsur: ttx +1 which review? | 14:00 |
prometheanfire | found it, ironiclient | 14:06 |
prometheanfire | +1'd | 14:06 |
dtantsur | thx! | 14:12 |
*** cdent has joined #openstack-release | 14:13 | |
peschk_l | o/ How do RCs work for projects with the "cycle-with-intermediary" release model ? I'd like to propose an RC but I get a "Version xxx looks like a pre-release and the release model does not allow for it" when I try to validate it locally | 14:16 |
peschk_l | We'd also be fine with a beta, but I get the same error. We just need some way to say "this will probably be the final release, but some fixes may be added" | 14:18 |
ttx | peschk_l: you either do intermediary releases (in which case you make multiple releases that are all consumable by users, and the "train" release just happens to be one of them) | 14:24 |
ttx | or you do ONE release per cycle, with RCs leading up to it | 14:25 |
*** mlavalle has joined #openstack-release | 14:25 | |
ttx | peschk_l: so it's OK to make a release, and add a minor if you spot some release critical bugs | 14:25 |
peschk_l | ttx: all right, thanks! | 14:26 |
*** ykarel has joined #openstack-release | 14:32 | |
*** AJaeger has joined #openstack-release | 14:33 | |
*** rpittau is now known as rpittau|afk | 14:34 | |
*** diablo_rojo has joined #openstack-release | 14:34 | |
openstackgerrit | Merged openstack/releases master: Release python-ironicclient 3.1.0 for Train https://review.opendev.org/684660 | 14:36 |
openstackgerrit | Luka Peschke proposed openstack/releases master: Make cloudkitty deliverables point to storyboard instead of launchpad https://review.opendev.org/684763 | 14:44 |
openstackgerrit | Luka Peschke proposed openstack/releases master: Release cloudkitty 11.0.0 https://review.opendev.org/684764 | 14:44 |
*** e0ne has quit IRC | 14:59 | |
*** e0ne has joined #openstack-release | 14:59 | |
*** dtantsur is now known as dtantsur|afk | 15:00 | |
*** tosky has quit IRC | 15:00 | |
*** tosky has joined #openstack-release | 15:01 | |
*** openstackgerrit has quit IRC | 15:21 | |
*** dulek has joined #openstack-release | 15:25 | |
*** tosky has quit IRC | 15:27 | |
*** ekcs has joined #openstack-release | 15:28 | |
*** weshay is now known as weshay_bbiab | 15:29 | |
*** ekcs has quit IRC | 15:29 | |
*** pcaruana has quit IRC | 15:30 | |
*** ykarel is now known as ykarel\afk | 16:04 | |
*** priteau has quit IRC | 16:23 | |
*** ykarel\afk is now known as ykarel | 16:23 | |
*** electrofelix has quit IRC | 16:27 | |
*** ykarel is now known as ykarel|away | 16:37 | |
*** ekcs has joined #openstack-release | 16:55 | |
*** openstackgerrit has joined #openstack-release | 16:58 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/releases master: Auto release request for networking-ovn https://review.opendev.org/684078 | 16:58 |
*** e0ne has quit IRC | 16:59 | |
openstackgerrit | Merged openstack/releases master: Adjutant first releases will be in Ussuri https://review.opendev.org/683401 | 17:05 |
*** witek has quit IRC | 17:07 | |
openstackgerrit | Merged openstack/releases master: Remove deliverable files for Neutron-LBaaS https://review.opendev.org/684291 | 17:07 |
openstackgerrit | Merged openstack/releases master: Release storlets for train https://review.opendev.org/684733 | 17:08 |
openstackgerrit | Merged openstack/releases master: Make cloudkitty deliverables point to storyboard instead of launchpad https://review.opendev.org/684763 | 17:08 |
openstackgerrit | Merged openstack/releases master: Release cloudkitty 11.0.0 https://review.opendev.org/684764 | 17:09 |
*** amoralej is now known as amoralej|off | 17:13 | |
*** ykarel|away has quit IRC | 17:19 | |
*** udesale has quit IRC | 17:19 | |
openstackgerrit | Merged openstack/releases master: Auto release request for ec2-api https://review.opendev.org/684059 | 17:22 |
*** cdent has quit IRC | 17:24 | |
*** armax has quit IRC | 18:18 | |
*** e0ne has joined #openstack-release | 18:25 | |
*** pkovar has quit IRC | 18:31 | |
*** ekcs has quit IRC | 18:43 | |
*** hberaud is now known as hberaud|gone | 18:46 | |
*** armax has joined #openstack-release | 18:50 | |
*** ricolin has quit IRC | 18:55 | |
*** e0ne has quit IRC | 19:23 | |
*** cdent has joined #openstack-release | 19:41 | |
openstackgerrit | Chris Dent proposed openstack/releases master: Auto release request for placement https://review.opendev.org/684088 | 19:44 |
*** radeks has quit IRC | 20:12 | |
*** diablo_rojo has quit IRC | 20:24 | |
*** ekcs has joined #openstack-release | 20:59 | |
*** diablo_rojo has joined #openstack-release | 21:06 | |
*** cdent has quit IRC | 21:12 | |
*** ekcs has quit IRC | 21:19 | |
*** ekcs has joined #openstack-release | 21:26 | |
*** weshay_bbiab is now known as weshay | 21:27 | |
*** dhellmann_ has joined #openstack-release | 21:27 | |
*** dhellmann has quit IRC | 21:28 | |
*** dhellmann_ is now known as dhellmann | 21:28 | |
openstackgerrit | Akihiro Motoki proposed openstack/releases master: Auto release request for neutron https://review.opendev.org/684084 | 21:39 |
openstackgerrit | Akihiro Motoki proposed openstack/releases master: Auto release request for neutron-tempest-plugin https://review.opendev.org/684016 | 21:41 |
*** diablo_rojo has quit IRC | 22:16 | |
*** diablo_rojo has joined #openstack-release | 22:16 | |
*** ekcs has quit IRC | 22:31 | |
*** ekcs has joined #openstack-release | 22:49 | |
openstackgerrit | Merged openstack/releases master: Auto release request for neutron-tempest-plugin https://review.opendev.org/684016 | 22:50 |
*** mriedem has quit IRC | 22:53 | |
*** dave-mccowan has joined #openstack-release | 23:28 | |
*** dave-mccowan has quit IRC | 23:33 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!