Friday, 2019-05-31

clarkbor just rm the file and let the cron sweep through00:00
clarkbzm* and zuul01 are in similar situation (I think zuul01 was legit using that until we did the base.yaml split out and took zuul out of the git playbook)00:00
*** ianychoi has joined #openstack-infra00:00
ianwyeah, i guess at least if we see this again it won't be a total surprise00:01
clarkbhttp://mirror.dfw.rax.opendev.org/ubuntu/ exists now too00:02
fungisolace in small victories? ;)00:06
fungii finished mowing the lawn after giving up halfway through yesterday. i'll take comfort in thaht00:06
*** hwoarang has quit IRC00:09
*** hwoarang has joined #openstack-infra00:10
*** weifan has quit IRC00:11
*** weifan has joined #openstack-infra00:11
ianwok ... ansible -v -f 10 'z*' -m shell -a "rm /etc/facter/facts.d/ansible.json" look ok?00:14
*** weifan has quit IRC00:15
*** lseki has quit IRC00:18
ianwwell, it's done ... so i think that should mean opendev mirror is full operational00:19
ianwproxy_80_access.log is probably the wrong name for that log file00:20
fungiif we grow the list of replacement opendev mirrors incrementally, mind the case sensitivity on the conditional check00:21
fungii just corrected the case on the region name since it wasn't immediately obvious to me how to make that an case-insensitive comparison00:21
openstackgerritIan Wienand proposed opendev/system-config master: mirror: rename 80/443 log files  https://review.opendev.org/66235700:23
ianwfungi: yeah, that one had never occurred to me00:24
*** rkukura has quit IRC00:32
*** rkukura has joined #openstack-infra00:32
*** hwoarang has quit IRC00:32
*** hwoarang has joined #openstack-infra00:33
openstackgerritTristan Cacqueray proposed zuul/zuul master: executor: run cleanup playbook on stop  https://review.opendev.org/66188100:33
*** Lucas_Gray has quit IRC00:34
*** rkukura has quit IRC00:42
*** hwoarang has quit IRC00:48
*** hwoarang has joined #openstack-infra00:49
openstackgerritPaul Belanger proposed zuul/zuul master: Discard GitHub PullReview if incomplete  https://review.opendev.org/66234700:55
*** mriedem_away has quit IRC00:57
*** hwoarang has quit IRC01:15
*** yamamoto has joined #openstack-infra01:16
*** hwoarang has joined #openstack-infra01:23
ianwAJaeger: so https://gerrit-dash-creator.readthedocs.io/en/latest/dashboards/index.html is back .. i had to go into the settings in RTD and flip it to use python301:25
ianwi'm not sure about the style though?  the old page used the old openstack theme, now it's a generic rtd thing with an ad01:26
ianwoh doh, you referenced that in the CL ... all good01:31
*** hwoarang has quit IRC01:35
ianwAJager: so i guess the only thing for us to note is that we can flip the RTD python building version by adding a config file and "python:" to make sure we use 3.x -- https://docs.readthedocs.io/en/stable/config-file/v2.html01:35
*** hwoarang has joined #openstack-infra01:35
ianwor, someone with access can flip it on the settings page, but the config file seems better overall01:35
*** michael-beaver has quit IRC01:40
*** gyee has quit IRC01:42
*** hwoarang has quit IRC01:57
*** hwoarang has joined #openstack-infra01:57
*** apetrich has quit IRC01:57
*** whoami-rajat has joined #openstack-infra01:57
*** igordc has quit IRC01:58
*** hwoarang has quit IRC02:08
*** hwoarang has joined #openstack-infra02:09
*** markvoelker has joined #openstack-infra02:10
*** markvoelker has quit IRC02:14
*** bnemec has quit IRC02:18
*** bnemec has joined #openstack-infra02:29
*** bobh has joined #openstack-infra02:34
*** gregoryo has joined #openstack-infra02:38
*** bobh has quit IRC02:39
*** hwoarang has quit IRC02:49
*** hwoarang has joined #openstack-infra02:49
*** ricolin has joined #openstack-infra02:54
*** hwoarang has quit IRC02:58
*** hwoarang has joined #openstack-infra02:59
*** nicolasbock has quit IRC02:59
*** hwoarang has quit IRC03:07
*** hwoarang has joined #openstack-infra03:07
*** dave-mccowan has quit IRC03:08
*** bnemec has quit IRC03:11
openstackgerritTristan Cacqueray proposed zuul/zuul master: docs: add cleanup-run documentation  https://review.opendev.org/66214703:18
*** hwoarang has quit IRC03:27
*** hwoarang has joined #openstack-infra03:28
*** bnemec has joined #openstack-infra03:29
*** ykarel has joined #openstack-infra03:52
*** Lucas_Gray has joined #openstack-infra04:09
*** Lucas_Gray has quit IRC04:16
*** Lucas_Gray has joined #openstack-infra04:21
*** toabctl has joined #openstack-infra04:41
openstackgerritMerged openstack/diskimage-builder master: allow the use of non-bzip compressed stages for building gentoo  https://review.opendev.org/66109804:42
*** hwoarang has quit IRC04:45
*** hwoarang has joined #openstack-infra04:45
*** raukadah is now known as chandankumar04:51
*** raissa has quit IRC04:56
*** kjackal has quit IRC05:01
*** tkajinam has quit IRC05:03
*** pcaruana has joined #openstack-infra05:03
*** tkajinam has joined #openstack-infra05:03
*** tkajinam has quit IRC05:04
*** ricolin has quit IRC05:05
*** igordc has joined #openstack-infra05:06
*** tkajinam has joined #openstack-infra05:07
*** rcernin has quit IRC05:07
openstackgerritTristan Cacqueray proposed zuul/zuul master: executor: run cleanup playbook on stop  https://review.opendev.org/66188105:19
*** dpawlik has joined #openstack-infra05:25
*** hwoarang has quit IRC05:25
*** hwoarang has joined #openstack-infra05:26
*** Qiming has joined #openstack-infra05:26
*** ramishra_ has quit IRC05:26
*** quiquell has joined #openstack-infra05:31
*** tkajinam_ has joined #openstack-infra05:33
*** hwoarang has quit IRC05:34
*** hwoarang has joined #openstack-infra05:34
*** tkajinam has quit IRC05:36
*** ramishra has joined #openstack-infra05:42
*** kjackal has joined #openstack-infra05:42
*** Lucas_Gray has quit IRC05:44
*** hwoarang has quit IRC05:44
*** hwoarang has joined #openstack-infra05:44
*** ykarel is now known as ykarel|afk05:47
*** dtantsur|afk is now known as dtantsur05:52
*** igordc has quit IRC05:52
*** hwoarang has quit IRC05:58
*** hwoarang has joined #openstack-infra05:59
*** yamamoto has quit IRC06:01
*** lpetrut has joined #openstack-infra06:03
*** dpawlik has quit IRC06:08
*** dpawlik has joined #openstack-infra06:08
openstackgerritIan Wienand proposed openstack/diskimage-builder master: fail early when lates build information can not be fetched  https://review.opendev.org/65789206:09
*** kopecmartin|off is now known as kopecmartin06:09
*** rcernin has joined #openstack-infra06:10
AJaegerianw: thanks for fixing https://gerrit-dash-creator.readthedocs.io/ !06:12
*** gregoryo has quit IRC06:20
AJaegerconfig-core, could you review https://review.opendev.org/662156 to add upper-constraints directly to post docs jobs, please?06:21
AJaegerI saw clarkb's earlier note - let's be careful what to merge to not create conflicts with the rename changes...06:21
openstackgerritMerged openstack/diskimage-builder master: bindep: add sudo  https://review.opendev.org/63591306:23
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Makes image caching more resilient  https://review.opendev.org/65647806:31
*** xek_ has joined #openstack-infra06:34
*** yamamoto has joined #openstack-infra06:36
*** e0ne has joined #openstack-infra06:36
*** yamamoto has quit IRC06:41
*** yamamoto has joined #openstack-infra06:41
*** dpawlik has quit IRC06:43
*** bobh has joined #openstack-infra06:45
*** ramishra has quit IRC06:47
*** armax has quit IRC06:49
*** ramishra has joined #openstack-infra06:49
*** bobh has quit IRC06:50
*** yamamoto has quit IRC06:51
*** ykarel|afk has quit IRC06:52
*** ykarel has joined #openstack-infra06:54
*** apetrich has joined #openstack-infra06:57
*** yamamoto has joined #openstack-infra06:59
openstackgerritIan Wienand proposed openstack/diskimage-builder master: bindep: exclude zypper from debian-stretch  https://review.opendev.org/63591007:02
*** ginopc has joined #openstack-infra07:07
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Increase size of EFI system partition (again)  https://review.opendev.org/64170207:09
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Increase size of EFI system partition (again)  https://review.opendev.org/64170207:10
*** tesseract has joined #openstack-infra07:12
*** yamamoto has quit IRC07:18
*** ccamacho has quit IRC07:24
*** yamamoto has joined #openstack-infra07:24
*** ccamacho has joined #openstack-infra07:24
*** roman_g has joined #openstack-infra07:25
*** dchen has quit IRC07:26
*** tosky has joined #openstack-infra07:33
*** tkajinam_ has quit IRC07:42
*** dpawlik has joined #openstack-infra07:46
*** jpich has joined #openstack-infra07:48
*** iurygregory has joined #openstack-infra07:50
*** adriancz has joined #openstack-infra07:52
*** rcernin has quit IRC07:52
openstackgerritMerged openstack/diskimage-builder master: Add version-less RHEL element for RHEL7 and RHEL8  https://review.opendev.org/64373107:56
openstackgerritMerged openstack/diskimage-builder master: Deprecate rhel7 in favor of rhel  https://review.opendev.org/64771008:01
*** happyhemant has joined #openstack-infra08:05
*** kjackal has quit IRC08:06
openstackgerritMerged openstack/diskimage-builder master: Add option to skip update packages  https://review.opendev.org/63606008:13
*** pkopec has joined #openstack-infra08:15
*** lucasagomes has joined #openstack-infra08:19
*** Lucas_Gray has joined #openstack-infra08:20
openstackgerritSlawek Kaplonski proposed openstack/openstack-zuul-jobs master: Add neutron-tempest-plugin to bgpvpn-bagpipe job  https://review.opendev.org/66240208:33
*** ginopc has quit IRC08:34
*** derekh has joined #openstack-infra08:40
*** janki has joined #openstack-infra08:42
*** kjackal has joined #openstack-infra08:44
openstackgerritMerged openstack/diskimage-builder master: fail early when lates build information can not be fetched  https://review.opendev.org/65789208:45
*** ramishra has quit IRC08:46
*** ramishra has joined #openstack-infra08:46
openstackgerritMerged openstack/diskimage-builder master: Makes image caching more resilient  https://review.opendev.org/65647808:54
openstackgerritMerged openstack/diskimage-builder master: Increase size of EFI system partition (again)  https://review.opendev.org/64170209:05
*** ykarel is now known as ykarel|lunch09:07
openstackgerritMerged openstack/diskimage-builder master: bindep: exclude zypper from debian-stretch  https://review.opendev.org/63591009:10
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213409:24
*** panda is now known as panda|ruck09:24
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213409:29
*** ociuhandu has joined #openstack-infra09:33
*** yboaron_ has quit IRC09:36
*** stakeda has quit IRC09:37
*** yboaron_ has joined #openstack-infra09:37
*** electrofelix has joined #openstack-infra09:38
*** ykarel|lunch is now known as ykarel09:39
*** janki has quit IRC09:46
*** janki has joined #openstack-infra09:48
*** janki has quit IRC09:49
*** janki has joined #openstack-infra09:50
*** kjackal has quit IRC09:52
*** janki has quit IRC09:52
*** kjackal has joined #openstack-infra09:53
*** janki has joined #openstack-infra09:53
*** Lucas_Gray has quit IRC09:55
*** yamamoto has quit IRC09:58
*** bobh has joined #openstack-infra10:00
*** janki has quit IRC10:02
*** janki has joined #openstack-infra10:03
*** janki has quit IRC10:04
*** bobh has quit IRC10:04
*** yboaron_ has quit IRC10:07
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213410:12
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213410:12
*** rfarr__ has joined #openstack-infra10:23
*** rfarr_ has joined #openstack-infra10:25
*** rfarr__ has quit IRC10:28
*** yamamoto has joined #openstack-infra10:32
*** yboaron_ has joined #openstack-infra10:38
AJaegermordred: do you have a governance change up that correspondents to https://review.opendev.org/#/c/657277/ ?10:39
*** ginopc has joined #openstack-infra10:39
*** yamamoto has quit IRC10:40
*** yboaron_ has quit IRC10:40
*** yboaron_ has joined #openstack-infra10:41
*** yboaron_ has quit IRC10:46
*** yboaron_ has joined #openstack-infra10:46
*** yboaron_ has quit IRC10:48
*** yboaron_ has joined #openstack-infra10:51
*** yboaron_ has quit IRC10:53
*** yboaron_ has joined #openstack-infra10:54
*** yboaron_ has quit IRC10:56
*** yboaron_ has joined #openstack-infra10:56
*** yboaron_ has quit IRC10:58
*** yboaron_ has joined #openstack-infra10:59
iceyper the comment on https://review.opendev.org/#/c/660908/ - how do we handle project renames for a project coming under governance from the 'x' namespace?11:02
*** yboaron_ has quit IRC11:04
AJaegericey: you need to make a change like https://review.opendev.org/644584 - see https://docs.openstack.org/infra/manual/creators.html#project-renames for full details.11:04
AJaegericey: I doubt we can squeeze you in, clarkb needs to answer that.11:05
iceyAJaeger: even though the only rename is s/x/openstack? ie, the current path is `https://opendev.org/x/charm-vault`, but it needs to become 'https://opendev.org/openstack/charm-vault'? - on that assumption, I'll start on the patch11:06
AJaegericey: but if we do, provide a change stacked on top of https://review.opendev.org/657277 - without touching the rest of the stack!11:06
iceyAJaeger: shouldn't kill me to wait :)11:06
AJaegericey: that rename needs a gerrit downtime, we will have one in 3 hours - but next one will come...11:06
*** yboaron_ has joined #openstack-infra11:07
iceyAJaeger: is there an easy way to identify the last entry in that stack? :-P11:07
AJaegericey: the one I gave you is the last entry ;)11:07
iceythe related-changes list suggests it's the charm-lxd rename?11:07
iceyah, the "fix missed renames" one?11:07
icey(you've linked me two in that series ;-P )11:08
AJaegerI said "on top of 657277" - and gave an easier example ;)11:08
iceythanks :)11:08
*** yboaron_ has quit IRC11:09
*** yboaron_ has joined #openstack-infra11:10
*** yboaron_ has quit IRC11:12
iceyAJaeger: seems like I should have the Depends-On to the other change as well which will, I suspect, force us to wait until the next go-round11:14
icey(the governance change)11:14
openstackgerritChris MacNaughton (icey) proposed openstack/project-config master: Rename x/charm-vault to openstack/charm-vault  https://review.opendev.org/66243211:15
*** yboaron_ has joined #openstack-infra11:15
*** yboaron_ has quit IRC11:16
*** yboaron_ has joined #openstack-infra11:17
*** yboaron_ has quit IRC11:22
*** dave-mccowan has joined #openstack-infra11:25
*** tjgresha has quit IRC11:30
*** tjgresha has joined #openstack-infra11:33
*** yboaron_ has joined #openstack-infra11:38
*** yboaron_ has quit IRC11:42
*** yboaron_ has joined #openstack-infra11:44
*** EmilienM is now known as EvilienM11:45
*** yboaron_ has quit IRC11:47
*** nicolasbock has joined #openstack-infra11:48
*** eharney has quit IRC11:50
*** lifeless has quit IRC11:51
*** lifeless has joined #openstack-infra11:52
*** kaiokmo has joined #openstack-infra11:56
*** rh-jelabarre has joined #openstack-infra12:06
*** iurygregory has quit IRC12:07
*** logan- has quit IRC12:07
*** logan- has joined #openstack-infra12:07
*** iurygregory has joined #openstack-infra12:08
*** hwoarang has quit IRC12:14
*** hwoarang has joined #openstack-infra12:14
*** openstack has joined #openstack-infra12:27
*** ChanServ sets mode: +o openstack12:27
*** bgmccollum has quit IRC12:28
*** bgmccollum has joined #openstack-infra12:31
*** yamamoto has joined #openstack-infra12:32
openstackgerritLucian Petrut proposed openstack/pbr master: Fix Windows support  https://review.opendev.org/66245012:35
*** yamamoto has quit IRC12:36
*** hwoarang has quit IRC12:39
*** hwoarang has joined #openstack-infra12:39
openstackgerritChris Dent proposed opendev/irc-meetings master: Cancel placement/scheduler meeting  https://review.opendev.org/66245112:40
*** rlandy has joined #openstack-infra12:41
*** sreejithp has quit IRC12:43
*** hjensas is now known as hjensas|afk12:44
*** hrw has joined #openstack-infra12:47
hrwmorning12:47
*** Goneri has joined #openstack-infra12:49
hrwIn Kolla project I have a patch which adds aarch64 CI job (based on Debian, written by someone who moved on to other job). I would like to add CentOS based one. can someone tell me do I have to create some node in infra or sth?12:51
*** adriancz has quit IRC12:51
hrwnevermind. probably found out12:56
*** ramishra has quit IRC12:59
*** ramishra has joined #openstack-infra12:59
*** mriedem has joined #openstack-infra13:05
*** yamamoto has joined #openstack-infra13:07
*** sreejithp has joined #openstack-infra13:07
*** sreejithp_ has joined #openstack-infra13:09
*** sreejithp_ has quit IRC13:09
*** sreejithp_ has joined #openstack-infra13:09
*** sreejithp_ has quit IRC13:09
*** eharney has joined #openstack-infra13:09
*** cloudnull has quit IRC13:11
*** d34dh0r53 has quit IRC13:11
*** sreejithp has quit IRC13:11
openstackgerritChris MacNaughton (icey) proposed openstack/project-config master: Rename x/charm-vault to openstack/charm-vault  https://review.opendev.org/66243213:12
*** d34dh0r53 has joined #openstack-infra13:13
*** d34dh0r53 has quit IRC13:15
*** cloudnull has joined #openstack-infra13:17
*** d34dh0r53 has joined #openstack-infra13:18
*** igordc has joined #openstack-infra13:19
*** aaronsheffield has joined #openstack-infra13:23
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213413:26
*** gfidente|afk is now known as gfidente13:28
*** kazsh has quit IRC13:32
*** kazsh has joined #openstack-infra13:33
openstackgerritMerged opendev/git-review master: Change StoryBoard URL to be a permalink  https://review.opendev.org/66145113:41
*** electrofelix has quit IRC13:43
*** yamamoto has quit IRC13:50
*** jcoufal has joined #openstack-infra13:50
*** yamamoto has joined #openstack-infra13:51
*** yamamoto has quit IRC13:51
*** yamamoto has joined #openstack-infra13:51
*** yamamoto has quit IRC13:56
fungiit's about time for me to stop the ansible cron on bridge.o.o in preparation for our rename maintenance in an hour13:59
*** sreejithp has joined #openstack-infra13:59
fungii've commented out the run_all.sh and run_cloud_launcher.sh entries in its crontab now14:00
*** yamamoto has joined #openstack-infra14:01
fungithe previous run_all.sh pass started at 13:45z, so i'll keep an eye out and make sure it doesn't start again after that completes14:01
AJaegergood morning, fungi!14:03
hrwI had 662456/2 started on debian-stretch-arm64. job failed, I fixed patch, sent new one and it queued because debian-stretch-arm64 node is still marked as 'building' while it's console.log shows only 'END OF STREAM' message. is there something I did wrong or is it normal operation?14:04
*** Lucas_Gray has joined #openstack-infra14:05
*** yamamoto has quit IRC14:05
hrwhttp://zuul.openstack.org/nodes shows: 0006892801 debian-stretch-arm64 ssh 66b734b5-b8f3-4677-b33e-fd46a31a8624 linaro-london building 30 minutes ago14:05
*** armax has joined #openstack-infra14:07
AJaegerhrw: for aarch64 CentOS, you need to create an image first, see https://opendev.org/openstack/project-config/src/branch/master/nodepool/nl03.openstack.org.yaml14:07
hrwAJaeger: job generates centos images, can be run on debian node without problem14:08
*** piotrowskim has quit IRC14:08
AJaegerhrw: you're talking Container image or VM image?14:08
hrwAJaeger: kolla runs on host and generates container images using other/same os. so host=debian, container=centos is perfectly fine14:09
hrwAJaeger: and good to see that there are debian-buster-arm64 too. will switch14:09
AJaegerhrw: I see14:09
AJaegeryou pushed a new patch for 66245, it's now 662456/3 - so the old job got killed14:10
hrwAJaeger: as I do all kolla builds on Debian it is easier for me to run CI on Debian host as I know that it works14:10
hrwAJaeger: ok14:10
AJaegerIf you only want ot experiment with single job, here's a trick: Remove all jobs from zuul.yaml for testing - and only add the new one to check queue. Once it works, add it as you do now ;)14:11
hrwAJaeger: right! will do it right now14:11
*** michael-beaver has joined #openstack-infra14:12
hrwAJaeger: so 'for tests' remove all normal jobs, does not define own one as 'experimental' but as normal. so zuul will by default run only my job on arm64. and once I have it working properly, readd normal jobs.14:14
AJaegerhrw: yes, that is my suggestion to not to have "check experimental" all the time and rerun jobs that are not neededed for now14:15
hrwAJaeger: great! thanks a lot14:15
*** chandankumar is now known as raukadah14:18
*** igordc has quit IRC14:18
hrw662456/4 in queue then14:19
fungiyeah, the experimental pipeline is more appropriate for jobs you expect to define once and then possibly run on multiple different changes over some extended period of time14:23
fungionce you have it working in the check pipeline of your wip change, you can always switch it to experimental before merging14:24
fungibut having it proposed to the check pipeline while you're working through it gives you the ability to see it run without having to merge it first14:25
fungiand as AJaeger says, while the change is wip you can remove all other jobs in that change if you want to avoid waiting on them to complete14:26
AJaegerfungi, do you have time for two short reviews for the infra-specs rename? https://review.opendev.org/662298 and https://review.opendev.org/662297 , please14:26
*** dpawlik has quit IRC14:27
*** weifan has joined #openstack-infra14:27
fungi#status notice The Gerrit service at https://review.opendev.org/ will be offline for briefly maintenance starting at 15:00 UTC (roughly 30 minutes from now); for details see http://lists.openstack.org/pipermail/openstack-discuss/2019-May/006684.html14:28
*** roman_g has quit IRC14:28
fungihrm, typo in that, but also statusbot is mia14:29
fungiwill restart14:29
*** openstackstatus has joined #openstack-infra14:30
*** ChanServ sets mode: +v openstackstatus14:30
fungi#status notice The Gerrit service at https://review.opendev.org/ will be offline briefly for maintenance starting at 15:00 UTC (roughly 30 minutes from now); for details see http://lists.openstack.org/pipermail/openstack-discuss/2019-May/006684.html14:30
openstackstatusfungi: sending notice14:30
*** weifan has quit IRC14:31
*** ginopc has quit IRC14:32
-openstackstatus- NOTICE: The Gerrit service at https://review.opendev.org/ will be offline briefly for maintenance starting at 15:00 UTC (roughly 30 minutes from now); for details see http://lists.openstack.org/pipermail/openstack-discuss/2019-May/006684.html14:32
AJaegerthanks, fungi!14:32
*** ginopc has joined #openstack-infra14:32
AJaegerfungi, I have to step out now for the next 30mins or so ;( Do you want to sneak in https://review.opendev.org/662432 for the rename - or is it too late and should wait for next time? If you support bringing it in, please check with clarkb.14:34
openstackstatusfungi: finished sending notice14:34
* AJaeger fears it's too late but wanted to bring attention to it - so that we don't merge it by accident if not desired.14:34
AJaegerSo, please -2 if it's too late...14:34
AJaegerchange itself LGTM14:34
clarkbYa I'd rather not change things at this point14:35
AJaegerthen I'll -2.14:35
fungithe governance addition does seem to have merged, so we could probably include it14:35
*** ginopc has quit IRC14:35
clarkbI already need to edit the rename playbook to deal with systemd14:35
fungiyeah, and the lack of passing tests14:35
fungilet's just plan to have another batch soon and this one can be at the top of the list14:35
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Add autohold-info CLI command  https://review.opendev.org/66248714:36
corvusthere isn't really a problem with the tests is there?14:36
AJaegericey: see above14:36
fungioh, right, no lack of passing tests, just lack of testability14:36
*** igordc has joined #openstack-infra14:36
fungiwe have to bypass zuul for these14:36
AJaegercorvus: it's based on a rename change that cannot be tested14:36
* AJaeger will step out now14:36
corvusright14:36
corvusi don't see a problem with doing it14:36
corvusit's only a change to our data file14:36
clarkbdoes it need to be rebased on the stack?14:37
AJaegerNo, it's on top14:37
* AJaeger removed his -2 - and really steps out now.14:37
fungiif clarkb's okay with adding it to the input file, i'm fine including it14:37
corvuslgtm if clarkb's ok14:38
clarkbok if you can review it for correctness now Im making tea and will prep a dir on bridge with edited playbook and repos.yaml (can include that rename if it looks good)14:38
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Store autohold requests in zookeeper  https://review.opendev.org/66111414:38
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Add autohold-info CLI command  https://review.opendev.org/66248714:38
*** lseki has joined #openstack-infra14:38
corvusto the best of my ability, it looks correct.  i'll happily propose the fixup patch if it turns out that 'v' does not come after 't'.  i'm really bad at alphabetizing.14:39
*** setuid has joined #openstack-infra14:40
iceycorvus: I'm in the same boat, I did my ABCs twice on that patch ;-)14:40
fungii rely heavily on the sort(1) utility14:40
corvusis there an etherpad or anything i should bring up?14:40
fungii'm getting it now14:41
fungicorvus: https://etherpad.openstack.org/p/project-renames-2019-05-3114:41
clarkbcorvus: fungi icey remote:   https://review.opendev.org/662097 Rename records for project renames on 20190531 updated with x/charm-vault rename14:42
fungii'll fix teh topic on the charm-vault change14:42
corvusi just did that14:42
corvustopic14:42
fungicool, you're ahead of me ;)14:42
iceymy bad on the topic - thanks :)14:43
fungiwe're not including 661803 in the maintenance, right?14:45
fungilooks like it doesn't actually involve a rename, it's just got that topic14:45
corvuscorrect, i don't think we should include it14:45
clarkbcorrect14:46
clarkbfungi: corvus /root/renames/20190531 on bridge has an updated rename playbook (I commented out the gerrit stop and everything after the start (inclusive with the start))14:46
fungicool14:46
clarkbI also have repos.yaml with a repos_first.yaml with a single project in it and a repos_second.yaml if we want to do it in two phases like we did with the great big migration14:47
fungithat sounds fine to me14:47
fungiless to clean up if things go sideways immediately14:47
clarkbI'm updating the etherpad with the commands to use that edited playbook now14:48
fungii've added myself to project bootstrappers so i can merge the changes, unless someone else was planning to take care of that14:49
*** npochet has joined #openstack-infra14:49
clarkbcorvus: one thing not yet in the etherpad is if we are going to stop zuul. Is that something you think we should do?14:52
fungiany objection to me removing the prophylactic wip votes from the rename changes now?14:52
clarkbfungi: nope14:53
clarkb(ansible puppet is disabled so should be safe)14:53
*** eharney has quit IRC14:53
*** ykarel is now known as ykarel|away14:53
corvuswe should really get https://review.opendev.org/588610 merged :/14:54
corvusclarkb: the only reason to stop zuul is to avoid gate resets14:55
clarkbcorvus: another question, using the edited playbook like we did last time  do you recall if there is anything special we have to do to make the role lookup stuff work?14:55
corvusif we think it'll be relatively quick, we could try it without stopping zuul.  i'm game.14:55
fungiwe've done it that way before14:55
openstackgerritMark Meyer proposed zuul/zuul master: Extend event reporting  https://review.opendev.org/66213414:55
fungisince the switch to gerrit online reindexing, the rename outages have been short enough that wasn't too disruptive for zuul14:56
corvusclarkb: what's role lookup stuff?14:56
*** lpetrut has quit IRC14:56
*** Lucas_Gray has quit IRC14:56
clarkbcorvus: in this case looks like it is actually an include_tasks14:56
clarkbfor gitea-rename-tasks.yaml14:56
clarkband ya looking at the root homedir we have a copy in there which I assume we pulled in so that the edited rename playbook would work last time14:57
clarkbI'll make a copy of that out of system-config in my renames dir now14:57
clarkband new version of that includes tasks out of a role14:58
clarkbmaybe it would be easier to have a clone of system-config and then edit that in place14:58
*** Lucas_Gray has joined #openstack-infra14:58
fungithe first several rename changes in the stack seem to be able to run and pass jobs... why are those different?14:59
fungiand should we still bypass gating on them anyway?14:59
clarkbfungi: they don't edit the zuul config14:59
clarkbfungi: zuul does pre merge sanity checking on config changes even for trusted repos14:59
clarkbso if a project that isn't part of the main.yaml shows up it will be an error15:00
fungithey edit zuul/main.yaml but not zuul.d/projects.yaml, so i guess the latter is the problem15:00
clarkbyes15:00
fungicool15:00
fungiso do we want them to merge normally, or just merge them all manually together?15:00
corvusall manually together i'd say15:01
fungicool, i'm ready to do those once we have a backup of the zuul pipelines (assuming we still think that's warranted)15:02
corvusi just made such a backup15:02
fungishould we status alert?15:02
clarkbya I can do the status alert15:02
fungithanks15:02
fungiokay, so ready for me to merge the project-rename stack?15:02
clarkbhow about #status alert Gerrit is now entering its maintenance window. Expect Gerrit outages in the near future. We will notify when it is back up and running.15:03
fungiwfm15:03
corvusclarkb: wfm15:03
clarkbalso I made a clone of system-config in my renames dir so that I don't have to copy a bunch of included tasks15:03
clarkband edited rename_repos.yaml in place there15:03
corvusclarkb: i'm ready for fungi to proceed; you?15:03
clarkbyup sending the alert now15:03
clarkb#status alert Gerrit is now entering its maintenance window. Expect Gerrit outages in the near future. We will notify when it is back up and running.15:04
openstackstatusclarkb: sending alert15:04
fungiyep, i'll go ahead with #415:04
openstackgerritFabien Boucher proposed zuul/zuul master: Disable gc in test_scheduler.TestExecutor  https://review.opendev.org/66131615:04
clarkbI'll start a screen session on bridge15:05
clarkbroot screen running15:05
openstackgerritSlawek Kaplonski proposed zuul/zuul-jobs master: Add role to fetch journal log from test node  https://review.opendev.org/64373315:05
openstackgerritMerged openstack/project-config master: Rename charm-lxd to charm-nova-lxd  https://review.opendev.org/64458415:06
openstackgerritMerged openstack/project-config master: Renaming of Public Cloud WG to Public Cloud SIG  https://review.opendev.org/65948915:06
openstackgerritMerged openstack/project-config master: Rename request: airship-in-a-bottle  https://review.opendev.org/65600815:06
openstackgerritMerged openstack/project-config master: Rename x/compute-hyperv to openstack/compute-hyperv  https://review.opendev.org/66168215:06
-openstackstatus- NOTICE: Gerrit is now entering its maintenance window. Expect Gerrit outages in the near future. We will notify when it is back up and running.15:07
*** ChanServ changes topic to "Gerrit is now entering its maintenance window. Expect Gerrit outages in the near future. We will notify when it is back up and running."15:07
openstackgerritMerged openstack/project-config master: Rename x/pyeclib and x/liberasurecode to openstack/*  https://review.opendev.org/65715515:07
openstackgerritMerged openstack/project-config master: Rename opendev projects we missed  https://review.opendev.org/65727715:07
openstackgerritMerged openstack/project-config master: Rename x/charm-vault to openstack/charm-vault  https://review.opendev.org/66243215:07
clarkbnow we wait for replication to complete15:07
corvusi think it's done?15:07
fungigerrit's queue does seem to indicate a git-upload-pack for /openstack/project-config15:08
corvusshow queue is empty and gitea is up to date15:08
corvusgit-upload-pack means someone is fetching it15:08
clarkbya I just checked gitea and it looks good from here15:08
fungiyep, i think we're set15:08
clarkbok I'll try to stop gerrit the normal way now?15:09
corvusclarkb: ++15:09
clarkband we'll figure it out if that doesn't work (as expected)15:09
clarkbas expected did not work15:09
clarkbfungi: did you end up having to kill it previously?15:09
fungiyeah, systemd is smart enough to believe it's not running because it thinks it never successfully started... can call the initscript directly or just sigterm15:10
clarkbok I'll run sudo /etc/init.d/gerrit stop15:10
*** kjackal has quit IRC15:10
openstackgerritFabien Boucher proposed zuul/zuul master: Set the DiskAccountant log as warning level  https://review.opendev.org/66099615:10
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Record held node IDs with authold request  https://review.opendev.org/66249815:10
openstackstatusclarkb: finished sending alert15:11
clarkbwaiting for it to stop now15:11
fungilooks like the java process is gone15:11
clarkband the command returned so we should be good15:11
clarkbI've got the git diff for my edited playbook up on the root screen now15:12
clarkbready to proceed with running the repos_first.yaml playbook?15:12
fungiyeah15:12
corvus++15:12
fungii've joined your screen session15:13
clarkbthat command look good? I will hit enter if so15:13
corvus++15:13
fungiyeah, i think so15:13
fungiand we're off to the races15:13
*** eharney has joined #openstack-infra15:14
fungiand... oof15:14
* AJaeger is back15:14
clarkbI feel like we actually ran into this last time. My omnibus must've missed something15:14
*** e0ne has quit IRC15:14
fungimind if i scroll up the buffer?15:15
*** Lucas_Gray has quit IRC15:15
corvusneed     - include_vars: "{{ repolist }}"15:15
corvusin that play15:15
clarkboh right15:15
clarkbfungi: go for it but corvus has the fix. I can edit it in15:16
fungino worries, i've made a very large xterm and am reattaching from that15:16
corvusclarkb: lgtm15:17
fungiwow, my massive terminal with tiny font is still not as wide as yours (by a few columns)15:17
clarkbfungi: I've pulled up my edits if you want to look before I run it again. Including commenting out the bits that did run successfully15:17
clarkbfungi: all the pixels :) sorry I use tiling wm that uses all available space and I give renames a dedicated workspace15:17
fungiahh, yep, looks good15:17
corvusthat window uses 1.25 of my monitors :)15:18
clarkbready to rerun the command?15:18
corvus++15:18
corvusi do feel like we're running into the same problems as last time15:18
fungiwell, this is on a 2560x1440 panel but it's rotated into portrait orientation so i only have 1440 pixels of width to play with (my window is several times taller than yours though)15:18
fungiand yeah, go for it15:19
clarkbcorvus: is var supposed to be vars ?15:19
corvusthat should just be "vars"15:19
corvusokay, this one doesn't seem familiar after all :)15:20
fungiahh15:20
clarkblike that?15:20
corvusyep15:20
clarkbnice thing about using a git repo this time is we'll get a git diff :)15:20
* fungi crosses fingers superstitiously15:20
clarkbready to rerun? I don't think I need to edit any tasks out?15:20
fungiyeah, ready15:21
corvus++15:21
fungiorg not in gitea_orgs15:21
*** jaosorior has quit IRC15:22
corvus*this* one seems familiar15:23
corvusdo we have a copy of what we ran during the great migration?15:23
corvusbecause istr this one is a more structural error15:23
fungiso gitea_orgs is a strictundefined type, which i guess means it's not set?15:23
clarkbcorvus: ya should still be in /root/15:23
clarkbfungi: ya I think that is it and I recall similar like corvus last time15:23
clarkblooks like we shortcutted last time and removed the check if we didn't need to rename any orgs15:25
clarkbin this case I don't think we are renaming any ogs. Shall we shortcut again and remove this section of the playbook?15:25
corvusok15:25
fungithat's expedient for now15:25
funginone of the current batch of changes is creating new orgs anyway, right?15:26
clarkbI don't think so but we should double check15:26
clarkbdo the edits I've made look correct? If so I'll pull up the repos.yaml files next and we can confirm no new orgs15:27
corvusclarkb: just the one task, right?15:27
fungiyeah, you do seem to have commented out the failing task15:27
clarkbcorvus: yup that does include_task setup-orgs15:27
corvus1 sec15:28
clarkbk15:28
corvuslgtm15:29
clarkbalright pulling up the repos.yaml files next to check there are no new orgs15:29
clarkbopenstack opendev airship x15:30
clarkball are preexisting so we should be good15:30
corvusagree15:30
clarkbshall I rerun the ansible command again?15:30
corvus++15:31
fungiyep15:31
AJaegerfungi, nothing creates new orgs15:31
clarkbfailure on gitea06 is "ok"15:31
corvusagree15:31
fungiaha, that's the one which scrolled by too fast for me to read15:31
fungiyeah, that looks far better15:32
clarkbok I'll remove my comments around the gerrit sections in the rename playbook and we can run this on the second list15:32
corvus++15:32
clarkbthat look good?15:33
corvusyep15:33
fungilgtm15:33
fungicrank it up15:33
clarkbcorvus: you ready too?15:33
corvusclarkb: ++15:33
fungiwoo!15:35
clarkbthe only failure is expected on gitea0615:35
* clarkb pulls up commented out bits of rename playbook so that we can see what is next15:35
corvusas an aside -- it seems zuul is behaving in a slightly different way than i would expect -- it's essentially frozen because it's throwing an exception about being unable to connect.  i think that ultimately the change it just tried to merge will fail to merge and there will be a gate reset, but in the mean time, it's paused and there shouldn't be any additional fallout15:36
clarkbk15:36
fungiinteresting behavior, and thanks for the update15:36
corvus(it is, however, writing log files as quickly as it can)15:36
clarkbso next step according to the playbook is to start gerrit, then rename groups and start reindexing15:36
clarkbshould I start gerrit now?15:37
fungiseems right15:37
corvus++15:37
fungiand yeah, let's see if the timeout change has fixed it15:37
clarkbok don't have a screen on review.o.o but I'll just go ahead and start it15:37
corvusfor posterity -- here's the zuul traceback: http://paste.openstack.org/show/752367/15:37
clarkbsudo systemctl start gerrit has been started15:37
fungii confirm it seems the process is running15:38
clarkbnot logging much though (not sure if that is normal or not)15:39
fungithis is fairly normal15:39
clarkbbut we also know it is slow to start15:39
fungigive it another minute or so15:39
clarkbthere it goes15:39
clarkb[2019-05-31 15:39:56,735] [main] INFO  com.google.gerrit.pgm.Daemon : Gerrit Code Review 2.13.12-11-g1707fec ready15:40
clarkbsystemd knows it is running too15:40
clarkbweb ui still coming up though15:40
fungimy webui reload is spinning now instead of the proxy throwing a 50015:40
fungiand now it's loading!15:40
fungiseems to be fully up15:41
AJaeger\o/15:41
*** Lucas_Gray has joined #openstack-infra15:41
clarkbok I'll sort out the group renames and reindexing now15:41
corvuszuul is receiving (replication) events15:41
clarkbdo you want me to start a root screen on gerrit for that?15:41
fungidid you want to just use ansible to trigger those bits, or you want to run them locally by hand?15:41
corvusas expected, the top gate change did not report, there was a gate reset and #2 was promoted15:42
clarkbI was thinking of just running them by hand15:42
clarkbbut I guess I can comment everything but that section out of the playbook and run it with repos_second.yaml?15:42
clarkbfungi: corvus ^ would you prefer I do it that way?15:42
fungifor some reason i thought gerrit group renames and triggering renidexing were done by the playbook15:42
corvusyeah, i think the group renames etc will be easiest to do with a one-off playbook15:43
clarkbfungi: they are, but they happen after the start which I commented out so I commented those tasks out too15:43
corvusshouldn't you run it for all the repos, not just second?15:43
fungiright, i was reacting more to the comment in the etherpad which was probably copied from a maintenance where we hadn't added those yet15:43
clarkbcorvus: I think it doesn't matter beacuse we just need the group data but ya I can run it against the master repos.yaml list15:43
clarkbworking on playbook now15:44
corvusah, i didn't recall which yaml files have the groups, but it sounds like we're on the same page15:44
fungioh, nevermind, the note in the etherpad is referring to the edited version of the playbook15:44
fungigot it15:44
openstackgerritAndreas Jaeger proposed openstack/project-config master: Remove unused ACLs  https://review.opendev.org/66191315:45
clarkbfungi: corvus ok one off playbook in root screen. If you can read and check that I'll prep it for running next15:45
corvusclarkb: ++15:46
clarkbok I'm going to remove the pause wait thing in there15:46
clarkbsince we have already confirmed gerrit is up15:46
clarkbediting done15:46
openstackgerritAndreas Jaeger proposed openstack/project-config master: Use retired ACL everywhere  https://review.opendev.org/66222315:46
clarkbfungi: ^ that look correct to you?15:47
fungichecking the reindex command15:47
fungilooks like index accounts no longer exists15:48
fungigerrit index --help says the actual parameters are activate, changes and start15:48
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Record held node IDs with autohold request  https://review.opendev.org/66249815:48
fungioh, wait15:48
clarkbfungi: I think that is the db name15:48
clarkband is an argument to the command15:49
fungiyeah, gerrit index start INDEX [--] [--force] [--help (-h)]15:49
*** bnemec is now known as beekneemech15:49
fungiso that should be correct15:49
clarkbthe rest of the playbook look good?15:49
fungiin the past i've seen `gerrit index start --force accounts` start to index changes so just making sure15:50
fungii think we moved --force to the end to fix that15:50
fungiall looks correct to me15:50
clarkbk15:50
*** gyee has joined #openstack-infra15:50
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Record held node IDs with autohold request  https://review.opendev.org/66249815:51
clarkbfungi: corvus command is queued up, ready to run that?15:51
fungiyep, go for it15:52
*** quiquell has quit IRC15:52
clarkbhostkey verification failed15:52
fungihost key verification15:52
clarkbbah15:52
fungiyeah15:52
corvusisn't that odd?15:53
corvusdoesn't ansible run stuff on that host all the time?15:53
fungidifferent port, different ssh host key15:53
clarkbcorvus: its ssh from gerrit2 to review.opendev.org port 29418 that is failing I think15:53
fungithis is the gerrit mina-sshd host key15:53
corvusgotcha15:53
funginot the openssh host key for the server15:53
clarkbwe probably have review.openstack.org accepted but not opendev.org15:53
corvusso we should log in as gerrit2 on review01, and make "gerrit ls-projects" work15:53
clarkb++15:54
clarkbI can do that15:54
corvusk15:54
corvusspecifically, log in as gerrit2 and make "ssh -p 29418 -i /home/gerrit2/review_site/etc/ssh_project_rsa_key openstack-project-creator@review.opendev.org gerrit ls-projects" work15:54
fungieven just the gerrit base command without ls-projects should return a help message from the service15:54
clarkbcorvus: yup done15:54
clarkbready to rerun the command?15:55
clarkbansible command in screen that is15:55
fungiyeah15:55
clarkband done15:55
fungimuch better15:55
clarkbthat concludes that "automated" portion of this rename15:56
fungi34716 tasks15:56
fungireindexing will likely wait behind replication15:56
clarkbthey run in different threads iirc15:56
fungiahh, okay, perhaps not then15:56
clarkbok so now we are mostly in the verification period15:56
corvusshould we status ok now?15:56
clarkbcorvus: ya I think we likely can if we are happy with how gerrit gitea and zuul are running15:57
fungii think so15:57
fungihas zuul regained sanity?15:57
corvusyeah15:57
clarkbok I'll send status ok now15:57
fungidid the change in question eventually fail to merge or did it go through once zuul reconnected?15:57
corvusit failed to merge15:57
clarkbI think corvus said it failed15:57
clarkb#status ok15:57
openstackstatusclarkb: sending ok15:57
fungigot it, i missed that in the rest of our troubleshooting. thanks!15:58
corvusi'm giving it a recheck now15:58
clarkbThe last major step in the rename is to reenable our puppetry15:58
corvusclarkb: you know you could have said "#status ok Gerrit maintenance complete"15:59
clarkbin theory we can do that now since we force merged and replicated prior to the restart15:59
clarkbcorvus: oh I did not know15:59
corvus(seeing as how we did say we would notify when it's up and running)15:59
*** ChanServ changes topic to "Discussion of OpenStack Developer and Community Infrastructure | docs http://docs.openstack.org/infra/ | bugs https://storyboard.openstack.org/ | source https://git.openstack.org/cgit/openstack-infra/ | channel logs http://eavesdrop.openstack.org/irclogs/%23openstack-infra/"15:59
clarkbya I thought it gave a canned message for some reason15:59
corvusif you don't give it a message, it just silently changes the topic back ^15:59
clarkbI can do a notice15:59
clarkbhow about #status notice Gerrit is back up and running again. Thank you for your patience.16:00
sreejithphello all, git clone is failing on https://review.opendev.org/airship/in-a-bottle  with the error "stderr: fatal: remote error: Git repository not found16:00
sreejithp"16:00
clarkbhaev to wait for status ok to complete first but otherwise ^ should work16:00
openstackgerritAndreas Jaeger proposed openstack/project-config master: Retire opendev-website (step 3)  https://review.opendev.org/66180316:00
corvusclarkb: yeah16:00
clarkbsreejithp: yup it was renamed. THe redirects only work on the gitea opendev.org side of things. YOu'll have to update your gerrit remotes16:00
AJaegersreejithp: we just renamed it to airship/airship-in-a-bottle16:00
fungisreejithp: the airship maintainers requested to have it renamed to https://review.opendev.org/airship/airship-in-a-bottle16:00
*** eharney has quit IRC16:01
corvussince we merged the p-c changes first, i think we can continue with the etherpad16:01
corvusnext is re-enable ansible cron?16:01
fungion it16:02
clarkbyup. I'm gonna do a local project config git update just to sanity check for myself16:02
clarkbbut should be safe16:02
sreejithpthanks all16:02
AJaegerzuul is still complaiing about the new repos - is that to be expected?16:02
clarkbchanges I expect are there16:02
*** rlandy is now known as rlandy|brb16:02
clarkbAJaeger: ya we haven't updated zuul's config yet. That happens as side effect of next change16:02
AJaegergreat!16:02
clarkbAJaeger: the step where fungi reenables ansible + puppet then puppet can update zuul config16:03
fungii've uncommented the run_all.sh and run_cloud_launcher.sh entries in the bridge.o.o root crontab16:03
fungishould start back up at 16:15z16:03
AJaegerclarkb: I see...16:03
fungiAJaeger: last sentence under #8 on https://etherpad.openstack.org/p/project-renames-2019-05-3116:03
clarkbstill waiting on statusbot to finish sneding its ok16:04
clarkblooks like it took ~6 minutes last time16:04
corvusafter we move the gitea project creation into python, we can write a test job that runs the rename playbook -- with the *actual* data for a rename.16:05
clarkbcorvus: that would be so nice16:05
clarkbI guess I should work on pushing some of my fixes up now16:05
openstackgerritClark Boylan proposed opendev/system-config master: More repo rename playbook fixes  https://review.opendev.org/66251016:07
clarkbcorvus: fungi ^ that captures the two fixes we needed but does not do anything with gitea orgs lists16:08
fungii guess we're still not sure what the fix is for the org creation piece?16:08
corvusi can take a stab at that16:09
*** Lucas_Gray has quit IRC16:10
*** kopecmartin is now known as kopecmartin|off16:10
clarkbthansk16:11
clarkbI'm guessing we need to copy a taks out of our regular gitea management into gitea-rename-tasks.yaml16:11
clarkb(the one that gets a list of orgs already in the system)16:11
*** Lucas_Gray has joined #openstack-infra16:12
openstackgerritJames E. Blair proposed opendev/system-config master: Fix gitea rename playbook org creation  https://review.opendev.org/66251116:12
corvusyep16:12
clarkbheh we are really overloading user namespaces aren't we16:13
*** mriedem is now known as mriedem_hangry16:14
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132716:15
openstackgerritAndreas Jaeger proposed openstack/project-config master: Fix gerrit/projects.yaml after rename  https://review.opendev.org/66251316:15
AJaegerinfra-root, here's a hot fix ^16:15
corvuswe need to stop ansible16:15
clarkbcorvus: go for it16:16
*** Ng is now known as Ng_16:17
clarkblooks like someone killed run_all.sh16:17
clarkbthere are ansible processes for base.yaml running16:17
clarkbbut I think those are ok?16:17
corvusi think everything is dead now16:18
clarkbyup I don't see anything now16:18
clarkbso whats up?16:18
*** jpich has quit IRC16:18
corvusAJaeger's hotfix16:18
corvuswe would have created x/ansible-role-cloud-launcher16:18
clarkbah looking now16:18
corvuslet's see if we did16:18
clarkbthe manage projects log on review.o.o is still empty so I don't think it got as far as gerrit16:19
corvusand we did16:19
openstackgerritAndreas Jaeger proposed openstack/project-config master: Sort zuul.d/projects.yaml again  https://review.opendev.org/66251416:19
corvusoh?16:19
corvushttps://review.opendev.org/#/admin/projects/x/ansible-role-cloud-launcher exists16:19
clarkbthat was its old name maybe we didn't rename it?16:19
AJaegerand 662514 is another hotfix (but not critical, just blocks us merging anything)16:19
clarkbya I made repos.yaml out of projects.yaml diffs16:20
AJaegerclarkb: we renamed it elsewhere, e.g. ACLs are renamed16:20
corvusoh16:20
corvuswe *did not* rename ansible-role-cloud-launcher16:20
clarkbso if it wasn't in there then it probably didn't get renamed the above url doesn't work if you replace x with opendev16:20
clarkbcorvus: ya I don't think we did16:20
clarkbso the real fix we want is to undo the other rename stuff then pick this up next time16:20
corvusAJaeger: your change is wrong then -- it should move it back to x/16:20
AJaegercorvus: can any of you take over - I need to step out for another 30mins now ;(16:21
clarkbI can help sort it out16:21
*** eharney has joined #openstack-infra16:21
fungii killed the remaining ansible remote processes16:21
AJaegerI can sort it...16:21
AJaegerI'm off by 30mins, have time.16:21
clarkbAJaeger: ok16:21
corvusi'll re-enable the ansible cron then?16:21
*** dtantsur is now known as dtantsur|afk16:22
clarkbcorvus: will manage projects just fail to run in that case because it won't have a acl file for that repo?16:22
clarkbif that is the behavior we expect then I think we might be ok with the cron running16:22
corvusfungi: thanks, though i'm a little curious about what processes you killed since my ps wasn't reporting any...16:22
fungissh: /root/.ansible/cp/7e582814a5 et cetera16:22
clarkbthe ssh control persist processes16:22
corvusfungi: oh, those are harmless and die on their own.  that's the controlpersist process16:23
fungiahh16:23
*** sgw1 has joined #openstack-infra16:23
fungii thought those were the remote ansible script runs16:23
corvusclarkb: hrm, i dunno.  want to force-merge a fix?16:23
clarkbcorvus: we'll still have to wait for replication to complete16:23
corvusfungi: nah, those look like: root     16757  0.0  0.0  47060  5452 ?        S    16:17   0:00 ssh -C -o ControlMaster=auto -o ControlPersist=60s -o KbdInteractiveAuthentication=no -o PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey -o PasswordAuthentication=no -o ConnectTimeout=10 -o ControlPath=/root/.ansible/cp/3954b8345e 2001:4800:7818:103:be76:4eff:fe04:976a /bin/sh -c16:23
corvus'/usr/bin/python && sleep 0'16:23
clarkbso probably don't need to force merge and just reenable cron later today or on monday?16:24
fungicorvus: thanks, noted16:24
clarkbit would be cool if there was a jump the queue replication command16:24
*** lucasagomes has quit IRC16:24
clarkbdown to 14k tasks now16:24
clarkb2k are indexing tasks16:25
openstackgerritAndreas Jaeger proposed openstack/project-config master: Fix ansible-role-cloud-launcher  https://review.opendev.org/66251316:25
AJaegerfixed both issues together ^16:25
clarkbI guess we can do an out of band replicate like we did in the past for refs/*16:25
corvusunfortunately, i don't think we can merge any changes to project-config until ansible runs and updates the zuul config16:26
clarkbwe might have to force merge anyway or we can update the zuul config via kick.sh?16:26
clarkbya16:26
corvuslet's force-merge it16:27
clarkbk16:27
fungii'm still trying to digest what transpired...16:27
corvusfungi: you still a boostrapper?  if so, when you're happy, you want to force-merge 66251316:27
fungiso did we or did we not rename x/ansible-role-cloud-launcher? looks like it's still named that ni gerrit16:27
fungiand yes, i'm still in bootstrappers16:28
clarkbfungi: x/ansible-role-cloud-launcher did all the steps to rename except for updating gerrit/projecs.yaml. I generated my repos.yaml for the rename playbook from projects.yaml diffs so we did not rename that project16:28
clarkbfungi: however we did update the zuul config for it and the gerrit acl config16:28
clarkbAJaeger's change undoes the updates so everything is back to x/ansible-role-cloud-launcher again16:28
fungigot it16:28
fungii was looking at the earlier iteration of that change16:28
*** ramishra has quit IRC16:29
clarkband we'll have to rename that project with the next group of renames16:30
clarkbnot a huge deal other than its making updating zuul's config awkward16:30
clarkbbut we can kick.sh our way out of that if we need to16:30
openstackgerritMerged openstack/project-config master: Fix ansible-role-cloud-launcher  https://review.opendev.org/66251316:30
fungigrr, gertty has decided it's offline again... just a sec16:30
fungithere it goes16:30
clarkbAJaeger: thank you for catching that16:31
clarkb13k ish tasks now and just over 2k are indexing tasks16:31
clarkbwe seem to be making reasonable progress16:31
AJaegerclarkb: my other change failed and looking at log files I noticed it. Sorry for missing it in review ;(16:32
clarkbI wonder if we could disable zuul's sanity checking of the config (force it to use last good config) so that jobs can run against updated data16:32
clarkbopenstackstatus never send the finished sending alert message16:35
corvuswhen we have the zuul playbook finished, this won't be a problem since we'll be updating zuul's config much faster16:36
clarkbre statusbot the logs for it indicate it is still running I think16:37
clarkbshould I try sending the notice now?16:37
clarkbI guess I should grep for tracebacks first16:37
*** ociuhandu_ has joined #openstack-infra16:38
clarkbnothing16:38
clarkbcorvus: fungi ^ thoughts on that. Should I try the notice or keep waiting?16:40
*** ociuhandu has quit IRC16:41
clarkbI know in the past it would have a sad if trying to process two events at once16:41
fungiclarkb: oh, did it never confirm the ok?16:41
clarkbnot that I can see in scrollback16:41
clarkbit confirmed the alert but not the ok16:41
AJaegerfungi, could you review https://review.opendev.org/#/c/662298/ to unbreak infra-manual building, please?16:42
*** ociuhandu_ has quit IRC16:42
fungiyeah, and it never parted/joined the channel in that time either16:42
fungilooks like it was processing them in alpha order and got as far as #zuul at 2019-05-31 16:01:29,59916:43
clarkbso maybe it completed16:44
fungi(from the last line of the log)16:44
fungii wonder if a #status ok with no message included doesn't generate a confirmation for some reason16:44
fungiyeah, i'd say go ahead with the notice16:44
fungiproblems in the past had to do with it losing track of channel topic changes, which look to have already completed16:45
fungialso, i think that particular bug was eventually fixed16:45
clarkb#status notice Gerrit is back up and running again. Thank you for your patience and sorry for the delay in this notification (we thought the statusbot was still busy updating channel topics).16:45
openstackstatusclarkb: sending notice16:45
*** rlandy|brb is now known as rlandy16:45
corvusit should send a 'finished' message even with no ok message16:45
clarkbthe notice is working fwiw16:46
clarkbalso down to 9.7k replication tasks16:46
-openstackstatus- NOTICE: Gerrit is back up and running again. Thank you for your patience and sorry for the delay in this notification (we thought the statusbot was still busy updating channel topics).16:46
clarkbnow just under 9.6k16:47
clarkbabout an hour and a half away from completion?16:48
clarkbthough that may depend on size of repos16:48
*** hwoarang has quit IRC16:48
fungiit will spend a lot more time on the larger repos, but those don't actually matter today16:48
*** hwoarang has joined #openstack-infra16:48
clarkbif that estimate is correct (or close) then we should have plenty of time today to turn on the cron today16:48
openstackstatusclarkb: finished sending notice16:49
fungias long as the reindexing tasks for the renamed repos complete, i think we're fine16:49
clarkbfungi: we also need the ansible-role-cloud-launcher fix to replicate so we can turn on ansible cron16:49
fungiwe didn't rename nova, neutron or openstack-manuals so when those are the only ones left (and typically they run for quite a while longer than the rest) we can presumably proceed16:50
AJaegerNow I need to step out...16:51
clarkbAJaeger: enjoy your evening and weekend. Thank you for all the help16:52
clarkbnow down to 8.6k replication tasks16:54
clarkbmoving faster than I mathed16:54
fungiso if there is no pending replication task for openstack/project-config we can go ahead i guess?16:56
clarkbstill waiting on those16:56
clarkbbut ya I guess so16:56
clarkbalso gitea06 is struggling. We may want to remove it from the replication list (later)16:58
clarkbnova and neutron are the last indexing projects16:59
fungi`... gerrit show-queue -w|grep -n openstack/project-config` says there are 1298 tasks ahead of the last gitea push for project-config16:59
*** derekh has quit IRC17:00
fungioh, though i guess we don't care about replication to gitea0617:02
fungiso more accurately, 1188 waiting replication tasks ahead of the final gitea replication of project-config excepting the one for gitea06 (which is a but farther down the queue)17:03
clarkbcorrect its just 07 we are waiting for now17:03
clarkbhttps://gitea01.opendev.org:3000/openstack/project-config/commit/6312597c060cb93689197a60140e32c31fd06edd does exist though (and I think everywhere but 06 and 07)17:03
clarkband github17:04
fungianother alternative would be to just take 07 out of haproxy temporarily?17:04
clarkbI think it should get there soon. It is in the osas now17:04
fungiif we don't want to wait17:04
fungiyeah17:04
fungii've been told that i have a hard stop for lunch at 17:30, lest christine gnaw off one of my arms17:06
clarkbdid we want to take a break now then reconvenve in ~1.5 hours or whatever to enable the cron?17:06
clarkbcorvus: ^ don't know what your schedule is like17:06
fungiwfm, having to type one-handed for the remainder of the afternoon would significantly impact my productivity17:06
openstackgerritMerged openstack/infra-manual master: Update infra-specs URL  https://review.opendev.org/66229817:08
corvusclarkb: sounds good17:08
clarkbI did a double check on 01-05 and 08 they all have the fix for ansible-role-cloud-launcher so just 06 and 0717:08
clarkbfungi: go have lunch then :)17:08
fungiyeah, still 1127 replication tasks ahead of it17:09
clarkbanother idea I had was for us to update our git remotes to be gerrit for things that drive stuff17:09
fungiwill bbiaw17:09
clarkbbecause then we wouldn't ahve to wait for replication17:09
fungiclarkb: once we get a shared backend for gitea we only need to replicate once17:09
clarkbfungi: thats true17:09
fungialso switching openstack/ repos over to github replication via zuul job means we don't need to do those either17:10
corvusthough it may not be much faster than this17:10
fungifair ;)17:10
*** xek_ has quit IRC17:12
fungi#status log manually removed nova 2013.1 release from pypi17:14
openstackstatusfungi: finished logging17:14
clarkbfungi: hurry before you lose your arm :P17:15
*** gfidente is now known as gfidente|afk17:16
clarkb5.3k total tasks now17:18
clarkbI think 07 may be slow due to disk io (it doesn't seem to be doing a ton of cpu)17:23
clarkbplenty of memory and disk available too17:24
*** haleyb has joined #openstack-infra17:24
haleybhave a question, maybe someone can point me in the right direction.  i'm trying to debug a period job failure that's failing to install the proper kernel-devel package.  Digging into it, the job is using 'fedora-latest', but the kernel booted in fc28, not fc29.  Instead of continuing I'd figured i'd ask if that looked wrong, especially since fc28 is now EOL17:28
clarkbhaleyb: a link to the job would be good, but there is a good chance that fedora-latest was never updated to be newer than 2817:29
clarkb*link to job log17:29
clarkbhrm no my local copy of the nodeset def says it is fedora-29 so ya logs would be a big help17:30
haleybclarkb: the log from my hacked job is http://logs.openstack.org/67/661067/2/check/networking-ovn-tempest-dsvm-ovs-master-fedora/8beacab/job-output.txt.gz17:30
haleybsearch for kernel-devel17:30
haleybi had to add the job as a standard one to get it to run on the patch17:30
clarkbthat job ran on a fedora-28 node according to ansible facts http://logs.openstack.org/67/661067/2/check/networking-ovn-tempest-dsvm-ovs-master-fedora/8beacab/zuul-info/host-info.controller.yaml so we can at least confirm that17:31
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132717:31
*** Lucas_Gray has quit IRC17:31
haleybclarkb: i thought my backtrack through the zuul definitions showed 'fedora-latest'17:32
corvushttp://zuul.openstack.org/job/networking-ovn-tempest-dsvm-ovs-master-fedora17:32
clarkbhaleyb: https://opendev.org/openstack/devstack/src/branch/master/.zuul.yaml#L51-L59 its teh devstack fedora latest nodeset17:33
clarkband that one hasn't updated17:33
clarkbso you can either upate it or use a different nodeset17:33
clarkbhaleyb:  https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/nodesets.yaml#L14-L18 is the generic one (note it doesn't ahve the controlloer/compute/etc labels that the devstack nodesets have)17:33
haleybclarkb: thanks, i guess i asked in the right channel :)  So is changing the devstack one to use fedora-29 a good start?17:35
clarkbprobably17:36
clarkbone thing to check is if tripleo uses that nodeset17:36
clarkbthey want to keep using 28 until centos 8 is a thing17:36
clarkbbut I'd probably just notify them and move forward since as you say it is eol17:37
haleybclarkb: getfedora.org says it is as of Wednesday17:37
*** hwoarang has quit IRC17:37
clarkbhaleyb: ya they knew that was a risk when they did the python3 testing on 2817:37
*** hwoarang has joined #openstack-infra17:38
clarkband they may have their own nodesets already17:38
openstackgerritMerged opendev/system-config master: More repo rename playbook fixes  https://review.opendev.org/66251017:39
haleyblooks like neutron, networking-ovn, barbican, devstack and novajoin use the devstack-single-node-fedora-latest nodeset17:39
* redrobot hears barbican in the distance; pokes head in17:40
haleybredrobot: i was just looking in codesearch for repos using the devstack-single-node-fedora-latest nodeset17:40
redrobothaleyb, gotcha.  Yeah we use in barbican to test the DogTag backend plugin, which is only available in Fedora IIUC17:41
haleybredrobot: as that is using f28 which just went EOL i was going to propose updating to f29, which may or may not solve the problem i'm chasing installing packages in a job17:42
*** gfidente|afk has quit IRC17:43
openstackgerritAndreas Jaeger proposed openstack/project-config master: Fix gerrit/projects.yaml sort order  https://review.opendev.org/66252717:43
redrobothaleyb, that would work for us.  We don't really care about the Fedora version.17:43
clarkbAJaeger: thanks. I don't think we need that one before we start ansible cron again but will be good to have working jobs :)17:44
clarkbgitea07 is in the olso projects now17:44
AJaegerclarkb: indeed, it just unblocks17:44
haleybredrobot: i'll propose it and try and get cross-repo test patches to see if things break, have no idea17:45
redrobothaleyb, cool.  I can help fix barbican things if they break.17:46
haleybredrobot: thanks, hopefully i can propose these today and check on them monday17:46
*** iurygregory has quit IRC17:46
*** hwoarang has quit IRC17:48
*** hwoarang has joined #openstack-infra17:48
*** raissa has joined #openstack-infra17:50
corvusclarkb: have you seed "dial tcp [2607:ff68:100:54:f816:3eff:fe6e:65ff]:5000: connect: permission denied" in a registry job yet?17:50
clarkbcorvus: no17:50
clarkbthe most recent issues I had seen were with the docker hub proxy on 500117:51
clarkb(causing it to fallback to docker hub then fail auth there)17:51
clarkbpermission denied there is interesting that almost reads like you can't open a socket17:51
clarkbsince when auth fails you get the your auth is bad message instead17:51
corvusyeah....that was trying to connect to a simulated intermediate registry -- (ie, it's a stand-in for insecure-ci-registry.opendev.org).  it was in limestone over ipv6, but there's an /etc/hosts entry for it, so skopeo is using a hostname.17:52
*** raissa has quit IRC17:53
clarkbis it possible skopeo is doing that namespacing for image transfers again and that doesn't have an ipv6 stack?17:53
*** raissa has joined #openstack-infra17:53
corvuswow.  i have run this job a lot, and it does not run on limestone often.17:55
clarkbopenstack/project-config is replicating to gitea07 now17:56
corvusclarkb: i found a previous run on limestone, it hit the same error :/17:57
openstackgerritAndreas Jaeger proposed openstack/project-config master: Rename x/ansible-role-cloud-launcher -> opendev/  https://review.opendev.org/66253017:58
corvusclarkb: does that mean we need to socat for the the transfer to the intermediate registry too?17:58
*** panda|ruck has quit IRC17:59
clarkbcorvus: in production if this is the issue it may be using ipv4 (because there is a record for that in dns)18:00
clarkbin which case we probably don't absolutely need to18:00
corvusclarkb: this is for the test i'm making18:00
clarkbyou might try it18:00
clarkbthough in the test env we hsould have private ipv4 that we could use too18:00
clarkb(and put that in /etc/hosts instead)18:01
*** pkopec_ has quit IRC18:01
*** panda has joined #openstack-infra18:01
corvusclarkb: do you think using "nodepool.private_ipv4 || nodepool.public_ipv4" is the way to go?18:02
clarkbI think so. To me that is preferable to the socat hack (whcih we should only really use where necessary)18:02
corvusk18:02
openstackgerritAndreas Jaeger proposed openstack/project-config master: Fix gerrit/projects.yaml sort order  https://review.opendev.org/66252718:02
*** kjackal has joined #openstack-infra18:04
*** tesseract has quit IRC18:04
clarkbproject-config still replicating18:06
*** tosky has quit IRC18:09
*** dave-mccowan has quit IRC18:09
corvusoh looks like private_ipv4 is always defined18:10
corvuseven if it's public18:10
openstackgerritBrian Haley proposed zuul/zuul-jobs master: Update repo file for fedora-29  https://review.opendev.org/66253618:11
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132718:11
clarkbhaleyb: is that required?18:12
clarkbI think pre 28 is when things changed18:12
clarkbso I think that will break 2818:12
haleybclarkb: i don't know, was just following what was done when 28 was added18:12
clarkbhaleyb: 29 is added and running and fine. The only thing that should need to be chagned is that devstack nodeset definition I linked you18:12
haleybclarkb: any idea how to overcome this zuul complaint in https://review.opendev.org/#/c/662529/ ?18:13
clarkbah the problem is what the stable branches ahve the nodesets too and they must match18:14
clarkbI don't know that -latest nodesets will work in branched repos like devstack give that constraint18:14
clarkbcorvus: ^18:14
clarkbhaleyb: zuul requires nodeset definitions to be the same acorss branches18:14
clarkbbut that change to a single branch breaks that rule18:14
clarkbso you might need to add a -29 nodeset and switch to that18:14
clarkbthen we can delete the -latest nodeset when it isn't used anymroe18:15
*** ykarel|away has quit IRC18:15
openstackgerritAndreas Jaeger proposed openstack/project-config master: Check ACLs for all namespaces  https://review.opendev.org/66253718:15
haleybfrom reading the commit message i thought the point was to avoid using the number, but i'm not going to fight it18:15
clarkbhaleyb: yes I think that was pabelanger's intent when he added the -latest nodeset but when he added them to project-config which is branchless that was fine. The problems arise when it got cargo culted into other repos18:16
clarkbif those repos are branched18:16
clarkbanother appraoch would be to move the devstack nodeset defs into openstack-zuul-jobs18:17
corvusthe process is supposed to be to remove nodesets from branches18:20
hrwhi again18:21
clarkbcorvus: ah so clean up stable/*18:21
corvus1) create branch from master 2) remove nodesets 3) update nodeset in master branch 4) repeat18:21
corvusyeah18:21
*** rkukura has joined #openstack-infra18:21
corvusthe idea is that nodesets aren't likely to change in the minutes after a branch creation, so if you do that, you won't hit an error.18:22
haleybcorvus: so these should move?  or should be removed from stable/* ?18:22
corvusif you put it off, then you're in this situation18:22
clarkbhrw: hello18:22
corvushaleyb: removed from stable18:22
hrwin 662456/4 I selected 'debian-buster-arm64' label for nodeset as it is listed in https://opendev.org/openstack/project-config/src/branch/master/nodepool/nl03.openstack.org.yaml as being present in 'linaro-london' cloud. But zuul gives me NODE_FAILURE on check.18:22
corvushaleyb: so if you remove all the nodesets from stable/* branches, then have your change to update the -latest nodeset depends-on those, all should be good.18:23
corvus(the alternative would be to ignore nodesets from stable branches, which would be more convenient, except that it would also be *way* more confusing)18:23
haleybcorvus: let my try that, think i just might have to remove it from stable/stein or will that then fail, and down the rabbit hole18:23
corvushaleyb: it should be perfectly fine to remove it from all stable branches, it's basically a noop already.  but you do need to remove it from every branch it appears in (i haven't checked too see which ones it does)18:24
corvushaleyb: to be clear, just the definition needs to be removed18:24
clarkbhrw: I don't see any buster images currently so something must be broken18:24
hrwhttp://zuul.openstack.org/nodes does not list 'debian-buster-arm64' (only 'ubuntu-xenial-arm64', 'ubuntu-bionic-arm64' and 'debian-stretch-arm64')18:24
clarkbstretch, bionic and xenial are present18:25
haleybcorvus: ack, i'll give it a try18:25
hrwclarkb: will switch back to stretch then18:25
clarkbhrw: we'll have to dig into why those buster images aren't being built. No build logs either https://nb03.openstack.org/18:27
clarkbbut right now I'm still pretty context switched into the project renaming so probably won't dig into that until later today or monday18:27
clarkbproject-config is still replicating18:27
hrwclarkb: no rush. stretch is fine too18:28
hrwthanks for all the work and have a good weekend18:29
corvusclarkb: looks like we're waiting on gitea07 and it is replicating p-c as we speak18:29
clarkbcorvus: yup18:29
clarkbits been going for a while18:29
corvusi wonder if there's a problem18:30
clarkbOne thing that happened before was when we had gitea07 restart when we did the big replication last time it was around the openstack/puppet repos18:30
clarkbpossible it didn't fully replicate refs/notes and refs/changes for project-config since that is alphabetically close18:30
clarkb(and now it is catching up?)18:30
clarkbnova is still reindexing btu is the only one left there18:31
corvusclarkb: yeah, work is happening on 07 with p-c18:31
corvusit's pushing old change refs18:31
clarkbya so that explains it. It was probably caught in that restart along with the puppet repos18:32
corvusps shows commands like: bash hooks/update refs/changes/02/559802/2 0000000000000000000000000000000000000000 a2f6d622089bfa34470b58ad76db01a3d6a3f59b18:32
corvusi don't think the refs are ordered, so it's hard to say it's progress18:32
clarkbdisk usage looks ok (I checked that because if it was replicating a lot of stuff it might overrun the disk again too)18:33
clarkbwe can check if master is updated18:34
* clarkb looks18:34
clarkbhttps://gitea07.opendev.org:3000/openstack/project-config/commit/6312597c060cb93689197a60140e32c31fd06edd exists and  https://gitea07.opendev.org:3000/openstack/project-config/commits/branch/master shows it as head18:35
clarkbso we are probably ok to proceed18:35
clarkboh its done18:36
corvusjust when i was about to be able to quantify it :)18:36
corvusin case you were wondering, project-config has about 35k refs18:36
clarkbwow18:36
clarkband we ended up with 8.6GB of disk free which GCing will clean up a bit from there18:37
clarkbso that all looks good18:37
*** whoami-rajat has quit IRC18:37
clarkbI think we can proceed whenever the humans are ready?18:37
clarkbdid we want to wait for fungi to return?18:37
* fungi returned a while back, sorry18:38
fungistill catching up18:38
clarkbnext cron run will be 18:45 UTC if we reenable before then18:38
corvusi think we're good18:38
corvusi will re-enable18:39
clarkbya I can't think of anything else to check (I individually checked the giteas for the ref we need)18:39
corvusre-enabled18:39
clarkb`tail -f /var/log/manage_projects.log` if you want to watch jeepyb there18:40
clarkbfungi: re github archival, you are thinking maybe add jroll to the access list there?18:41
*** yamamoto has joined #openstack-infra18:41
clarkbmy one concern with this is we were previously trying to reduce our account surface area in github (to avoid people accidenting github pushes/merges/etc)18:42
clarkbso we may need to have a think about whether or not we use a second shared account (not shared by opendev but for openstack specifically) or require second admin accounts or something18:42
jrollI would be terrified of screwing that up, but am willing to try if y'all prefer18:42
jrollcan always revoke my access when complete18:42
clarkbpersonally I'm happy for openstack to self manage it, but I think the general please don't use your real account rules we were slowly moving towards were decent (I say this and I haven't removed my account yet so shame on me :) )18:43
fungiclarkb: adding jroll wfm as long as it wfh18:43
fungii think (at least) sdague was already given access to that org previously to aid in curation18:44
*** mriedem_hangry is now known as mriedem18:44
jrollI'm fine with doing it (and can create another account if you prefer), if one or two of you can help review my logic/list in the email I sent to the ML a bit ago18:44
clarkbjroll: really the concern is that if you are working on your github fork of say nova or ironic that you don't accidentally push your company fork to upstream nova in github18:45
clarkbjroll: and if you are aware of those concerns then its probably fine because you'll be careful :)18:45
jrollclarkb: for sure, understandable18:45
clarkband ya we can help review the plan and details18:45
jrollexcellent18:46
*** yamamoto has quit IRC18:46
clarkbansible should've started18:46
clarkbyes seems to be starting base.yaml18:47
fungiyeah, watching the run_all log18:47
fungilooks... busy18:47
* fungi can't read that fast18:47
clarkbthis will take like 15 minutes iirc18:49
clarkb(base.yaml)18:49
clarkbthough we did clean it up a bit so maybe shorter now18:49
openstackgerritAndreas Jaeger proposed openstack/project-config master: Check ACLs for all namespaces  https://review.opendev.org/66253718:50
*** bobh has joined #openstack-infra18:51
clarkbdoing exim then snmpd then iptables then onto the service stuff and puppet stuff18:53
*** eharney has quit IRC18:54
clarkbbase is done18:55
openstackgerritAndreas Jaeger proposed openstack/project-config master: Use retired ACL everywhere  https://review.opendev.org/66222318:55
openstackgerritAndreas Jaeger proposed openstack/project-config master: Remove unused ACLs  https://review.opendev.org/66191318:55
clarkbits doing dns servers now. which we should check after to make sure the renames didn't end up in unhappyness18:56
clarkbThey should follow the redirects18:56
clarkbgitea running now18:58
clarkbproject-config on bridge updated properly18:59
clarkbgitea looked good19:00
*** openstackgerrit has quit IRC19:01
fungiyeah, all seems fine so far19:01
clarkbjeepyb is running now and project-config on review was updated19:01
clarkbyup looking good19:02
clarkbreview is done19:05
clarkbit is doing afs now19:05
clarkbthen I think zuul is in the else block now19:06
clarkbnova is still indexing but we are now down to 8 tasks in the gerrit queue19:07
clarkbother than that indexing I think we are caught up19:07
clarkbzuul should get puppeted soon then we can recheck changes19:08
clarkbtimburke: ^ we need to update the zuul main.yaml on the zuul scheduler server that should happen shortly then you can recheck your changes19:09
*** openstackgerrit has joined #openstack-infra19:10
openstackgerritSlawek Kaplonski proposed zuul/zuul-jobs master: Add role to fetch journal log from test node  https://review.opendev.org/64373319:10
fungii left a similar comment on 66230319:11
clarkbpuppet is just now starting to run on zuul0119:12
clarkband its done19:12
fungiwe should be clear to recheck those changes now? i can do that19:12
clarkbhttps://review.opendev.org/#/c/662225/2 I just rechecked that one as a canary but y a I think you can recheck them all19:13
fungi662303 was the only other i was aware of19:13
clarkbI think all of AJaeger's fixes above too?19:14
AJaegerfungi, could you review this infra-specs change so that all links work again, please? https://review.opendev.org/#/c/662297/19:14
clarkbhrm we still have errors in the zuul web ui19:14
clarkbmaybe it is loading its config?19:14
AJaegerclarkb: mine should be fine (with exception of system-config) - but I'll double check now19:14
clarkbthere are a ton of merger jobs being logged by the scheduler so that may be it19:15
AJaegerclarkb: http://zuul.openstack.org/projects still shows x/pyeclib19:15
clarkbAJaeger: ya I think it hasn't finished reloading its new config yet19:15
AJaegeragreed19:16
timburkethanks for running that down clarkb and AJaeger -- fwiw, i'm not in a huge rush or anything; we don't have anything we're trying to push through on those projects right now19:16
AJaegerand "Last reconfigured" is not new yet...19:16
clarkbaccording ot the logs I think it is done19:17
AJaegertimburke: It's a broken zuul job that will show up as error on zuul's status page - so, I'd like to see it fixed ;)19:17
timburkefair enough :-)19:17
clarkbbut web ui is not updated yet19:17
corvusi think it's still churning19:17
*** imacdonn has quit IRC19:17
AJaegertimburke: but no need for you to stay longer for it - can wait until Monday...19:18
*** imacdonn has joined #openstack-infra19:18
clarkblast reconfigured did just update19:18
AJaegerand we have http://zuul.openstack.org/project/openstack/pyeclib ;)19:18
corvus2019-05-31 19:18:23,469 DEBUG zuul.ConfigLoader: Created layout id a255741dea884c67ae73f3fbfd12d7a119:18
AJaeger\o/19:18
AJaegerstill our 5 config errors ;(19:18
corvus(the delay is the re-enqueing of changes after loading the config)19:19
clarkbAJaeger: ya I think those might be cached? I'm not convinced yet that they are still a problem19:19
clarkb662225 got far enough to start jobs at least19:20
AJaegertimburke: you can +2A the change if you like - if it passes, it will be fine ;)19:21
clarkbas did pyeclib19:21
openstackgerritMerged openstack/project-config master: Fix gerrit/projects.yaml sort order  https://review.opendev.org/66252719:21
timburke🎉19:21
corvusthat's one heck of a cache19:21
fungi"cache"19:21
clarkboh I see its not complaining about projects not existing anymore19:21
clarkbbut the other errors are probably actually errors19:21
clarkbso some of them did go away19:21
clarkbsystem-config-run-dns is not defined apparently ( as an example)19:22
clarkband os-http isn't in zuul/main.yaml at all19:22
clarkbso a few things to clean up I guess19:22
corvusagree those look legit19:23
clarkb662225 fixes system-config-run-dns19:24
clarkb(or should)19:24
clarkband ya os-http is just removed from project-config/zuul/main.yaml is the reason for that we are retiring it?19:25
AJaegeros-http was moved - so that's wrong ;(19:26
* AJaeger sends another hotfix19:26
clarkbya it was moved but maybe it is also being retired? Simple fix in either case19:26
*** happyhemant has quit IRC19:27
openstackgerritAndreas Jaeger proposed openstack/project-config master: Readd x/os-http  https://review.opendev.org/66254919:27
AJaegerclarkb: it was only moved...19:28
clarkbAJaeger: ok so your fix is the correct thing19:29
clarkbit has my +219:29
corvusand +319:30
AJaegerthanks19:30
corvusAJaeger: thank you19:31
AJaegercould you also review clarkb's record of the renames? https://review.opendev.org/661845 and https://review.opendev.org/#/c/662097 ?19:31
corvusi also +3d the system-config change19:31
* AJaeger can approve these19:32
clarkbI updated the second one with the additional project this morning too19:33
clarkb(so it should be good to go)19:33
AJaegerclarkb: yeah - both approved now.19:33
* AJaeger added a few reviews to https://etherpad.openstack.org/p/project-renames-2019-05-3119:34
clarkblooking19:35
clarkbAJaeger: do we need to set the var for https://review.opendev.org/#/c/662537/2 in tox.ini ?19:37
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132719:37
corvusimma gonna get some lunch19:37
clarkbanyone else want to review https://review.opendev.org/#/c/662297/ whcih is a cleanup for urls in our infra specs? If not I'll just approve it19:37
clarkbalso https://review.opendev.org/#/c/662293/1/docs-site/infra-index.html.tmpl for linking to the specs19:38
clarkbI think all of the more urgent once that fix zuul's running config have been approved though which is great19:38
*** jcoufal has quit IRC19:38
fungihrm, yeah, i can't tell from http://logs.openstack.org/37/662537/2/check/project-config-gerrit/b16c388/job-output.txt.gz#_2019-05-31_18_54_31_175193 whether it actually did anything19:40
clarkbfungi: corvus: I'm going to detach from the screen now19:40
AJaegerclarkb: we could remove that var completely, looks like historic relict19:40
clarkbI'm out of the screen last one in should close it :)19:41
AJaegerfungi: it runs's here http://logs.openstack.org/37/662537/2/check/project-config-gerrit/b16c388/job-output.txt.gz#_2019-05-31_18_55_10_85380019:41
clarkbAJaeger: I wonder if the intent was to force openstack to follow certain rules but not force stackforge19:42
openstackgerritMerged openstack/infra-manual master: Follow rename of sandbox repos  https://review.opendev.org/66222619:42
clarkbif the rules are relaxed enough we can probably get away without that distinction and just check for sane things19:42
AJaegerclarkb: it passes everywhere, so we could test it everywhere19:42
clarkbk19:42
clarkbthen ya maybe we just want to run it everywhere and if we run into a case where that isn't helpful anymore add in the separation again19:43
fungiAJaeger: yeah, i saw the "are valid" result, but there's no indication of whether it possibly checked 0 acls and all were valid19:43
AJaegerfungi: shall I add some logging?19:43
fungii guess we can assume it didn't sit on its thumbs for 30 seconds, so i'm okay with it19:44
clarkb#status log Performed repo renames. Some were cleanup from opendev migration and others were normal reorgs.19:44
openstackstatusclarkb: finished logging19:44
fungithe gap suggests there was *something* going on anyway19:44
* AJaeger will udpate19:45
fungiAJaeger: even just a count of acls in the summary might be helpful, but i don't think it's urgent19:45
clarkbI'm going to go for a bike ride and find lunch as soon as the kids get home19:45
clarkbbut I think things are looking good and the fixes we need are on their way in19:46
fungimostly concerned that if the variable in that loop is undefined it will short-circuit silently19:46
fungiwhereas at least the static list didn't run the same risk19:46
clarkbya19:47
*** slaweq has quit IRC19:48
clarkbThanks everyone for the help on this. Much smoother than last time :)19:48
clarkband with the bug fixes we pushed up hopefully even better the next time19:48
fungiyes, huge thanks to everyone!19:48
fungii need to go run some errands i skipped during lunch, but glad to see things are basically working now19:49
clarkbI'll be out for probably an hour or so in just a few minutes. When I get back will help ensure the rest of these fixups get in and if I have time take a quick look at why arm64 buster images aren't happening19:50
openstackgerritMerged openstack/project-config master: Readd x/os-http  https://review.opendev.org/66254919:50
openstackgerritAndreas Jaeger proposed openstack/project-config master: Check ACLs for all namespaces  https://review.opendev.org/66253719:53
AJaegerfungi: should be much better now ^19:53
*** slaweq has joined #openstack-infra19:54
*** hjensas|afk is now known as hjensas19:54
AJaegertimburke: and pyeclib jobs work fine ;) - on https://review.opendev.org/66230319:54
*** dmsimard has quit IRC19:55
*** dmsimard has joined #openstack-infra19:55
*** dmsimard has quit IRC19:56
fungiAJaeger: ooh, awesome thanks!19:56
*** eharney has joined #openstack-infra19:57
openstackgerritAndreas Jaeger proposed openstack/project-config master: Retire opendev-website (step 3)  https://review.opendev.org/66180319:57
AJaegerand this retires last openstack-infra repo ^19:57
*** mattw4 has joined #openstack-infra19:58
*** slaweq has quit IRC19:58
*** dmsimard has joined #openstack-infra20:01
*** dmsimard has quit IRC20:01
*** Ng_ is now known as Ng20:03
fungiokay, popping back out to run errands now20:04
*** e0ne has joined #openstack-infra20:07
*** xek_ has joined #openstack-infra20:11
*** e0ne has quit IRC20:12
*** e0ne has joined #openstack-infra20:13
AJaegerso, Zuul has been reconfigured and now knows x/os-http - one error less ;)20:14
* AJaeger waves good night20:16
clarkbgood night. thanjs again20:19
*** dmsimard has joined #openstack-infra20:21
*** dpawlik has joined #openstack-infra20:23
corvusWOW!  https://review.opendev.org/661327  works :)20:24
corvusthat's one doozy of a job.  note the 145 tasks in the main playbook20:24
corvusthere's still one TODO in there, which is to figure out a way to verify the image has been pushed to the intermediate registry.20:27
corvusprobably can do an ansible uri module call or something20:27
corvushowever the push command did run: http://logs.openstack.org/27/661327/42/check/zuul-jobs-test-registry/e42ceca/ara-report/result/3441ba5e-5dd5-4226-b8ee-523588dd544f/20:28
*** bobh has quit IRC20:29
*** e0ne has quit IRC20:30
openstackgerritMerged opendev/system-config master: Follow opendev renames  https://review.opendev.org/66222520:30
*** dpawlik has quit IRC20:31
corvuslooks like the only remaining errors are in pyeclib20:33
*** rkukura has quit IRC20:45
*** rfolco has quit IRC20:45
*** kjackal has quit IRC20:47
*** dklyle has quit IRC20:58
clarkband pyeclib fixes ahve since merged. No more errors21:10
clarkbdo we need to restart nodepool builders to pick up new images? I wonder if that is why we don't have debian buster for arm64 yer21:13
corvusclarkb: not under normal circumstances21:13
corvusit should still watch its config file21:14
clarkbhrm the config for it is complete as far as I can tell. We have a label, provider, and diskimage21:14
clarkband that file is up to date on nb03.o.o21:14
clarkbhrm we have filled the disk on that builder though21:14
clarkbI'll start by cleaning that up21:15
clarkbok I think part of the issue here is we have >2 ready iamges for each image on that builder I'm guessing because of the clouds that went away21:18
clarkbfungi: any idea where we are in cleaning those up?21:19
clarkbI think mordreds reorg of clouds.yamls may actually clean some of that up but probably not on the nodepool config side?21:19
clarkbthis may end up being a next week thing as the thread is already getting long :)21:19
clarkbat this point I expect we'll have to sanitize the db manually since the cloud went poof21:21
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132721:21
corvusclarkb: if you have a minute, can you take a quick look at that? ^ it should be working now (previous patchset passed, latest is cleanup and a bit more checking).  i think the next thing is to find a permanent home for it, but it'd be good to get a second set of eyes on it now before i go too far down that road.21:23
clarkbcan do21:23
corvusin particular, feel free to say "wtf add comments here" :)21:24
*** rkukura has joined #openstack-infra21:25
clarkbcorvus: seems sane. There are a couple bits that I wonder if we need them to be test specific like setting up certs and user credentials. Don't the existing roles already do that?21:34
*** jtomasek has quit IRC21:37
corvusclarkb: not quite in exactly the way we need for the test.  i would like to refactor them so we can reduce that duplication.  we could either continue that work on this change, or after we create the job.21:39
corvus(i didn't want to do it before now because i wanted testing for that kind of refactoring :)21:40
clarkbthat seems like a reasonable compromise21:40
*** Lucas_Gray has joined #openstack-infra21:43
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132721:48
*** xek_ has quit IRC21:48
*** pcaruana has quit IRC21:54
*** sreejithp has quit IRC22:08
*** sreejithp has joined #openstack-infra22:09
clarkbfungi: if you are still around https://review.opendev.org/#/c/662511/ would be a good one to get in before we forget22:10
*** hwoarang has quit IRC22:15
*** hwoarang has joined #openstack-infra22:16
*** _erlon_ has quit IRC22:19
*** sreejithp has quit IRC22:20
*** rkukura has quit IRC22:31
*** rkukura has joined #openstack-infra22:31
*** rlandy has quit IRC22:32
*** rkukura has quit IRC22:34
*** yamamoto has joined #openstack-infra22:44
*** yamamoto has quit IRC22:48
*** michael-beaver has quit IRC22:52
fungii was not around but am back now and taking a look23:01
*** rfarr_ has quit IRC23:08
clarkbI'm doing another pass of the changes AJaeger put on the etherpad now23:09
clarkbmake sure nothing needs a recheck or new review23:09
clarkbfungi: do you want to look at https://review.opendev.org/#/c/662297/ before I approve it? cleanup of infra-specs for new urls23:13
*** mattw4 has quit IRC23:15
fungiyeah, looking now23:17
clarkbI annotated the etherpad with the current state of things for all the changes there23:17
corvusi approved the remaining ones23:21
clarkb\o/23:22
clarkbspec update now approved too23:22
clarkbhopefully that means AJaeger returns to work on monday with a slightly cleaner slate/smaller backlog23:23
*** aaronsheffield has quit IRC23:23
*** raissa has quit IRC23:24
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132723:26
openstackgerritMerged opendev/infra-specs master: Update site URL  https://review.opendev.org/66229723:33
*** panda has quit IRC23:37
openstackgerritMerged opendev/system-config master: Fix gitea rename playbook org creation  https://review.opendev.org/66251123:37
clarkblooking at the calendar I'll refresh certs week after next23:42
openstackgerritMerged openstack/project-config master: Check ACLs for all namespaces  https://review.opendev.org/66253723:43
openstackgerritMerged openstack/project-config master: Remove unused ACLs  https://review.opendev.org/66191323:43
openstackgerritMerged openstack/project-config master: Use retired ACL everywhere  https://review.opendev.org/66222323:43
openstackgerritMerged openstack/project-config master: Update infra-specs link  https://review.opendev.org/66229323:43
*** gyee has quit IRC23:44
*** rh-jelabarre has quit IRC23:53

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