opendevreview | Merged openstack/releases master: Proposing Xena RC1 for ec2-api https://review.opendev.org/c/openstack/releases/+/808617 | 01:28 |
---|---|---|
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for aodh https://review.opendev.org/c/openstack/releases/+/808598 | 01:52 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for ceilometer https://review.opendev.org/c/openstack/releases/+/808606 | 01:52 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for heat https://review.opendev.org/c/openstack/releases/+/808629 | 01:52 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for heat-agents https://review.opendev.org/c/openstack/releases/+/808626 | 01:52 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for masakari-monitors https://review.opendev.org/c/openstack/releases/+/808641 | 01:53 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for networking-hyperv https://review.opendev.org/c/openstack/releases/+/808686 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for magnum-ui https://review.opendev.org/c/openstack/releases/+/808635 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for magnum https://review.opendev.org/c/openstack/releases/+/808636 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for zun-ui https://review.opendev.org/c/openstack/releases/+/808741 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for zun https://review.opendev.org/c/openstack/releases/+/808742 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for networking-bgpvpn https://review.opendev.org/c/openstack/releases/+/808683 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for networking-bagpipe https://review.opendev.org/c/openstack/releases/+/808681 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for networking-odl https://review.opendev.org/c/openstack/releases/+/808688 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for neutron-dynamic-routing https://review.opendev.org/c/openstack/releases/+/808690 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for cyborg https://review.opendev.org/c/openstack/releases/+/808613 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for compute-hyperv https://review.opendev.org/c/openstack/releases/+/808611 | 02:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for trove https://review.opendev.org/c/openstack/releases/+/808735 | 02:25 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for trove-dashboard https://review.opendev.org/c/openstack/releases/+/808734 | 02:25 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for kuryr-libnetwork https://review.opendev.org/c/openstack/releases/+/808634 | 02:25 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for kuryr-kubernetes https://review.opendev.org/c/openstack/releases/+/808633 | 02:25 |
*** ysandeep|out is now known as ysandeep | 04:18 | |
*** hberaud_ is now known as hberaud | 08:31 | |
hberaud | Info about the latest job failure: the pipeline failed with `tag-releases`, however, the git tag, the pypi version and the tarball have been properly created. https://tarballs.opendev.org/openstack/kuryr-kubernetes/kuryr-kubernetes-5.0.0.0rc1.tar.gz https://opendev.org/openstack/kuryr-kubernetes/src/tag/5.0.0.0rc1 https://pypi.org/project/kuryr-kubernetes/5.0.0.0rc1/ . | 08:48 |
hberaud | I'm sure what is exaclty the error. The doc job has been skipped. Can we reenqueue this job to update the doc accordingly to the latest version of this deliverable? (@fungi) | 08:52 |
hberaud | the failing job: https://zuul.opendev.org/t/openstack/build/de8986f2fc3146dc90507955580e51ec the release job failure announce: http://lists.openstack.org/pipermail/release-job-failures/2021-September/001566.html | 08:54 |
elodilles | hberaud: it seems that the 'Add Python3 yoga unit tests' patch creation failed as kuryr-kubernetes has a gitignore rule that ignores hidden directories (thus .zuul.d - https://opendev.org/openstack/kuryr-kubernetes/src/branch/master/.gitignore#L49 ) | 09:12 |
elodilles | hberaud: i've uploaded the change + fix (to avoid the same failure in the future): https://review.opendev.org/c/openstack/kuryr-kubernetes/+/809160 | 09:25 |
hberaud | elodilles: excellent thank you | 09:36 |
elodilles | np, i was curious what caused the problem and finally i spotted it :) | 09:49 |
opendevreview | Lajos Katona proposed openstack/releases master: Proposing Xena RC1 for neutron https://review.opendev.org/c/openstack/releases/+/808693 | 10:03 |
*** ykarel_ is now known as ykarel | 11:11 | |
opendevreview | Lajos Katona proposed openstack/releases master: Proposing Xena RC1 for neutron https://review.opendev.org/c/openstack/releases/+/808693 | 11:21 |
opendevreview | Lajos Katona proposed openstack/releases master: Proposing Xena RC1 for neutron https://review.opendev.org/c/openstack/releases/+/808693 | 11:26 |
*** ykarel_ is now known as ykarel | 11:28 | |
fungi | elodilles: good eye! | 12:04 |
fungi | hberaud: they'll need to re-tag with the new fix, looks like | 12:04 |
fungi | at least i don't see any 5.0.0.0rc1 in openstack/kuryr-kubernetes yet | 12:04 |
opendevreview | Merged openstack/releases master: Release Glance stable branches https://review.opendev.org/c/openstack/releases/+/808803 | 12:05 |
*** ysandeep is now known as ysandeep|brb | 12:12 | |
elodilles | fungi: :) | 12:32 |
elodilles | fungi: actually the tagging went well i think | 12:33 |
elodilles | fungi: see hberaud's link | 12:33 |
elodilles | * links | 12:33 |
elodilles | it is there both @ opendev and @ pypi | 12:34 |
opendevreview | Lajos Katona proposed openstack/releases master: [vpnaas] Transition stein, rocky, queens to EOL https://review.opendev.org/c/openstack/releases/+/809180 | 12:43 |
hberaud | yes https://opendev.org/openstack/kuryr-kubernetes/src/tag/5.0.0.0rc1 | 12:44 |
*** amoralej is now known as amoralej|lunch | 12:45 | |
fungi | interesting, i didn't see the tag when i looked earlier | 13:01 |
fungi | https://opendev.org/openstack/kuryr-kubernetes/tags | 13:02 |
fungi | oh, nevermind, it's in the tags drop-down selector | 13:02 |
hberaud | np | 13:02 |
fungi | i'll see if i can dig into why gitea's "tags" page is so incomplete | 13:03 |
hberaud | ok | 13:04 |
armstrong | Hello hberaud | 13:06 |
hberaud | armstrong: o/ | 13:06 |
armstrong | You did this week tasks already ? | 13:06 |
hberaud | yes | 13:08 |
armstrong | Ok | 13:08 |
*** ysandeep|brb is now known as ysandeep | 13:08 | |
hberaud | I created this series on Monday https://review.opendev.org/q/topic:"xena-rc1-deadline" | 13:08 |
*** ykarel is now known as ykarel|afk | 13:18 | |
vishalmanchanda | hberaud: Hi herve if a project follows cycle-with-intermediary release model then after which week it cannot accept any code changes. | 13:19 |
hberaud | vishalmanchanda: Feature freeze for CWI was on R-5, we are on R-3. However you can still propose bugfix by only requesting a RFE because we past the requirement freeze too. | 13:22 |
vishalmanchanda | hberaud: Do we need to request for RFE for trivial changes as well? | 13:24 |
vishalmanchanda | hberaud: I am asking because we are merging a series of patches to add Django3.2 support in horizon(xena). | 13:24 |
*** amoralej|lunch is now known as amoralej | 13:25 | |
vishalmanchanda | hberaud: I hope it's fine from release point of view. | 13:25 |
opendevreview | Lajos Katona proposed openstack/releases master: [vpnaas] Transition stein, rocky, queens to EOL https://review.opendev.org/c/openstack/releases/+/809180 | 13:50 |
smcginnis | vishalmanchanda: That sounds like a somewhat major change that could introduce a lot of risk at this point in the release. | 13:59 |
smcginnis | vishalmanchanda: If it's needed to even work with current horizon, then a case could be made for it. | 14:00 |
smcginnis | vishalmanchanda: But otherwise that sounds like something better to wait until Y development opens up so you have the whole cycle to sort out any issues. | 14:00 |
vishalmanchanda | smcginnis: ok, fair enough thanks for response. | 14:03 |
smcginnis | np | 14:03 |
*** ykarel|afk is now known as ykarel | 14:07 | |
vishalmanchanda | smcginnis: I have one more question what if we already merged few of them like https://review.opendev.org/q/topic:%2522django3-support%2522+status:merged | 14:10 |
vishalmanchanda | smcginnis: these are trivial changes and doesn't impact current implementation, as I have also tested locally. | 14:11 |
vishalmanchanda | smcginnis: Should I revert them now, please suggest? | 14:12 |
vishalmanchanda | smcginnis: I have merged above patches because We got a mail on open-discuss about Horizon Wallaby and Xena completely broken on Debian http://lists.openstack.org/pipermail/openstack-discuss/2021-September/024873.html | 14:15 |
opendevreview | Gregory Thiemonge proposed openstack/releases master: Proposing Xena RC1 for octavia https://review.opendev.org/c/openstack/releases/+/808710 | 14:26 |
opendevreview | Takashi Kajinami proposed openstack/releases master: Proposing Xena RC1 for storlets https://review.opendev.org/c/openstack/releases/+/808731 | 14:50 |
opendevreview | Radosław Piliszek proposed openstack/releases master: Release final python-masakariclient for Xena https://review.opendev.org/c/openstack/releases/+/808949 | 14:52 |
opendevreview | Abhishek Kekane proposed openstack/releases master: Release glance-tempest-plugin for Xena https://review.opendev.org/c/openstack/releases/+/808782 | 15:08 |
whoami-rajat | hi #openstack-release , I have a doubt regarding stable branch releases, if we have released a branch N, which doesn't contain a fix, and later we are releasing N-1 with that fix, is it acceptable? I don't see anything regarding that mentioned in the stable branch doc https://docs.openstack.org/project-team-guide/stable-branches.html | 15:14 |
whoami-rajat | smcginnis, hberaud elodilles ^^ | 15:15 |
*** ykarel is now known as ykarel|away | 15:20 | |
*** ysandeep is now known as ysandeep|away | 15:23 | |
tosky | just to be clear, in the case above, the fix is both in N and N-1, it's just that N-1 with the fix is released before N with the fix - is that acceptable? | 15:32 |
whoami-rajat | my wordings might not be clear, when i say "doesn't contain the fix" i mean the released version and not the branch itself | 15:49 |
elodilles | whoami-rajat: this is a valid concern. the best is to release in a way that this situation does not happen | 16:00 |
opendevreview | Dmitriy Rabotyagov proposed openstack/releases master: Release OpenStack-Ansible Wallaby https://review.opendev.org/c/openstack/releases/+/808790 | 16:01 |
*** marios is now known as marios|out | 16:01 | |
elodilles | whoami-rajat: for example if the releases (N, N-1, N-2) happen at more or less at the same time, then probably this problematic case should not happen | 16:02 |
elodilles | whoami-rajat: or, if you see that an N-1 release would contain such fixes that are not yet released on N, then release N as well (again, if needed) | 16:04 |
whoami-rajat | elodilles, that's what my thought was, to release in order of N then N-1, and N-1 should not be released with changes not part of N release | 16:05 |
whoami-rajat | elodilles, can we add that point in the stable maint doc if everyone agrees it's the right approach? | 16:05 |
whoami-rajat | s/everyone/release team | 16:05 |
elodilles | sometimes maybe it is acceptable, but best is to avoid such cases, at least that is my understanding | 16:06 |
elodilles | yes, we can add a note to stable policy that the best is to avoid this issue. it does not harm if it is explained | 16:07 |
whoami-rajat | for critical cases i agree but in general for normal patches, i think it just creates mess and would become a problem while upgrading | 16:07 |
whoami-rajat | ok, i will propose a patch for that, thanks for the info elodilles | 16:08 |
elodilles | whoami-rajat: cool, thanks, please add me as reviewer | 16:09 |
whoami-rajat | elodilles, sure, thanks | 16:09 |
*** amoralej is now known as amoralej|off | 16:13 | |
hberaud | vishalmanchanda: stable branch isn't yet cut for horizon so, I would argue that you could leave these commits as they are, and we will cut your stable branch on your latest tag (2.0.0) (https://review.opendev.org/q/topic:%2522django3-support%2522+status:merged) | 16:19 |
hberaud | vishalmanchanda: in other words, when we will switch your master from xena to yoga your patches will be only available on yoga | 16:20 |
hberaud | vishalmanchanda: it will avoid you to miss with revert and revert of revert | 16:21 |
hberaud | s/miss/mess/ | 16:22 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for ansible-role-atos-hsm https://review.opendev.org/c/openstack/releases/+/808594 | 16:39 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for ansible-role-lunasa-hsm https://review.opendev.org/c/openstack/releases/+/808596 | 16:48 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for watcher-dashboard https://review.opendev.org/c/openstack/releases/+/808736 | 16:49 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for watcher https://review.opendev.org/c/openstack/releases/+/808737 | 16:57 |
vishalmanchanda | hberaud: ack. | 17:00 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for zaqar-ui https://review.opendev.org/c/openstack/releases/+/808738 | 17:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for zaqar https://review.opendev.org/c/openstack/releases/+/808739 | 17:04 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for neutron https://review.opendev.org/c/openstack/releases/+/808693 | 17:23 |
opendevreview | Merged openstack/releases master: Proposing Xena RC1 for storlets https://review.opendev.org/c/openstack/releases/+/808731 | 17:35 |
opendevreview | Abhishek Kekane proposed openstack/releases master: Proposing Xena RC1 for glance https://review.opendev.org/c/openstack/releases/+/808625 | 17:42 |
opendevreview | Merged openstack/releases master: Release OpenStack-Ansible Victoria https://review.opendev.org/c/openstack/releases/+/808924 | 17:44 |
opendevreview | Merged openstack/releases master: Release OpenStack-Ansible Ussuri https://review.opendev.org/c/openstack/releases/+/808785 | 17:44 |
prometheanfire | hberaud: does ceilometer have a freeze exception? iirc it should, they are used as a lib (like horizon) | 18:43 |
opendevreview | Douglas Mendizábal proposed openstack/releases master: Release Xena RC1 for barbican https://review.opendev.org/c/openstack/releases/+/808600 | 18:45 |
opendevreview | Douglas Mendizábal proposed openstack/releases master: Release Xena RC1 for barbican https://review.opendev.org/c/openstack/releases/+/808600 | 19:04 |
*** osmanlicilegi is now known as Guest0 | 21:20 | |
*** prometheanfire is now known as Guest1 | 21:20 | |
*** promethe- is now known as prometheanfire | 21:53 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!