Tuesday, 2021-03-23

*** tosky has quit IRC00:09
*** hamalq has quit IRC00:56
*** rcernin has quit IRC02:28
*** rcernin has joined #openstack-oslo02:39
*** rcernin has quit IRC05:25
*** rcernin has joined #openstack-oslo05:26
*** ralonsoh has joined #openstack-oslo06:26
*** Luzi has joined #openstack-oslo06:34
*** rcernin has quit IRC07:46
*** sboyron has joined #openstack-oslo08:04
*** gokhani has joined #openstack-oslo08:13
*** rpittau|afk is now known as rpittau08:13
*** rcernin has joined #openstack-oslo08:23
*** tosky has joined #openstack-oslo09:04
*** tosky has quit IRC10:13
*** tosky has joined #openstack-oslo10:14
openstackgerritHervé Beraud proposed openstack/oslo.db stable/wallaby: Update .gitreview for stable/wallaby  https://review.opendev.org/c/openstack/oslo.db/+/78136110:21
openstackgerritHervé Beraud proposed openstack/oslo.db stable/wallaby: move flake8 as a pre-commit local target  https://review.opendev.org/c/openstack/oslo.db/+/78240210:21
openstackgerritHervé Beraud proposed openstack/oslo.db stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby  https://review.opendev.org/c/openstack/oslo.db/+/78136210:28
hberaudelod: I think that the we needs bandit fix on wallaby too ^10:39
hberaudc.f these patches10:39
elodhberaud: well, bandit shouldn't have any problem from ussuri and newer branches10:44
elodhberaud: let me check the issue there10:45
hberaudERROR: Could not find a version that satisfies the requirement bandit<1.7.0,>=1.6.010:45
hberaudERROR: No matching distribution found for bandit<1.7.0,>=1.6.010:45
elod$ grep bandit *txt10:45
elodlower-constraints.txt:bandit==1.4.010:45
elodtest-requirements.txt:bandit>=1.6.0,<1.7.0 # Apache-2.010:45
elodthe problem is the contradicting lower constraints10:46
elod(the error printout is a bit misleading)10:46
elodif you bump bandit to 1.6.0 in lower-constraints.txt, then it should be OK10:46
*** rcernin has quit IRC10:46
elodand there will be other conflicts as I see :S10:50
elodhberaud: so these are "caused" (or maybe rather "revealed") by the new pip resolver10:55
elodactually there can be many such issues around projects that had no patches lately11:00
elod(and even here, as I bump bandit, to match with test-req, then we must bump pyyaml version, due to it. and after that, pip resolver is still running for ages, as it is trying to backtrack lots of things....)11:01
*** yoctozepto has quit IRC11:02
*** yoctozepto has joined #openstack-oslo11:03
hberaudack11:06
*** rcernin has joined #openstack-oslo11:09
*** elod has quit IRC11:13
*** rcernin has quit IRC11:17
*** rcernin has joined #openstack-oslo11:18
*** elod has joined #openstack-oslo11:21
*** rcernin has quit IRC11:25
openstackgerritDaniel Bengtsson proposed openstack/automaton master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/automaton/+/78244011:53
*** brinzhang has quit IRC11:56
*** brinzhang has joined #openstack-oslo11:57
openstackgerritDaniel Bengtsson proposed openstack/castellan master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/castellan/+/78244211:59
openstackgerritDaniel Bengtsson proposed openstack/devstack-plugin-amqp1 master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/devstack-plugin-amqp1/+/78244312:00
openstackgerritDaniel Bengtsson proposed openstack/devstack-plugin-kafka master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/devstack-plugin-kafka/+/78244412:00
openstackgerritDaniel Bengtsson proposed openstack/oslo.cache master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.cache/+/78244512:00
openstackgerritDaniel Bengtsson proposed openstack/oslo.config master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.config/+/78244612:01
openstackgerritDaniel Bengtsson proposed openstack/oslo.context master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.context/+/78244712:01
openstackgerritDaniel Bengtsson proposed openstack/oslo.metrics master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.metrics/+/78244812:01
openstackgerritDaniel Bengtsson proposed openstack/oslo.policy master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.policy/+/78244912:02
openstackgerritDaniel Bengtsson proposed openstack/oslo.serialization master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.serialization/+/78245012:02
openstackgerritDaniel Bengtsson proposed openstack/oslo.service master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.service/+/78245112:02
openstackgerritDaniel Bengtsson proposed openstack/oslo.upgradecheck master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.upgradecheck/+/78245212:03
openstackgerritDaniel Bengtsson proposed openstack/oslo.utils master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.utils/+/78245312:03
openstackgerritDaniel Bengtsson proposed openstack/oslo.versionedobjects master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.versionedobjects/+/78245412:04
openstackgerritDaniel Bengtsson proposed openstack/oslo.vmware master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.vmware/+/78245512:04
openstackgerritDaniel Bengtsson proposed openstack/stevedore master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/stevedore/+/78245612:05
openstackgerritDaniel Bengtsson proposed openstack/tooz master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/tooz/+/78245712:05
openstackgerritDaniel Bengtsson proposed openstack/debtcollector master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/debtcollector/+/78245812:06
openstackgerritDaniel Bengtsson proposed openstack/etcd3gw master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/etcd3gw/+/78245912:06
openstackgerritDaniel Bengtsson proposed openstack/futurist master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/futurist/+/78246012:06
openstackgerritDaniel Bengtsson proposed openstack/microversion-parse master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/microversion-parse/+/78246112:07
openstackgerritDaniel Bengtsson proposed openstack/openstack-doc-tools master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/openstack-doc-tools/+/78246212:07
openstackgerritDaniel Bengtsson proposed openstack/openstackdocstheme master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/openstackdocstheme/+/78246312:08
openstackgerritDaniel Bengtsson proposed openstack/os-api-ref master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/os-api-ref/+/78246412:08
openstackgerritDaniel Bengtsson proposed openstack/oslo.concurrency master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.concurrency/+/78246512:08
openstackgerritDaniel Bengtsson proposed openstack/oslo.log master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.log/+/78246612:09
openstackgerritDaniel Bengtsson proposed openstack/oslo.reports master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.reports/+/78246712:09
openstackgerritDaniel Bengtsson proposed openstack/oslo.rootwrap master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.rootwrap/+/78246812:10
openstackgerritDaniel Bengtsson proposed openstack/oslotest master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslotest/+/78246912:10
openstackgerritDaniel Bengtsson proposed openstack/osprofiler master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/osprofiler/+/78247012:10
openstackgerritDaniel Bengtsson proposed openstack/sphinx-feature-classification master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/sphinx-feature-classification/+/78247112:12
openstackgerritDaniel Bengtsson proposed openstack/taskflow master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/taskflow/+/78247212:13
*** hemna has quit IRC12:21
*** hemna has joined #openstack-oslo12:21
*** hemna has quit IRC12:23
*** hemna has joined #openstack-oslo12:23
openstackgerritHervé Beraud proposed openstack/oslo.db stable/wallaby: move flake8 as a pre-commit local target  https://review.opendev.org/c/openstack/oslo.db/+/78240212:29
openstackgerritHervé Beraud proposed openstack/oslo.db stable/wallaby: Update .gitreview for stable/wallaby  https://review.opendev.org/c/openstack/oslo.db/+/78136112:29
openstackgerritHervé Beraud proposed openstack/oslo.db stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby  https://review.opendev.org/c/openstack/oslo.db/+/78136212:29
openstackgerritHervé Beraud proposed openstack/oslo.db stable/wallaby: Upgrade bandit in lower-constraints  https://review.opendev.org/c/openstack/oslo.db/+/78247612:29
openstackgerritDaniel Bengtsson proposed openstack/oslo.limit master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.limit/+/78225713:01
openstackgerritDaniel Bengtsson proposed openstack/oslo.limit master: Fix requirements issues  https://review.opendev.org/c/openstack/oslo.limit/+/76773413:09
openstackgerritDaniel Bengtsson proposed openstack/oslo.limit master: Fix requirements issues  https://review.opendev.org/c/openstack/oslo.limit/+/76773413:11
hberaudmoguimar: please can you review these changes they are needed to unlock our gates on wallaby => https://review.opendev.org/c/openstack/oslo.db/+/77657213:38
moguimarhberaud, ack13:47
hberaudthanks13:48
moguimarhberaud, single approve?13:49
hberaudI already approved once13:49
hberaudyeah so either I single approve it or you final +W13:50
hberaudas you want13:50
hberaudI think we can consider this patch as one of the "dropping lc" story13:51
openstackgerritElod Illes proposed openstack/oslo.db stable/wallaby: Remove contradictions in lower-constraints  https://review.opendev.org/c/openstack/oslo.db/+/78247614:26
elodhberaud: updated your patch to make the lc job pass ^^^^14:26
hberaudack14:27
hberaudthanks14:27
elodnp14:27
elodactually I saw that you removed the lc job from oslo.db. but it seems that is still runs (part of a template maybe)14:28
hberaudelod: however I wonder if your solution isn't simply to backport https://review.opendev.org/c/openstack/oslo.db/+/77657214:28
elodhberaud: no. and I don't know whether it would be enough14:29
hberaudok14:29
hberaudis it possible to deactivate the template part?14:30
hberaudelod: ^14:30
hberaudI suppose that yes by modifying something in project-config or something like that on .zuul.d14:31
elodwell, tbh I don't know the best solution,14:32
elodone option is to set it simply non-voting14:32
elodbut that's not resource efficient14:32
moguimargood job damani, nice coverage in the first pass of patches14:32
elodother way is to remove the template and add its parts only which are required14:33
damanimoguimar, thanks a lot14:33
damanii will answer mail14:34
elodbut maybe it would be better to check with the infra team (who maintain the templates)14:34
damaniand i will continue to work on it14:34
elodhberaud: btw, was there any decision regarding LC jobs at last TC meeting?14:34
elodI haven't checked yet14:34
elodhberaud: I'm asking because if it was decided that lc will be dropped, then it could be removed from the template. otherwise lots of other projects will have the same issue14:36
elodhberaud: I'm just reading: http://eavesdrop.openstack.org/meetings/tc/2021/tc.2021-03-18-15.01.log.html#l-14614:38
hberaudI see14:39
*** Luzi has quit IRC14:53
bnemechberaud: damani: The deadline for signing up for the PTG is in a couple of days and we don't currently have anything but the retrospective. Do we still want to reserve space?14:56
damanibnemec, i think hberaud would like propose topics14:56
damanibnemec, and maybe moguimar i think14:57
damanimoguimar, hberaud, do you confirm ?14:57
hberaudI already propose topics but they are some sort of retrospecitve14:57
moguimarI couldn't really think of any topic14:57
moguimarI think we can keep up with the usual plan14:57
moguimarbook a slot overlaping our weekly meeting14:58
moguimarand do a video call14:58
hberaud+114:58
moguimarso folks can look at each other for a change other than a black screen with text14:58
damaniok15:06
damaniseems good15:07
damanibnemec, moguimar, can i have a review on this patch please https://review.opendev.org/c/openstack/oslo.limit/+/767734 ?15:08
*** ebbex has quit IRC15:35
*** purplerbot has quit IRC15:37
*** ebbex has joined #openstack-oslo15:37
*** irclogbot_3 has quit IRC15:37
*** cognifloyd has quit IRC15:37
*** irclogbot_2 has joined #openstack-oslo15:39
openstackgerritElod Illes proposed openstack/oslo.db stable/wallaby: Unblock wallaby gate  https://review.opendev.org/c/openstack/oslo.db/+/78240215:43
elodhberaud: anyway, I have squashed the two patch together to unblock the gate ^^^15:43
*** adriant7 has joined #openstack-oslo15:44
*** adriant has quit IRC15:46
*** adriant7 is now known as adriant15:46
elodhberaud: I hope you don't mind o:)15:49
hberaudnp15:49
hberaudthank you elod15:49
hberauddamani: FYI ^15:50
*** irclogbot_2 has quit IRC15:51
*** irclogbot_1 has joined #openstack-oslo15:53
damanielod, hberaud, why not remove it ?15:58
damanihberaud, elod, i mean the lower-constraints file15:59
eloddamani hberaud : ohh, I thought openstack-lower-constraints-jobs was removed already when stable/wallaby was cut, but now as I double-checked, it's still there. hberaud , then forget what I said about the job-template stuff :S16:04
damanielod, hberaud, yes still there16:05
eloddamani: anyway, this patch will pass on gate, and now that lc is fixed I think it will be OK (even in long term, I think, but if not, it can be removed later when we see issue)16:06
damanielod, ok16:06
elodbut it's up to you. :) you can remove lc now as well16:07
*** gokhani has quit IRC16:11
*** cognifloyd has joined #openstack-oslo16:16
*** kgiusti has left #openstack-oslo16:54
*** iurygregory has quit IRC17:02
*** rpittau is now known as rpittau|afk17:26
*** iurygregory has joined #openstack-oslo17:29
*** hamalq has joined #openstack-oslo18:18
*** hemna has quit IRC18:39
*** hemna has joined #openstack-oslo18:50
openstackgerritMerged openstack/oslo.reports master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.reports/+/78246718:53
*** sboyron has quit IRC19:22
*** hemna has quit IRC19:25
*** hemna has joined #openstack-oslo19:26
openstackgerritElod Illes proposed openstack/castellan stable/rocky: Cap Bandit below 1.6.0 and update Sphinx requirement  https://review.opendev.org/c/openstack/castellan/+/78259519:57
*** sean-k-mooney has quit IRC20:24
*** rcernin has joined #openstack-oslo21:01
*** rcernin has quit IRC21:16
openstackgerritElod Illes proposed openstack/oslo.log stable/pike: [stable-only] Cap bandit to 1.6.2  https://review.opendev.org/c/openstack/oslo.log/+/78261321:34
*** ralonsoh has quit IRC21:36
*** fnordahl has quit IRC21:37
*** rcernin has joined #openstack-oslo21:46
*** rcernin has quit IRC22:10
*** rcernin has joined #openstack-oslo22:10
*** hoonetorg has joined #openstack-oslo22:53
openstackgerritMerged openstack/oslo.concurrency master: Move flake8 as a pre-commit local target.  https://review.opendev.org/c/openstack/oslo.concurrency/+/78246522:53
*** hamalq has quit IRC23:55

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