Friday, 2020-04-10

openstackgerritJeremy Stanley proposed openstack/project-config master: Revert "Introduce job for granular GitHub mirroring"  https://review.opendev.org/71883900:14
*** gary_perkins has quit IRC00:24
*** gary_perkins has joined #openstack-infra00:24
openstackgerritMerged openstack/project-config master: Revert "Introduce job for granular GitHub mirroring"  https://review.opendev.org/71883900:32
*** ysandeep|out is now known as ysandeep00:34
*** zxiiro has quit IRC00:56
*** gyee has quit IRC01:09
*** eharney has quit IRC01:12
*** yamamoto has joined #openstack-infra01:32
*** Goneri has quit IRC01:47
*** lbragstad_ has joined #openstack-infra01:59
*** lbragstad has quit IRC02:02
*** __ministry has joined #openstack-infra02:02
*** apetrich has quit IRC02:09
*** rh-jelabarre has quit IRC02:19
*** yamamoto has quit IRC02:21
*** yamamoto_ has joined #openstack-infra02:21
*** yamamoto_ has quit IRC02:23
*** yamamoto has joined #openstack-infra02:58
*** ociuhandu has joined #openstack-infra03:04
*** ociuhandu has quit IRC03:09
prometheanfirehmmm03:19
prometheanfireTASK [lxc_hosts : Ensure networking includes interfaces.d]03:19
prometheanfirenetworkd still not a thing?03:19
* prometheanfire blames cloudnull03:21
*** ysandeep is now known as ysandeep|off03:28
openstackgerritMohammed Naser proposed openstack/project-config master: Add infra-prod-update-system-config to deploy  https://review.opendev.org/71887703:31
prometheanfirehuh03:35
prometheanfire/bin/sh: 1: /usr/bin/apt-listchanges: not found03:35
*** ramishra has joined #openstack-infra03:38
*** ramishra has quit IRC03:38
*** ramishra has joined #openstack-infra03:39
prometheanfirewell..., gues I know what I'm doing https://bugs.launchpad.net/openstack-ansible/+bug/183953503:40
openstackLaunchpad bug 1839535 in openstack-ansible "debian9 container image is missing apt-listchanges package, but has the apt.conf.d/20listchanges config file, causing failure." [Undecided,New] - Assigned to Dmitriy Rabotyagov (noonedeadpunk)03:40
prometheanfiredebian 10 too03:40
fungithe apt-listchanges is priority standard, so may not be on a minimal image03:41
fungierm the apt-listchanges package is priority standard03:42
*** yamamoto has quit IRC03:42
*** HenryG has joined #openstack-infra03:44
*** rcernin has joined #openstack-infra03:46
*** yamamoto has joined #openstack-infra03:51
*** yamamoto has quit IRC04:14
*** cdearborn has quit IRC04:17
*** yamamoto has joined #openstack-infra04:18
*** evrardjp has quit IRC04:37
*** evrardjp has joined #openstack-infra04:37
prometheanfirehmm meant that for openstack-ansible04:53
prometheanfirehttps://review.opendev.org/71888204:53
openstackgerritIan Wienand proposed zuul/zuul-jobs master: ensure-pip: export ensure_pip_virtualenv_command  https://review.opendev.org/71822404:53
openstackgerritIan Wienand proposed zuul/zuul-jobs master: fetch-zuul-cloner: use ensure-pip  https://review.opendev.org/71788204:53
openstackgerritIan Wienand proposed zuul/zuul-jobs master: fetch-subunit-output test: use ensure-pip  https://review.opendev.org/71822504:53
openstackgerritIan Wienand proposed zuul/zuul-jobs master: ensure-tox: use ensure-pip role  https://review.opendev.org/71766304:53
openstackgerritIan Wienand proposed zuul/zuul-jobs master: Update Fedora to 31  https://review.opendev.org/71765704:53
*** rcernin has quit IRC05:03
*** armax has quit IRC05:23
*** armax has joined #openstack-infra05:34
*** lmiccini has joined #openstack-infra05:40
*** armax has quit IRC06:02
*** dpawlik has joined #openstack-infra06:20
*** ociuhandu has joined #openstack-infra06:35
*** yamamoto has quit IRC06:37
*** ociuhandu has quit IRC06:40
*** yamamoto has joined #openstack-infra06:44
*** kota_ has quit IRC06:48
*** kota_ has joined #openstack-infra06:49
*** yamamoto has quit IRC06:58
*** yamamoto has joined #openstack-infra06:59
*** jcapitao has joined #openstack-infra06:59
*** slaweq has joined #openstack-infra07:00
*** ociuhandu has joined #openstack-infra07:04
*** jlvillal has quit IRC07:08
*** jlvillal has joined #openstack-infra07:08
*** ociuhandu has quit IRC07:09
*** pgaxatte has joined #openstack-infra07:14
*** ociuhandu has joined #openstack-infra07:14
*** ociuhandu has quit IRC07:15
*** yamamoto has quit IRC07:15
*** dklyle has quit IRC07:16
*** ociuhandu has joined #openstack-infra07:16
*** yamamoto has joined #openstack-infra07:17
*** ociuhandu has quit IRC07:19
*** ociuhandu has joined #openstack-infra07:19
*** tosky has joined #openstack-infra07:32
*** ricolin has joined #openstack-infra07:34
*** rpittau|afk is now known as rpittau07:36
*** Tengu has quit IRC07:39
*** Tengu has joined #openstack-infra07:41
*** slaweq has quit IRC07:51
*** hashar has joined #openstack-infra07:52
*** ramishra has quit IRC08:04
*** dchen has quit IRC08:11
*** roman_g has quit IRC08:18
*** ociuhandu has quit IRC08:21
*** pkopec has joined #openstack-infra08:22
*** ociuhandu has joined #openstack-infra08:23
*** tkajinam has quit IRC08:25
*** ociuhandu has quit IRC08:28
*** xek has joined #openstack-infra08:32
*** jtomasek has joined #openstack-infra08:37
*** jtomasek has quit IRC08:38
openstackgerritBernard Cafarelli proposed openstack/project-config master: Update Grafana dashboards for stable Neutron releases  https://review.opendev.org/71867608:51
*** gfidente|afk is now known as gfidente09:09
*** jtomasek has joined #openstack-infra09:23
*** ociuhandu has joined #openstack-infra09:32
*** ociuhandu has quit IRC09:36
noonedeadpunkAJaeger: yeah, I was aware about other places to clean, just thought we can postpone these fixes a bit and they shouldn't be breaking things09:36
AJaegernoonedeadpunk: yeah, those can wait - ther equired-projects not09:38
*** yamamoto has quit IRC10:01
*** yamamoto has joined #openstack-infra10:01
*** yamamoto has quit IRC10:02
*** ociuhandu has joined #openstack-infra10:04
*** persia_ is now known as persia10:07
*** yamamoto has joined #openstack-infra10:13
*** rpittau is now known as rpittau|bbl10:19
*** yamamoto has quit IRC10:26
*** pgaxatte has quit IRC10:32
*** pgaxatte has joined #openstack-infra10:39
*** yamamoto has joined #openstack-infra10:52
*** yamamoto has quit IRC10:53
*** jcapitao is now known as jcapitao_lunch10:58
*** pkopec_ has joined #openstack-infra11:19
*** pkopec has quit IRC11:22
*** yamamoto has joined #openstack-infra11:34
*** pkopec_ has quit IRC11:45
*** pkopec_ has joined #openstack-infra11:46
*** ociuhandu has quit IRC11:50
*** ociuhandu has joined #openstack-infra11:51
*** ociuhandu has quit IRC11:56
*** rpittau|bbl is now known as rpittau11:58
*** yamamoto has quit IRC11:59
*** yamamoto has joined #openstack-infra12:12
*** jcapitao_lunch is now known as jcapitao12:18
openstackgerritThierry Carrez proposed openstack/project-config master: [check-approval] Use committer instead of owner  https://review.opendev.org/71899412:36
*** pkopec_ has quit IRC13:01
*** pkopec_ has joined #openstack-infra13:02
*** ociuhandu has joined #openstack-infra13:09
*** zigo has quit IRC13:16
*** prometheanfire has quit IRC13:16
*** cdearborn has joined #openstack-infra13:23
*** yamamoto has quit IRC13:38
*** psachin has joined #openstack-infra13:45
*** dklyle has joined #openstack-infra13:55
*** rh-jelabarre has joined #openstack-infra13:57
*** yamamoto has joined #openstack-infra14:00
*** zigo has joined #openstack-infra14:03
*** slaweq has joined #openstack-infra14:18
*** yamamoto has quit IRC14:23
*** psachin has quit IRC14:24
*** yamamoto has joined #openstack-infra14:27
*** slaweq has quit IRC14:27
*** ociuhandu has quit IRC14:28
*** ociuhandu has joined #openstack-infra14:28
*** slaweq has joined #openstack-infra14:29
*** ociuhandu has quit IRC14:33
*** slaweq has quit IRC14:33
*** ociuhandu has joined #openstack-infra14:34
openstackgerritSean McGinnis proposed openstack/openstack-zuul-jobs master: Add Python3 project templates for Victoria release  https://review.opendev.org/71903114:38
*** ociuhandu has quit IRC14:38
*** bdodd has quit IRC14:42
*** bdodd has joined #openstack-infra14:43
*** yamamoto has quit IRC14:44
*** pgaxatte has quit IRC14:46
*** yamamoto has joined #openstack-infra14:46
openstackgerritSean McGinnis proposed openstack/openstack-zuul-jobs master: Add Python3 project templates for Victoria release  https://review.opendev.org/71903114:51
*** yamamoto has quit IRC14:51
AJaegersmcginnis: the TC page you reference mentions py37 but your change omits it14:53
AJaegersmcginnis: I should have read next paragraph - all fine.14:53
smcginnisAJaeger: Note the text below that says it is only for integration tests.14:53
smcginnis;)14:53
*** rpittau is now known as rpittau|afk14:57
AJaegerconfig-core, please review 719031  ^15:01
openstackgerritMerged openstack/openstack-zuul-jobs master: Add Python3 project templates for Victoria release  https://review.opendev.org/71903115:13
*** ociuhandu has joined #openstack-infra15:24
*** lmiccini has quit IRC15:27
*** yamamoto has joined #openstack-infra15:29
*** gyee has joined #openstack-infra15:31
*** ociuhandu has quit IRC15:35
*** ociuhandu has joined #openstack-infra15:35
*** yamamoto has quit IRC15:36
*** Goneri has joined #openstack-infra15:40
*** ociuhandu has quit IRC15:40
openstackgerritMohammed Naser proposed openstack/project-config master: Drop dependencies from manage-projects  https://review.opendev.org/71887715:46
*** jcapitao has quit IRC16:00
*** armax has joined #openstack-infra16:01
-openstackstatus- NOTICE: review.opendev.org is being restarted for scheduled maintenance; see http://lists.opendev.org/pipermail/service-announce/2020-April/000003.html16:05
openstackgerritMerged openstack/project-config master: Drop dependencies from manage-projects  https://review.opendev.org/71887716:06
*** slaweq has joined #openstack-infra16:12
*** artom has quit IRC16:17
openstackgerritMohammed Naser proposed openstack/project-config master: Revert "Revert "Introduce job for granular GitHub mirroring""  https://review.opendev.org/71904716:24
*** ociuhandu has joined #openstack-infra16:29
*** evrardjp has quit IRC16:37
*** evrardjp has joined #openstack-infra16:37
*** slaweq has quit IRC16:40
*** prometheanfire has joined #openstack-infra16:48
*** dpawlik has quit IRC16:53
openstackgerritMohammed Naser proposed openstack/project-config master: Revert "Revert "Introduce job for granular GitHub mirroring""  https://review.opendev.org/71904716:57
*** __ministry has quit IRC16:59
-openstackstatus- NOTICE: etherpad.openstack.org will be offline for about 30 minutes while it is migrated to a new server with a new hostname; see http://lists.opendev.org/pipermail/service-announce/2020-April/000003.html17:02
openstackgerritAndreas Jaeger proposed zuul/zuul-jobs master: Fix check_jobs_documented linter  https://review.opendev.org/71905417:09
*** hashar has quit IRC17:20
*** diablo_rojo has joined #openstack-infra17:45
*** marcosilva has joined #openstack-infra17:48
*** ociuhandu has quit IRC17:52
*** admcleod has quit IRC17:52
-openstackstatus- NOTICE: The etherpad migration is still in progress; revised estimated time of completion 18:30 UTC17:52
*** admcleod has joined #openstack-infra17:53
*** marcosilva has quit IRC17:56
*** melwitt has quit IRC17:59
*** mugsie has quit IRC17:59
*** EmilienM has quit IRC17:59
*** cmurphy has quit IRC17:59
*** hwoarang has quit IRC17:59
*** cyberpear has quit IRC17:59
*** ricolin has quit IRC18:03
*** melwitt has joined #openstack-infra18:03
*** mugsie has joined #openstack-infra18:03
*** cmurphy has joined #openstack-infra18:03
*** EmilienM has joined #openstack-infra18:03
*** hwoarang has joined #openstack-infra18:03
*** cyberpear has joined #openstack-infra18:03
*** jtomasek has quit IRC18:06
*** EmilienM has quit IRC18:12
*** EmilienM has joined #openstack-infra18:13
*** jtomasek has joined #openstack-infra18:23
*** slaweq has joined #openstack-infra18:30
*** jamesmcarthur has joined #openstack-infra18:54
*** jamesmcarthur has quit IRC19:30
*** jamesmcarthur has joined #openstack-infra19:35
smcginnisPer http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014055.html - could I request assistance in deleting the openstack/cinder driverfixes/mitaka and driverfixes/newton branches?19:38
*** jamesmcarthur has quit IRC19:40
fungismcginnis: happy to help, just a moment19:45
*** markmcclain has quit IRC19:45
*** jamesmcarthur has joined #openstack-infra19:45
*** markmcclain has joined #openstack-infra19:46
fungi#status log deleted openstack/cinder branch driverfixes/mitaka formerly at a77f17e3778377a0d7aee9bf412554551a6b843519:47
openstackstatusfungi: finished logging19:48
fungismcginnis: "branch openstack/cinder,refs/heads/driverfixes/newton has open changes"19:48
fungiplease abandon those and i'll be happy to try again19:48
smcginnisfungi: On it!19:48
smcginnisfungi: OK, abandoned on WIP patch that was still out there.19:50
fungi#status log deleted openstack/cinder branch driverfixes/newton formerly at b9f6cd23ed7c806487d5df065b19741aecd3643819:50
openstackstatusfungi: finished logging19:50
smcginnisfungi: Thanks!19:50
fungismcginnis: done, i'll reply on the ml19:50
smcginnisExcellent19:50
fungiand sent19:51
smcginnisAlways nice to wrap up pending items before the holiday. I had that one pending way too long.19:52
*** artom has joined #openstack-infra19:55
*** nhicher has quit IRC20:02
*** nhicher has joined #openstack-infra20:02
-openstackstatus- NOTICE: Due to a database migration error, etherpad.opendev.org is offline until further notice.20:09
*** aedc_ has joined #openstack-infra20:18
*** aedc has quit IRC20:20
*** aedc_ has quit IRC20:28
*** sshnaidm|off has quit IRC20:46
*** pkopec_ has quit IRC20:48
*** jamesmcarthur_ has joined #openstack-infra20:57
*** jamesmca_ has joined #openstack-infra20:58
melwittcan anyone sanity check us whether this is recommended way to handle required-projects on stable branch? https://review.opendev.org/71910520:59
clarkbmelwitt: I think you only need the override checkout if you intend to test train placement with queens nova or whatever. Basically if hte branches aren't already aligned20:59
*** jamesmcarthur has quit IRC21:00
clarkbmelwitt: bceause zuul should use the same branch for that dep as the branch for the change in question. The exception to that is if the branch does not exist it will fall back to master21:00
melwittclarkb: ok, what we're seeing is that on stable/train, it's checking out placement from master21:00
melwittif we don't do this21:00
melwittoh, hm21:00
melwittplacement has a stable/train branch so...21:01
gmannclarkb: and i think that is same for other functional job in nova which has placement as required_project21:01
clarkbthe branch selection should always be based on the triggering event21:01
gmannmelwitt: clarkb - https://zuul.opendev.org/t/openstack/build/b460a8c59ad64e57b871aa3c00638b01/log/job-output.txt#28321:01
clarkbare these jobs running in periodic maybe? since that doesn't have chagne event data21:01
*** ociuhandu has joined #openstack-infra21:01
*** jamesmcarthur_ has quit IRC21:02
melwittno these are plain functional test jobs. now, I'm not sure why we need required-projects on our func test jobs, I was going to guess because we use test fixtures from placement?21:02
fungior because you want to install placement from source instead of releases on pypi?21:02
melwittthis is the example change where it failed https://review.opendev.org/71893421:03
clarkbmelwitt: what required-projects will do is ensure /home/zuul/src/review.opendev.org/openstack/placement is in place next to /home/zuul/src/review.opendev.org/openstack/nova21:03
clarkband they should both be checked out to the branch that triggered the event21:03
clarkbmelwitt: ya I think because that chagne was to train placement would be checked out to train too and you can remove the override checkout directive21:03
gmannand that is how other job with required_project does- checkout the stable branch version of required_project repo21:04
melwittsorry, I'm confused. that change (without override-checkout) is checking out placement from master21:04
clarkbmelwitt: in this job https://zuul.opendev.org/t/openstack/build/40f44994e05f4466b7576c0e22d0b13a ?21:05
gmannhttps://zuul.opendev.org/t/openstack/build/b460a8c59ad64e57b871aa3c00638b01/log/job-output.txt#78221:05
gmannclarkb: ^^21:05
melwittclarkb: I was looking at this one https://zuul.opendev.org/t/openstack/build/b460a8c59ad64e57b871aa3c00638b0121:05
clarkbhttps://zuul.opendev.org/t/openstack/build/40f44994e05f4466b7576c0e22d0b13a/log/job-output.txt#184 says its set to checkout train21:05
*** ociuhandu has quit IRC21:06
clarkbhttps://zuul.opendev.org/t/openstack/build/b460a8c59ad64e57b871aa3c00638b01/log/job-output.txt#196 same thing for that job so we need to work from there and figure out what is grabbing master I guess21:06
melwittthe -py36 failure is legit. I need to update my patch for that one21:06
gmannclarkb: melwitt ohh here is issue - https://github.com/openstack/nova/blob/master/tox.ini#L7321:07
melwittclarkb: yeah that's what gmann was saying earlier in -nova, he said it's not using the placement that was already cloned from train21:07
clarkbhttps://zuul.opendev.org/t/openstack/build/b460a8c59ad64e57b871aa3c00638b01/log/job-output.txt#562 yes that21:07
gmannhttps://github.com/openstack/nova/blob/stable/train/tox.ini#L8721:07
melwittoh, good spot21:07
clarkbits never using the zuul version21:07
gmannin stable train aslo tox use master as dep21:07
clarkbthat needs to be udpated to use the zuul version if you want to avoid doing unnecessary clones and to test with depends on21:08
melwittgmann: do you know how to tell it to use the zuul version? I don't21:09
clarkbI think what you can do is have it check and env var and default to the opendev url21:09
gmanni think we can rely on required-projects or just add placement in test-requirements.txt21:10
clarkbthen in your job set the env var to be /home/zuul/src/review.opendev.org/openstack/placement21:10
melwittgmann: yeah, I wish I knew why it's being done this way. mriedem did it so I'm sure there was a good reason21:10
clarkbhttps://github.com/openstack/nova/blob/master/tox.ini#L29 the constraints file is selected in a similar way to what I describe21:10
melwittah ok21:11
clarkbmelwitt: and in the tox jobs should be something that sets that constraints env var to use openstack/requiremetns from zuul21:11
clarkbI think you can directly adopt that system21:11
fungialso this may predate publication of services to pypi, so just having placement be a normal package requirement when running locally on your dev environment may suffice these days?21:13
*** gfidente has quit IRC21:13
clarkbunless you want to use depends on21:13
fungihow would you depends-on in your local dev environment?21:13
clarkbit wouldn't but it doesn't with constraints either21:14
clarkbyou'd probably do a tox deps install only, then install version of placement you want, then run tox tests only21:14
fungiright, it would just go by the supported placement versions declared in requirements.txt if running locally21:14
fungiand in zuul, tox-siblings would make sure the required-projects are installed from source, yeah?21:15
fungi(using the checkout zuul has provided)21:16
clarkbyup thats a good point I always forget about siblings21:16
clarkbso ya listing it as a normal dep is probably simplest here21:16
fungibut it's friday evening after a long day/week so entirely possible i shouldn't be engineering complex solutions at this stage21:16
gmannohk it is only required for functional tests so putting in requirement file is not good. i thought it is or both functional and unit deps21:17
fungitest-requirements though?21:17
melwittyeah the code comment from mriedem says we don't want it for unit tests, only func tests21:17
clarkbor just list it in tox.ini and siblings should do the correct thing21:17
fungiyeah, in the deps var21:17
gmanntest-requirements also should not as unit test end up require it21:17
clarkbthe issue here is you are saying "install master placement" but if you say install placement version range from train it should work21:18
gmannyeah deps var as it is currently but with u-c21:18
fungii actually think our historical test-requirements.txt model is terribly inflexible anyway. for my personal projects i just put tox-environment-specific requirements in the deps for each individual testenv21:19
melwittclarkb, gmann: ok, so it sounds like we need to update the deps in tox.ini to use the zuul version, that way we don't have to hard code each stable branch's line with literally stable/<release>?21:20
clarkbmelwitt: and you don't even need to use the zuul version explicitly if you just list placement as a dep and tox-siblings is in use21:21
clarkblet me check the job log to see if it was in use there21:21
melwittoh. sorry, I know almost nothing about this until today21:21
melwittso the dep "git+https://opendev.org/openstack/placement#egg=openstack-placement" is not what you mean by listing placement as a dep21:23
gmannenv:PLACEMENT_VERSION: /home/zuul/src/review.opendev.org/openstack/placement}21:24
clarkbcorrect because that literally means do a git clone and install what you get21:24
gmannthis way ?21:24
clarkbgmann: also we don't need that because of tox siblings which I think are enabled by default21:24
gmannohk great21:24
clarkball you should need to do is add placement>x,<y21:24
clarkbto ensure it installs the train version range on train21:25
clarkbthen the tox siblings handling will look to see if it needs to install tox from zuul for you21:25
melwitthm, that means we'd need different stable-only patches for each branch21:25
gmannclarkb: do we still bed version cap things there ?or just deps placement'' and it will pick form zuul which will be stable branch checkout21:26
clarkbgmann: you need it for the local run21:26
gmannyeah what melwitt mentioned, we should avid that21:26
clarkbsince tox can be run outside of a zuul context21:26
melwittso maybe gmann earlier suggestion would be better so we just have the same thing we don't have to do a bunch of different things21:26
clarkbfwiw its not different than curating requirements.txt21:26
clarkbyou are just doing it in tox.ini to avoid installing it all the time21:26
gmannohk for local run I think we reply on env they run and install proper version21:27
gmannyeah21:27
gmanns/reply/rely21:28
*** xek has quit IRC21:43
*** diablo_rojo has quit IRC21:54
*** ociuhandu has joined #openstack-infra22:13
*** jamesdenton has joined #openstack-infra22:15
*** jamesden_ has quit IRC22:16
-openstackstatus- NOTICE: Maintenance on etherpad.opendev.org is complete and the service is available again22:25
*** ociuhandu has quit IRC22:29
*** ociuhandu has joined #openstack-infra22:30
*** ociuhandu has quit IRC22:35
*** smarcet has joined #openstack-infra22:53
*** hashar has joined #openstack-infra22:58
*** jamesden_ has joined #openstack-infra23:00
*** jamesdenton has quit IRC23:04
*** hashar has quit IRC23:13
*** smarcet has left #openstack-infra23:18
*** jamesmca_ has quit IRC23:19
*** jamesmcarthur has joined #openstack-infra23:20
*** tosky has quit IRC23:20
*** rcernin has joined #openstack-infra23:20
*** jamesmcarthur has quit IRC23:25
*** jamesmcarthur has joined #openstack-infra23:33
*** yamamoto has joined #openstack-infra23:36
*** yamamoto has quit IRC23:40

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