Tuesday, 2021-03-23

*** tosky has quit IRC00:09
*** johnthetubaguy has quit IRC00:16
*** HongKong has joined #openstack-release00:17
*** johnthetubaguy has joined #openstack-release00:22
openstackgerritzhongshengping proposed openstack/releases master: Puppet OpenStack wallaby Milestone 3  https://review.opendev.org/c/openstack/releases/+/78160102:18
*** evrardjp has quit IRC03:33
*** evrardjp has joined #openstack-release03:33
*** HongKong has quit IRC03:44
*** bdodd has quit IRC03:57
*** diablo_rojo has quit IRC04:02
*** armstrong has joined #openstack-release04:09
*** vishalmanchanda has joined #openstack-release04:09
*** ykarel has joined #openstack-release04:28
*** whoami-rajat_ has joined #openstack-release04:57
*** EmilienM has quit IRC05:13
*** ykarel has quit IRC05:23
*** ykarel has joined #openstack-release05:25
*** ricolin_ has joined #openstack-release06:17
*** armstrong has quit IRC06:19
*** ricolin_ has quit IRC06:23
*** xarlos has quit IRC07:15
*** slaweq_ has joined #openstack-release07:15
*** eolivare has joined #openstack-release07:37
*** slaweq_ is now known as slaweq07:53
*** e0ne has joined #openstack-release07:57
*** whoami-rajat_ is now known as whoami-rajat08:03
*** sboyron has joined #openstack-release08:04
*** rpittau|afk is now known as rpittau08:13
*** e0ne has quit IRC08:15
openstackgerritHervĂ© Beraud proposed openstack/releases master: Proposing Wallaby RC1 for magnum  https://review.opendev.org/c/openstack/releases/+/78167008:54
*** ykarel is now known as ykarel|lunch08:57
brinzhanghberaud: hi, I found the some patches reported by OpenStack Release Bot, but the pep8 failed, it's said "rsync: change_dir "/home/zuul//src/opendev.org/openstack/cyborg/.tox/pep8/log" failed: No such file or directory (2)"09:02
brinzhanghberaud: do you know what happened? such as https://review.opendev.org/c/openstack/cyborg/+/78164509:02
* hberaud look09:04
*** tosky has joined #openstack-release09:04
hberaudbrinzhang: I think that the rsync error is a side effect triggered by an other issue with tools/check-cherry-picks.sh during the pep8 job execution (`Stable branch requires either cherry-pick -x headers or [stable-only] tag!`)09:10
hberaudI can reproduce the issue locally09:10
hberaudhowever I've no idea of what is the goal of `check-cherry-picks`09:11
brinzhanghberaud: me too, but I dont find howto resolve it now :(09:11
hberaudthis is one of your tools and I don't know what is expected here09:11
hberaudI think that you should debug is execution step by step to see what's the culprit09:12
brinzhangyeah, doing now09:12
hberaudthis is a shell script so maybe by using `set -x` you'll have a better understanding of what's went wrong09:13
hberaudbut I think that's the problem is somewhere here09:14
brinzhangyou mean, using "tox -e pep8 set -x"?09:14
hberaudlet me show you09:14
hberaudbrinzhang: http://paste.openstack.org/show/803814/09:16
brinzhangack, thanks, will try09:16
elodbrinzhang hberaud : i am surprised that cherry-pick-check is in the cyborg repository :)09:17
elodactually it is a tool added to nova repo,09:17
elodto force stable-policy way of working:09:17
hberaudas the goal of https://review.opendev.org/c/openstack/cyborg/+/781645 is to create the right git config for wallaby is stable branch I suppose that your shell script dedicated to stable branch isn't compatible09:17
elod* use cherry-pick -x09:18
elod* always backport to N-1 branch from branch N09:18
hberaudI think that they have copied this script09:18
hberaudhowever some pieces are surely missing09:18
brinzhangelod: yes, I think using this shell in order to using "tox -e fast8", and the raise out of control09:19
brinzhangbut I am not sure, need to be debug09:19
elodbrinzhang: I know the issue09:19
elodbrinzhang: you need [stable-only] tag09:20
hberaudit have been added 6 months ago https://opendev.org/openstack/cyborg/commit/bb4fd104a3cc21ae93d67f1b64910c77bd79c66409:20
elodas that kind of patch is not cherry-picked from master, so it needs a [stable-only] tag09:20
elodin the commit message09:20
hberaudI see09:21
hberaudbrinzhang: so I think you just need to update the commit message of this patch09:21
hberaudaccordingly to elod's explanation ^09:22
hberaudgood to know09:22
hberaudthanks elod for these details :)09:22
brinzhangyou mean just need to update this patch's commit message? such as "Update .gitreview for [stable/wallaby]'09:23
hberaud"[stable-only] Update .gitreview for stable wallaby"09:23
brinzhangok, let me try09:23
elodbrinzhang: yes, e.g.: https://review.opendev.org/c/openstack/nova/+/75431909:24
elodhberaud: :)09:24
brinzhangelod, hberaud: ack, I think yes09:25
brinzhangI will test in my local09:25
elodwell, this script was added by the nova stable cores, so as being a nova stable core I'm familiar with it :)09:25
brinzhangelod, hberaud: yes, it's ok now, may I need to update the shell and add some comments inline.09:26
hberaudbrinzhang: can't hurt09:27
brinzhangelod: thanks, as I said, we think it's useful to us, so I introduced it09:27
brinzhanghberaud: ^^09:28
hberaudhowever https://opendev.org/openstack/cyborg/commit/bb4fd104a3cc21ae93d67f1b64910c77bd79c664 isn't in victoria this is why we didn't see the same error with victoria09:28
elodbrinzhang: no problem :) and yes, it could help to better track backports :)09:28
*** vishalmanchanda has quit IRC09:28
brinzhanghberaud: yes, we introduced it in wallaby release09:30
brinzhangelod: agree. thank you very much elod, hberaud, I will update the commit message and allow the Bot commit to go09:30
elodbrinzhang: although I suggest to compare the script with nova, as there were some fixes (nova (master)$ git log --oneline -- tools/check-cherry-picks.sh)09:33
elodbrinzhang: might worth to cherry-pick to cyborg those fixes09:33
brinzhangelod: ack, will diff later, and adding the update info09:33
elodbrinzhang: and also, this is really for stable branches, so needs to be backported all the way to stable branches09:34
brinzhangelod: there are also some json and arqs.inc changed, I am not sure it's easy to backport, but I will try ^^09:36
elodbrinzhang: I recommend then to backport only the script (with a [stable-only] tag :))09:41
elodbrinzhang: and feel free to ping me if you need help with this check script (or other stable reviews)09:43
brinzhangelod: ok, after I backport that commit, I will add you to review and ping you here :)09:44
elodgood-good :)09:53
brinzhanghberaud: did you remember the python-cyborgclient 1.3.1.dev1 required error? it has happened again in https://review.opendev.org/c/openstack/python-cyborgclient/+/781789, the Bot patch, it seems https://review.opendev.org/c/openstack/python-cyborgclient/+/781603 didnt fix that unknown issue09:54
hberaudYou didn't backport https://review.opendev.org/c/openstack/python-cyborgclient/+/781603 on stable/wallaby09:55
hberaudhttps://review.opendev.org/c/openstack/python-cyborgclient/+/781603 is only on master09:55
brinzhangwow, yes09:56
brinzhangI ignored it, her fix was after wallaby was released.09:58
*** ykarel|lunch is now known as ykarel10:01
openstackgerritMerged openstack/releases master: Proposing Wallaby RC1 for tacker-horizon  https://review.opendev.org/c/openstack/releases/+/78176010:12
*** tosky has quit IRC10:13
*** tosky has joined #openstack-release10:14
*** e0ne has joined #openstack-release10:16
brinzhangelod: I was backport that patch to victoria, and diff the updated info from nova/tools, and added you as the reviewer, if you are free pls review it10:35
brinzhangelod: I was cherry-picked until stable/ussuri, others branch need to be check the conflicts after we merged the ussuri backport10:36
elodbrinzhang: ack, I will review them10:44
brinzhangelod: thanks10:44
*** yoctozepto has quit IRC11:02
*** yoctozepto has joined #openstack-release11:03
*** vishalmanchanda has joined #openstack-release11:11
*** elod has quit IRC11:13
*** elod has joined #openstack-release11:21
*** brinzhang has quit IRC11:56
*** brinzhang has joined #openstack-release11:57
*** EmilienM has joined #openstack-release12:05
*** e0ne has quit IRC12:36
*** ykarel has quit IRC12:38
*** ykarel has joined #openstack-release12:38
*** nweinber has joined #openstack-release12:39
*** ykarel_ has joined #openstack-release12:44
*** ykarel has quit IRC12:47
*** mgoddard has quit IRC12:53
*** ykarel_ is now known as ykarel13:17
*** ykarel_ has joined #openstack-release13:22
*** ykarel has quit IRC13:24
*** ykarel_ has quit IRC13:26
openstackgerritHervĂ© Beraud proposed openstack/releases master: Proposing Wallaby RC1 for monasca-transform  https://review.opendev.org/c/openstack/releases/+/78169613:32
*** e0ne has joined #openstack-release13:44
*** bdodd has joined #openstack-release13:45
*** mgoddard has joined #openstack-release13:57
*** e0ne has quit IRC14:00
*** whoami-rajat has quit IRC15:03
*** frickler has quit IRC15:37
*** irclogbot_2 has quit IRC15:37
*** irclogbot_3 has quit IRC15:37
*** frickler has joined #openstack-release15:37
*** dirk has quit IRC15:38
*** dirk has joined #openstack-release15:39
*** irclogbot_1 has joined #openstack-release15:39
*** irclogbot_2 has joined #openstack-release15:39
*** adriant7 has joined #openstack-release15:44
*** adriant has quit IRC15:46
*** adriant7 is now known as adriant15:46
*** irclogbot_1 has quit IRC15:49
*** irclogbot_2 has quit IRC15:51
*** irclogbot_1 has joined #openstack-release15:53
*** irclogbot_2 has joined #openstack-release15:53
openstackgerritMerged openstack/releases master: Proposing Wallaby RC1 for kuryr-libnetwork  https://review.opendev.org/c/openstack/releases/+/78166416:53
openstackgerritMerged openstack/releases master: Proposing Wallaby RC1 for glance  https://review.opendev.org/c/openstack/releases/+/78165317:00
*** iurygregory has quit IRC17:02
*** rpittau is now known as rpittau|afk17:26
openstackgerritMerged openstack/releases master: trivial - fix week number  https://review.opendev.org/c/openstack/releases/+/78107117:26
*** armstrong has joined #openstack-release17:28
*** iurygregory has joined #openstack-release17:29
*** vishalmanchanda has quit IRC18:17
*** eolivare has quit IRC18:24
*** dtantsur is now known as dtantsur|afk18:55
*** dtantsur|afk is now known as dtantsur|afk|afk18:55
*** dtantsur|afk|afk is now known as dtantsur|afk18:55
*** sboyron has quit IRC19:22
*** armstrong has quit IRC19:38
*** e0ne has joined #openstack-release20:36
*** nweinber has quit IRC21:05
*** e0ne has quit IRC21:31
*** e0ne has joined #openstack-release21:32
*** e0ne has quit IRC21:35
*** fnordahl has quit IRC21:37
*** jbadiapa has quit IRC21:45
*** slaweq has quit IRC21:52
*** slaweq has joined #openstack-release21:54
*** diablo_rojo has joined #openstack-release21:56
*** slaweq has quit IRC23:05
*** slaweq has joined #openstack-release23:06
*** johnthetubaguy has quit IRC23:36
*** johnthetubaguy has joined #openstack-release23:38
*** johnthetubaguy has quit IRC23:49
*** johnthetubaguy has joined #openstack-release23:57

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!