Wednesday, 2019-07-10

*** jamesmcarthur has joined #openstack-infra00:03
mordredmmm. brunswick stew00:03
*** bobh has quit IRC00:05
clarkbrouter_solicitation_delay maybe we can set that sysctl to delay things long enough for the test to get NM up00:05
clarkbthe default is 1 I think this may be the race00:05
mordredyeah - seems reasonable00:05
clarkbbasically interface driver/hardware is brought up in kernel then we have 1 second + transmit time of ra and processing time to get NM up00:05
clarkband within qemu slowness that is hard00:05
clarkbon a normal machine its much quicker00:06
clarkbI'll make a new image tomorrow with that value set to ~30 and try again00:06
*** jamesmcarthur has quit IRC00:07
*** ijw has quit IRC00:07
*** slaweq has joined #openstack-infra00:08
*** bobh has joined #openstack-infra00:12
*** slaweq has quit IRC00:13
*** bobh has quit IRC00:17
*** jamesmcarthur has joined #openstack-infra00:21
*** rlandy has quit IRC00:25
*** altlogbot_3 has joined #openstack-infra00:28
*** zbr has quit IRC00:34
*** altlogbot_3 has quit IRC00:37
*** jamesmcarthur has quit IRC00:38
*** zbr has joined #openstack-infra00:43
*** weifan has quit IRC00:47
*** jamesmcarthur has joined #openstack-infra00:47
*** weifan has joined #openstack-infra00:48
*** weifan has quit IRC00:52
*** jamesmcarthur has quit IRC00:52
*** bobh has joined #openstack-infra00:52
*** jamesmcarthur has joined #openstack-infra00:55
*** bobh has quit IRC00:58
*** mriedem has quit IRC00:59
*** yamamoto has joined #openstack-infra01:00
*** altlogbot_1 has joined #openstack-infra01:06
*** jamesmcarthur has quit IRC01:09
*** imacdonn has quit IRC01:13
*** imacdonn has joined #openstack-infra01:14
*** jamesmcarthur has joined #openstack-infra01:14
*** altlogbot_1 has quit IRC01:15
*** jamesmcarthur has quit IRC01:19
*** jamesmcarthur_ has joined #openstack-infra01:21
*** gyee has quit IRC01:22
*** jamesmcarthur_ has quit IRC01:23
*** jamesmcarthur has joined #openstack-infra01:27
*** jamesmcarthur has quit IRC01:32
*** jamesmcarthur has joined #openstack-infra01:36
*** lathiat has joined #openstack-infra01:39
*** jamesmcarthur has quit IRC01:41
*** lathiat has quit IRC01:42
*** lathiat has joined #openstack-infra01:43
*** jamesmcarthur has joined #openstack-infra01:43
*** jamesmcarthur has quit IRC01:45
*** pcrews has joined #openstack-infra01:50
*** jamesmcarthur has joined #openstack-infra01:50
*** rkukura has quit IRC01:54
*** bhavikdbavishi has joined #openstack-infra01:54
*** jamesmcarthur has quit IRC01:54
*** bhavikdbavishi1 has joined #openstack-infra01:57
*** bobh has joined #openstack-infra01:58
*** apetrich has quit IRC01:58
*** bhavikdbavishi has quit IRC01:58
*** bhavikdbavishi1 is now known as bhavikdbavishi01:58
*** bobh has quit IRC02:02
*** yamamoto has quit IRC02:03
*** jamesmcarthur has joined #openstack-infra02:07
*** rkukura has joined #openstack-infra02:08
*** jamesmcarthur has quit IRC02:09
*** jamesmcarthur has joined #openstack-infra02:09
*** jamesmcarthur has quit IRC02:14
*** altlogbot_0 has joined #openstack-infra02:18
*** irclogbot_3 has joined #openstack-infra02:18
*** jamesmcarthur_ has joined #openstack-infra02:22
*** bhavikdbavishi has quit IRC02:23
*** altlogbot_0 has quit IRC02:25
*** irclogbot_3 has quit IRC02:26
*** jamesmcarthur_ has quit IRC02:26
*** tkajinam has quit IRC02:29
*** tkajinam has joined #openstack-infra02:29
*** factor has quit IRC02:34
*** factor has joined #openstack-infra02:35
*** jamesmcarthur has joined #openstack-infra02:56
*** xinranwang has joined #openstack-infra03:02
*** psachin has joined #openstack-infra03:02
*** dklyle has quit IRC03:19
*** jamesmcarthur has quit IRC03:19
*** jamesmcarthur has joined #openstack-infra03:20
*** bhavikdbavishi has joined #openstack-infra03:26
*** yamamoto has joined #openstack-infra03:36
*** njohnston has quit IRC03:53
*** irclogbot_1 has joined #openstack-infra04:06
*** bobh has joined #openstack-infra04:11
*** irclogbot_1 has quit IRC04:12
*** altlogbot_0 has joined #openstack-infra04:12
*** bobh has quit IRC04:16
*** altlogbot_0 has quit IRC04:17
*** udesale has joined #openstack-infra04:27
*** jamesmcarthur has quit IRC04:32
*** yamamoto has quit IRC04:33
*** factor has quit IRC04:36
*** yamamoto has joined #openstack-infra04:38
*** armax has quit IRC04:40
*** jamesmcarthur has joined #openstack-infra04:46
*** ykarel has joined #openstack-infra04:47
*** whoami-rajat has joined #openstack-infra04:52
AJaegerclarkb: sorry for the pasto. https://review.opendev.org/#/c/669932/ is the second part of the tox-molecule change but I just gave a -1 on the current change set. I think zbr is on the right track in general.05:01
*** altlogbot_2 has joined #openstack-infra05:02
*** irclogbot_2 has joined #openstack-infra05:06
*** diablo_rojo has quit IRC05:08
*** bobh has joined #openstack-infra05:09
*** raukadah is now known as chandankumar05:14
*** kjackal has joined #openstack-infra05:20
*** altlogbot_2 has quit IRC05:21
*** bobh has quit IRC05:22
*** irclogbot_2 has quit IRC05:22
*** ykarel_ has joined #openstack-infra05:31
*** janki has joined #openstack-infra05:33
*** ykarel has quit IRC05:33
*** pcaruana has joined #openstack-infra05:37
*** kjackal has quit IRC05:40
*** xinranwang has quit IRC05:41
*** jamesmcarthur has quit IRC05:50
*** lei-zh has joined #openstack-infra05:56
*** lei-zh has quit IRC05:58
*** kjackal has joined #openstack-infra05:58
*** lei-zh has joined #openstack-infra05:59
*** rcernin has quit IRC06:10
*** rcernin has joined #openstack-infra06:11
*** pcaruana has quit IRC06:20
*** lei-zh1 has joined #openstack-infra06:29
*** lei-zh has quit IRC06:31
*** haleyb has quit IRC06:40
*** janki has quit IRC06:40
*** dpawlik has joined #openstack-infra06:44
*** xek has joined #openstack-infra06:48
*** slaweq has joined #openstack-infra06:54
*** gtema has quit IRC06:55
*** gtema has joined #openstack-infra06:56
*** janki has joined #openstack-infra06:57
*** tosky has joined #openstack-infra07:02
*** altlogbot_0 has joined #openstack-infra07:03
*** rpittau|afk is now known as rpittau07:04
*** lei-zh1 has quit IRC07:05
*** lei-zh1 has joined #openstack-infra07:06
*** altlogbot_0 has quit IRC07:08
*** lei-zh has joined #openstack-infra07:08
*** lei-zh1 has quit IRC07:10
*** witek has joined #openstack-infra07:11
*** pgaxatte has joined #openstack-infra07:17
*** kaisers has quit IRC07:24
*** ykarel__ has joined #openstack-infra07:34
*** kaisers has joined #openstack-infra07:36
*** ykarel_ has quit IRC07:36
*** ykarel__ is now known as ykarel07:37
*** irclogbot_0 has joined #openstack-infra07:39
*** lucasagomes has joined #openstack-infra07:43
*** kaisers has quit IRC07:43
*** irclogbot_0 has quit IRC07:43
*** kaisers has joined #openstack-infra07:48
*** traskat has joined #openstack-infra07:51
*** pcaruana has joined #openstack-infra07:54
*** bobh has joined #openstack-infra07:57
*** iurygregory has quit IRC07:58
*** bobh has quit IRC08:02
*** gtema has quit IRC08:04
*** gtema has joined #openstack-infra08:05
*** dchen has quit IRC08:11
*** ralonsoh has joined #openstack-infra08:13
*** tkajinam has quit IRC08:14
*** ykarel is now known as ykarel|lunch08:15
*** priteau has joined #openstack-infra08:17
*** dtantsur|afk is now known as dtantsur08:19
*** Adri2000 has quit IRC08:20
*** derekh has joined #openstack-infra08:20
*** Adri2000 has joined #openstack-infra08:20
AJaegerzbr: why are you not adding a template in 669932?08:26
zbrAJaeger: we had some talks and it seems that we will want to use templates only for the final grouping of jobs. That is because they cannot be overriden.08:28
zbrwes gave me a good example: when using template we lose the ability to temporary disable the voting on gate to make a job passing. with inhereted jobs is easier.08:29
AJaegerzbr: yes, that is true.08:30
zbrbut do not worry, we will migrate to more templates really soon, as we will use them a lot.08:30
AJaegerzbr: I would expect the non-voting to be not needed on the other hand;)08:30
AJaegerzbr: and not beeing able to inherit from openstack-zuul-jobs is a mess IMHO. You could move the job to another repo instead...08:31
*** Fidde has joined #openstack-infra08:31
*** iurygregory has joined #openstack-infra08:31
zbryep, is a PITA but luky that one is not complex. i will follow this approach and see how it goes.08:31
zbrdefining jobs that can work on two different zuul instances is a bit tricky08:32
AJaegeryep - so question is, whether this is needed in this case... Moving it would allow inheritance. Your change to openstack-zuul-jobs wouldn't have been needed in this case (but we didn't know that yesterday ;)08:35
*** Goneri has joined #openstack-infra08:42
*** janki has quit IRC08:43
*** pkopec has joined #openstack-infra08:43
*** janki has joined #openstack-infra08:51
*** irclogbot_2 has joined #openstack-infra08:57
*** irclogbot_2 has quit IRC09:01
*** ykarel|lunch is now known as ykarel09:03
*** ccamacho has joined #openstack-infra09:03
*** janki has quit IRC09:07
*** ociuhandu has joined #openstack-infra09:09
*** janki has joined #openstack-infra09:09
*** altlogbot_1 has joined #openstack-infra09:11
*** altlogbot_1 has quit IRC09:14
*** rcernin has quit IRC09:17
*** aedc has joined #openstack-infra09:21
*** lei-zh has quit IRC09:25
*** altlogbot_2 has joined #openstack-infra09:25
*** altlogbot_2 has quit IRC09:28
*** altlogbot_3 has joined #openstack-infra09:29
*** priteau has quit IRC09:30
*** lei-zh has joined #openstack-infra09:31
*** ricolin_phone has joined #openstack-infra09:32
*** altlogbot_3 has quit IRC09:34
*** ykarel is now known as ykarel|afk09:35
*** irclogbot_1 has joined #openstack-infra09:37
*** irclogbot_1 has quit IRC09:40
*** gtema_ has joined #openstack-infra09:41
*** ricolin_phone has quit IRC09:42
*** sshnaidm|afk is now known as sshnaidm|ruck09:42
*** gtema has quit IRC09:43
*** altlogbot_3 has joined #openstack-infra09:45
*** bhavikdbavishi has quit IRC09:45
*** yamamoto has quit IRC09:47
*** altlogbot_3 has quit IRC09:50
*** ykarel|afk is now known as ykarel09:52
iurygregoryHello infra team, does anyone have an idea how I can solve my problem with requirements-check in https://review.opendev.org/#/c/669885/ ? =)10:00
* iurygregory going for lunch brb10:00
jonher^ "ERROR: Requirement for package 'prometheus-client' has no lower bound"10:04
jonheralso this: http://logs.openstack.org/85/669885/3/check/requirements-check/a789032/job-output.txt.gz#_2019-07-10_09_34_46_80843110:05
*** electrofelix has joined #openstack-infra10:05
*** aedc has quit IRC10:09
openstackgerritMatthieu Huin proposed zuul/zuul master: web: add tenant and project scoped, JWT-protected actions  https://review.opendev.org/57690710:11
openstackgerritMatthieu Huin proposed zuul/zuul master: Allow operator to generate auth tokens through the CLI  https://review.opendev.org/63619710:11
openstackgerritMatthieu Huin proposed zuul/zuul master: Zuul CLI: allow access via REST  https://review.opendev.org/63631510:11
*** irclogbot_1 has joined #openstack-infra10:23
*** irclogbot_1 has quit IRC10:26
*** Goneri has quit IRC10:28
*** lei-zh has quit IRC10:30
*** lei-zh has joined #openstack-infra10:30
*** pgaxatte has quit IRC10:32
*** bobh has joined #openstack-infra10:35
*** lei-zh has quit IRC10:35
*** yamamoto has joined #openstack-infra10:37
*** bobh has quit IRC10:40
iurygregoryjonher, so I need to add uWSGI to oepnstack/requirements?10:40
*** kjackal has quit IRC10:45
*** yamamoto has quit IRC10:46
*** janki has quit IRC10:55
*** tesseract has joined #openstack-infra11:08
AJaegeriurygregory: there's an #openstack-requirements channel that can help you as well...11:13
AJaegeriurygregory: for the ERROR that jonher mentions, you need to have a lower-bound in the requirements file. So, instead of "prometheus-client", write "prometheus-client >=1.0" or whatever version is appropriate11:15
AJaegeriurygregory: and yes, every line in there needs to be in requirements repo11:15
*** apetrich has joined #openstack-infra11:16
*** bhavikdbavishi has joined #openstack-infra11:17
*** bobh has joined #openstack-infra11:19
jamespageAJaeger: I'll get onto that plumgrid charm tidyup this week11:19
jamespageAJaeger: whilst we're talking about retiring projects - what's the easiest way to keep stable/* branches gating and just switch the master branch to a noop?11:20
*** lucasagomes has quit IRC11:22
*** bobh has quit IRC11:23
*** kjackal has joined #openstack-infra11:24
AJaegerjamespage: see how django does it, you can have in project-config noop: branches: master11:26
iurygregoryAJaeger, tks o/11:33
*** aedc has joined #openstack-infra11:49
iurygregoryAJaeger, do you may know wich file I need to update in the requirements repository?11:49
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: Avoid confusing rsync errors when source folders are missing  https://review.opendev.org/67004411:50
*** udesale has quit IRC11:51
*** udesale has joined #openstack-infra11:52
chandankumarclarkb: AJaeger https://review.opendev.org/#/c/669641/ please have a look at this review, thanks :-)11:55
AJaegeriurygregory: check the history of the repo on what was done previously ;)11:57
*** rfolco has joined #openstack-infra11:57
AJaegerchandankumar: I do not review diskimage-builder11:57
AJaegerchandankumar: better ask on #openstack-dib11:58
chandankumarAJaeger: sure, thanks :-)11:59
*** ekultails has joined #openstack-infra12:00
*** rh-jelabarre has joined #openstack-infra12:04
*** Goneri has joined #openstack-infra12:04
*** pgaxatte has joined #openstack-infra12:10
*** gtema_ has quit IRC12:16
*** gtema has joined #openstack-infra12:17
*** gtema_ has joined #openstack-infra12:18
*** gtema__ has joined #openstack-infra12:21
*** gtema has quit IRC12:21
*** gtema_ has quit IRC12:23
*** rlandy has joined #openstack-infra12:24
openstackgerritKevin Carter (cloudnull) proposed opendev/system-config master: Add proxy for registry.access.redhat  https://review.opendev.org/66642312:24
*** gtema has joined #openstack-infra12:24
*** gtema_ has joined #openstack-infra12:25
*** gtema__ has quit IRC12:26
*** gtema__ has joined #openstack-infra12:27
*** gtema___ has joined #openstack-infra12:28
*** gtema has quit IRC12:29
*** gtema_ has quit IRC12:30
openstackgerritKevin Carter (cloudnull) proposed opendev/system-config master: Add proxy for registry.access.redhat  https://review.opendev.org/66642312:31
*** gtema has joined #openstack-infra12:31
*** gtema__ has quit IRC12:31
*** gtema_ has joined #openstack-infra12:33
*** gtema___ has quit IRC12:33
*** ykarel is now known as ykarel|afk12:33
*** gtema__ has joined #openstack-infra12:34
*** gtema___ has joined #openstack-infra12:35
*** gtema has quit IRC12:36
*** eharney has joined #openstack-infra12:37
*** gtema_ has quit IRC12:37
*** gtema__ has quit IRC12:38
*** gtema___ has quit IRC12:39
*** gtema has joined #openstack-infra12:40
openstackgerritMatthieu Huin proposed zuul/zuul master: Zuul CLI: allow access via REST  https://review.opendev.org/63631512:42
*** gtema_ has joined #openstack-infra12:43
*** rfarr_ has quit IRC12:43
*** tosky__ has joined #openstack-infra12:44
*** gtema__ has joined #openstack-infra12:44
*** tosky is now known as Guest8732412:44
*** tosky__ is now known as tosky12:44
*** gtema has quit IRC12:44
*** gtema has joined #openstack-infra12:45
*** Guest87324 has quit IRC12:46
*** gtema has joined #openstack-infra12:47
*** rfarr_ has joined #openstack-infra12:47
*** gtema has quit IRC12:47
*** priteau has joined #openstack-infra12:47
*** gtema_ has quit IRC12:47
*** gtema has joined #openstack-infra12:48
*** gtema__ has quit IRC12:48
*** aaronsheffield has joined #openstack-infra12:50
*** gtema_ has joined #openstack-infra12:52
*** gtema has quit IRC12:53
*** gtema has joined #openstack-infra12:53
*** gtema__ has joined #openstack-infra12:55
*** gtema___ has joined #openstack-infra12:57
*** gtema_ has quit IRC12:57
*** gtema has quit IRC12:58
*** gtema_ has joined #openstack-infra12:58
*** gtema has joined #openstack-infra12:59
*** gtema__ has quit IRC13:00
*** gtema___ has quit IRC13:01
*** gtema__ has joined #openstack-infra13:02
*** gtema_ has quit IRC13:02
*** gtema_ has joined #openstack-infra13:03
*** gtema has quit IRC13:03
*** gtema has joined #openstack-infra13:04
*** irclogbot_1 has joined #openstack-infra13:06
*** gtema___ has joined #openstack-infra13:06
*** gtema__ has quit IRC13:06
*** gtema_ has quit IRC13:08
*** irclogbot_1 has quit IRC13:08
*** gtema_ has joined #openstack-infra13:08
*** gtema has quit IRC13:09
jamespageAJaeger: thanks13:09
*** gtema___ has quit IRC13:10
*** gtema has joined #openstack-infra13:10
*** altlogbot_0 has joined #openstack-infra13:11
*** gtema__ has joined #openstack-infra13:13
*** gtema_ has quit IRC13:13
*** gtema_ has joined #openstack-infra13:14
*** gtema has quit IRC13:15
*** gtema has joined #openstack-infra13:16
*** yamamoto has joined #openstack-infra13:16
*** altlogbot_0 has quit IRC13:16
*** gtema___ has joined #openstack-infra13:17
*** irclogbot_3 has joined #openstack-infra13:17
*** gtema__ has quit IRC13:17
*** sthussey has joined #openstack-infra13:18
*** gtema_ has quit IRC13:18
*** gtema_ has joined #openstack-infra13:20
*** gtema has quit IRC13:20
*** yamamoto has quit IRC13:20
*** tosky__ has joined #openstack-infra13:20
*** tosky has quit IRC13:20
*** tosky__ is now known as tosky13:20
openstackgerritJames Page proposed openstack/project-config master: nova-lxd: noop jobs as part of project retirement  https://review.opendev.org/67005813:21
*** gtema___ has quit IRC13:21
*** gtema has joined #openstack-infra13:22
*** irclogbot_3 has quit IRC13:22
*** gtema__ has joined #openstack-infra13:23
*** gtema___ has joined #openstack-infra13:24
*** gtema_ has quit IRC13:24
*** haleyb has joined #openstack-infra13:25
*** gtema has quit IRC13:26
*** gtema_ has joined #openstack-infra13:26
*** gtema__ has quit IRC13:28
*** goldyfruit has joined #openstack-infra13:28
*** gtema___ has quit IRC13:29
openstackgerritMonty Taylor proposed opendev/system-config master: [WIP] Translate gitea project creation to python  https://review.opendev.org/67006013:30
*** gtema has joined #openstack-infra13:30
mordredcorvus: ^^ there's the first pass - I'm sure there's bugs, it's totally untested locally - but I *think* it should be *mostly8 there13:30
*** gtema_ has quit IRC13:31
*** mriedem has joined #openstack-infra13:31
*** aedc has quit IRC13:33
*** gtema_ has joined #openstack-infra13:33
*** gtema has quit IRC13:35
*** altlogbot_2 has joined #openstack-infra13:36
*** gtema has joined #openstack-infra13:36
*** altlogbot_2 has quit IRC13:36
*** gtema_ has quit IRC13:38
*** gtema__ has joined #openstack-infra13:38
*** gtema_ has joined #openstack-infra13:39
*** gtema has quit IRC13:40
*** gtema___ has joined #openstack-infra13:40
*** gtema has joined #openstack-infra13:41
*** gtema__ has quit IRC13:42
*** Goneri has quit IRC13:43
*** gtema_ has quit IRC13:43
*** gtema__ has joined #openstack-infra13:43
*** tosky___ has joined #openstack-infra13:44
*** gtema_ has joined #openstack-infra13:45
*** gtema___ has quit IRC13:45
*** gtema has quit IRC13:46
*** gtema has joined #openstack-infra13:46
*** yolanda has quit IRC13:46
*** tosky has quit IRC13:47
*** gtema___ has joined #openstack-infra13:47
*** gtema__ has quit IRC13:48
*** gtema__ has joined #openstack-infra13:48
*** gtema____ has joined #openstack-infra13:49
*** gtema_ has quit IRC13:49
*** gtema_ has joined #openstack-infra13:50
*** gtema has quit IRC13:50
*** gtema has joined #openstack-infra13:51
*** gtema___ has quit IRC13:51
*** gtema__ has quit IRC13:53
*** Lucas_Gray has joined #openstack-infra13:53
*** gtema____ has quit IRC13:54
*** whoami-rajat has quit IRC13:54
*** whoami-rajat has joined #openstack-infra13:54
openstackgerritMonty Taylor proposed opendev/system-config master: [WIP] Translate gitea project creation to python  https://review.opendev.org/67006013:54
*** gtema_ has quit IRC13:55
*** bobh has joined #openstack-infra13:55
*** gtema has quit IRC13:55
*** aedc has joined #openstack-infra13:57
*** ykarel|afk is now known as ykarel13:59
*** gtema has joined #openstack-infra14:00
*** bobh has quit IRC14:00
*** gtema_ has joined #openstack-infra14:03
*** dklyle has joined #openstack-infra14:04
*** jamesmcarthur has joined #openstack-infra14:04
*** gtema__ has joined #openstack-infra14:05
*** jamesmcarthur has quit IRC14:05
*** Goneri has joined #openstack-infra14:05
*** jamesmcarthur has joined #openstack-infra14:05
*** gtema has quit IRC14:05
*** gtema has joined #openstack-infra14:06
*** gtema___ has joined #openstack-infra14:07
*** gtema_ has quit IRC14:08
*** gtema_ has joined #openstack-infra14:08
*** gtema__ has quit IRC14:09
*** gtema has quit IRC14:10
*** adriancz has quit IRC14:10
*** gtema___ has quit IRC14:12
*** lseki has joined #openstack-infra14:14
*** tosky___ is now known as tosky14:15
*** bobh has joined #openstack-infra14:25
*** ykarel is now known as ykarel|away14:25
*** yamamoto has joined #openstack-infra14:29
AJaegerconfig-core, two project retirements for review, please: https://review.opendev.org/#/c/668854 and https://review.opendev.org/#/c/669883/14:29
*** bobh has quit IRC14:30
*** ykarel|away has quit IRC14:30
*** dklyle has quit IRC14:30
corvusAJaeger, zbr, weshay: you can override the voting attribute in the project pipeline config of a job which is in a template (ie, you can set it non-voting in gate of a specific project even if it's in a template on that project)14:35
zbrcorvus: thanks, that is good to know.14:35
*** dpawlik has quit IRC14:37
*** dklyle has joined #openstack-infra14:38
*** altlogbot_1 has joined #openstack-infra14:39
*** altlogbot_1 has quit IRC14:42
*** altlogbot_1 has joined #openstack-infra14:51
*** Fidde has quit IRC14:52
fungiwhat's harder, i guess, is to set it non-voting for the check pipeline and remove it from the gate pipeline14:52
corvusthat is true14:52
fungii wonder if there would be benefit to a pipeline option which basically strips out non-voting jobs from the buildset?14:53
fungior if there would be a more user-friendly way to tackle that14:54
fungi(so that if the openstack tenant wants a policy of "no non-voting jobs in the gate pipeline" it's just a simple toggle in the config)14:55
corvusfungi: yeah, that could be a thing.14:55
corvusmordred: comment on 67006014:55
weshayah.. that is cool, thanks zbr14:56
*** altlogbot_1 has quit IRC14:56
weshayand corvus14:56
cloudnullqq - I'm not seeing our post jobs run (like docs getting republished) and I suspect job layout is to blame, would anyone mind giving https://opendev.org/openstack/tripleo-ansible/src/branch/master/zuul.d/layout.yaml#L49-L59 a look and telling me what I'm doing wrong?14:57
cloudnullAsking because after getting https://review.opendev.org/#/c/669940/ merged I haven't seen any changes in our published docs14:58
cloudnulland in watching the zuul post queue, I've not seen anything pop up for tripleo-ansible14:58
clarkbcloudnull: you can chek the zuul dashaboard's build page to see when the last build was14:59
corvushttp://zuul.openstack.org/builds?project=openstack%2Ftripleo-ansible&pipeline=post14:59
corvusthat looks pretty empty14:59
cloudnull^14:59
clarkbI'm not sure files matchers work in post?14:59
clarkb(would depend on whether or not post trigger event has that info)15:00
openstackgerritJames Page proposed openstack/project-config master: nova-lxd: noop jobs as part of project retirement  https://review.opendev.org/67005815:00
openstackgerritMonty Taylor proposed opendev/system-config master: [WIP] Translate gitea project creation to python  https://review.opendev.org/67006015:01
mordredcorvus: thanks!15:01
corvusclarkb, cloudnull: they don't.  that's likely the problem.  that will probably change in the future, but the dev working on that is afk for a while.15:02
*** gtarnaras has joined #openstack-infra15:02
corvus(though, the future change is merely likely to ignore file matchers in post)15:02
clarkbrouter_solicitation_delay value of 30 allows network manager to function15:03
clarkbI'm going to try and figure out how large of a delay is necessary now. Then I'll get a change up to dib which glean can depends on to change that I guess :/15:03
*** armax has joined #openstack-infra15:04
corvuscloudnull: so for now, i'd just drop the matchers; but if you are interested in going down that road, you could look into some of the promote jobs i've been working on.  they would be able to have file matchers.15:04
corvuscloudnull: https://opendev.org/zuul/zuul-sphinx/src/branch/master/.zuul.yaml  (note that we don't have openstack versions of those yet, just basic opendev ones)15:05
corvusall still a work in progress15:05
clarkbcouple things with what those file matchers seem to be trying to do: For docs jobs in many cases there is autodoc'ing of code in which case only publishing when the docs dir updates isn't necessarily correct. Also for the tarball items in the root of the dir determine how the tarballs are built so it sn't quite correct either to only tarball when the source dir changes15:05
*** tdasilva has joined #openstack-infra15:05
clarkbit may be the case that this particularrepo is fine with that because it isn't autodocing and doesn't use python sdists but in general I think we have to be careful with rules like that15:05
*** mattw4 has joined #openstack-infra15:07
cloudnullcorvus excellent, thats a simple fix, at least for me :)15:07
*** altlogbot_1 has joined #openstack-infra15:07
*** dklyle has quit IRC15:09
AJaegerclarkb, could you put these two project retirements on your review queue, please: https://review.opendev.org/#/c/668854 and https://review.opendev.org/#/c/669883/ ?15:11
*** altlogbot_1 has quit IRC15:12
*** ykarel|away has joined #openstack-infra15:12
clarkbAJaeger: yes. I'm trying to finish up this glean test problem then will look at reviews15:13
AJaegerthanks, clarkb !15:16
cloudnullcorvus clarkb are there WIP changes folks could help out with to make post jobs have the ability to be filtered?15:18
* cloudnull happy to help out if at all possible (or if useful) 15:18
*** irclogbot_3 has joined #openstack-infra15:19
*** irclogbot_3 has quit IRC15:22
*** yolanda has joined #openstack-infra15:23
clarkbok 10 seconds is not enough time but 15 seconds appears to be15:23
clarkbnow to write a change15:23
corvuscloudnull: i don't think there's anything tee'd up for that; https://review.opendev.org/660856 is related and we're waiting for tristanC to return, but i think that even with that, we won't have the file information available for the ref-updated events which trigger post; i think that another change would be necessary for that, and i don't recall seeing one out there.15:30
*** altlogbot_2 has joined #openstack-infra15:31
cloudnullcorvus sounds good, if I can help let me know.15:31
*** altlogbot_2 has quit IRC15:34
*** mattw4 has quit IRC15:34
corvuscloudnull: we use the supercedent pipeline manager with post, which i think is incompatible with the idea of file matchers.  supercedent allows us to only run jobs for the latest merge if there happen to be several merges queued up.  so if we skip items, and the items we skip would have matche jobs, and the items we do run don't trigger jobs, then we will have not run a job we should have.15:34
corvuscloudnull: given that, i think "don't have file matchers in post" may be the best long-term strategy.  (supercedent does allow us to skip a significant number of jobs)15:35
*** mattw4 has joined #openstack-infra15:35
*** ykarel|away has quit IRC15:36
openstackgerritClark Boylan proposed openstack/diskimage-builder master: Set router solicitation delay with using NM  https://review.opendev.org/67010215:37
openstackgerritClark Boylan proposed opendev/glean master: network-manager: add network-pre dependencies  https://review.opendev.org/66977315:39
*** irclogbot_2 has joined #openstack-infra15:39
*** mattw4 has quit IRC15:39
clarkbcorvus: ^ lets see if that fixes it15:39
*** Lucas_Gray has quit IRC15:39
cloudnullcorvus understood. thanks for the info on the supercedent pipeline (TIL).15:40
chandankumarclarkb: please have a look on https://review.opendev.org/#/c/669641/ dib RHEL8 change15:40
*** Goneri has quit IRC15:41
*** ykarel|away has joined #openstack-infra15:42
*** roman_g has quit IRC15:42
*** irclogbot_2 has quit IRC15:42
chandankumarclarkb: thanks :-)15:44
*** diablo_rojo has joined #openstack-infra15:44
*** Lucas_Gray has joined #openstack-infra15:44
*** roman_g has joined #openstack-infra15:45
*** altlogbot_1 has joined #openstack-infra15:45
clarkbAJaeger: stephenfin do we want to leave https://review.opendev.org/#/c/668854/3/specs/specs.yaml in place for linkage to historical docs?15:46
clarkbwe use that to generate index.html type files right?15:46
stephenfinif it is, then yes15:46
stephenfinI just did the usual search and destroy (all references)15:46
clarkbya I want to say that is used to geneate the specs.openstack.org page index or similar15:47
clarkbAJaeger should know for sure though15:47
openstackgerritMonty Taylor proposed opendev/system-config master: [WIP] Translate gitea project creation to python  https://review.opendev.org/67006015:47
*** altlogbot_1 has quit IRC15:48
*** ykarel|away has quit IRC15:48
AJaegerclarkb: yes, we use it for the index page. I would remove it since the content is stale - but I'm fine wiht leaving it as well15:49
clarkbI guess the published content behind the index won't go away so direct links will work15:49
clarkbstephenfin: ^ do you think having the index up for historical but no longer current docs is good or bad?15:50
*** pgaxatte has quit IRC15:50
*** ykarel has joined #openstack-infra15:50
AJaegerclarkb: correct, published content removal needs an admin15:50
stephenfinFor specs, it's probably useful15:50
openstackgerritJames E. Blair proposed opendev/system-config master: DNM: debug a weird python version issue  https://review.opendev.org/67010615:50
*** Vadmacs has joined #openstack-infra15:51
AJaegerwant to update the link to say "(frozen, historic)" or something similar?15:51
openstackgerritMonty Taylor proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006015:52
stephenfinAJaeger: Sure15:52
*** kjackal has quit IRC15:52
corvusmordred: i re-pushed your change as a new one and set an autohold to try to figure out why it isn't running on python315:52
mriedemAJaeger: do you know if something has changed recently(ish) wrt the tools/test-setup.sh script for setting up mysql/postgresql?15:53
mriedemhttp://logs.openstack.org/71/659371/2/gate/openstack-tox-pep8/f361eb9/job-output.txt.gz#_2019-07-09_21_23_29_11484815:53
*** Goneri has joined #openstack-infra15:53
openstackgerritStephen Finucane proposed openstack/project-config master: Retire docs-specs  https://review.opendev.org/66885415:54
clarkbmriedem: we stopped installed bindep-fallback.txt by default a few weeks ago so you need to have those packages in your bindep.txt file15:54
mriedemclarkb: ah ok15:54
clarkb(or you can switch to the legacy-base base parent job)15:54
clarkbconsidering that is the shared pep8 job updating bindep.txt is the best path forward15:54
mordredcorvus: okie15:55
*** altlogbot_3 has joined #openstack-infra15:55
fungibut also, installing and configuring mysql and postgresql to be able to run flake8 still seems like there's a lot of potential for optimization15:57
*** altlogbot_3 has quit IRC15:58
AJaegerclarkb: I would not base openstack-tox-pep8 onto legacy-base15:58
mriedemfungi: i'm not sure we're controlling that in sqlalchemy-migrate15:58
mriedemlooks like the " test-setup : Check if project's tools/test-setup.sh exists" task runs,15:59
mriedemand if true, runs tools/test-setup.sh15:59
mriedemregardless of it being a pep8 job or not15:59
AJaegermriedem: I really suggest to add bindep.txt file to the repo16:00
*** _erlon_ has joined #openstack-infra16:00
mriedemlooks like it's also doing that on nova's pep8 job16:00
mriedemhttp://logs.openstack.org/12/669912/2/check/openstack-tox-pep8/89ef469/ara-report/file/273b1918-6bf4-4d90-8a49-a37315ef5deb/#line-116:00
mriedemAJaeger: yeah i can do that16:00
AJaegerwe could remove it from pep8, but it would still break on py27 and others...16:01
corvusis there anything in test-setup.sh that does need to run for pep8?16:01
mriedemsure, i was just replying to fungi's comment for optimizing the pep8 job16:01
mriedemthat it's not unique to sqla-migrate16:01
*** iurygregory has quit IRC16:01
*** udesale has quit IRC16:01
corvusbecause i'm wondering if we could remove or disable the 'test-setup' role for pep8 jobs16:02
corvus(alternately, we could pass in a variable or argument to the script in the pep8 job so it could have a conditional to avoid running the db setup: https://zuul-ci.org/docs/zuul-jobs/general-roles.html#rolevar-test-setup.test_setup_environment )16:03
mriedemyou'll have to copy that change to all projects that have a tools/test-setup.sh script16:03
AJaegercorvus: I don't see why pep8 would need the setup, but bindep is needed anyhow for py27 testing16:04
mriedemif bindep.txt is in the root of the repo will infra just pick it up automatically?16:05
mriedems/infra/zuul/16:05
mordredyup16:05
mordredwell - the jobs will, but yes16:05
mriedemah i se http://logs.openstack.org/71/659371/2/gate/openstack-tox-pep8/f361eb9/ara-report/file/4fcc0dee-3363-49af-9624-34c86bb2f08f/#line-116:05
openstackgerritJames E. Blair proposed opendev/system-config master: DNM: debug a weird python version issue  https://review.opendev.org/67010616:05
fungiyeah, the cases i'm aware of for test-setup.sh are setting db permissions (e.g. nova) and creating special filesystems (e.g. swift) for unit tests, so not relevant to static analyzers... but that's not to say someone *hasn't* done something with that api which is relevant to a static analyzer16:06
AJaegercorvus: FYI, roles/test-setup/ is in zuul-jobs and used by the unittests job which is parent of tox.16:06
AJaegerSo, we would need to add a variable for disabling it16:07
fungiright, it's baked pretty far down in the ancestry of those jobs16:07
clarkbfungi: I think that is a side effect of sharing a parent job for tox16:07
corvuseither that, or come to terms with the idea that "pep8" is not "unit testing"16:07
AJaeger;)16:08
fungiclarkb: i concur. it was a choice made for simplicity over efficiency, and it's probably fine as is16:08
*** ijw has joined #openstack-infra16:08
fungibut making it switchable or preemptible also seems fine16:08
mordred++16:08
corvusdoes pep8 need bindep?16:09
clarkbcorvus: yes see mriedem's job that precipitated this16:09
mordredit might16:09
AJaegercorvus: it might - it installs *all* requirements16:09
clarkbwell I guess that does the wrong thing though16:09
fungiclarkb: well, that's only because it was running the test-setup.sh script16:09
clarkbin general though pep8 jobs can install packages that need system deps16:09
AJaegerso, mriedem will need bindep for other jobs.16:10
mordredyah. like libxml16:10
corvusso i'm picking up the task "can we make pep8 more efficient"16:10
mordredin this case, that mriedem was using the pep8 job is a detail - py27 and py35 are broken for that repo too16:10
mordredcorvus: cool16:10
fungii think the corner case where it could would be if you set your tox.ini to want defaultpython=3.7 and ran it on bionic where you need the python3.7 package installed explicitly, that might be one example (contrived though i'll admit)16:10
corvusmy question is: in an ideal world, would a theoretically pure pep8 job need bindep?16:10
mordredyes16:10
corvusok thx (to help me fix that in memory, have a handy example of why?)16:11
mordredbecause of things like libxml and pyflakes wanting to be able to import python files16:11
corvusgotach16:11
*** kjackal has joined #openstack-infra16:11
fungioh, or yeah, if the tox.ini is (mis?)configured to install the project being checked16:11
*** gtarnaras has quit IRC16:11
clarkbright in order for linters to check if imports and names are valid they check that the libs exist and have those names and to install some libs you need system deps16:11
fungithat said, you shouldn't need to install nova into the venv to run pep8 against it16:11
fungioh, does it actually?16:12
clarkbya pep8 the command does not need it16:12
mordredyeah16:12
clarkbbut what we really run is hacking/pyflakes/pycodestyle etc and some of those tools do16:12
fungiahh16:12
mordredand - those are actually some of the useful tests16:12
mriedemhttps://docs.openstack.org/infra/bindep/usage.html doesn't mention "bindep test"16:13
AJaegerSo, we need bindep.txt but do not need tools/test-setup.sh for pep816:13
mordredAJaeger: ++16:13
clarkbmriedem: that is because its convention of openstack not of the tool16:13
*** Lucas_Gray has quit IRC16:13
clarkbmriedem: the profile names are arbitrary16:13
mriedemmy question is,16:13
mriedemif i get this16:13
mriedembindep runtests: commands[0] | bindep test16:13
mriedemMissing packages:16:13
mriedem    mysql-client mysql-server postgresql postgresql-client16:13
mriedemdoes that mean the packages are missing on my local vm?16:13
AJaegermriedem: yes16:13
mriedemok16:13
clarkbmriedem: yes and you can feed that lin to apt-get install -y16:13
mriedemb/c they are in bindep.txt16:13
fungithat said, i've got a project whose tox.ini does skipsdist=True and explicitly sets deps=flake8 in its linting testenv, and it works16:13
fungiso at least basic flake8 doesn't choke when the project and its deps aren't installed into the venv16:14
clarkbfungi: do you have an empty deps line too?16:14
mordredfungi: yes - I do not believe it needs them to be installed16:14
mordredjust that it needs to be able to attempt to import things that the things import16:14
funginot empty, it's got deps=flake8 because it invokes flake8 from the testenv16:15
mordredso I agree16:15
AJaegerbbl16:15
mriedemok i'll see if this works https://review.opendev.org/67011316:16
*** gyee has joined #openstack-infra16:16
fungitechnically the example project has a deps list with two entries because it also uses the flake8-bugbear plugin16:17
*** kjackal has quit IRC16:20
fungioh, the other bit of magic involved in that is setting usedevelop=False for that testenv16:22
fungi(in case it's set true in the base testenv)16:22
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add test_setup_skip role variable  https://review.opendev.org/67011916:24
fungipip list in the resulting venv claims all it installed was attrs,entrypoints,flake8,flake8-bugbear,mccabe,pip,pycodestyle,pyflakes,setuptools16:24
openstackgerritJames E. Blair proposed openstack/openstack-zuul-jobs master: Skip test-setup.sh in pep8 jobs  https://review.opendev.org/67012016:25
corvusthere's the skip flag approach ^16:25
corvusi think the reparent approach would not make us happy16:26
*** psachin has quit IRC16:26
*** bobh has joined #openstack-infra16:26
mordredcorvus: ++16:27
clarkb\o/ I think the centos functest for glean unit change will pass now16:28
mordredclarkb: \o/16:28
clarkbits cleaning up after the job but the run playbook seemed to exit successfully according to the console log16:29
clarkbyup just went green16:29
clarkbok so I think that means I/we want to clean up that stack that adds better debugging info (optionally) then we can review and hopefully merge some changes then make a glean release (and a dib release?)16:30
corvusclarkb: let me know when you need me16:30
*** ginopc has quit IRC16:30
clarkbcorvus: maybe start by reviewing https://review.opendev.org/#/c/670102/1 and checking if the approach there seems like a fair compromise?16:31
*** bobh has quit IRC16:31
clarkband if that looks good I'll go through and remove WIP tags on commits and stuff and we can get the whole set happy16:32
*** gtema has joined #openstack-infra16:33
clarkbcorvus: or as an alternative we can disconnect the centos glean fixing from the journal console log debug improvements16:34
clarkbI actually think that is a good idea as it will separate the two concerns and we can work them independently16:34
openstackgerritMerged openstack/diskimage-builder master: [RHEL-8] Set _clear_old_files=0 in install-pip element  https://review.opendev.org/66964116:35
corvusclarkb: reading that bug -- why not go with the approach in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=755202#331 ?16:35
openstackDebian bug 755202 in network-manager "network-manager: keeps creating and using new connection "eth0" that does not work" [Important,Open]16:35
clarkbcorvus: because that fully disables ipv616:36
clarkbthe comment says NM seems to configure ipv6 for them anyway, but in testing on our centos image that wasn't the case16:37
corvusclarkb: oh, the comment says that NM ... ok that's what i was looking for, thx16:37
*** gtema_ has quit IRC16:37
fungithat could also be a debian vs centos difference16:37
fungibehavior-wise16:37
clarkbya they have different plugins for network manager and stuff16:38
clarkbjust as soon as 669773 reports its current patchset I'll push a new patchset that decouples it from the additional debugging in the jobs16:39
clarkblooks like its opensuse job had a zk timeout and will fail16:40
clarkb(but I want to have the green for fedora and centos posted :) )16:40
*** gtema has left #openstack-infra16:40
openstackgerritClark Boylan proposed opendev/glean master: network-manager: add network-pre dependencies  https://review.opendev.org/66977316:40
fungiit did get working network though, so that's sufficient proof16:41
openstackgerritMerged openstack/project-config master: Drop tempest-tripleo-ui  https://review.opendev.org/66988316:41
clarkbfungi: well and opensuse is unaffected by NM ( we don't NM them)16:41
*** mattw4 has joined #openstack-infra16:41
clarkbhttps://review.opendev.org/669773 should be ready for review now as well16:41
fungiaha, right. that too ;)16:41
clarkbI'm going to get a bike ride in while I wait on test results16:41
*** factor has joined #openstack-infra16:41
cloudnullso i got the post jobs fixed, however, now I have a new issue (the wheel of progress i know) http://zuul.openstack.org/build/487e06c0b3b049cc8ca0005fbee74a69 seems I need to pass in a couple additional requirements to the post job, is there an easy way to do that?16:42
*** Goneri has quit IRC16:45
fungicloudnull: looks like you need ansible installed into that tox venv where sphinx is run16:46
openstackgerritMerged zuul/zuul-jobs master: Add test_setup_skip role variable  https://review.opendev.org/67011916:47
fungicloudnull: http://logs.openstack.org/fb/fbb57dc619061b008610cda53d1a2d0ccfb37aa3/post/publish-openstack-sphinx-docs/487e06c/job-output.txt.gz#_2019-07-10_16_35_11_08939016:47
fungithat job will install any packages included in doc/requirements.txt so you could just add a line in there?16:48
cloudnull++16:48
cloudnullwill do16:48
cloudnullthanks fungi16:48
fungiany time!16:48
*** altlogbot_2 has joined #openstack-infra16:49
*** pkopec_ has joined #openstack-infra16:51
*** ricolin has quit IRC16:52
*** pkopec has quit IRC16:53
*** altlogbot_2 has quit IRC16:54
*** ykarel is now known as ykarel|away16:56
corvusmordred: i think i've got it -- the new nodepool feature that passes ansible_python_interpreter to zuul: http://logs.openstack.org/06/670106/2/check/system-config-run-gitea/19dfbc4/zuul-info/inventory.yaml16:56
corvusmordred: it causes ansible_python_interpreter to show up in the inventory file as a host-specific hostvar, which overrides the group var16:57
*** dtantsur is now known as dtantsur|afk16:57
AJaegercorvus: want to do the same change for tox-pep8 in zuul-jobs?16:57
*** Goneri has joined #openstack-infra16:58
corvusmordred: (the write-inventory role passes that variable through to the inner-inventory we create from the outer-inventory, so it's there in /etc/ansible/inventory on fake-bridge16:58
AJaegercorvus: I can do it ...16:58
corvusAJaeger: yeah, if you want to take that, it sounds like a good idea16:58
*** derekh has quit IRC17:01
corvusmordred: because of that change, zuul is now always going to set an ansible_python_interpreter in its own inventory.  however, for system-config, we always set the ansible_python_interpreter in the playbook/host_vars or playbook/group_vars files (never in our inventory file).  so as long as that's the case, i think we can tell write-inventory to strip ansible_python_interpreter for us.17:01
corvus(there's one exception to that -- and that's the "trusty" node that we run on system-config-run-base -- we may just need to move that setting out of .zuul.yaml and actually make a playbook/host_vars/trusty file17:02
corvusi think i know what to do, change incoming17:03
corvusmordred: good talk, thanks :)17:03
*** weifan has joined #openstack-infra17:03
openstackgerritAndreas Jaeger proposed zuul/zuul-jobs master: Skip test-setup.sh in pep8 jobs  https://review.opendev.org/67013317:03
*** witek has quit IRC17:03
AJaegermordred, pabelanger, this addresses your comments on the original change as well ^17:04
*** weifan has quit IRC17:04
*** weifan has joined #openstack-infra17:04
mordredcorvus: glad I could help!17:05
AJaegerpabelanger: please see backscroll why we go with https://review.opendev.org/#/c/670120/17:08
AJaegerpabelanger: from the discussion we had, I think 670133  is fine.17:08
*** weifan has quit IRC17:09
pabelangerAJaeger: I don't mind if we change, but given it is zuul-jobs, we need a window of notification I think. As changing it affects all zuuls17:09
pabelanger670120 is fine, as it only affects openstack17:09
openstackgerritJames E. Blair proposed opendev/system-config master: DNM: debug a weird python version issue  https://review.opendev.org/67010617:10
openstackgerritJames E. Blair proposed opendev/system-config master: Exclude ansible_python_interpreter from write-inventory  https://review.opendev.org/67013717:10
openstackgerritMerged openstack/openstack-zuul-jobs master: Skip test-setup.sh in pep8 jobs  https://review.opendev.org/67012017:10
AJaegercorvus: do you think we need a notification for 670133 ?17:11
corvusyep17:11
corvusAJaeger, pabelanger: looks like a 2-week minimum for that one: https://zuul-ci.org/docs/zuul-jobs/policy.html#deprecation-policy17:13
pabelangeryah17:13
pabelangerI'll audit jobs here now17:13
corvusAJaeger: i can send the mail if you want17:13
AJaegerthanks, corvus !17:13
* AJaeger sets WIP17:13
AJaegercorvus: 25th of July as waiting time?17:14
corvusAJaeger: ++17:14
*** aedc has quit IRC17:15
pabelangerokay, so we look to be good. I cannot see anything use it17:15
AJaegerpabelanger: thanks for checking17:15
AJaegerpabelanger: and we also don't expect anyone *needing* it for pep817:15
pabelangerAJaeger: yah, depends if they are really doing 'pep8' testing, or using the entry point for pep8 link linting.17:16
pabelangerbut agree17:16
mordredcorvus: nice patch17:17
corvusemail sent17:20
AJaegerthanks, corvus !17:22
corvusmordred: the real patch has moved on to the next error!17:23
mordredcorvus: woot!17:23
*** rpittau is now known as rpittau|afk17:23
*** weifan has joined #openstack-infra17:24
mordredcorvus: how about I rebase my version of the patch on yours, remove the try/import and fix that issue?17:24
corvusmordred: hrm... i vote you stick to your branch right now until my thing is proved correct.... in case i messed up, that'll let us keep working on yours17:25
mordredcorvus: kk17:25
openstackgerritMonty Taylor proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006017:27
* AJaeger is keeping an eye on http://zuul.openstack.org/builds?job_name=openstack-tox-pep8 to see whether we get failures, so far all good...17:33
*** Fidde has joined #openstack-infra17:41
corvusmordred: looks like the python3 fix worked: http://logs.openstack.org/06/670106/3/check/system-config-run-gitea/3e86d49/bridge.openstack.org/ara-report/result/f2a1b603-4a93-4f0e-9c49-2d6acd25270c/  (that's the error i'd expect after fixing py3 with that old version of the patch)17:43
corvusclarkb, fungi: so we should be ready to +3 https://review.opendev.org/67013717:44
corvusmordred: and next PS of yours you can rebase and drop the try/except17:44
corvusmordred: and the next error is ready for your inspection: http://logs.openstack.org/60/670060/6/check/system-config-run-gitea/e269311/bridge.openstack.org/ara-report/result/d5a79a8f-4fd6-4e92-9a74-4fb06e032347/17:46
mordredcorvus: yeah - looking at that now17:46
mordredcorvus: bwahahaha. copy-pasta issue17:47
openstackgerritMonty Taylor proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006017:47
*** dpawlik has joined #openstack-infra17:47
mordredcorvus: the interdiff on that might be poop - that was the rebase and the fix - the fix was on line 179 to change from request.get to request.request17:48
corvusmordred: were you going to drop or leave the try/except?17:48
mordredoh right17:48
openstackgerritMonty Taylor proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006017:48
corvusmordred: one more thing i just saw (inline comment)17:49
*** ociuhandu has quit IRC17:53
clarkbok back17:53
clarkbhttps://review.opendev.org/#/c/670102/ adds the router solicitation delay to dib when using NetworkManager and passes tests now if someone else wants to review it (thanks corvus !)17:53
mordredcorvus: fixed17:55
*** weifan has quit IRC17:58
clarkbhrm the fedora-29 job on the child of ^ may not be happy. Maybe 15 seconds isn't always enough of a wait :/17:59
mordredclarkb: boo18:00
clarkbI'll try to be paitent and see if it eventually goes through18:00
clarkbok it +1'd18:04
clarkbcrises averted so https://review.opendev.org/#/c/670102/ and https://review.opendev.org/#/c/669773/ are ready for people to review and approve if they are happy with the approach18:04
clarkbthen I can tag a glean (tagging dib is actually less important since this fix works on fn without the ipv6 delay)18:05
*** irclogbot_0 has joined #openstack-infra18:05
fungiyeah, if 10 was generally broken and 15 was generally working, i have a feeling upping it more might help avoid nondeterministic results when that job gets scheduled on slower node providers18:05
fungibut we can always start there18:05
*** irclogbot_0 has quit IRC18:08
openstackgerritMerged opendev/system-config master: Exclude ansible_python_interpreter from write-inventory  https://review.opendev.org/67013718:11
openstackgerritMonty Taylor proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006018:12
*** weifan has joined #openstack-infra18:17
*** ijw has quit IRC18:24
*** weifan has quit IRC18:25
*** Goneri has quit IRC18:25
*** ijw has joined #openstack-infra18:25
*** weifan has joined #openstack-infra18:28
mordredcorvus: we have a new error - and now it looks like an actual http error18:31
corvusmordred: link?18:31
corvusnm18:32
corvusmordred: the force_basic_auth for that one is key i think18:33
corvusmordred: i think some of those tasks use the api, where we can use basic auth.  and others use the web api, which is tokens and crsf18:34
corvusmordred: maybe there's a way to use the rest api with a login token?  i'm not sure...18:35
*** xek has quit IRC18:35
corvusmordred: nope, i think it's just basic or "api token" which is not the same thing as a web ui login token18:35
corvusmordred: so i think that's going to need a separate request method to do the basic auth reqs18:36
*** xek has joined #openstack-infra18:36
*** Lucas_Gray has joined #openstack-infra18:37
*** weifan has quit IRC18:38
mordredcorvus: hrm. I thought passing auth=(user,pass) to requests was the same as forcing basic auth - lemme go back and look at that ...18:38
mordredcorvus: I was also thinking maybe it was unhappy about the // in the poorly-joined url18:39
corvusmordred: oh, i see... then i think the issue may be that gitea doesn't do the 401 dance there18:39
corvusmordred: that's why we had to do "force_basic_auth" in ansible18:39
mordredgotcha18:39
*** weifan has joined #openstack-infra18:39
corvusmordred: to say "go ahead and send this with an auth header even though it doesn't look like you need it"18:39
mordrednod18:40
mordredcorvus: I'm not seeing an option for "force_basic_auth" in requests - it looks like it should just send it regardless18:43
*** weifan has quit IRC18:44
mordredcorvus: I'm going to try urljoining to see if that fixes the 404 - and if it doesn't, going to try switching to a requests.session in the class, which someone on the internet thinks might be a better choice18:44
corvuswell, it's called force_basic_auth in ansible18:44
corvusi very much doubt it's the double /18:45
mordredme too18:45
mordredbut I don't see anything like that in requests - either in the docs or in the actual code18:45
openstackgerritJeremy Houser proposed openstack/project-config master: New repo for ranger-tempest-plugin  https://review.opendev.org/67015918:47
openstackgerritMonty Taylor proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006018:50
AJaegerclarkb: want to review the docs-specs retire change again, please? https://review.opendev.org/66885418:51
*** ijw has quit IRC18:52
*** ijw has joined #openstack-infra18:52
*** jamesmcarthur has quit IRC18:57
*** gtema has joined #openstack-infra18:57
*** priteau has quit IRC18:59
*** altlogbot_1 has joined #openstack-infra19:01
*** iurygregory_ has joined #openstack-infra19:02
*** ralonsoh has quit IRC19:03
mordredcorvus: I have another change lined up to try after this one - but I want to see the results of this one first19:04
*** altlogbot_1 has quit IRC19:04
*** iurygregory_ is now known as iurygregory19:04
openstackgerritJeremy Houser proposed openstack/project-config master: New repo for ranger-tempest-plugin  https://review.opendev.org/67015919:04
*** aedc has joined #openstack-infra19:08
*** Fidde has quit IRC19:09
*** ykarel|away has quit IRC19:09
clarkbAJaeger: done19:11
mordredcorvus: ok. inexplicably the double // fix seems to have fixed it ... and now we get a 404 later on /api/v1/orgs/opendev/repos - which makes me think maybe the python is going too quick the gitea hasn't 100% caught up with itself - so maybe we should be retrying19:12
mordredcorvus: any reason you can think of to not just wrap all of the gitea rest calls in a retry? they should all be largely idempotent, yeah?19:13
openstackgerritMerged openstack/diskimage-builder master: Set router solicitation delay with using NM  https://review.opendev.org/67010219:14
AJaegerthanks, clarkb19:14
clarkbNow just need reviews for https://review.opendev.org/#/c/669773/19:14
openstackgerritJeff Liu proposed zuul/zuul-operator master: Create tasks that spin up kubernetes and run the operator  https://review.opendev.org/66802919:15
*** tesseract has quit IRC19:16
mordredcorvus: actuallky - that makes no sense - the call to /api/v1/orgs/opendev/repos is after all of the interactions with org teams19:16
zbrwho maintains doc8 now? can we inject some life into it? I am willing to spend few minutes on it to keep it from collapse19:16
*** elod has quit IRC19:16
mordredcorvus: is it possible that get /api/v1/orgs/opendev/repos now returns a 404 if the repo list is empty?19:17
*** elod has joined #openstack-infra19:18
openstackgerritJeremy Houser proposed openstack/project-config master: New repo for ranger-tempest-plugin  https://review.opendev.org/67015919:19
*** irclogbot_2 has joined #openstack-infra19:19
*** weifan has joined #openstack-infra19:21
*** eharney has quit IRC19:21
openstackgerritMerged openstack/project-config master: Retire docs-specs  https://review.opendev.org/66885419:22
*** irclogbot_2 has quit IRC19:22
*** whoami-rajat has quit IRC19:24
*** weifan has quit IRC19:26
fungizbr: looks like it's an independent project managed by these folks: https://review.opendev.org/#/admin/groups/doc8-core19:26
fungireach out to them and see if one is willing to add you?19:27
fungizbr: alternatively, it's distributed under a free license so you could always just fork it19:27
zbryeah, I will send an email, sadly being only me it is unlikely to help, it needs at least one more to join. I really doubt any of current cores has interest in it.19:28
zbri don't think I need it so much to go for a fork19:29
fungican't hurt to put out a call for help on openstack-discuss if there are lots of openstack projects using it19:29
zbryep, i will send an email there and cc the current cores. maybe others may want to help. i doubt it requires more than an hour per month.19:30
mordredcorvus: nevermind. logic error. fix coming19:31
mordredcorvus: I had an inverted if19:34
openstackgerritMonty Taylor proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006019:34
openstackgerritJeremy Houser proposed openstack/project-config master: New repo for ranger-tempest-plugin  https://review.opendev.org/67015919:46
*** bhavikdbavishi has quit IRC19:46
openstackgerritJeff Liu proposed zuul/zuul-operator master: Create tasks that spin up kubernetes and run the operator  https://review.opendev.org/66802919:47
openstackgerritJeremy Houser proposed openstack/project-config master: New repo for ranger-tempest-plugin  https://review.opendev.org/67015919:47
*** iurygregory has quit IRC19:48
*** openstackgerrit has quit IRC19:49
*** openstackgerrit has joined #openstack-infra19:50
openstackgerritJeremy Houser proposed openstack/project-config master: New repo for ranger-tempest-plugin  https://review.opendev.org/67015919:50
*** weifan has joined #openstack-infra19:53
*** factor has quit IRC19:55
openstackgerritMonty Taylor proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006019:56
*** weifan has quit IRC19:57
clarkbI'm starting to debug the gentoo lack of ipv6 on fn now20:01
clarkbbooting gentoo test node20:01
*** Lucas_Gray has quit IRC20:06
*** factor has joined #openstack-infra20:06
*** Lucas_Gray has joined #openstack-infra20:08
*** Vadmacs has quit IRC20:17
*** dpawlik has quit IRC20:27
*** altlogbot_1 has joined #openstack-infra20:27
*** jamesmcarthur has joined #openstack-infra20:29
*** kjackal has joined #openstack-infra20:30
*** altlogbot_1 has quit IRC20:32
*** weifan has joined #openstack-infra20:34
*** weifan has quit IRC20:38
*** eharney has joined #openstack-infra20:42
*** dpawlik has joined #openstack-infra20:44
*** bobh has joined #openstack-infra20:45
*** hwoarang has quit IRC20:47
*** pcaruana has quit IRC20:48
*** dpawlik has quit IRC20:48
*** hwoarang has joined #openstack-infra20:48
*** xek_ has joined #openstack-infra20:51
*** xek has quit IRC20:52
*** dklyle has joined #openstack-infra20:55
*** bobh has quit IRC20:56
*** kjackal has quit IRC20:57
openstackgerritJeff Liu proposed zuul/zuul-operator master: Create tasks that spin up kubernetes and run the operator  https://review.opendev.org/66802920:58
*** ociuhandu has joined #openstack-infra20:59
*** mattw4 has quit IRC21:00
*** mattw4 has joined #openstack-infra21:00
*** cloudnull has quit IRC21:01
*** cloudnull has joined #openstack-infra21:02
*** altlogbot_3 has joined #openstack-infra21:03
*** ociuhandu has quit IRC21:08
*** altlogbot_3 has quit IRC21:08
*** xek_ has quit IRC21:11
*** weifan has joined #openstack-infra21:15
*** altlogbot_1 has joined #openstack-infra21:17
*** weifan has quit IRC21:19
openstackgerritJeff Liu proposed zuul/zuul-operator master: Create tasks that spin up kubernetes and run the operator  https://review.opendev.org/66802921:21
*** weifan has joined #openstack-infra21:22
*** altlogbot_1 has quit IRC21:22
*** michael-beaver has joined #openstack-infra21:24
openstackgerritJames E. Blair proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006021:29
corvusmordred: ^ i pushed up a new PS with a fix for the latest21:29
*** dklyle has quit IRC21:29
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add install-devstack test job  https://review.opendev.org/67019521:36
openstackgerritClark Boylan proposed opendev/glean master: Enable RAs with gentoo when using dhcpv6-stateless  https://review.opendev.org/67019721:38
clarkbprometheanfire: ^ can you take a look at that? I think that is the fix for the gentoo ipv6 only cloud problem21:39
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Normalize test jobs yaml  https://review.opendev.org/67019821:44
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-authorized-keys test job  https://review.opendev.org/67019921:44
*** factor has quit IRC21:45
*** factor has joined #openstack-infra21:46
*** irclogbot_3 has joined #openstack-infra21:47
*** goldyfruit has quit IRC21:47
prometheanfireclarkb: not familiar with that interface type, but looks good if it operates the same as slaac21:49
clarkbprometheanfire: ya I think the way it is supposed to work is it RAs for addr and route(r) info then dhcps for stuff like dns21:50
clarkbfungi: ^ that right?21:50
fungicorrect21:52
fungislaac+ra determine address and routes, dhcpv6 provides dns resolvers et cetera21:52
*** irclogbot_3 has quit IRC21:52
clarkbcool then I think I got that right21:53
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-fileserver test job  https://review.opendev.org/67020121:55
*** rh-jelabarre has quit IRC21:59
prometheanfireclarkb: yep, cool22:02
*** ekultails has quit IRC22:02
clarkbfungi: mordred corvus https://review.opendev.org/#/c/669773/6 can I get eyes on that change?22:08
prometheanfireclarkb: oh, be sure to hit the same section for the netifrc (openrc) version22:13
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-fileserver test job  https://review.opendev.org/67020122:14
*** aedc has quit IRC22:15
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-fileserver test job  https://review.opendev.org/67020122:17
fungioh, yep, i had already looked at 669773 but hadn't voted on it22:18
fungiapproved22:18
clarkbprometheanfire: I don't think there is similar code in that path22:18
clarkbprometheanfire: I think it will just accept ras by default due to sysctl22:19
*** rlandy is now known as rlandy|bbl22:19
fungiyeah, the kernel normally dtrt by magic22:19
*** slaweq has quit IRC22:20
prometheanfireok22:21
clarkb(I did look and couldn't find one)22:21
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-authorized-keys test job  https://review.opendev.org/67019922:22
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-fileserver test job  https://review.opendev.org/67020122:22
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-gpgkey test job  https://review.opendev.org/67020622:22
prometheanfireclarkb: ya, the write_gentoo stuff doesn't look like it even supports ipv6 at first glance22:23
clarkbya it relies on default kernel behavior with router advertisements I think22:24
clarkbwhich is what centos/rhel/fedora do too22:25
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-launchpad-credentaials test job  https://review.opendev.org/67020722:27
*** _erlon_ has quit IRC22:30
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-sshkey test job  https://review.opendev.org/67020822:32
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-fileserver test job  https://review.opendev.org/67020122:45
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-gpgkey test job  https://review.opendev.org/67020622:47
*** rcernin has joined #openstack-infra22:48
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-launchpad-credentaials test job  https://review.opendev.org/67020722:50
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add add-sshkey test job  https://review.opendev.org/67020822:51
openstackgerritJames E. Blair proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006022:54
corvusmordred: ^ pushed another simple typo fix22:54
*** aaronsheffield has quit IRC22:59
*** altlogbot_0 has joined #openstack-infra23:03
*** tkajinam has joined #openstack-infra23:06
*** altlogbot_0 has quit IRC23:08
*** tosky has quit IRC23:10
*** altlogbot_0 has joined #openstack-infra23:13
*** jamesmcarthur has quit IRC23:16
*** altlogbot_0 has quit IRC23:18
*** jamesmcarthur has joined #openstack-infra23:19
*** altlogbot_2 has joined #openstack-infra23:19
*** dchen has joined #openstack-infra23:21
*** jamesmcarthur has quit IRC23:24
*** altlogbot_2 has quit IRC23:24
*** altlogbot_2 has joined #openstack-infra23:25
*** jamesmcarthur has joined #openstack-infra23:25
*** bobh has joined #openstack-infra23:29
openstackgerritMerged opendev/glean master: network-manager: add network-pre dependencies  https://review.opendev.org/66977323:30
clarkbalrihgt it is late enough in the day that I don't want to tag glean now and then disappear23:30
*** altlogbot_2 has quit IRC23:30
clarkbif we can get https://review.opendev.org/#/c/670197/ in before tomorrow too late I can tag it with both fixes23:31
clarkbthen we can turn on fn again23:31
*** mattw4 has quit IRC23:32
*** michael-beaver has quit IRC23:34
*** bobh has quit IRC23:34
*** jamesmcarthur has quit IRC23:43
*** weifan has quit IRC23:53
openstackgerritJames E. Blair proposed opendev/system-config master: Translate gitea project creation to python  https://review.opendev.org/67006023:53
*** ijw has quit IRC23:55

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!