Wednesday, 2020-02-05

*** dchen has joined #openstack-infra00:00
*** Goneri has joined #openstack-infra00:05
*** mattw4 has quit IRC00:06
openstackgerritIan Wienand proposed openstack/project-config master: Update service/opendev-zuul key  https://review.opendev.org/70587300:19
fungiianw: looking through the current tarballs sites, i'm not sure we're publishing zuul release tarballs anywhere besides pypi00:24
ianwfungi: that ... could be right.  we don't have, say /afs/openstack.org/project/opendev.org/releases00:25
fungiwe're only publishing branch tip artifacts at https://tarballs.opendev.org/zuul/zuul/ and the last release artifacts published in https://tarballs.openstack.org/zuul/ seems to be from mid-last year00:26
fungii think all releases from july onward (around the same time we set up the branch tip tarballs via promote pipeline to tarballs.opendev.org) are only on pypi00:27
fungii suspect that was not intentional00:27
*** armax has quit IRC00:29
ianwfungi: that would actually be the jobs i'm say aren't used @ https://review.opendev.org/#/c/705367/ ?00:29
fungiyep, that change is what got me looking00:33
fungiwell, or zuul-release-python needs modifying to also copy files to the tarballs afs volume00:34
fungizuul-release-python also isn't creating detached openpgp signatures for the files it's generating00:36
fungilooks like https://review.opendev.org/670388 was the change which removed use of the older release-zuul-python job00:37
ianwhttps://zuul.opendev.org/t/zuul/build/9f8770685cef4a9f882705e078990833 looks like the last run of zuul-release-python00:38
fungi(in favor of opendev-release-python which was then replaced by zuul-release-python in the later https://review.opendev.org/676466 change)00:39
fungiright, i looked through the log of that one a few minutes ago00:40
fungiit doesn't seem to do anything with signing artifacts or copying them to afs00:41
funginot sure if those roles were missed when the job was written or needed recreating00:41
ianwahhh, ok00:41
ianwor dropped for $reason?00:43
*** jamesmcarthur has joined #openstack-infra00:45
fungiit's not clear from review comments that was the case at least00:47
fungibut i'm still digging in history00:49
fungihttps://review.openstack.org/649199 implemented opendev-release-python and skipped tarball uploading because it predated creation of tarballs.opendev.org00:51
fungii suspect that's it, job was not completed once the new tarballs site was created00:53
ianwfungi: ok, what do you think, file a story?00:56
ianwas we mentioned before, it would be good to switch more to the opendev jobs, but i think after the extant jobs are publishing to afs00:57
*** jamesmcarthur has quit IRC01:00
fungiyeah, i get the impression that when corvus wrote 649199 he expected someone to add tarball site publication later before switching zuul to it, and then we just forgot to double-check that months later when reviewing 67038801:05
fungii wonder if it's as simple as just adding the missing roles for signing and copying to afs?01:06
*** rcernin has joined #openstack-infra01:06
*** jamesmcarthur has joined #openstack-infra01:10
*** slaweq_ has joined #openstack-infra01:11
*** michael-beaver has quit IRC01:13
ianwfungi: i don't think so because the release job builds (https://opendev.org/opendev/base-jobs/src/branch/master/playbooks/release-python/run.yaml) while the existing tarballs jobs use promote pipeline?01:14
ianwi don't think it's unsolvable, but just an impedance mis-match01:14
*** slaweq_ has quit IRC01:16
fungizuul-release-python (which is parented on opendev-release-python) runs in the release pipeline01:16
fungiopendev-promote-python is what zuul runs in the promote pipeline01:17
fungiunless i'm misunderstanding you01:17
*** Lucas_Gray has quit IRC01:18
ianwfungi: that's correct. opendev-promote-python is what uploads the tarball currently01:18
ianwthis is where it's copied ... https://opendev.org/opendev/base-jobs/src/branch/master/playbooks/artifacts/promote.yaml#L3901:20
fungiahh, i see, and the signing and copying for that aren't implemented as roles, they're baked into its playbooks01:20
fungi(or the copying anyway, and signing isn't implemented at all?)01:20
ianwyep, that was what i was going to say :)01:20
ianwi haven't seen anything related to gpg secrets in the opendev repos01:21
*** jamesmcarthur has quit IRC01:22
*** ociuhandu has joined #openstack-infra01:31
*** ociuhandu has quit IRC01:35
*** armax has joined #openstack-infra01:36
*** vesper has joined #openstack-infra01:38
*** vesper11 has quit IRC01:39
openstackgerritIan Wienand proposed opendev/system-config master: Update letsencrypt_gid documentation  https://review.opendev.org/70587801:43
*** jamesmcarthur has joined #openstack-infra01:45
*** Goneri has quit IRC01:47
*** igordc has joined #openstack-infra01:52
*** gyee has quit IRC02:00
ianwkevinz: rather than try and maintain two builders, i feel like we just replace nb0302:05
kevinzianw: you mean setup nb03 at linaro-us side and remove the linaro-london side right?02:06
kevinzI think it is a good approach02:07
ianwyep, let me look at starting the server ... it's just about the only operation we have non-root can't do :)02:09
ianwkevinz: hrm, we'll need a xenial image in there, our builders don't support bionic yet02:12
ianwwe can actually get our cloud launcher to do that i guess02:12
kevinzianw: you mean that upload an xenial image to linaro-us? I can help to move it02:15
openstackgerritIan Wienand proposed opendev/system-config master: linaro-us : add Xenial image  https://review.opendev.org/70587902:15
ianwkevinz: ^ that should upload it.  i think we can quick approve it due to the non-production status of all this02:16
*** armax has quit IRC02:16
*** armax has joined #openstack-infra02:16
*** armax has quit IRC02:17
kevinzcool02:18
ianwi just uploaded it anyway, to get this going02:20
*** jamesmcarthur has quit IRC02:22
ianwok, launch it, let's see what happens02:22
ianwkevinz: hrm, so that's not booting02:27
ianwkevinz: http://paste.openstack.org/show/789145/ ... look familiar ?  it's dropped into a shell i guess02:28
kevinzianw: let me check02:28
ianwmaybe i got the properties on the image wrong?02:31
*** roman_g has quit IRC02:33
kevinzianw: could you try without any properties?02:35
ianwhrm, yep, let me delete them02:35
*** rfolco|bbl has quit IRC02:40
ianwkevinz: hrm, it still seem stuck? http://paste.openstack.org/show/789146/02:42
kevinzianw: sorry in a meeting now, will sort out the problem soon02:49
kevinzafter the meeting02:49
ianwok, np02:49
*** rlandy has quit IRC03:03
openstackgerritMerged opendev/base-jobs master: Migrate to upload-afs-roots role  https://review.opendev.org/70537103:08
kevinzianw: back. I tested and it still stuck...03:09
kevinzI will test one from uk builder03:09
*** slaweq_ has joined #openstack-infra03:11
*** apetrich has quit IRC03:12
*** rh-jelabarre has quit IRC03:13
*** slaweq_ has quit IRC03:16
*** armax has joined #openstack-infra03:19
openstackgerritMerged openstack/project-config master: Remove publish-zuul-python-branch-tarball and release-zuul-python  https://review.opendev.org/70536703:27
openstackgerritMerged openstack/project-config master: Migrate to upload-afs-roots role  https://review.opendev.org/70537003:27
openstackgerritMerged openstack/project-config master: upload-artifacts-with-afs: use upload-afs-synchronzie role  https://review.opendev.org/70537403:27
*** ykarel|away is now known as ykarel03:30
*** rcernin has quit IRC03:31
*** gagehugo has quit IRC03:45
*** ramishra has joined #openstack-infra03:50
*** gagehugo has joined #openstack-infra03:59
*** ramishra has quit IRC04:01
*** armax has quit IRC04:03
ianwkevinz: i'm not sure what the difference would be?04:18
*** raukadah is now known as chkumar|rover04:22
*** ramishra has joined #openstack-infra04:24
*** dSrinivas has joined #openstack-infra04:42
*** adriant has quit IRC04:45
*** iokiwi has quit IRC04:45
*** adriant has joined #openstack-infra04:50
*** udesale has joined #openstack-infra04:53
*** bnemec has joined #openstack-infra04:53
*** hongbin has joined #openstack-infra04:55
ianwi feel like hrw helped me with this before04:56
*** rcernin has joined #openstack-infra04:57
ianw2020-02-05 02:28:20.559265 | bridge.openstack.org | TASK [install-kubectl : Install kubectl] ***************************************04:58
ianw2020-02-05 02:28:20.574481 | bridge.openstack.org | Wednesday 05 February 2020  02:28:20 +0000 (0:00:11.462)       0:00:16.763 ****04:58
ianw2020-02-05 02:51:40.272354 | RUN END RESULT_TIMED_OUT: [untrusted : opendev.org/opendev/system-config/playbooks/zuul/run-base.yaml@master]04:58
*** rcernin has quit IRC04:58
ianwhave to keep an eye on that in the gate ... it looks like that hung the test ... probably just upstream issues04:58
*** rcernin has joined #openstack-infra04:58
*** igordc has quit IRC04:59
*** slaweq_ has joined #openstack-infra05:11
*** slaweq_ has quit IRC05:15
*** hongbin has quit IRC05:28
*** soniya29 has joined #openstack-infra05:29
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-infra05:34
kevinzianw: I use this image: https://cloud-images.ubuntu.com/xenial/current/xenial-server-cloudimg-arm64-uefi1.img, with uefi support, can boot into kernel, but failed to boot the system05:38
kevinzianw: I will talk with hrw later today05:38
kevinzAlso I test image got from uk cloud, could not boot either..05:39
ianwkevinz: yeah, that's what i uploaded  ... that would be great.  i feel like we needed those property flags originally in the cn cloud; but then i see in the uk cloud it doesn't have them05:39
ianwmaybe there's some magic config happening behind the scenes05:39
kevinzianw: that's what I got now :http://paste.openstack.org/show/789152/05:40
ianwhuh, that got into the kernel which i didn't see ... but that may be a serial to console issue?05:43
openstackgerritMerged zuul/zuul-jobs master: Use unique loop vars to avoid conflicts  https://review.opendev.org/70533705:43
openstackgerritMerged zuul/zuul-jobs master: Link the policy to the CONTRIBUTING.rst file  https://review.opendev.org/70448405:50
*** rcernin has quit IRC06:03
*** redrobot has quit IRC06:04
*** tetsuro has joined #openstack-infra06:06
*** redrobot has joined #openstack-infra06:06
openstackgerritMerged openstack/project-config master: Add "Review-Priority" gerrit label to devstack  https://review.opendev.org/70554706:10
*** tetsuro has quit IRC06:10
*** lmiccini has joined #openstack-infra06:13
*** tetsuro has joined #openstack-infra06:15
*** tetsuro has quit IRC06:27
openstackgerritJan Kubovy proposed zuul/zuul master: WIP: Store unparsed branch config in Zookeeper  https://review.opendev.org/70571606:36
*** tetsuro has joined #openstack-infra06:58
*** roman_g has joined #openstack-infra07:00
*** tetsuro has quit IRC07:01
*** tetsuro has joined #openstack-infra07:01
*** tetsuro has quit IRC07:02
*** tetsuro has joined #openstack-infra07:03
*** slaweq_ has joined #openstack-infra07:11
*** slaweq_ has quit IRC07:16
*** tetsuro has quit IRC07:17
*** tetsuro has joined #openstack-infra07:19
*** tetsuro has joined #openstack-infra07:20
*** tetsuro has quit IRC07:31
*** tetsuro has joined #openstack-infra07:32
*** pgaxatte has joined #openstack-infra07:37
*** ociuhandu has joined #openstack-infra07:38
*** jaosorior has joined #openstack-infra07:38
*** ykarel is now known as ykarel|lunch07:42
*** dpawlik has joined #openstack-infra07:49
*** ociuhandu has quit IRC07:50
openstackgerritFelix Schmidt proposed zuul/zuul master: Report retried builds via sql reporter.  https://review.opendev.org/63350107:52
openstackgerritFelix Schmidt proposed zuul/zuul master: Store information about gate resets in MQTT and SQL reporter  https://review.opendev.org/69667007:52
*** dpawlik has quit IRC07:53
*** jaosorior has quit IRC07:53
*** iurygregory has quit IRC07:53
*** dpawlik has joined #openstack-infra07:56
*** tetsuro has quit IRC07:57
*** tetsuro has joined #openstack-infra08:00
*** slaweq_ has joined #openstack-infra08:00
*** jaosorior has joined #openstack-infra08:06
*** slaweq__ has joined #openstack-infra08:09
*** slaweq_ has quit IRC08:10
*** iurygregory has joined #openstack-infra08:11
*** jtomasek has quit IRC08:12
*** tkajinam has quit IRC08:12
openstackgerritTobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check  https://review.opendev.org/64455708:12
*** tetsuro has quit IRC08:14
*** slaweq has joined #openstack-infra08:14
*** tesseract has joined #openstack-infra08:16
*** slaweq__ has quit IRC08:16
*** tommylikehu has quit IRC08:22
*** hashar has joined #openstack-infra08:27
*** ociuhandu has joined #openstack-infra08:30
*** rcernin has joined #openstack-infra08:31
*** jtomasek has joined #openstack-infra08:34
*** rpittau|afk is now known as rpittau08:34
*** ociuhandu has quit IRC08:34
openstackgerritSimon Westphahl proposed zuul/zuul master: Spec for allowing circular dependencies  https://review.opendev.org/64330908:36
*** ociuhandu has joined #openstack-infra08:36
openstackgerritSimon Westphahl proposed zuul/zuul master: Add optional support for circular dependencies  https://review.opendev.org/68535408:36
*** ociuhandu has quit IRC08:40
*** gfidente has joined #openstack-infra08:41
*** ralonsoh has joined #openstack-infra08:42
*** ociuhandu has joined #openstack-infra08:44
*** gshippey has joined #openstack-infra08:45
*** rcernin has quit IRC08:48
*** apetrich has joined #openstack-infra08:48
*** slaweq has quit IRC08:48
*** slaweq_ has joined #openstack-infra08:49
*** ociuhandu has quit IRC08:49
*** ykarel|lunch is now known as ykarel08:52
*** jpena|off is now known as jpena08:53
*** rcernin has joined #openstack-infra08:53
*** xek has joined #openstack-infra08:54
*** ociuhandu has joined #openstack-infra08:59
*** lucasagomes has joined #openstack-infra09:02
*** slaweq__ has joined #openstack-infra09:06
*** zxiiro has quit IRC09:06
*** slaweq_ has quit IRC09:07
*** rcernin has quit IRC09:07
*** ociuhandu has quit IRC09:13
*** pkopec has joined #openstack-infra09:15
*** rcernin has joined #openstack-infra09:16
kevinzianw: there?09:18
openstackgerritTobias Henkel proposed openstack/diskimage-builder master: Fix cache-url -f  https://review.opendev.org/70590209:19
kevinzianw: hrw and I tested that boot the xenial with ipv4 subnet works fine. But for both IPv4 and IPv6, it could not boot..09:19
*** rcernin has quit IRC09:20
*** slaweq__ is now known as slaweq09:25
*** rcernin has joined #openstack-infra09:26
*** ociuhandu has joined #openstack-infra09:32
*** rcernin has quit IRC09:33
*** ociuhandu has quit IRC09:38
*** lmiccini has quit IRC09:39
*** lmiccini has joined #openstack-infra09:40
*** derekh has joined #openstack-infra09:40
openstackgerritSimon Westphahl proposed zuul/zuul master: Spec for allowing circular dependencies  https://review.opendev.org/64330910:04
*** rfolco|bbl has joined #openstack-infra10:08
*** ociuhandu has joined #openstack-infra10:14
*** ramishra has quit IRC10:15
openstackgerritChandan Kumar (raukadah) proposed zuul/zuul-jobs master: Added bzip2 package for CentOS 8  https://review.opendev.org/70591110:18
*** ociuhandu has quit IRC10:20
*** ramishra has joined #openstack-infra10:21
*** slaweq_ has joined #openstack-infra10:40
*** hashar has quit IRC10:41
*** slaweq has quit IRC10:42
openstackgerritRiccardo Pittau proposed openstack/diskimage-builder master: Allow python3 to be used in CentOS 7  https://review.opendev.org/70592010:48
*** slaweq__ has joined #openstack-infra11:01
*** dciabrin has joined #openstack-infra11:02
*** slaweq_ has quit IRC11:03
*** rpittau is now known as rpittau|bbl11:10
*** udesale_ has joined #openstack-infra11:13
*** pkopec has quit IRC11:14
*** udesale has quit IRC11:16
*** ramishra has quit IRC11:18
*** ramishra has joined #openstack-infra11:22
*** dciabrin has quit IRC11:28
*** psachin has joined #openstack-infra11:30
*** pcaruana has quit IRC11:37
*** ahosam has joined #openstack-infra11:39
*** ahosam has quit IRC11:46
*** pcaruana has joined #openstack-infra11:50
*** ociuhandu has joined #openstack-infra11:52
*** ociuhandu has quit IRC11:58
*** dciabrin has joined #openstack-infra12:01
*** ociuhandu has joined #openstack-infra12:05
*** derekh has quit IRC12:07
*** derekh has joined #openstack-infra12:08
*** ociuhandu has quit IRC12:11
*** tosky has joined #openstack-infra12:12
*** ociuhandu has joined #openstack-infra12:17
*** slaweq__ has quit IRC12:19
*** ociuhandu has quit IRC12:22
*** slaweq__ has joined #openstack-infra12:23
*** jpena is now known as jpena|lunch12:33
*** soniya29 has quit IRC12:38
*** Lucas_Gray has joined #openstack-infra12:40
*** rh-jelabarre has joined #openstack-infra12:45
*** rfolco|bbl is now known as rfolco12:45
*** ociuhandu has joined #openstack-infra12:57
*** Lucas_Gray has quit IRC13:01
*** rlandy has joined #openstack-infra13:01
*** Lucas_Gray has joined #openstack-infra13:02
*** nhicher has quit IRC13:05
*** nhicher has joined #openstack-infra13:06
*** slaweq has joined #openstack-infra13:11
*** slaweq__ has quit IRC13:13
*** ociuhandu has quit IRC13:13
*** rpittau|bbl is now known as rpittau13:15
*** smcginnis|FOSDEM is now known as smcginnis13:19
*** jamesmcarthur has joined #openstack-infra13:20
*** ociuhandu has joined #openstack-infra13:22
openstackgerritRiccardo Pittau proposed openstack/diskimage-builder master: Allow python3 to be used in CentOS 7  https://review.opendev.org/70592013:23
*** lpetrut has joined #openstack-infra13:24
*** ociuhandu has quit IRC13:27
*** ykarel is now known as ykarel|away13:27
*** ociuhandu has joined #openstack-infra13:28
*** ociuhandu has quit IRC13:29
*** electrofelix has joined #openstack-infra13:29
*** ociuhandu has joined #openstack-infra13:29
*** jpena|lunch is now known as jpena13:35
*** jamesmcarthur has quit IRC13:37
*** Lucas_Gray has quit IRC13:38
*** Lucas_Gray has joined #openstack-infra13:39
*** dciabrin has quit IRC13:45
*** jamesmcarthur has joined #openstack-infra13:47
*** smarcet has joined #openstack-infra13:49
*** pkopec has joined #openstack-infra13:52
mordredfungi: is there some trick these days to getting console log to work on rax vms?13:53
*** zxiiro has joined #openstack-infra13:53
mordredor, rather, the emergency serial console thing?13:53
fungivia the web browser, i pull up the server instance details and then there's an actions drop-down to let you choose the emergency console13:54
fungithat should open in a new window/tab though maybe you have a pop-up blocker or something rejecting that?13:54
openstackgerritFelix Schmidt proposed zuul/zuul master: Implement github checks API  https://review.opendev.org/70516813:55
fungiat any rate, it usually comes up for me as a black rectangle because the screen has gone to sleep, but giving it input focus and then hitting enter usually wakes it up13:55
*** liuyulong has joined #openstack-infra13:58
mordredhrm13:59
*** dciabrin has joined #openstack-infra14:00
*** eharney has joined #openstack-infra14:01
*** rpittau is now known as rpittau|mtg14:03
stephenfinmordred, fungi, clarkb, <others>: I'm thinking we should hold off on removing py27 support from pbr because afaict we don't pin pbr anywhere and we'd break *everyone* still using Python 2.7. Correct? https://review.opendev.org/#/c/70569314:04
*** dtantsur is now known as dtantsur|brb14:05
fungistephenfin: quite possibly. we implement it as a setup_requires so it's fetched out of band from normal install_requires14:08
fungiand you're right, in openstack/nova stable/rocky for example, we pass it as setup_requires=['pbr>=2.0.0'] so no version cap14:09
mordredyeah - let's hold off on pbr14:15
mordredalso - it doesn't really change much14:15
*** pcaruana has quit IRC14:17
mordredfungi: so - in rax dfw - review-dev01.opendev.org seems stuck in rebooting state and I'm not sure how to debug further (I get nothing, not even a black box on the emergency console)14:17
fungimake a review-dev02? is there benefit in testing out a fresh redeployment?14:18
fungidid you do a hard reboot/server reboot --force?14:18
fungior just a "soft" reboot?14:19
*** ykarel|away is now known as ykarel14:20
fungii'll see if i have any luck with the console14:20
fungimordred: same, console just gives me "loading console" and a spinner14:24
fungipretty sure the server isn't in a state where its console is reachable by nova14:25
*** rpittau|mtg is now known as rpittau14:25
*** psachin has quit IRC14:25
*** ociuhandu has quit IRC14:29
*** Lucas_Gray has quit IRC14:30
mordredfungi: *awesome*14:31
mordredfungi: and yeah - this is the new server I made yesterday :)14:31
fungiouch14:31
mordredso, I can totally blow it away and build it again - that's just a bummer14:31
fungiyeah, that's not great14:32
mordredit was in read-only root filesystem mode this morning14:32
fungisounds related14:32
mordredso I tried just rebooting it since it wasn't in production or anything14:32
mordredand things have not gone well since14:32
*** ociuhandu has joined #openstack-infra14:33
*** Lucas_Gray has joined #openstack-infra14:33
*** jamesdenton has joined #openstack-infra14:34
*** ociuhandu has quit IRC14:37
openstackgerritAlfredo Moralejo proposed zuul/zuul-jobs master: Add CentOS8 High Availability repository  https://review.opendev.org/70597314:45
openstackgerritAlfredo Moralejo proposed zuul/zuul-jobs master: Add CentOS8 High Availability repository  https://review.opendev.org/70597314:47
mordredfungi: oh - and of course, it's not going to let me delete it because it's in rebooting state14:51
*** sshnaidm|afk is now known as sshnaidm14:51
mordredfungi: I'm going to have to file a ticket aren't I?14:51
clarkbmordred: did you hard reboot it?14:51
mordredclarkb: yes14:52
mordredclarkb: and it's stuck in rebooting - for about an hour now14:52
clarkbok sometimes the avpi event for spft reboot gets ignored by the VM and the dtate doesnt change thereeither14:52
*** ociuhandu has joined #openstack-infra14:54
mordredclarkb, fungi: I was able to delete it14:57
*** Lucas_Gray has quit IRC14:58
openstackgerritClark Boylan proposed opendev/system-config master: Update project doc to reflect OpenDev changes  https://review.opendev.org/70348814:59
mordredWOW15:00
mordredsince yesterday the base xenial image we were using has gone away or been renamed :)15:00
fungineat15:05
ttxHmm, I'm confused. Where should I go to define a new label that Zuul can vote on?15:07
mordredsuper weird - if I do an image show on the uuid reported by the image for review01 - it totally shows up with the name that's expected - but I can't find it in launch.py15:07
ttxLooks like the Gerrit ACL has some, but maps them to groups15:07
mordredbut - there is a new one that replaces PVHVM with Cloud - I'm trying that one :(15:08
fungittx: yes, that's precisely how it works15:08
ttxCan't find where for example you allow Zuul to vote Verified+115:08
fungiit's in the all-projects acl, which is not managed by project-config15:08
*** ykarel is now known as ykarel|away15:08
fungijust a sec15:08
ttxah. That would.. explain my grep fail15:09
clarkband then in the zuul pipeline config you define actions for succes/failure and that includes the gerrit category and value15:09
fungittx: https://docs.openstack.org/infra/system-config/gerrit.html#access-controls15:10
ttxclarkb: yeah that part I got right15:10
*** Lucas_Gray has joined #openstack-infra15:11
fungittx: that document contains the project.config file we use for configuring gerrit's all-projects metaproject15:11
fungittx: i think you want "group Continuous Integration Tools"15:12
ttxOK looks like the group I shall use would be... yes that15:12
ttxThanks!15:12
clarkbmordred: stephenfin re pbr correct we cannot drop python2 from there unless we want to break all of the valid users of pbr with python215:12
*** eharney has quit IRC15:13
stephenfinclarkb: Cool. You're all in agreement so yeah, we'll hold off on this for another year (or even two)15:14
clarkbstephenfin: or 10 :) dependso n how long openstack stable branches live plus some leeway probably15:14
stephenfin:)15:15
clarkbtrain in particular is expected to be supported for a long time because it does python2 right?15:16
clarkband we don't want to break that15:16
stephenfinI would imagine so, yes. In hindsight having an upper bound of the next major release across projects wouldn't have been a terrible idea15:17
clarkbit doesn't work15:17
stephenfinbut no way am I going to submit the hundreds of patches needed to do that15:17
clarkbin fact doing that breaks all kinds of stuff15:17
stephenfinoh?15:17
clarkbI want to say we pinned all pbr usage to <1.0.0 way back when. Then 1.0.0 released and due to bootstrap methods for setup_requires 1.0.0 was pulled down anyway but then running setup.py failed beacuse 1.0.0 was not in the valid range15:18
mordredyeah - pbr is a special pony15:18
clarkbthe problem is specifically with setup_requires15:18
mordredit's why we're so cautious about making changes to it15:19
mordredmaybe one day with the new whatever system for specifying libraries that take care of building distributions we can rework it15:19
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add index_links option to zuul manifest  https://review.opendev.org/70558015:19
clarkbmordred: ya that should actually respect versions and even verifies checksums!15:19
stephenfinah, I thought it was so we wouldn't be blocked on older branches if there was an urgent change needed in pbr15:20
stephenfinTIL15:20
fungiit's possible setup_requires versioning works now with pip shadowing easy_install15:20
fungibut that'll still only be the case for people using pip from the last year or two i think15:21
*** Lucas_Gray has quit IRC15:21
*** Wryhder has joined #openstack-infra15:21
stephenfinsomewhat related, apparently people have been hired by the PSF to improve pip and fix e.g. the horrific dependency resolver it has15:21
stephenfinthey were asking for guinea pigs in the Python devroom at FOSDEM15:21
openstackgerritMonty Taylor proposed zuul/zuul-jobs master: Google logs: Link to index.html files  https://review.opendev.org/70558615:21
*** Wryhder is now known as Lucas_Gray15:22
*** ociuhandu_ has joined #openstack-infra15:22
clarkbstephenfin: I try to keep up to date on that bug (I was the original filer)15:23
clarkb988 iirc15:23
stephenfinNot sure if it's the bug, but the wiki page for the work is here https://wiki.python.org/psf/Pip2020DonorFundedRoadmap15:24
clarkbhttps://github.com/pypa/pip/issues/988 is the bug15:25
*** eernst has joined #openstack-infra15:25
*** eernst has quit IRC15:25
*** ociuhandu has quit IRC15:25
fungistephenfin: yep, they've been funding sumana and the rest of the crew at changeset to help with pypa (pip, warehouse/pypi, et cetera) stuff15:26
clarkbI hope they succeed and have more productivity in that code base than I15:26
stephenfinfungi: Yeah, I saw the work on supporting tokens in warehouse15:27
*** ociuhandu_ has quit IRC15:27
*** chkumar|rover is now known as raukadah15:27
stephenfinI'm hoping they add support for user groups at some point. I tried doing it but it was too much for a weekend project and the ground kept shifting under my feet :(15:27
*** lpetrut has quit IRC15:28
clarkbstephenfin: groups for managing pypi packages? that could be helpful for projects with lots of packages15:30
stephenfinyup, for those that don't have it all automated15:30
stephenfin(the sphinx-doc and sphinx-contrib organizations on GitHub, in my case)15:31
openstackgerritMonty Taylor proposed opendev/zone-opendev.org master: Update review-dev01.opendev.org IP addresses  https://review.opendev.org/70598515:31
mordredclarkb, fungi: ^^15:31
clarkbmy struggle has typically been with their review culture. I don't know if we are spoiled or my expectations are wrong, but I basically got told to take a hike on my last PR and that my code wasn't even worthy of proper review because there was so much wrong with it15:32
clarkbso now I hope others will succeed in making pip better15:32
clarkbmordred: you'll need to update the inventory file too (and possibly clean out the ansible cache)15:33
openstackgerritMonty Taylor proposed opendev/system-config master: Add review-dev01.opendev.org  https://review.opendev.org/70566915:33
mordredyup ^^15:33
Shrewsclarkb: i had a similar unfortunate experience years ago in a java-based community project that shall remain nameless.  :(15:33
openstackgerritMerged zuul/zuul-jobs master: Add upload-logs-gcs role  https://review.opendev.org/70371115:34
mordredclarkb: re ansible cache - what do you mean?15:34
clarkbmordred: if ansible had previously talked to that host it would think the IP address was the old one15:34
clarkbmordred: though maybe it wasn't the IP and it was other facts15:35
clarkbrm'ing the file in /var/cache/ansible (or wherever it goes) seemed tofix it in the past15:35
openstackgerritMonty Taylor proposed opendev/system-config master: Get LE certs for review.o.o  https://review.opendev.org/70567015:35
openstackgerritMonty Taylor proposed opendev/system-config master: Remove review-dev01.openstack.org  https://review.opendev.org/70567115:35
openstackgerritMonty Taylor proposed opendev/system-config master: Use LE certs for Apache  https://review.opendev.org/70569015:35
mordredclarkb: nod. will do15:35
clarkbin this case looks like we hadn't added it to the inventory yet anyway so should be fine15:35
mordredOH FFS15:35
mordredclarkb: don't approve that yet15:36
clarkbI won't15:36
mordredI ...15:36
mordredI didn't replace "servername" in the launch readme15:36
mordredso I just created "servername.opendev.org"15:36
clarkbha15:36
mordredyeah15:36
clarkbdrink more coffee?15:36
* mordred sighs15:36
mordredyes15:36
openstackgerritJames E. Blair proposed zuul/zuul master: Add disallowed-labels tenant option  https://review.opendev.org/70585615:36
*** ramishra has quit IRC15:37
*** ianychoi_ is now known as ianychoi15:40
openstackgerritMerged zuul/zuul-jobs master: Add index_links option to zuul manifest  https://review.opendev.org/70558015:40
*** osmanlicilegi has quit IRC15:42
*** osmanlicilegi has joined #openstack-infra15:42
openstackgerritMerged zuul/zuul-jobs master: Google logs: Link to index.html files  https://review.opendev.org/70558615:44
*** armax has joined #openstack-infra15:47
*** pcaruana has joined #openstack-infra15:49
openstackgerritThierry Carrez proposed openstack/project-config master: Add release-approval pipeline  https://review.opendev.org/70599015:50
fungioh this is neat, apparently rackspace has started listing their ipv6 netblocks in the spamhaus pbl as well15:51
*** dciabrin has quit IRC15:52
fungiso now a lot of our sysadmins are bouncing cronspam back from our servers15:52
clarkbwhile I'm not actually sitting in a boat on a river in the rain freezing, I've got sick kid at home so will probably end up being half here anyway15:52
fungii guess i'll try to work on getting all the ones i'm seeing bounces for delisted15:52
clarkbfungi: thanks!15:52
*** dtantsur|brb is now known as dtantsur15:53
mordredfungi: oh joy!15:56
*** smarcet has quit IRC15:57
openstackgerritMonty Taylor proposed opendev/zone-opendev.org master: Update review-dev01.opendev.org IP addresses  https://review.opendev.org/70598515:57
mordredclarkb, fungi: _now_ that's valid15:57
openstackgerritMonty Taylor proposed opendev/system-config master: Add review-dev01.opendev.org  https://review.opendev.org/70566915:59
openstackgerritMonty Taylor proposed opendev/system-config master: Get LE certs for review.o.o  https://review.opendev.org/70567015:59
openstackgerritMonty Taylor proposed opendev/system-config master: Remove review-dev01.openstack.org  https://review.opendev.org/70567115:59
openstackgerritMonty Taylor proposed opendev/system-config master: Use LE certs for Apache  https://review.opendev.org/70569015:59
*** ociuhandu has joined #openstack-infra16:01
*** Goneri has joined #openstack-infra16:02
*** eharney has joined #openstack-infra16:03
fungi#status log delisted ipv6 address of storyboard01.opendev.org from the spamhaus pbl16:03
fungihrm, looks like there's something else which also needs fixing16:03
*** openstackstatus has joined #openstack-infra16:05
*** ChanServ sets mode: +v openstackstatus16:05
fungi#status log restarted statusbot after failing to return from a 2020-02-04 11:16:50 utc netsplit16:05
openstackstatusfungi: finished logging16:05
fungi#status log delisted ipv6 address of storyboard01.opendev.org from the spamhaus pbl16:05
*** ociuhandu has quit IRC16:05
openstackstatusfungi: finished logging16:05
mordredinfra-root: could I get some eyes on https://review.opendev.org/#/c/705985 real quick (and its parent) - I need to get that updated so I can keep rolling on review-dev16:06
fungimordred: regarding 705662, there's a reason it didn't have a caa rr yet16:12
fungiwe'll have to add those after we cut over to letsencrypt-issued certs for review.opendev.org16:13
*** smarcet has joined #openstack-infra16:13
mordredfungi: ah. so I should ditch that change for now?16:13
fungioh, actually, wait, i'm confusing that with cert pinning16:13
fungiit's *probably* safe to add now, the caa record tells certificate authorities that if they're not letsencrypt.org then they shouldn't issue a cert for review.o.o and such16:14
fungias long as we don't try to ask namecheap for another cert for review.o.o then it shouldn't be an issue16:14
mordrednod. well, yeah, I think we're going to be on LE before the existing one expires16:15
mordredlet me rephrase, we *BETTER* be on LE before the namecheap expires in april16:15
*** ramishra has joined #openstack-infra16:16
fungiokay, both of those are approved16:17
*** lmiccini has quit IRC16:18
*** slaweq_ has joined #openstack-infra16:23
*** jamesmcarthur has quit IRC16:24
*** udesale_ has quit IRC16:24
*** slaweq has quit IRC16:25
openstackgerritMerged opendev/zone-opendev.org master: Clean up LE entry for review.o.o  https://review.opendev.org/70566216:25
mordredfungi: woot - thanks!16:26
openstackgerritMerged opendev/zone-opendev.org master: Update review-dev01.opendev.org IP addresses  https://review.opendev.org/70598516:27
fungiinfra-root: can we delete mirror01.mtl01.inap.opendev.org? i see mirror.mtl01.inap.opendev.org is currently a cname to mirror02 and there is no dns record for mirror01, so cronspam deliveries from it are bouncing due to lack of an address rr16:27
*** smarcet has quit IRC16:29
*** dciabrin has joined #openstack-infra16:31
mordredfungi: I do not know the answer - but my hunch would be that it's ok given those informations16:32
corvusditto16:32
*** smarcet has joined #openstack-infra16:33
*** psachin has joined #openstack-infra16:38
cgoncalveshey infra! is it possible to add project specific tasks on stable branch create to openstack bot? like it does to update the requirements URL in tox.ini16:38
fungicool, i'll probably go ahead and turn it off after i return from lunch, and then delete it if nothing breaks16:38
*** jamesmcarthur has joined #openstack-infra16:38
fungiin the meantime, i'm going to go find lunch. back in an hour or so and will continue going through cronspam bounces to see what else needs delisting in the pbl16:39
*** jamesmcarthur_ has joined #openstack-infra16:41
*** jamesmcarthur has quit IRC16:44
*** smarcet has quit IRC16:46
*** sean-k-mooney has quit IRC16:51
*** dciabrin has quit IRC16:53
*** lucasagomes has quit IRC16:53
*** artom has quit IRC16:54
*** jamesmcarthur_ has quit IRC16:55
*** psachin has quit IRC16:58
*** jamesmcarthur has joined #openstack-infra16:58
*** Lucas_Gray has quit IRC17:04
*** jamesmcarthur has quit IRC17:05
*** eernst has joined #openstack-infra17:05
*** pgaxatte has quit IRC17:06
*** zxiiro has quit IRC17:07
*** zxiiro has joined #openstack-infra17:08
*** iurygregory has quit IRC17:08
*** Lucas_Gray has joined #openstack-infra17:11
*** soniya29 has joined #openstack-infra17:24
*** smarcet has joined #openstack-infra17:25
*** jamesmcarthur has joined #openstack-infra17:27
*** gyee has joined #openstack-infra17:29
*** evrardjp has quit IRC17:33
*** evrardjp has joined #openstack-infra17:34
*** igordc has joined #openstack-infra17:34
*** eernst has quit IRC17:35
*** sean-k-mooney has joined #openstack-infra17:36
*** sean-k-mooney has quit IRC17:41
*** sean-k-mooney has joined #openstack-infra17:43
*** vdrok has quit IRC17:45
*** vdrok has joined #openstack-infra17:45
*** dtantsur is now known as dtantsur|afk17:48
*** sean-k-mooney1 has joined #openstack-infra17:48
*** sean-k-mooney has quit IRC17:48
*** sean-k-mooney1 is now known as sean-k-mooney17:49
openstackgerritTobias Henkel proposed zuul/zuul master: Add spec for scale out scheduler  https://review.opendev.org/62147917:49
*** jaosorior has quit IRC17:49
*** jpena is now known as jpena|off17:53
*** jamesmcarthur has quit IRC17:56
mordredfungi, corvus: if you have a sec - I ran the letsencrypt playbook against the new review-dev01.opendev.org - I got a key and a csr in the letsencrypt certs dir ... but no cer or ca.cer file (I'm looking at security.openstack.org's certs as an example from an apache config perspective)17:56
mordreddid I do something wrong?17:56
corvusmordred: did it run on adns too?17:57
*** gfidente is now known as gfidente|afk17:58
corvus(i've never run letsencrypt with a restricted host list; i've only let the regular cron job run)17:58
mordredcorvus: ah! that's probably the reason for sure17:59
mordredI see this in the log:17:59
mordred[Wed Feb  5 17:52:58 UTC 2020] review-dev.opendev.org:Verify error:Incorrect TXT record17:59
mordredcorvus: so - yeah - that's probably right on :)17:59
*** TheJulia has quit IRC18:01
*** TheJulia has joined #openstack-infra18:01
corvusmordred: i bet if you added adns to the list it'd work (but make sure you don't step on a cron run)18:01
*** rpittau is now known as rpittau|afk18:01
mordredcorvus: yeah - I feel like maybe just landing the patch that adds the inventory/letsencrypt stuff would be better at this point - it's not like landing those will break anything in prod at the moment18:02
corvus++18:02
openstackgerritCarlos Goncalves proposed openstack/project-config master: Update branch checkout and u-c.txt for Octavia  https://review.opendev.org/70607018:02
mordredinfra-root: if you have a sec, landing https://review.opendev.org/#/c/705669 and https://review.opendev.org/#/c/704498 would be nice18:02
*** johnsom has quit IRC18:03
openstackgerritMerged opendev/git-review master: Make it possible to specify who is notified  https://review.opendev.org/70286118:03
*** johnsom has joined #openstack-infra18:03
*** jamesmcarthur has joined #openstack-infra18:03
*** artom has joined #openstack-infra18:06
*** soniya29 has quit IRC18:07
*** mattw4 has joined #openstack-infra18:09
*** tesseract has quit IRC18:11
openstackgerritCarlos Goncalves proposed openstack/project-config master: Update branch checkout and u-c.txt for Octavia  https://review.opendev.org/70607018:18
*** lathiat has quit IRC18:22
*** lathiat has joined #openstack-infra18:22
*** Lucas_Gray has quit IRC18:26
openstackgerritMerged opendev/system-config master: Add Apache to Ansible for Gerrit  https://review.opendev.org/70449818:33
*** masayukig has quit IRC18:34
*** masayukig has joined #openstack-infra18:34
fungi#status log halted mirror01.mtl01.inap.opendev.org (which does not appear to be in production) in preparation for deletion18:38
openstackstatusfungi: finished logging18:38
*** mattw4 has quit IRC18:48
*** eharney has quit IRC18:49
*** ralonsoh has quit IRC18:54
*** jamesmcarthur has quit IRC18:54
fungi#status log delisted ipv6 address of mirror-update01.opendev.org from the spamhaus pbl18:54
openstackstatusfungi: finished logging18:54
*** jamespage has quit IRC18:56
*** mattw4 has joined #openstack-infra18:56
*** jamespage has joined #openstack-infra18:56
cmurphyjust saw this https://zuul.opendev.org/t/openstack/build/c3f1711ecd9545abbddc00a6d419469c not sure if this is something to be concerned about?18:58
*** electrofelix has quit IRC18:59
AJaegercmurphy: that is interesting - so, the docs job run in gate but promote job could not find it. Definitely worth investigating...19:00
*** gshippey has quit IRC19:01
AJaegerinfra-root, https://zuul.opendev.org/t/openstack/build/6ef0429860004c05a3e40b0150db318e is the gate job - and has no artificats. Any ideas what happened?19:01
openstackgerritMerged opendev/system-config master: Add review-dev01.opendev.org  https://review.opendev.org/70566919:01
fungiAJaeger: we also collected no logs19:03
fungiinterestingly, opendev.org/opendev/base-jobs/playbooks/tox-docs/post.yaml did collect logs19:04
fungifollowed by opendev.org/opendev/base-jobs/playbooks/base/post.yaml which was a no-op on log collection19:04
*** dpawlik has quit IRC19:04
AJaegerfungi: json seems to be available - console shows content? That's confusing :(19:04
*** dpawlik has joined #openstack-infra19:05
*** mattw4 has quit IRC19:06
fungi#status log delisted ipv4 address of nb01.opendev.org from the spamhaus pbl19:08
openstackstatusfungi: finished logging19:09
*** artom has quit IRC19:15
*** jamesmcarthur has joined #openstack-infra19:15
*** mattw4 has joined #openstack-infra19:19
openstackgerritJames E. Blair proposed zuul/zuul master: Extract the watcher from git driver  https://review.opendev.org/70609219:21
*** larsks has left #openstack-infra19:28
*** sean-k-mooney has quit IRC19:29
openstackgerritCarlos Goncalves proposed openstack/project-config master: Update branch checkout and u-c.txt for Octavia  https://review.opendev.org/70607019:30
*** sean-k-mooney has joined #openstack-infra19:32
*** masayukig has quit IRC19:38
*** lathiat has quit IRC19:38
*** vdrok has quit IRC19:39
*** zxiiro has quit IRC19:39
*** jamespage has quit IRC19:39
*** TheJulia has quit IRC19:39
mordredinfra-root: booyah! https://review-dev.opendev.org/#/q/status:open19:40
*** johnsom has quit IRC19:40
mordredI ran the playbook from the tip of the stack19:41
openstackgerritMonty Taylor proposed opendev/system-config master: Update letsencrypt_gid documentation  https://review.opendev.org/70587819:41
mordredinfra-root: so, https://review.opendev.org/#/c/705878 and parents are ready for review I believe19:42
mordredfwiw - I copied the home dir over from review-dev.openstack.org since we expect to be updating review.o.o in place19:44
*** xek_ has joined #openstack-infra19:48
*** pkopec has quit IRC19:49
*** xek has quit IRC19:51
openstackgerritJames E. Blair proposed zuul/zuul master: Extract the watcher from git driver  https://review.opendev.org/70609220:00
*** eharney has joined #openstack-infra20:04
*** jtomasek has quit IRC20:06
*** artom has joined #openstack-infra20:17
*** dklyle has quit IRC20:19
*** david-lyle has joined #openstack-infra20:19
*** jamesmcarthur has quit IRC20:21
ianwAJaeger/ fungi : is there a problem with artifacts?  i can see another instance of promote job failing20:27
ianwsystem-config : this job should have made the docs -> https://zuul.opendev.org/t/openstack/build/89dd023531884a3c9cddfccaa2782116/console20:27
ianwthis job failed to find it -> https://zuul.opendev.org/t/openstack/build/b483f3e6d07c43508e8d84e40a72b6c820:28
AJaegerianw: see backscroll from cmurphy20:29
AJaegernobody else mentioned anything, but let's check20:29
AJaegerianw: http://zuul.opendev.org/t/openstack/builds?pipeline=promote20:29
ianwyeah, looks the same as cmurphy comment20:29
AJaegerThat looks bad ;(20:29
AJaegerinfra-root, promote pipeline is broken, most recent ones all failed20:30
ianwyeah ... ok first thing was zuul restarted recently?20:30
AJaegerfirst failure at 15:46 today, and complete failure since 17:02 today20:31
AJaegerianw: thanks for checking!20:31
ianwnothing in status ... zuul-jobs might be the next suspect20:31
ianwclarkb's loop change is the only one, ... maybe, when did that merge?20:32
AJaegerwe merged at 15:46 this one: https://review.opendev.org/70558620:32
AJaegercorvus, could your stack be related to breaking promote jobs? ^20:33
fungihttps://review.opendev.org/705580 merged at 15:40z and added an option to the generate-zuul-manifest role20:33
fungithat's a more likely cause than 705586 which only touches the upload-logs-gcs role20:34
ianwyeah, that looks like a good one to investigate20:34
AJaegerfungi: agreed, 705580 looks more likely - both in the same stack20:35
ianwif library/generate_manifest.py raises an exception, i don't think we see it in the logs and it might cause the manifest to not be made, but hte job still pass20:36
fungii think we'll see it on the executor though, right?20:36
* fungi checks logs20:37
fungibuild 89dd023531884a3c9cddfccaa2782116 ran on ze0520:39
openstackgerritMerged openstack/project-config master: Add release-approval pipeline  https://review.opendev.org/70599020:39
ianwyeah, unfortunately nothing there i can see20:40
AJaegerlet me propose reverts for now - while we investigate...20:41
fungi2020-02-05 18:38:34,185 DEBUG zuul.AnsibleJob: [e: b631f24b90bf427b9d143f5ee1870f4f] [build: 89dd023531884a3c9cddfccaa2782116] Sending result: {"result": "SUCCESS", "warnings": [], "data": {"zuul": {"artifacts": [{"url": "zuul-manifest.json", "metadata": {"type": "zuul_manifest"}, "name": "Zuul Manifest", "index_links": false}, {"url": "docs-html.tar.bz2", "metadata": {"type": "docs_archive"},20:41
fungi"name": "Docs archive"}, {"url": "docs/", "metadata": {"type": "docs_site"}, "name": "Docs preview site"}], "log_url": "https://c426c48a68cc95805513-73204bd61763b348e9171435eeed8553.ssl.cf1.rackcdn.com/705669/7/gate/opendev-tox-docs/89dd023/"}}}20:41
openstackgerritAndreas Jaeger proposed zuul/zuul-jobs master: Revert "Add index_links option to zuul manifest"  https://review.opendev.org/70610620:41
fungithat seems to have successfully set index_links: false20:42
fungiso i'm not seeing any evidence of a problem therwe20:42
fungiindeed, https://c426c48a68cc95805513-73204bd61763b348e9171435eeed8553.ssl.cf1.rackcdn.com/705669/7/gate/opendev-tox-docs/89dd023/ has the logs20:42
fungimaybe adding index_links broke the dashboard somehow?20:42
fungior something broke the mysql reporter?20:43
*** slaweq_ has quit IRC20:44
*** dkehn has joined #openstack-infra20:45
ianwhttps://c426c48a68cc95805513-73204bd61763b348e9171435eeed8553.ssl.cf1.rackcdn.com/705669/7/gate/opendev-tox-docs/89dd023/zuul-manifest.json looks all ok20:45
ianwi wonder if adding the "index_links" as a separate variable has broken some sort of json parser20:45
ianwnot variable, key in the output dict i mean20:46
fungiyeah, i think either 1. the manifest isn't being recorded in the db, or 2. it's not able to be displayed by the dashboard20:46
fungistill digging20:46
AJaegerfungi: keep in mind that hte promote jobs is not finding the artifact20:46
ianwi don't see any errors in browser console20:47
AJaegerSo, displaying in the dashboard is not he only problem20:47
*** slaweq_ has joined #openstack-infra20:47
AJaeger"Could not find or access '/var/lib/zuul/builds/c3f1711ecd9545abbddc00a6d419469c/work/docs-html.tar.bz2' on the Ansible Controller."20:47
fungigood point AJaeger, so artifacts/manifest are missing from the build metadata i guess20:47
AJaegerfungi, yes, that could explain the missing display in dashboard as well20:48
fungii'm looking in the db now20:50
ianw2020-02-05 19:01:44,618 DEBUG zuul.SQLReporter: Result data did not pass artifact schema validation:20:52
AJaegerianw: great20:52
AJaegerWe have {"index_links": false, "tree":20:52
ianwhang on, that might be a red herring20:52
AJaegervs {"tree": [{20:52
AJaegerin zuul-manifest.json at the beginning20:53
ianwyeah, i think it has something to do with this20:53
fungiokay, so the build with uuid 89dd023531884a3c9cddfccaa2782116 has id 921717920:54
fungiand the zuul_artifact table has 0 records where build_id=921717920:54
AJaegerianw, fungi, I pushed up the revert for review in case you want to merge it. I'm signing off now, hope you find the culprit!20:55
ianwi think this is it -> https://opendev.org/zuul/zuul/src/branch/master/zuul/lib/artifacts.py#L2420:55
AJaegerThanks for digging further20:55
fungicompared to working build 3235693177904546aaaa6fca586d75be whose id is 9203227 and has several corresponding records in the zuul_artifact table20:55
fungiso we're failing to record the manifest records in the zuul_artifact table20:56
AJaegerianw: lines 18-21 I guess20:56
mordredianw: I think v.Extra should let it have extra fields, right?20:56
mordredoh - 18-2120:56
AJaegerianw: we change from name, url, metadata to name, url, index_links, metadata20:56
AJaegerso, that clearly fails the check20:57
ianwyeah -- i think we have to revert the original, then i think zuul will need a restart to handle it before adding it again20:57
AJaegerSo, revert - update zuul, restart, revert ervert?20:57
mordredyeah - so - I think we should land the revert so that we can re-revert once there is a fix20:57
mordredAJaeger: yah20:57
* AJaeger waves good night...20:58
fungithanks AJaeger!20:59
fungimordred: ianw: can you summarize your suspicions in a comment on 706106?20:59
ianwfungi: i'll propose the change in zuul, that probably gets attention20:59
fungiwfm21:00
fungithanks!!!21:00
openstackgerritIan Wienand proposed zuul/zuul master: Handle index_links in manifest  https://review.opendev.org/70611521:03
corvusstop please21:04
corvusi see the error and it's a one-liner21:05
corvusgive me just a sec to propose a fix21:05
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Move index_links to correct task in generate_zuul_manifest  https://review.opendev.org/70611621:07
corvusianw, fungi, AJaeger, mordred: ^21:07
corvusthis was never supposed to touch the artifact data21:07
ianwheh, and so that gets in there as a bool, and everything still passes ... i see :)21:09
corvusianw: i don't follow that21:10
ianwi mean the new value gets into the manifest a boolean, which does trigger the artifact schema failing, rather than raising an error or exception, just a bug that it nicely hides itself21:11
corvusthe extra field in the manifest is fine; the artifact storage doesn't care what the content of the artifact is.21:13
corvusthe artifact storage only cared that we were giving it extra garbage21:14
*** slaweq_ has quit IRC21:15
corvusi -2d the other changes to prevent merging; i think we should abandon them.  sorry for the bug, and thanks for the zuul-jobs revert (which would have stopped the errors as well)21:15
fungithanks for the quick fix! reviewing now21:16
ianwcorvus: if you have a sec would you mind a +2 on https://review.opendev.org/#/c/705873/1 -- that's the re-encoding of the key we discussed yesterday21:17
corvusianw: those characters look correct to me! :)21:17
ianwyes, more just the theory of it :)21:18
ianwafter that's tested, i think we're right to switch the tarball publishing jobs and will propose final changes21:18
*** tosky has quit IRC21:18
*** smarcet has left #openstack-infra21:19
openstackgerritJames E. Blair proposed zuul/zuul master: Extract the watcher from git driver  https://review.opendev.org/70609221:21
ianwinfra-root: actually one more that adds the tarballs sites to ansible (won't change anything till dns updated)21:22
ianwhttps://review.opendev.org/#/c/704913/21:22
ianwoh, hang on, that's gone into merge conflict, i guess the apache stuff, just a minute21:23
openstackgerritIan Wienand proposed opendev/system-config master: Add tarballs.<openstack|opendev>.org to static.opendev.org  https://review.opendev.org/70491321:24
*** xek_ has quit IRC21:27
*** rfolco has quit IRC21:27
openstackgerritMerged openstack/project-config master: Update service/opendev-zuul key  https://review.opendev.org/70587321:29
openstackgerritMerged zuul/zuul-jobs master: Move index_links to correct task in generate_zuul_manifest  https://review.opendev.org/70611621:31
*** rfolco has joined #openstack-infra21:31
*** hwoarang has quit IRC21:35
*** rfolco has quit IRC21:35
*** hwoarang has joined #openstack-infra21:37
*** dciabrin has joined #openstack-infra21:40
*** liuyulong has quit IRC21:41
openstackgerritTristan Cacqueray proposed zuul/zuul master: add_host: enable using ansible_fqdn and ansible_private_key_file  https://review.opendev.org/70612421:52
corvustristanC: if you use ssh-add you can avoid leaving the key sitting around on the executor during the job.  eg https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/add-fileserver/tasks/main.yaml#L12-L1622:00
corvustristanC: anyway, i can't think of a reason not to add that, but let's ask mordred and tobiash to review22:02
mordredcorvus: should we wait for an ssh-add based respin?22:02
mordredoh - wait22:02
mordredyeah - I can't see any reason not to add that patch22:03
corvusthe ssh-add was more of an alternative to consider22:03
mordredyeah22:03
* mordred groks now22:03
corvusi can't remember if we disallowed those args intentionally, or just didn't think to add them because we used ssh-add originally22:04
mordredinfra-root: my internet went out for no good reason (there is an alarm light on the fiber modem - so I'm guessing that has something to do with it) - so I'm phone-tethered for the new little bit until it can get resolved22:04
mordredcorvus: I *think* it's the latter22:04
mordred(yes, I have tried turning it off and turning it back on again)22:05
fungimordred: have you considered relocating it above sea level?22:11
fungicould be barnacles eating the packets22:11
openstackgerritJames E. Blair proposed zuul/zuul master: Gerrit: add polling support for refs  https://review.opendev.org/70613822:29
*** yolanda has joined #openstack-infra22:35
openstackgerritDonny Davis proposed openstack/project-config master: Temporarly disable Fort Nebula general purpose nodes  https://review.opendev.org/70614222:41
donnydOh wow - I did such a bad job spelling temporarily22:41
donnydwhere is the shaky head emoticon22:41
openstackgerritJames E. Blair proposed zuul/zuul master: Extract allow/disallow filter into util function  https://review.opendev.org/70614422:44
*** rh-jelabarre has quit IRC22:46
*** tkajinam has joined #openstack-infra22:48
openstackgerritMerged zuul/zuul-jobs master: Add CentOS8 High Availability repository  https://review.opendev.org/70597322:50
*** vdrok has joined #openstack-infra23:00
*** TheJulia has joined #openstack-infra23:00
*** lathiat has joined #openstack-infra23:00
*** masayukig has joined #openstack-infra23:00
*** jamespage has joined #openstack-infra23:01
*** johnsom has joined #openstack-infra23:02
*** zxiiro has joined #openstack-infra23:02
*** jamesmcarthur has joined #openstack-infra23:03
*** jamesmcarthur has quit IRC23:06
donnydianw: FN needs to have some major refactoring done, so I am pretty sure this will be the first of a couple of changes necessary to support said changes23:12
ianwdonnyd: ok, np23:12
*** rlandy is now known as rlandy|bbl23:25
*** sean-k-mooney has quit IRC23:28
*** sean-k-mooney has joined #openstack-infra23:29
*** ociuhandu has joined #openstack-infra23:30
*** ociuhandu has quit IRC23:35
*** dciabrin has quit IRC23:36
*** dSrinivas has quit IRC23:43
*** hwoarang has quit IRC23:51
*** hwoarang has joined #openstack-infra23:53

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