*** tosky has quit IRC | 00:09 | |
*** hamalq has quit IRC | 00:56 | |
*** rcernin has quit IRC | 02:28 | |
*** rcernin has joined #openstack-oslo | 02:39 | |
*** rcernin has quit IRC | 05:25 | |
*** rcernin has joined #openstack-oslo | 05:26 | |
*** ralonsoh has joined #openstack-oslo | 06:26 | |
*** Luzi has joined #openstack-oslo | 06:34 | |
*** rcernin has quit IRC | 07:46 | |
*** sboyron has joined #openstack-oslo | 08:04 | |
*** gokhani has joined #openstack-oslo | 08:13 | |
*** rpittau|afk is now known as rpittau | 08:13 | |
*** rcernin has joined #openstack-oslo | 08:23 | |
*** tosky has joined #openstack-oslo | 09:04 | |
*** tosky has quit IRC | 10:13 | |
*** tosky has joined #openstack-oslo | 10:14 | |
openstackgerrit | Hervé Beraud proposed openstack/oslo.db stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/oslo.db/+/781361 | 10:21 |
---|---|---|
openstackgerrit | Hervé Beraud proposed openstack/oslo.db stable/wallaby: move flake8 as a pre-commit local target https://review.opendev.org/c/openstack/oslo.db/+/782402 | 10:21 |
openstackgerrit | Hervé Beraud proposed openstack/oslo.db stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/oslo.db/+/781362 | 10:28 |
hberaud | elod: I think that the we needs bandit fix on wallaby too ^ | 10:39 |
hberaud | c.f these patches | 10:39 |
elod | hberaud: well, bandit shouldn't have any problem from ussuri and newer branches | 10:44 |
elod | hberaud: let me check the issue there | 10:45 |
hberaud | ERROR: Could not find a version that satisfies the requirement bandit<1.7.0,>=1.6.0 | 10:45 |
hberaud | ERROR: No matching distribution found for bandit<1.7.0,>=1.6.0 | 10:45 |
elod | $ grep bandit *txt | 10:45 |
elod | lower-constraints.txt:bandit==1.4.0 | 10:45 |
elod | test-requirements.txt:bandit>=1.6.0,<1.7.0 # Apache-2.0 | 10:45 |
elod | the problem is the contradicting lower constraints | 10:46 |
elod | (the error printout is a bit misleading) | 10:46 |
elod | if you bump bandit to 1.6.0 in lower-constraints.txt, then it should be OK | 10:46 |
*** rcernin has quit IRC | 10:46 | |
elod | and there will be other conflicts as I see :S | 10:50 |
elod | hberaud: so these are "caused" (or maybe rather "revealed") by the new pip resolver | 10:55 |
elod | actually there can be many such issues around projects that had no patches lately | 11: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 IRC | 11:02 | |
*** yoctozepto has joined #openstack-oslo | 11:03 | |
hberaud | ack | 11:06 |
*** rcernin has joined #openstack-oslo | 11:09 | |
*** elod has quit IRC | 11:13 | |
*** rcernin has quit IRC | 11:17 | |
*** rcernin has joined #openstack-oslo | 11:18 | |
*** elod has joined #openstack-oslo | 11:21 | |
*** rcernin has quit IRC | 11:25 | |
openstackgerrit | Daniel Bengtsson proposed openstack/automaton master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/automaton/+/782440 | 11:53 |
*** brinzhang has quit IRC | 11:56 | |
*** brinzhang has joined #openstack-oslo | 11:57 | |
openstackgerrit | Daniel Bengtsson proposed openstack/castellan master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/castellan/+/782442 | 11:59 |
openstackgerrit | Daniel Bengtsson proposed openstack/devstack-plugin-amqp1 master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/devstack-plugin-amqp1/+/782443 | 12:00 |
openstackgerrit | Daniel Bengtsson proposed openstack/devstack-plugin-kafka master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/devstack-plugin-kafka/+/782444 | 12:00 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.cache master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.cache/+/782445 | 12:00 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.config master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.config/+/782446 | 12:01 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.context master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.context/+/782447 | 12:01 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.metrics master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.metrics/+/782448 | 12:01 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.policy master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.policy/+/782449 | 12:02 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.serialization master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.serialization/+/782450 | 12:02 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.service master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.service/+/782451 | 12:02 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.upgradecheck master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.upgradecheck/+/782452 | 12:03 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.utils master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.utils/+/782453 | 12:03 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.versionedobjects master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.versionedobjects/+/782454 | 12:04 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.vmware master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.vmware/+/782455 | 12:04 |
openstackgerrit | Daniel Bengtsson proposed openstack/stevedore master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/stevedore/+/782456 | 12:05 |
openstackgerrit | Daniel Bengtsson proposed openstack/tooz master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/tooz/+/782457 | 12:05 |
openstackgerrit | Daniel Bengtsson proposed openstack/debtcollector master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/debtcollector/+/782458 | 12:06 |
openstackgerrit | Daniel Bengtsson proposed openstack/etcd3gw master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/etcd3gw/+/782459 | 12:06 |
openstackgerrit | Daniel Bengtsson proposed openstack/futurist master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/futurist/+/782460 | 12:06 |
openstackgerrit | Daniel Bengtsson proposed openstack/microversion-parse master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/microversion-parse/+/782461 | 12:07 |
openstackgerrit | Daniel Bengtsson proposed openstack/openstack-doc-tools master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/openstack-doc-tools/+/782462 | 12:07 |
openstackgerrit | Daniel Bengtsson proposed openstack/openstackdocstheme master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/openstackdocstheme/+/782463 | 12:08 |
openstackgerrit | Daniel Bengtsson proposed openstack/os-api-ref master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/os-api-ref/+/782464 | 12:08 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.concurrency master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.concurrency/+/782465 | 12:08 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.log master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.log/+/782466 | 12:09 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.reports master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.reports/+/782467 | 12:09 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.rootwrap master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.rootwrap/+/782468 | 12:10 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslotest master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslotest/+/782469 | 12:10 |
openstackgerrit | Daniel Bengtsson proposed openstack/osprofiler master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/osprofiler/+/782470 | 12:10 |
openstackgerrit | Daniel Bengtsson proposed openstack/sphinx-feature-classification master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/sphinx-feature-classification/+/782471 | 12:12 |
openstackgerrit | Daniel Bengtsson proposed openstack/taskflow master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/taskflow/+/782472 | 12:13 |
*** hemna has quit IRC | 12:21 | |
*** hemna has joined #openstack-oslo | 12:21 | |
*** hemna has quit IRC | 12:23 | |
*** hemna has joined #openstack-oslo | 12:23 | |
openstackgerrit | Hervé Beraud proposed openstack/oslo.db stable/wallaby: move flake8 as a pre-commit local target https://review.opendev.org/c/openstack/oslo.db/+/782402 | 12:29 |
openstackgerrit | Hervé Beraud proposed openstack/oslo.db stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/oslo.db/+/781361 | 12:29 |
openstackgerrit | Hervé Beraud proposed openstack/oslo.db stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/oslo.db/+/781362 | 12:29 |
openstackgerrit | Hervé Beraud proposed openstack/oslo.db stable/wallaby: Upgrade bandit in lower-constraints https://review.opendev.org/c/openstack/oslo.db/+/782476 | 12:29 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.limit master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.limit/+/782257 | 13:01 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.limit master: Fix requirements issues https://review.opendev.org/c/openstack/oslo.limit/+/767734 | 13:09 |
openstackgerrit | Daniel Bengtsson proposed openstack/oslo.limit master: Fix requirements issues https://review.opendev.org/c/openstack/oslo.limit/+/767734 | 13:11 |
hberaud | moguimar: please can you review these changes they are needed to unlock our gates on wallaby => https://review.opendev.org/c/openstack/oslo.db/+/776572 | 13:38 |
moguimar | hberaud, ack | 13:47 |
hberaud | thanks | 13:48 |
moguimar | hberaud, single approve? | 13:49 |
hberaud | I already approved once | 13:49 |
hberaud | yeah so either I single approve it or you final +W | 13:50 |
hberaud | as you want | 13:50 |
hberaud | I think we can consider this patch as one of the "dropping lc" story | 13:51 |
openstackgerrit | Elod Illes proposed openstack/oslo.db stable/wallaby: Remove contradictions in lower-constraints https://review.opendev.org/c/openstack/oslo.db/+/782476 | 14:26 |
elod | hberaud: updated your patch to make the lc job pass ^^^^ | 14:26 |
hberaud | ack | 14:27 |
hberaud | thanks | 14:27 |
elod | np | 14:27 |
elod | actually 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 |
hberaud | elod: however I wonder if your solution isn't simply to backport https://review.opendev.org/c/openstack/oslo.db/+/776572 | 14:28 |
elod | hberaud: no. and I don't know whether it would be enough | 14:29 |
hberaud | ok | 14:29 |
hberaud | is it possible to deactivate the template part? | 14:30 |
hberaud | elod: ^ | 14:30 |
hberaud | I suppose that yes by modifying something in project-config or something like that on .zuul.d | 14:31 |
elod | well, tbh I don't know the best solution, | 14:32 |
elod | one option is to set it simply non-voting | 14:32 |
elod | but that's not resource efficient | 14:32 |
moguimar | good job damani, nice coverage in the first pass of patches | 14:32 |
elod | other way is to remove the template and add its parts only which are required | 14:33 |
damani | moguimar, thanks a lot | 14:33 |
damani | i will answer mail | 14:34 |
elod | but maybe it would be better to check with the infra team (who maintain the templates) | 14:34 |
damani | and i will continue to work on it | 14:34 |
elod | hberaud: btw, was there any decision regarding LC jobs at last TC meeting? | 14:34 |
elod | I haven't checked yet | 14:34 |
elod | hberaud: 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 issue | 14:36 |
elod | hberaud: I'm just reading: http://eavesdrop.openstack.org/meetings/tc/2021/tc.2021-03-18-15.01.log.html#l-146 | 14:38 |
hberaud | I see | 14:39 |
*** Luzi has quit IRC | 14:53 | |
bnemec | hberaud: 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 |
damani | bnemec, i think hberaud would like propose topics | 14:56 |
damani | bnemec, and maybe moguimar i think | 14:57 |
damani | moguimar, hberaud, do you confirm ? | 14:57 |
hberaud | I already propose topics but they are some sort of retrospecitve | 14:57 |
moguimar | I couldn't really think of any topic | 14:57 |
moguimar | I think we can keep up with the usual plan | 14:57 |
moguimar | book a slot overlaping our weekly meeting | 14:58 |
moguimar | and do a video call | 14:58 |
hberaud | +1 | 14:58 |
moguimar | so folks can look at each other for a change other than a black screen with text | 14:58 |
damani | ok | 15:06 |
damani | seems good | 15:07 |
damani | bnemec, moguimar, can i have a review on this patch please https://review.opendev.org/c/openstack/oslo.limit/+/767734 ? | 15:08 |
*** ebbex has quit IRC | 15:35 | |
*** purplerbot has quit IRC | 15:37 | |
*** ebbex has joined #openstack-oslo | 15:37 | |
*** irclogbot_3 has quit IRC | 15:37 | |
*** cognifloyd has quit IRC | 15:37 | |
*** irclogbot_2 has joined #openstack-oslo | 15:39 | |
openstackgerrit | Elod Illes proposed openstack/oslo.db stable/wallaby: Unblock wallaby gate https://review.opendev.org/c/openstack/oslo.db/+/782402 | 15:43 |
elod | hberaud: anyway, I have squashed the two patch together to unblock the gate ^^^ | 15:43 |
*** adriant7 has joined #openstack-oslo | 15:44 | |
*** adriant has quit IRC | 15:46 | |
*** adriant7 is now known as adriant | 15:46 | |
elod | hberaud: I hope you don't mind o:) | 15:49 |
hberaud | np | 15:49 |
hberaud | thank you elod | 15:49 |
hberaud | damani: FYI ^ | 15:50 |
*** irclogbot_2 has quit IRC | 15:51 | |
*** irclogbot_1 has joined #openstack-oslo | 15:53 | |
damani | elod, hberaud, why not remove it ? | 15:58 |
damani | hberaud, elod, i mean the lower-constraints file | 15:59 |
elod | damani 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 :S | 16:04 |
damani | elod, hberaud, yes still there | 16:05 |
elod | damani: 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 |
damani | elod, ok | 16:06 |
elod | but it's up to you. :) you can remove lc now as well | 16:07 |
*** gokhani has quit IRC | 16:11 | |
*** cognifloyd has joined #openstack-oslo | 16:16 | |
*** kgiusti has left #openstack-oslo | 16:54 | |
*** iurygregory has quit IRC | 17:02 | |
*** rpittau is now known as rpittau|afk | 17:26 | |
*** iurygregory has joined #openstack-oslo | 17:29 | |
*** hamalq has joined #openstack-oslo | 18:18 | |
*** hemna has quit IRC | 18:39 | |
*** hemna has joined #openstack-oslo | 18:50 | |
openstackgerrit | Merged openstack/oslo.reports master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.reports/+/782467 | 18:53 |
*** sboyron has quit IRC | 19:22 | |
*** hemna has quit IRC | 19:25 | |
*** hemna has joined #openstack-oslo | 19:26 | |
openstackgerrit | Elod Illes proposed openstack/castellan stable/rocky: Cap Bandit below 1.6.0 and update Sphinx requirement https://review.opendev.org/c/openstack/castellan/+/782595 | 19:57 |
*** sean-k-mooney has quit IRC | 20:24 | |
*** rcernin has joined #openstack-oslo | 21:01 | |
*** rcernin has quit IRC | 21:16 | |
openstackgerrit | Elod Illes proposed openstack/oslo.log stable/pike: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/oslo.log/+/782613 | 21:34 |
*** ralonsoh has quit IRC | 21:36 | |
*** fnordahl has quit IRC | 21:37 | |
*** rcernin has joined #openstack-oslo | 21:46 | |
*** rcernin has quit IRC | 22:10 | |
*** rcernin has joined #openstack-oslo | 22:10 | |
*** hoonetorg has joined #openstack-oslo | 22:53 | |
openstackgerrit | Merged openstack/oslo.concurrency master: Move flake8 as a pre-commit local target. https://review.opendev.org/c/openstack/oslo.concurrency/+/782465 | 22:53 |
*** hamalq has quit IRC | 23:55 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!