Thursday, 2019-10-24

*** jamesmcarthur has joined #openstack-infra00:00
clarkbhrm there is a refelect in that so ya maybe its not type checked enough at build and its looking it up and failing00:00
openstackgerritIan Wienand proposed opendev/system-config master: Add mirror01.mtl01.opendev.org  https://review.opendev.org/69077100:00
mordredit's a bug00:02
mordredif you're doing this as an unlogged in user00:02
mordredthe bug was fixed in other places00:02
ianwmordred: tfr -- mind checking in on the prior one too https://review.opendev.org/#/c/673724/1 ... really does the same thing, but makes the opensafs client  package install explicitly first00:05
ianws/first/second00:05
*** apetrich has quit IRC00:06
*** dciabrin_ has quit IRC00:06
*** armax has quit IRC00:09
*** rlandy has quit IRC00:10
*** rfolco|rover has joined #openstack-infra00:10
*** gyee has quit IRC00:11
mordredclarkb: https://github.com/go-gitea/gitea/pull/865300:12
mordredclarkb: I'm thinking maybe let's revert ...00:12
mordredOR - we could change the playbook to try the "check for root user" check to send the root user auth ... the code path for 404 would still work right, and if the root user exists, we should be able to authenticate as it00:13
*** otsukahy has joined #openstack-infra00:14
mordredof course - if we send auth and the user doesn't exist - we're likely to get a different code than 40400:14
*** jamesmcarthur has quit IRC00:16
mordrednope. that just results in ctx.User being nil00:16
*** dciabrin has joined #openstack-infra00:16
*** jamesmcarthur has joined #openstack-infra00:17
*** yamamoto has quit IRC00:19
*** otsukahy has quit IRC00:19
*** prometheanfire has quit IRC00:20
clarkbmeaning it sendsback a 404?00:20
*** prometheanfire has joined #openstack-infra00:20
clarkbI'm happy with a revert if that us the easiest thibg00:20
mordredI think a revert is easiest for now00:20
mordredand we can try again when that pr lands00:20
mordredif you send a user:pass as a non-existent user, you still get the 50000:21
*** markvoelker has joined #openstack-infra00:21
mordredassuming I curl'd right: curl --basic -u mordred:foo -k -H https://localhost:3000/api/v1/users/root00:21
clarkbgotit00:21
mordred(ignore the -H)00:22
*** jamesmcarthur_ has joined #openstack-infra00:26
openstackgerritMonty Taylor proposed opendev/system-config master: Revert "Update gitea to 1.9.4"  https://review.opendev.org/69077200:26
mordredclarkb, corvus, ianw: ^^00:26
*** jamesmcarthur has quit IRC00:26
mordredincidentally, our testing didn't catch this because our testing doesn't run the role twice - so we dont run through the "root user exists" codepath00:26
*** jamesmcarthur_ has quit IRC00:29
*** jamesmcarthur has joined #openstack-infra00:30
*** jamesmcarthur has quit IRC00:30
*** jamesmcarthur has joined #openstack-infra00:32
*** zhurong has quit IRC00:36
openstackgerritMerged opendev/zone-opendev.org master: Add mirror01.mtl01.inap.opendev.org  https://review.opendev.org/69076700:40
*** jamesmcarthur has quit IRC00:40
*** jamesmcarthur has joined #openstack-infra00:40
*** otsukahy has joined #openstack-infra00:52
*** otsukahy has quit IRC00:57
*** weifan has joined #openstack-infra01:00
*** mriedem_afk is now known as mriedem01:02
*** mriedem has quit IRC01:02
openstackgerritMerged opendev/system-config master: openafs: avoid pulling in client package before kernel modules  https://review.opendev.org/67372401:03
openstackgerritMerged opendev/system-config master: base-server: disable install of suggests and recommends packages  https://review.opendev.org/67373901:03
*** weifan has quit IRC01:14
*** xek_ has quit IRC01:21
openstackgerritJulia Kreger proposed openstack/diskimage-builder master: Enable IPv6 DHCPv6  https://review.opendev.org/69077701:23
*** markvoelker has quit IRC01:28
*** derekh has quit IRC01:51
*** derekh has joined #openstack-infra01:51
*** spsurya has joined #openstack-infra01:59
*** markvoelker has joined #openstack-infra02:00
*** anteaya has quit IRC02:04
*** markvoelker has quit IRC02:05
*** ricolin has joined #openstack-infra02:12
*** tinwood has quit IRC02:18
*** tinwood has joined #openstack-infra02:20
openstackgerritMerged openstack/project-config master: Update ceph grafana dashboard for nova master branch  https://review.opendev.org/69046602:24
*** jamesmcarthur has quit IRC02:25
*** jamesmcarthur has joined #openstack-infra02:30
openstackgerritMerged opendev/system-config master: Add mirror01.mtl01.opendev.org  https://review.opendev.org/69077102:41
*** xinranwang has joined #openstack-infra03:04
*** jamesmcarthur has quit IRC03:10
*** jamesmcarthur has joined #openstack-infra03:11
*** ykarel has joined #openstack-infra03:11
*** jamesmcarthur has quit IRC03:15
*** diablo_rojo has quit IRC03:18
openstackgerritIan Wienand proposed openstack/project-config master: Enable inap-mtl01  https://review.opendev.org/69071403:28
openstackgerritIan Wienand proposed openstack/project-config master: Use opendev.org mirror for mtl01.inap region  https://review.opendev.org/69078803:28
*** psachin has joined #openstack-infra03:39
ianwclarkb: ^ ready to go i'd say03:39
*** jamesmcarthur has joined #openstack-infra03:42
*** derekh has quit IRC03:42
*** derekh has joined #openstack-infra03:43
clarkb+2'd but I cant watch it asit is nearbedtime03:48
*** ykarel has quit IRC03:54
ianwhaha at this time i expected it to be a "for tomorrow" message anyway :)04:00
*** lathiat has joined #openstack-infra04:01
*** yamamoto has joined #openstack-infra04:05
adriantAJaeger: hey, I've just run into a weird issue between busybox/GNU find because of some old bash you added to Aodh tests in this commit: https://github.com/openstack/aodh/commit/faada69444988c66a09ade812e10fa2ab5a0adae04:09
*** yamamoto has quit IRC04:10
adriantAJaeger: is it safe to just assume whatever this was solving isn't an issue anymore and just remove the line from the tox.ini file: https://github.com/openstack/aodh/blob/master/tox.ini#L38 ?04:10
*** ykarel has joined #openstack-infra04:15
*** jerryz has quit IRC04:20
*** jamesmcarthur has quit IRC04:27
*** jamesmcarthur has joined #openstack-infra04:31
*** udesale has joined #openstack-infra04:38
*** jamesmcarthur has quit IRC04:39
openstackgerritIan Wienand proposed opendev/system-config master: [wip] test arm64 mirror  https://review.opendev.org/69079804:44
*** kjackal has joined #openstack-infra04:48
*** kjackal has quit IRC04:49
*** yamamoto has joined #openstack-infra04:50
*** jamesmcarthur has joined #openstack-infra04:52
*** tkajinam_ has joined #openstack-infra05:03
*** efried has quit IRC05:04
*** tkajinam has quit IRC05:05
openstackgerritIan Wienand proposed opendev/system-config master: [wip] test arm64 mirror  https://review.opendev.org/69079805:10
*** efried has joined #openstack-infra05:11
*** slaweq has joined #openstack-infra05:13
*** jamesmcarthur has quit IRC05:13
*** surpatil has joined #openstack-infra05:18
*** lmiccini has joined #openstack-infra05:18
*** jtomasek has joined #openstack-infra05:18
*** jamesmcarthur has joined #openstack-infra05:20
*** slaweq has quit IRC05:29
*** jamesmcarthur has quit IRC05:30
openstackgerritMerged openstack/project-config master: Remove fedora-28 nodes  https://review.opendev.org/69043205:31
lennybfrickler: Is there any good instructions how to create new/custom cirros image for openstack?05:45
*** tkajinam__ has joined #openstack-infra05:52
*** tkajinam_ has quit IRC05:54
AJaegeradriant: yes, remove it. We test that now in infra directly05:57
*** psachin has quit IRC05:58
*** dpawlik has joined #openstack-infra05:59
*** udesale has quit IRC06:01
*** markvoelker has joined #openstack-infra06:02
*** psachin has joined #openstack-infra06:02
*** dpawlik has quit IRC06:03
*** markvoelker has quit IRC06:06
*** udesale has joined #openstack-infra06:10
openstackgerritMerged openstack/project-config master: Use opendev.org mirror for mtl01.inap region  https://review.opendev.org/69078806:13
mordredclarkb, corvus: the gitea patch has been merged and lunny suggested backporting it to 1.9 and 1.10 - so hopefully it'll be in a release soon. maybe if it lands in release/v1.9 we can build an image from that branch until there's either a 1.9 point release with it, or a 1.10 release with it06:15
*** dpawlik has joined #openstack-infra06:15
*** ykarel is now known as ykarel|afk06:15
*** dpawlik has quit IRC06:20
*** dpawlik has joined #openstack-infra06:23
*** slaweq has joined #openstack-infra06:26
*** surpatil has quit IRC06:30
*** iurygregory has joined #openstack-infra06:34
*** tkajinam__ is now known as tkajinam06:34
*** pkopec has joined #openstack-infra06:35
*** zzehring has quit IRC06:40
*** pcaruana has joined #openstack-infra06:41
*** aedc has quit IRC06:54
*** jaicaa has quit IRC07:02
*** jaicaa has joined #openstack-infra07:05
*** tosky has joined #openstack-infra07:12
*** surpatil has joined #openstack-infra07:12
*** kopecmartin|off is now known as kopecmartin07:13
*** tesseract has joined #openstack-infra07:18
*** ccamacho has joined #openstack-infra07:29
*** ykarel|afk is now known as ykarel07:31
openstackgerritMerged opendev/system-config master: Revert "Update gitea to 1.9.4"  https://review.opendev.org/69077207:32
*** xinranwang has quit IRC07:33
*** gfidente has joined #openstack-infra07:34
openstackgerritMonty Taylor proposed opendev/system-config master: Revert "Revert "Update gitea to 1.9.4""  https://review.opendev.org/69087207:34
openstackgerritMonty Taylor proposed opendev/system-config master: Update gitea to v1.10  https://review.opendev.org/69087307:34
*** surpatil is now known as surpatil|lunch07:36
*** trident has quit IRC07:46
*** trident has joined #openstack-infra07:51
*** rpittau|afk is now known as rpittau07:52
*** apetrich has joined #openstack-infra07:55
*** apetrich has quit IRC07:57
*** apetrich has joined #openstack-infra08:00
*** kjackal has joined #openstack-infra08:02
*** amoralej|off is now known as amoralej08:04
*** lucasagomes has joined #openstack-infra08:05
*** ralonsoh has joined #openstack-infra08:07
*** dchen has quit IRC08:11
*** aedc has joined #openstack-infra08:12
*** kaisers has quit IRC08:13
openstackgerritMonty Taylor proposed zuul/nodepool master: Handle newer nova microversions  https://review.opendev.org/69087608:19
*** tkajinam has quit IRC08:19
*** xek_ has joined #openstack-infra08:29
*** ykarel is now known as ykarel|lunch08:31
*** hwoarang has quit IRC08:35
openstackgerritMonty Taylor proposed opendev/system-config master: Revert "Revert "Update gitea to 1.9.4""  https://review.opendev.org/69087208:36
*** hwoarang has joined #openstack-infra08:36
*** hwoarang has quit IRC08:40
*** hwoarang has joined #openstack-infra08:48
*** xek_ has quit IRC09:02
*** surpatil|lunch is now known as surpatil09:11
*** jaosorior has joined #openstack-infra09:15
*** yamamoto has quit IRC09:16
*** yamamoto has joined #openstack-infra09:22
*** yamamoto has quit IRC09:22
*** yamamoto has joined #openstack-infra09:22
*** ykarel|lunch is now known as ykarel09:25
*** yamamoto has quit IRC09:26
*** e0ne has joined #openstack-infra09:33
openstackgerritIvan Kolodyazhny proposed openstack/openstack-zuul-jobs master: Add openstack-tox-pep8 to openstack-python3-ussuri-jobs template.  https://review.opendev.org/68899709:54
*** markvoelker has joined #openstack-infra10:04
*** aedc has quit IRC10:06
*** markvoelker has quit IRC10:09
*** udesale has quit IRC10:19
*** dpawlik has quit IRC10:19
*** yamamoto has joined #openstack-infra10:26
*** yamamoto has quit IRC10:30
*** otsukahy has joined #openstack-infra10:30
*** ykarel_ has joined #openstack-infra10:40
openstackgerritWitold Bedyk proposed openstack/project-config master: Grant permissions for Monasca UI to horizon-core  https://review.opendev.org/69091810:42
*** yamamoto has joined #openstack-infra10:42
*** ykarel has quit IRC10:43
*** udesale has joined #openstack-infra10:44
*** dpawlik has joined #openstack-infra10:44
*** ykarel_ is now known as ykarel10:45
*** yamamoto has quit IRC10:54
*** yamamoto has joined #openstack-infra10:57
*** amoralej is now known as amoralej|lunch11:00
*** roman_g has joined #openstack-infra11:06
*** prometheanfire has quit IRC11:09
noonedeadpunkhi:) I have kinda question regarding new upper-constraints url. the thing is, that we're using releases SHA instead of branch name to be sure about requirements used with one particular osa release.11:09
noonedeadpunkbut, new url https://releases.openstack.org/constraints/upper/ don't understand providing it SHA11:10
*** prometheanfire has joined #openstack-infra11:11
openstackgerritWitold Bedyk proposed openstack/project-config master: Grant permissions for Monasca UI to horizon-core  https://review.opendev.org/69091811:11
noonedeadpunkofc we can use https://opendev.org/openstack/requirements/raw/5d2dc701ab2fe7e87d78e4d96eb444766eef6725/upper-constraints.txt but to be consistent with everyone, maybe it's possible to add redirect?11:11
*** yamamoto has quit IRC11:12
noonedeadpunkI just didn't found anything relevant in https://opendev.org/opendev/system-config so maybe you can point me to where these redirects are?11:12
noonedeadpunkoh, ok found it:)11:13
*** otsukahy has quit IRC11:22
*** aedc has joined #openstack-infra11:27
AJaegernoonedeadpunk: the upper-constrains URL comes from release team, best to talk with them directly11:28
openstackgerritIvan Kolodyazhny proposed openstack/openstack-zuul-jobs master: Add openstack-tox-pep8 to openstack-python3-ussuri-jobs template.  https://review.opendev.org/68899711:28
*** roman_g has quit IRC11:35
openstackgerritIvan Kolodyazhny proposed openstack/openstack-zuul-jobs master: Add openstack-tox-pep8 to openstack-python3-ussuri-jobs template.  https://review.opendev.org/68899711:36
*** derekh has quit IRC11:37
noonedeadpunkAJaeger: oh, I see, thanks11:42
*** janki has joined #openstack-infra11:44
openstackgerritWitold Bedyk proposed openstack/project-config master: Grant permissions for Monasca UI to horizon-core  https://review.opendev.org/69091811:46
*** yamamoto has joined #openstack-infra11:46
*** larainema has joined #openstack-infra11:48
*** yamamoto has quit IRC11:54
mtreinishrm_work: I do not have access, I was given write access to the github repo a few months ago but haven't been added on pypi for anything yet11:59
*** amoralej|lunch is now known as amoralej12:01
*** jamesmcarthur has joined #openstack-infra12:11
noonedeadpunkcores, can you kindly review our initial os_murano stuff? https://review.opendev.org/#/q/topic:os_murano+status:open12:12
noonedeadpunkWe've announced it for train, and I guess we need to backport this when it'll be merged12:12
*** roman_g has joined #openstack-infra12:13
*** ykarel is now known as ykarel|afk12:15
*** xek_ has joined #openstack-infra12:16
*** psachin has quit IRC12:18
*** rlandy has joined #openstack-infra12:18
*** aedc has quit IRC12:21
*** jamesmcarthur has quit IRC12:28
*** yamamoto has joined #openstack-infra12:28
noonedeadpunkAnd we also need to unfreeze https://review.opendev.org/#/c/686574/12:30
*** markvoelker has joined #openstack-infra12:31
*** psachin has joined #openstack-infra12:32
*** yamamoto has quit IRC12:35
*** kjackal has quit IRC12:39
*** jaosorior has quit IRC12:42
*** kjackal has joined #openstack-infra12:43
AJaegernoonedeadpunk: are you sure that you're in the right channel?12:44
noonedeadpunkwow, no, I'm not :(12:44
noonedeadpunksorry:(12:44
roman_gHello team. I was checking documentation here https://docs.openstack.org/infra/manual/creators.html#add-gerrit-permissions , and couldn't find a way to add rules.pl file to the project's refs/meta/config branch. Need to set up some additional verification for voting. Also couldn't find any usage of rules.pl file in OpenStack repositories (may be my search skills are not so good). Could you guide12:45
roman_gme where to look for to get rules.pl file added to refs/meta/config branch and merged?12:45
AJaegerroman_g: rules.pl? What is that? I doubt we use that at all...12:46
roman_gTo be more specific, need to enforce 2x Code-Review +2 from non-authors to be able to merge change.12:47
AJaegerroman_g: all config is in project-config/gerrit/acls/ - and that's what we support12:47
roman_gThis one: https://gerrit-review.googlesource.com/Documentation/config-project-config.html#file-rules_pl12:47
AJaegerroman_g: we handle that as social contract, not by tool support currently.12:47
roman_gAJaeger: I know.12:47
roman_gFile needs to be added here: https://review.opendev.org/gitweb?p=airship/docs.git;a=tree;h=refs/meta/config;hb=refs/meta/config12:48
AJaegerroman_g: so, I guess it needs extensive rework - including work in jeepyb.12:48
AJaegerroman_g: best discuss with rest of infra team later (once US west coast is awake)12:48
*** jamesmcarthur has joined #openstack-infra12:48
roman_gOK. Thank you, AJaeger.12:48
roman_gI hope it would just need some minor 'OpenDev Project Creator' jobs modification.12:49
*** iokiwi has quit IRC12:51
*** adriant has quit IRC12:51
*** adriant has joined #openstack-infra12:52
openstackgerritIvan Kolodyazhny proposed openstack/openstack-zuul-jobs master: Add openstack-tox-pep8 to openstack-python3-ussuri-jobs template.  https://review.opendev.org/68899712:53
fungiroman_g: we haven't supported prolog scripting for project labels up to now, and no the project creation isn't done via any ci job, it's done with custom python utilities in a the opendev/jeepyb project, triggered by configuration management12:53
fungijust to be clear, you're saying you're entrusting approval rights to people who can't be trusted to not approve changes nobody else has reviewed?12:54
fungiand also you're sure you'll never need to approve an emergency fix when there's only one core reviewer around?12:54
fungian alternative (which we've used in a variety of governance-related projects for example) might be to restrict the workflow +1 vote to a different group than the code-review +2 vote12:57
*** surpatil has quit IRC12:57
fungithe script which updates refs/meta/config in gerrit is this: https://opendev.org/opendev/jeepyb/src/branch/master/jeepyb/cmd/manage_projects.py13:00
*** goldyfruit has joined #openstack-infra13:00
roman_gfungi: we have unexperienced developers and need to ensure they don't rush to merge everything they have just wrote. And we want Gerrit to enforce rule (we have seen breaches).13:01
*** ykarel|afk is now known as ykarel13:02
fungii see. another alternative is to not grant workflow +1 (or code-review +2) access for inexperienced developers13:02
*** eharney has joined #openstack-infra13:05
*** yamamoto has joined #openstack-infra13:10
*** jamesmcarthur has quit IRC13:13
*** janki has quit IRC13:13
*** otsukahy has joined #openstack-infra13:19
*** jamesmcarthur has joined #openstack-infra13:20
*** mriedem has joined #openstack-infra13:23
*** onovy has quit IRC13:25
*** roman_g has quit IRC13:26
*** psachin has quit IRC13:28
*** ykarel is now known as ykarel|afk13:29
*** spsurya has quit IRC13:29
*** rh-jelabarre has joined #openstack-infra13:30
*** yamamoto has quit IRC13:30
*** zzehring has joined #openstack-infra13:30
*** yamamoto has joined #openstack-infra13:31
*** yamamoto has quit IRC13:31
*** slaweq has quit IRC13:32
*** otsukahy has quit IRC13:34
*** e0ne has quit IRC13:35
*** e0ne has joined #openstack-infra13:36
*** onovy has joined #openstack-infra13:37
*** sreejithp has joined #openstack-infra13:39
*** sgw has joined #openstack-infra13:43
*** ykarel|afk is now known as ykarel13:48
*** amoralej is now known as amoralej|mtg14:00
openstackgerritMatt Riedemann proposed openstack/project-config master: Update ceph grafana dashboard for cinder master branch  https://review.opendev.org/69098014:01
*** yamamoto has joined #openstack-infra14:03
*** jpena|off is now known as jpena14:05
*** jamesmcarthur has quit IRC14:06
*** jpena is now known as jpena|off14:07
*** dpawlik has quit IRC14:11
openstackgerritIvan Kolodyazhny proposed openstack/openstack-zuul-jobs master: Update openstack-python3-ussuri-jobs-* templates to test with Python3   https://review.opendev.org/68899714:12
*** yamamoto has quit IRC14:14
*** jamesmcarthur has joined #openstack-infra14:17
*** ykarel has quit IRC14:18
*** pcaruana has quit IRC14:19
*** ykarel has joined #openstack-infra14:20
mriedemclarkb: i've been seeing http://status.openstack.org/elastic-recheck/#1808010 a lot this week. thinking out loud, https://review.opendev.org/#/c/619319/ might have introduced that when we started setting the non-0 disk size on the tempest flavors in devstack,14:21
mriedemsince we're using tiny cirros images, the root disk on the flavor is just 1GiB,14:21
mriedembefore that it was disk=0 so the root disk was whatever was needed for the size of the image at spawn,14:21
mriedemi'm wondering if we should bump those tempest flavors up 1GiB to account for extra space needed, like the config drive14:22
mriedemkeep seeing that growroot failure14:22
*** larainema has quit IRC14:25
*** pkopec has quit IRC14:28
clarkbmriedem: config drive is a separate device right? the flavor probably isnt impacted by that. However adding another half a gig probably wont hurt14:29
*** xek_ has quit IRC14:30
*** xek_ has joined #openstack-infra14:30
mriedemyeah that's true14:31
openstackgerritIvan Kolodyazhny proposed openstack/openstack-zuul-jobs master: Update openstack-python3-ussuri-jobs-* templates to test without Python3  https://review.opendev.org/68899714:32
ttxAJaeger: you were looking into retiring old, now-unofficial repositories still present in the opensatck/ namespace (think Fuel). I think we can proceed, now that Train is behind us... Let me know if I can be of any help there14:35
*** bnemec has quit IRC14:35
*** pkopec has joined #openstack-infra14:35
*** bnemec has joined #openstack-infra14:37
*** pkopec has quit IRC14:40
fungiyeah, it mainly needs to be brought up with the tc to make sure it's something they want to do at this point, but their self-imposed window to opt out of tc jurisdiction over deliverables remaining in the openstack namespace since the opendev shuffle has now expired with the train release behind them14:43
*** kopecmartin is now known as kopecmartin|off14:46
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Test buildset registry with k8s and docker  https://review.opendev.org/68928014:47
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Adjust buildset registry container name  https://review.opendev.org/69099214:47
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Don't sudo when saving container logs in registry test  https://review.opendev.org/69099314:47
*** evrardjp_ is now known as evrardjp14:48
*** amoralej|mtg is now known as amoralej14:49
AJaegerttx, I only mentioned fuel and I reached out to SergeyLukjanov and there are some internal uses of the repos but those should "soon" move away. Slow discussion going on there... Let's wait a bit14:49
fungithe tc could choose to retire them now, but could also choose to give them a little more time to stop depending on those copies of their repositories14:51
*** mattw4 has joined #openstack-infra14:52
fungibut also, fuel is just one example (though perhaps the largest example) of former openstack deliverables which did not get their repos retired14:52
*** armax has joined #openstack-infra14:52
ttxAJaeger: I ran an analysis recently and here was the gap between official and openstack/ : https://etherpad.openstack.org/p/MGNTt7YkZ514:52
*** xek_ has quit IRC14:53
*** xek_ has joined #openstack-infra14:53
*** roman_g has joined #openstack-infra14:55
*** jcoufal has joined #openstack-infra14:58
*** ccamacho has quit IRC14:59
*** Goneri has joined #openstack-infra15:01
*** gouthamr_ is now known as gouthamr15:01
ttxlooks like there are no surprises there15:02
fungiit's a lot less to clean up than i would have expected, really15:03
*** yolanda has quit IRC15:03
*** mattw4 has quit IRC15:07
*** priteau has joined #openstack-infra15:09
*** slaweq has joined #openstack-infra15:10
*** yolanda has joined #openstack-infra15:11
*** jaosorior has joined #openstack-infra15:13
*** jaosorior has quit IRC15:16
clarkbroman_g: fungi ya the current setup is intended to be rigid enough that we have some checks in place while still being flexible enough to give people what they need to address emergencies without involving gerrit admins15:18
rosmaitafungi: who should i talk to about launchpad administration? i'm looking to get the Maintainer for os-brick changed from "OpenStack Administrators" to "Cinder Drivers"15:18
fungirosmaita: i think as long as cinder drivers is owned by openstack administrators then that's fine (we have that situation on some other projects too)15:20
fungithat at least allows someone in openstack administrators to add themselves temporarily to cinder drivers to check/fix configuration problems with the lp project15:20
rosmaitacinder drivers already maintains cinder & python-cinderclient, so i imagine that relationship is set up correctly15:21
roman_gclarkb: I see. Thank you.15:21
clarkbfungi: roman_g also I think if we wanted to add prolog rules we should add them globally then projects can opt into those functions in their acls as per usual15:21
clarkbif we do it globally we may not need to hvae jeepyb manage it15:22
rosmaitafungi: i just checked, cinder drivers is owned by OS Administrators15:22
roman_gfungi: clarkb would a patch to https://opendev.org/opendev/jeepyb/src/branch/master/jeepyb/cmd/manage_projects.py to support rules.pl be a good idea?15:22
clarkb(I think you can drop prolog files into review_site somewhere for them to be picked up globally)15:22
fungirosmaita: i can fix os-brick in a few minutes then15:23
fungithanks for checking!15:23
rosmaitafungi: no rush, thanks in advance15:23
clarkbroman_g: as I just mentioned I think my preference would be for them to be added as global rules that projects can opt into using via acl config rather than project specific rules. I believe that if we do it that way we can simply drop the prolog files into the review_site config and don't need to update tools15:23
fungiclarkb: ahh, so they don't have to be pushed directly into refs/meta/config? neat15:23
clarkbfungi: from memory no15:23
fungivery cool. so it just comes down to whether that's a feature the opendev sysadmins feel comfortable supporting15:24
roman_gclarkb: if we create global rules, then there is no way to enable/disable them per repo. Rules are enforced from top.15:25
*** slaweq has quit IRC15:25
clarkbroman_g: you add them as functions in your acl config15:25
clarkbthe currnet function we use is a built in one which requires highest vote value at least once and no lowest vote values for submit15:26
clarkbyou would simply say "use this other function from the prolog that is loaded" I believe that bit works the same if the prolog is project specific or global15:26
* clarkb is trying to find docs on it though15:26
clarkbaha the "global" config goes in All-Projects refs/meta/config15:29
roman_ghttps://gerrit-review.googlesource.com/Documentation/config-labels.html#label_function - this one?15:30
clarkbso the mechanism is the same either way, more a matter of scoping via destination15:30
clarkbroman_g: ya though that says the name is ignored so maybe I've misunderstood how you apply prolog to projects15:31
clarkbthat is a bit disappointing that it is implicit and the explicit config is ignored15:32
clarkbwe could make global config but then it would apply globally I guess and we can't name custom filters?15:34
roman_gthat's what I thought15:35
roman_gI would just add directory rules/ here: https://opendev.org/openstack/project-config/src/branch/master/gerrit15:36
roman_gwith per-project subdirs15:36
roman_glike there are for acl's15:36
roman_gand just keep rules.pl there15:37
*** otsukahy has joined #openstack-infra15:38
*** gyee has joined #openstack-infra15:38
clarkbthat is the easy part :) the files can go just about anywhere. We'd need to map them into gerrit (existing tool that does this is manage-projects for acls)15:39
melwittanybody know why there's no data in https://graphite.opendev.org for example stats_counts.zuul.tenant.openstack.pipeline.check.project.opendev_org.openstack_nova.stable_train.job there's no tempest-integrated-compute job there but it is running in real life, example: https://review.opendev.org/69071115:39
clarkbwe'd also need to ensure our config allows it and that the limits are not too large to cause problems, but big enough that the rules work15:39
melwittI've been using graphite to know how to add jobs to a grafana dashboard for ceph and have noticed data seems missing for some things http://grafana.openstack.org/d/-iKINcImz/ceph-failure-rate?orgId=115:40
roman_gclarkb: exactly, I wanted simply to modify manage_projects.pl so that it copies rules.pl to the right place if it is present15:40
roman_gclarkb: default config allows rules15:40
clarkbroman_g: I get that, I'm pointing out it not "simple"15:41
clarkbroman_g: we also need to update our gerrit config and ensure the limits imposed by the config are correct for our setup15:41
clarkbmelwitt: as a sanity check http://zuul.openstack.org/builds?job_name=tempest-integrated-compute&pipeline=check&project=openstack%2Fnova shows results (so one would expect graphite to have data too)15:42
clarkbroman_g: I think we should also consider whether or not this is the best solution to the problem you are trying to address15:42
clarkbroman_g: this was fungi's point earlier15:42
clarkbbasically if people shouldn't be approving changes we can remove their approval rights instead15:42
melwittclarkb: yeah, I don't know what's going on with graphite. it's missing jobs for train. and I don't know you to link you to a path there15:43
clarkbwhile still having the flexiblity for those that can approv echanges to do so in an emergency when others are traveling or on vacation or otherwise unable to help15:43
clarkbinfra-root I am going to approve https://review.opendev.org/#/c/690714/2 to enable inap15:44
corvusclarkb, fungi: fyi, new work to replace prolog: https://docs.google.com/presentation/d/1fUjnmngWLFaexZKlN5XmL9EmOX8Nh_P0yJcThnlJ3Lk/15:44
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Fixes generation of documentation for jobvars  https://review.opendev.org/69100815:45
*** iurygregory has quit IRC15:46
clarkbcorvus: looks like those would be written as java and added to gerrit as plugins. Fwiw the consumption of those prolog less rules seems to reflect how I thought prolog rules were implemented. Create a named entity then in your acl file apply that named entity as the submit function15:49
clarkb(for that reason I'm in favor of this change)15:49
*** tesseract has quit IRC15:49
clarkbthey list some other very good reasons for the change in the deck too15:50
corvusclarkb: yeah, i think the new model is good.  the simple-submit-rules plugin is supposed to be a collection of simple things (so could probably add most things we'd want to that).  it only has one rule now.  but also, the lua scripting rules plugin is interesting too.15:50
corvusi think we get that with 2.1615:50
corvusclarkb: i'm going to wait for 690714 to land, then restart zuul15:51
clarkbthat said re tests for prolog, prolog is such an easy language to have tests for. I wonder if that is simply a lack of examples15:51
fungialso the switch from prolog to lua will be nice. at least the latter is a little more readable15:51
corvusyeah, and maybe hard to integrate with the java testing framework15:51
clarkbyou basically define a set of facts then run your predicates and assert yes or no15:52
clarkbcorvus: ok15:52
clarkbfungi: also you don't have to melt your brain to understand the "flow"15:52
*** ykarel is now known as ykarel|away15:53
clarkbroman_g: given ^ and the current work to upgrade gerrit I think a better path forward may be to rely on prolog less rules, then we can do what I described earlier. Have a set of global rules that projects opt into using15:53
clarkbroman_g: then we avoid adding tech debt we won't want in the near future15:53
*** goldyfruit has quit IRC15:54
openstackgerritMerged zuul/nodepool master: Handle newer nova microversions  https://review.opendev.org/69087615:54
*** goldyfruit has joined #openstack-infra15:56
*** diablo_rojo has joined #openstack-infra15:56
clarkbmelwitt: if you use the graphite.opendev.org browser thing on the right that job doesn't show up at all under that path15:58
clarkband yet it is there at http://zuul.openstack.org/builds?job_name=tempest-integrated-compute&pipeline=check&project=openstack%2Fnova&branch=stable%2Ftrain15:59
clarkb/dev/mapper/main-graphite 1008G 1008G     0 100% /var/lib/graphite/storage15:59
clarkbI think ^ is the problem15:59
*** diablo_rojo has quit IRC15:59
openstackgerritMerged openstack/project-config master: Enable inap-mtl01  https://review.opendev.org/69071416:00
sean-k-mooneyfungi: just saw the reference to https://launchpad.net/bugs/1734320  on openstack meeting by the way.16:00
openstackLaunchpad bug 1734320 in OpenStack Compute (nova) "Eavesdropping private traffic" [Undecided,In progress] - Assigned to sean mooney (sean-k-mooney)16:00
*** diablo_rojo has joined #openstack-infra16:00
melwittclarkb: whoa yeah16:00
sean-k-mooneyfungi: the status of that has not changed16:00
clarkbcorvus: with the switcharoo on garphite "paths" post zuulv3 are there old paths we can delete files for on the grpahite server?16:00
clarkbcorvus: I wonder if we're doubling up on storage needs because of the old paths and the new paths coexisting16:00
corvusclarkb: almost certainly; and it's probably been long enough we aren't interested in the old data16:01
clarkbcorvus: ++16:01
corvusprobably a find for timestamps is the best approach?16:01
corvusi'm going to restart zuul now16:01
clarkbcorvus: that make ssense. Let me see what a find that looks for anything older than a year finds16:01
openstackgerritMerged zuul/zuul-jobs master: Fixes generation of documentation for jobvars  https://review.opendev.org/69100816:01
sean-k-mooneyfungi: at least not since febuary. the nova and neutron patch have been avaliabel and  pauses more or less since then. feel free to ping me later if you want more details16:02
*** auristor has quit IRC16:04
*** rpittau is now known as rpittau|afk16:05
fungisean-k-mooney: no problem, i was mostly pointing out that there's been renewed discussion on that bug report16:08
clarkbthis is not a very fast find16:08
* clarkb waits patiently16:08
*** mattw4 has joined #openstack-infra16:09
mwhahahais zuul.o.o 503ing for anyone else or is it just me16:09
openstackgerritIvan Kolodyazhny proposed openstack/openstack-zuul-jobs master: Update openstack-python3-ussuri-jobs-* templates to test without Python2  https://review.opendev.org/68899716:09
corvusmwhahaha: it's restarting16:09
mwhahahaah16:09
sean-k-mooneyfungi: ah ok. i would like to get the final to patches merged eventullay but m3-release crazyness was a thing so i had to step back form it again.16:11
sean-k-mooneyclarkb: in case you did not see my reply thanks for the tip on glean and root login16:12
clarkbsean-k-mooney: I did, glad you got it working. Re ipv6 it should work if your cloud does stateless config16:13
fungisean-k-mooney: as for ipv6 on centos, yeah that's been a struggle, mostly to do with races between kernel handling and network-manager16:13
clarkbsean-k-mooney: glean doesn't know how to configure static addresses for ipv6 on centos16:13
*** dkehn_ has joined #openstack-infra16:13
clarkbfungi: that is dynamic stateless config specific and we have it working now16:13
fungiyep16:14
sean-k-mooneywell my issue seams to be even running dhclient -6 does not work16:14
*** e0ne has quit IRC16:14
fungisean-k-mooney: is that environment doing stateful or stateless dhcp6?16:14
sean-k-mooneybut for my ubuntu 18.04 vms it just works16:14
clarkbsean-k-mooney: hte interface should be network manager managed (because thats the only option on rhel/centos 8 now I guess)16:14
sean-k-mooneystateful dhcp616:14
clarkbsean-k-mooney: there is a good chance that you have to do stateful dhcpv6 through NM16:14
sean-k-mooneyya maybe16:15
sean-k-mooneyi mean i can alwasys chagne it its just my dev cloud at home16:15
sean-k-mooneybut ill see if i can figure it out at the weekend16:15
*** mriedem is now known as mriedem_away16:16
sean-k-mooneyhave ye had any better look with statels rdavd16:16
sean-k-mooney*stateless16:16
*** lmiccini has quit IRC16:16
clarkbI don't know what NM uses under the covers16:16
clarkbit should be working with whatever NM uses to receive and configure RAs16:16
sean-k-mooneyright but when using neutron stateflu dhcp6 i guess RA adversisme of ips is disable16:17
*** lucasagomes has quit IRC16:17
corvus#status log restarted all of zuul on commit 360f1e9fd32e7aa5eec7135838cb5e14f0cac6ae16:17
openstackstatuscorvus: finished logging16:17
sean-k-mooneyanyway now that i can log in i can play with it16:17
*** ianychoi has joined #openstack-infra16:21
*** auristor has joined #openstack-infra16:21
*** jcoufal has quit IRC16:25
fungisean-k-mooney: i wonder, with stateful, what would be involved in getting that added to nova's network metadata (so it's available in the config drive)? might be easy to add support for that directly to glean and then not need dhclient to get involved at all16:28
clarkbinfra-root can we get reviews on https://review.opendev.org/#/c/689248/ to clarify the state of third party ci docs16:29
clarkbfungi: I believe the info is already in the config drive16:29
corvusclarkb: done16:29
clarkbtyty16:30
fungiclarkb: oh, neat. couldn't glean just treat it the same way it does v4 in that case?16:30
sean-k-mooneyfungi: oh so like have glean staticaly configure the static adress ya that could work16:31
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Adjust buildset registry container name  https://review.opendev.org/69099216:31
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Don't sudo when saving container logs in registry test  https://review.opendev.org/69099316:31
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Test buildset registry with k8s and docker  https://review.opendev.org/68928016:31
clarkbfungi: I think glean should configure NM to do dhcpv6 in that case, but it does not currently know how to do that16:31
sean-k-mooneyfungi: did i imaging it or is that something that can already be done with ipv4 if you disable dhcp on the network16:31
sean-k-mooneyclarkb: i mean that too. the network manager config had dhcp enabled16:32
fungiclarkb: oh, i meant telling nova to incorporate the addresses/routes into the network metadata and then have glean set it as if it were static addressing16:32
fungirather than actually using dhcp16:33
sean-k-mooneyit looks like this http://paste.openstack.org/show/785555/16:33
*** ykarel|away has quit IRC16:33
sean-k-mooneyfungi: it might already be there16:33
sean-k-mooneywhere would i check?16:33
fungimount the configdrive and look in the network metadata json16:33
clarkbfungi: I don't think glean should do that. It should configure the network as told by config drive16:33
clarkbfungi: fwiw I think it would work just fine if you told NM to do dhvpcv616:34
sean-k-mooneyya its there16:34
fungiclarkb: right, maybe i'm not being clear. i guess i'm asking why stateful dhcp6 is any different there than static addressing16:34
clarkbfungi: bceause the cloud told us to use dhcpv616:34
sean-k-mooneyhttp://paste.openstack.org/show/785556/16:34
* fungi tries to rephrase again16:35
fungiwhat is the benefit to configuring the cloud to do stateful dhcp6 instead of static assignment, if the vm is capable of ingesting the static configuration anyway16:35
sean-k-mooney"type": "ipv6_dhcpv6-stateful"16:35
clarkbfungi: it is not capable of ingesting the static config16:35
fungithere we go16:35
clarkbfungi: neither static config nor dhcpv6 are implemented on red hat platofrms in glean16:36
clarkb(for ipv6)16:36
fungiso the other part of my question was "what is needed to get the static addressing usable"16:36
sean-k-mooneyso the could is telling glean/cloud init to configre dhcpv6 in stateful mode16:36
fungiand if the cloud were set for static v6, couldn't glean fairly easily set those static addresses the same way as for v416:36
clarkbfungi: glean would need to be updated to handle the static address type from config drive and write out the appropriate NM /etc/sysconfig/ config to configure those addresses on those interfaces16:37
clarkbfungi: it needs work in glean, but it isdoable16:37
fungicool, we're on the same page. that's what i was trying to get to16:37
fungiso if stateful dhcp6 was meant as a workaround for missing static v6 support, then it might make more sense to focus on the static v6 case16:37
clarkbI don't know why anyone would choose stateful dhcvp6 in neutron fwiw16:38
clarkbI don't think you really get much from it?16:38
clarkbmaybe dns server config?16:39
*** igordc has joined #openstack-infra16:39
sean-k-mooneyfungi: for what its woth i just chose stateful dhcpv6 for my neutron subnet because i understand its better router table usage16:39
clarkbbenj_: inap is in the rotation again. Thank you!16:40
clarkbbenj_: I have noticed that our maxTotalInstances quota is 10 which means we are only using 10 instances out of the configured 19016:40
clarkbbenj_: additionally maxTotalRamSize allows for 160 instances of 8GB ram (our flavor size) and maxTotalCores allows for 125 intsances at our flavor core count (8)16:42
clarkbbenj_: if you have time it would be great to adjust those numbers to reflect what you are able to supply then we can update the nodepool max-servers number if necessary to match16:43
clarkbbenj_: also let me know if I can help16:43
fungi#status log updated the owner for https://launchpad.net/os-brick from https://launchpad.net/~openstack-admins to https://launchpad.net/~cinder-drivers at rosmaita's request (the former already owns the latter, so no control is lost)16:44
openstackstatusfungi: finished logging16:45
*** yamamoto has joined #openstack-infra16:45
*** whoami-rajat has joined #openstack-infra16:45
clarkbfungi: fwiw I think it owuld be equally easy to add dhcpv6 support as it would be for static config (in fact dhcpv6 may be slightly easier as you are only setting flags not specific values)16:45
clarkbfungi: I think the red hat support in glean needs a lot of work though16:45
sean-k-mooneyi think the same behavior as ipv4 would make sense.16:47
sean-k-mooneyif dhcp is disabeld in the usbnet but you have the ip info staticaly configure it16:47
clarkbreally its only not there becaus eno one has added it16:47
sean-k-mooneybut if dhcp is enabled the configure dhcpv616:47
benj_clarkb: I forgot the quotas have been reseted. I'm updating them.16:48
clarkbbenj_: thank you!16:48
sean-k-mooneyclarkb: ya i assumed that was the main reason16:48
*** yamamoto has quit IRC16:49
openstackgerritMerged opendev/puppet-openstackci master: Add a warning to the third party CI doc  https://review.opendev.org/68924816:50
*** tosky has quit IRC16:52
clarkbbenj_: I see the update16:53
clarkbnow to see if nodepool/shade/sdk will pick up on that update without a restart16:53
benj_clarkb: You should be good now. I'll be interested in knowing if it works without a restart.16:55
clarkbbenj_: looks like nodepool caught up it is now booting many more instances16:55
clarkbbenj_: I think we should be set now. I'll keep an eye on it though16:56
benj_clarkb: same on my side.16:56
clarkbfungi: did the debian mirror updating get set back to normal update process? I assume we are happy with that state now if so? Any changes need to get in to close that?16:58
*** priteau has quit IRC16:59
*** igordc has quit IRC17:01
*** igordc has joined #openstack-infra17:02
clarkb#status log reenabled inap mtl01 in nodepool17:02
openstackstatusclarkb: finished logging17:02
*** jaosorior has joined #openstack-infra17:03
fungiclarkb: images are building successfully last i checked (modulo the day or so where we broke all image builds across the board)17:04
*** xek_ has quit IRC17:05
*** xek_ has joined #openstack-infra17:05
fungiwe have ready debian-buster and debian-stretch images from ~4 hours ago17:06
fungithough the debian-buster-arm64 and debian-stretch-arm64 images have been broken for around a week, looks like17:07
*** xek_ has quit IRC17:07
fungithat may be something gone sideways on nb03 though17:07
clarkbI think it ran out of disk17:07
*** xek_ has joined #openstack-infra17:07
clarkbfungi: are we updating the mirror with the regularl cron and don't need to merge config fixes? or if not are those changes up for review?17:07
clarkbstats.timers in graphite is 894GB17:09
clarkbstill waiting on my find to finish17:09
fungicron logs indicate updates are happening. 688788 would be good to get merged17:10
fungias for changes we need, mainly someone needs to work out how to get the security mirror writing to a different subpath so we can drop the symlinks17:10
fungiit's probably just an option in the reprepro config we need to set17:11
*** markvoelker has quit IRC17:12
clarkbapproved 68878817:13
*** udesale has quit IRC17:13
*** mriedem_away is now known as mriedem17:14
*** eernst has joined #openstack-infra17:16
clarkbbenj_: we are seeing timeouts for server deletions I think that might be caused by (or possibly contribute to?) some thrashing that we are seeing there. That said in the past we've seen that when we unleash a bunch of new uncached images onto a cloud and it need a little bit to settle down17:16
clarkbbenj_: I think I'll continue to watch it for a bit longer before deciding action needs to be taken17:16
*** ralonsoh has quit IRC17:18
clarkb`find ./ -type f -mtime 365` says there are no files more than a year old in grpahite17:19
clarkbit finally returned with no results17:19
clarkbmy du trolling shows that stats.timers is the bulk of the data though so I'm digging around in there now17:19
corvusclarkb: when did the great opendev rename happen?17:21
clarkbcorvus: april 19?17:21
clarkbmid april17:21
clarkbdo I need to make that the cut off on the find?17:22
corvusclarkb: yeah, i think if we already got the zuulv3 move, that's probably the next big stats reorg17:22
clarkbok lets try 176 days then (delta between today and may 1)17:23
clarkbfwiw I think the opensatck timers may be the big cost based on poking around with du17:23
*** armax has quit IRC17:25
*** markvoelker has joined #openstack-infra17:26
*** jamesmcarthur has quit IRC17:29
*** otsukahy has quit IRC17:30
*** jamesmcarthur has joined #openstack-infra17:31
clarkbbenj_: | fault                  | {'message': 'Build of instance 52cf6048-c143-4e38-baab-46b0fc387100 aborted: Failed to allocate the network(s) with error No fixed IP addresses available for network: 5a52a95d-c531-4393-ad81-df5ae6ce9010, not rescheduling.', 'code': 500, 'created': '2019-10-24T17:08:05Z'} |17:35
openstackgerritMerged opendev/system-config master: Further split Debian and Ubuntu reprepro configs  https://review.opendev.org/68878817:35
clarkb274GB are stats.timers.openstack which means ~600GB is zuul I think17:35
clarkbre the network ip addr failure, the subnet is a /24 with 243 addresses in the allocation pool17:37
clarkbI think maybe we are thrsahing quickly enough that ip addrs aren't getting freed?17:37
clarkbmaybe we should reduce max servers to say 100 and see if that makes it happier?17:37
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Store a list of held nodes per held build in hold request  https://review.opendev.org/68246617:39
*** otsukahy has joined #openstack-infra17:40
openstackgerritJames E. Blair proposed opendev/system-config master: Set zuul_work_dir in gerrit master job  https://review.opendev.org/69102917:40
clarkbany idea what sends stats.timers.openstack? is that sdk?17:41
clarkbthere is a separate stats.timers.nodepool which is much smaller (1.7GB)17:41
corvusclarkb: that seems likely17:42
corvuswe pass statsd info into sdk17:42
clarkbhttp://codesearch.openstack.org/?q=openstack.api.compute&i=nope&files=&repos= seems to say probably sdk too17:43
*** jamesmcarthur has quit IRC17:43
corvusit's the only thing that can time individual api calls now that we use the shade layer in nodepool17:43
clarkbabout 270GB of that is compute specific17:43
clarkbI'm not sure we need 270GB of nova api timing stats17:44
corvusi'm also not sure we use them?17:44
clarkbpossible that the nodepool grafana dashboards use some of that info but I agree codeserach didn't pop anything up matching it if os17:44
corvusoh, we used to but we didn't update the grafana dashboards17:45
*** jaosorior has quit IRC17:45
clarkbare we able to put a retention policy on this so that we onl yhave say 4 weeks of data?17:45
corvusweird, i wonder why that never came up with all the fn activity...17:46
*** otsukahy has quit IRC17:46
clarkbzuul is also the other 600GB or so and we probably want to figure retention out for zuul too, but I figured the openstack stuff was easier because it is mostly admin facing17:46
*** otsukahy has joined #openstack-infra17:47
*** jerryz has joined #openstack-infra17:48
clarkbI need to step out for a bit but I think that helps narrow down the graphite cleanup focus17:48
corvusclarkb: retentions are here https://opendev.org/opendev/puppet-graphite/src/branch/master/manifests/init.pp17:49
*** jaosorior has joined #openstack-infra17:50
clarkbthanks17:50
clarkbI assume we could do a more specific retention for stats.timers and do less retention of those?17:50
corvusyep17:50
clarkbcool I'll work on a patch for that if no one beats me to it. It will be a bit. I've got a dr appointment at noon and need to get ready for that17:51
*** weifan has joined #openstack-infra17:52
*** amoralej is now known as amoralej|off17:52
*** otsukahy has quit IRC17:52
clarkbinap seems to be settling down now17:52
clarkb53 in use17:52
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Remove --id option for autohold_delete/autohold_info  https://review.opendev.org/69103317:53
openstackgerritJeremy Stanley proposed opendev/storyboard master: Regular expression support for CORS and OAuth ACLs  https://review.opendev.org/69103417:53
*** eernst has quit IRC17:53
*** jamesmcarthur has joined #openstack-infra17:54
*** ricolin has quit IRC17:55
openstackgerritClark Boylan proposed opendev/puppet-graphite master: Reduce retention of stats.timers  https://review.opendev.org/69103617:56
clarkbcorvus: ^ something like that maybe?17:56
clarkbI've got to pop out now feel free to update with better retention rules17:57
*** jamesmcarthur has quit IRC17:59
*** armax has joined #openstack-infra18:02
*** tosky has joined #openstack-infra18:03
*** otsukahy has joined #openstack-infra18:05
*** jaosorior has quit IRC18:09
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Make the buildset registry port configurable  https://review.opendev.org/68924018:10
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Adjust buildset registry container name  https://review.opendev.org/69099218:10
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Don't sudo when saving container logs in registry test  https://review.opendev.org/69099318:10
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Test buildset registry with k8s and docker  https://review.opendev.org/68928018:10
*** otsukahy has quit IRC18:10
*** jaosorior has joined #openstack-infra18:11
openstackgerritSTEW TY proposed openstack/project-config master: Update Project Configs for x/ranger  https://review.opendev.org/69103818:15
*** otsukahy has joined #openstack-infra18:17
*** jaosorior has quit IRC18:20
*** mriedem has quit IRC18:20
*** mriedem has joined #openstack-infra18:22
*** otsukahy has quit IRC18:22
corvusmordred, clarkb, fungi:   https://review.opendev.org/691029  should fix the failure we saw with the gerrit build when it ran against the checks plugin repo18:30
corvussince the new zuul pipeline config is in place, i think we're good to try again after that lands18:31
*** otsukahy has joined #openstack-infra18:40
fungii'm stepping out to run some errands and grab late lunch/early dinner, but should be back later18:41
*** kjackal has quit IRC18:42
openstackgerritMerged zuul/nodepool master: AWS driver: add ability to determine AMI id using filters  https://review.opendev.org/68320518:45
*** otsukahy has quit IRC18:47
openstackgerritSTEW TY proposed openstack/project-config master: Update Project Configs for x/ranger  https://review.opendev.org/69103818:59
*** pkopec has joined #openstack-infra19:01
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Remove unused imagemin build step  https://review.opendev.org/69105019:02
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Update selenium-standalone and gifsicle  https://review.opendev.org/69105119:02
*** rascasoft has quit IRC19:07
openstackgerritMerged opendev/system-config master: Set zuul_work_dir in gerrit master job  https://review.opendev.org/69102919:08
*** rascasoft has joined #openstack-infra19:09
*** weifan has quit IRC19:10
*** kjackal has joined #openstack-infra19:13
*** whoami-rajat has quit IRC19:15
*** otsukahy has joined #openstack-infra19:16
*** otsukahy has quit IRC19:21
openstackgerritDavid Shrewsbury proposed zuul/zuul master: Remove --id option for autohold_delete/autohold_info  https://review.opendev.org/69103319:32
rm_workmtreinish: yeah ok, seems like lifeless is who has access to the pypi19:37
*** weifan has joined #openstack-infra19:43
*** weifan has quit IRC19:49
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Test buildset registry with k8s and docker  https://review.opendev.org/68928019:50
*** jtomasek has quit IRC19:51
*** goldyfruit_ has joined #openstack-infra19:57
openstackgerritSTEW TY proposed openstack/project-config master: Update Project Configs for x/ranger  https://review.opendev.org/69103819:59
*** goldyfruit has quit IRC19:59
*** tosky_ has joined #openstack-infra20:00
*** otsukahy has joined #openstack-infra20:01
*** tosky has quit IRC20:03
*** slaweq has joined #openstack-infra20:04
*** otsukahy has quit IRC20:07
*** iokiwi has joined #openstack-infra20:08
*** yamamoto has joined #openstack-infra20:11
*** jamesmcarthur has joined #openstack-infra20:14
*** otsukahy has joined #openstack-infra20:15
*** yamamoto has quit IRC20:16
*** Goneri has quit IRC20:18
*** otsukahy has quit IRC20:19
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Test buildset registry with k8s and docker  https://review.opendev.org/68928020:20
*** weifan has joined #openstack-infra20:22
*** otsukahy has joined #openstack-infra20:22
openstackgerritMonty Taylor proposed opendev/system-config master: Update gitea to v1.10  https://review.opendev.org/69087320:23
clarkbinfra-root https://review.opendev.org/#/c/691036/ should hopefully fix graphites lack of free disk20:24
clarkbif you hvae a moment to revie wthat it would be great20:25
clarkbinap seems to have stablized while I was out20:26
clarkbthat is good20:26
*** weifan has quit IRC20:26
benj_clarkb: Not sure where the network issues you saw earlier came from. But things seem to be runnning smooth now.20:27
*** otsukahy has quit IRC20:27
clarkbbenj_: my hunch is that builds timed out due to demand the the images not being cached by nova computes yet20:28
clarkbwhich led to nodepool trying to build more nodes than there were IPs for (due to delay in cleaning things up fully?)20:28
clarkbbut ya seems happy now20:28
benj_clarkb: our guess too on our side.20:28
*** weifan has joined #openstack-infra20:29
*** kjackal has quit IRC20:32
mordredclarkb, corvus: the patch above to update gitea to 1.10 (which hasn't been released yet) actually did catch an api change - so our testing is non-terrible20:33
clarkbdoes 1.10 include your fix?20:33
mordredyeah. actually - release/v1.9 and release/v1.10 and master all now include my fix20:33
*** jamesmcarthur has quit IRC20:34
mordredso i pushed up https://review.opendev.org/#/c/690872/ which would get us on to what will be 1.9.5 when it releases - and then 690873 would do the same but with the 1.10 series, which is likely to release today ish? (they normally release on the 25th)20:35
*** rh-jelabarre has quit IRC20:35
*** weifan has quit IRC20:36
*** weifan has joined #openstack-infra20:37
*** KeithMnemonic has quit IRC20:41
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Adds go jobs and roles.  https://review.opendev.org/69111420:45
*** weifan has quit IRC20:46
*** weifan has joined #openstack-infra20:46
openstackgerritMerged opendev/puppet-graphite master: Reduce retention of stats.timers  https://review.opendev.org/69103620:51
*** KeithMnemonic has joined #openstack-infra20:52
*** weifan has quit IRC21:00
*** weifan has joined #openstack-infra21:01
openstackgerritMerged zuul/zuul master: Store a list of held nodes per held build in hold request  https://review.opendev.org/68246621:04
openstackgerritMerged zuul/zuul master: Remove --id option for autohold_delete/autohold_info  https://review.opendev.org/69103321:08
*** otsukahy has joined #openstack-infra21:09
*** aedc has joined #openstack-infra21:10
*** sreejithp has quit IRC21:13
*** otsukahy has quit IRC21:13
*** markvoelker has quit IRC21:20
*** rfolco|rover has quit IRC21:25
*** e0ne has joined #openstack-infra21:37
*** goldyfruit_ has quit IRC21:38
*** armax has quit IRC21:38
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Test buildset registry with k8s and docker  https://review.opendev.org/68928021:39
*** slaweq has quit IRC21:44
*** e0ne has quit IRC21:49
*** e0ne has joined #openstack-infra21:49
*** e0ne has quit IRC21:56
*** slaweq has joined #openstack-infra21:56
*** rcernin has quit IRC22:03
*** apetrich has quit IRC22:03
*** markvoelker has joined #openstack-infra22:03
*** goldyfruit has joined #openstack-infra22:03
*** gfidente has quit IRC22:06
*** igordc has quit IRC22:08
openstackgerritMonty Taylor proposed opendev/system-config master: DNM Update gitea to v1.10  https://review.opendev.org/69087322:11
*** slaweq has quit IRC22:13
*** pkopec has quit IRC22:18
clarkbthe graphite storage-schemas.conf file has been updated on graphite.o.o22:23
clarkbdisk usage is where it was22:23
clarkbI am going to restart graphite and carbon to see if that is necessary to pick up the new config22:23
corvusyeah, i'm not sure what needs to happen to trigger that22:23
clarkbit may also be necessary to manually clean up files and only new files will have the new retention? will cross that bridge if we get there22:23
corvusit might be on the next write?22:23
corvusor that22:23
corvusthat's actually sounding familiar22:23
clarkbI'll start first with service restart22:23
corvusclarkb: https://stackoverflow.com/questions/30106553/carbon-graphite-apply-new-retention-to-old-data22:24
clarkbthanks22:26
clarkbI guess I'll run that under stats.timers.openstack.api to start22:26
clarkb(I didn't end up restartin gservices yet either as I didn't figure out what they were called before you shared that22:26
*** markvoelker has quit IRC22:26
corvusclarkb: i guess the best thing would be to stop, resize, start ?22:27
clarkb++22:27
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Test buildset registry with k8s and docker  https://review.opendev.org/68928022:27
clarkbcarbon-cache is the service name fwiw22:27
clarkband statsd22:28
clarkbI think I stop both of those then do resize?22:28
clarkb`find ./ -type f -name '*.wsp' -exec whisper-resize.py --nobackup {} 10s:8h 60s:7d 1h:90d 1d:1y \;` is my command22:29
clarkbgoing to run that in /var/lib/graphite/storage/whisper/stats/timer/openstack22:29
clarkbstopping carbon-cache and statsd now22:30
openstackgerritMerged openstack/diskimage-builder master: Fix syntax error in selinux-fixfiles-restore  https://review.opendev.org/68943022:30
clarkbwell that failed due to no space left on device. I killed the find22:31
clarkbI guess I need to make a bit of room first22:31
*** sgw has quit IRC22:33
mordredclarkb: :(22:34
lifelessrm_work: I do? for what?22:34
*** otsukahy has joined #openstack-infra22:34
clarkbok I cleared up ~15MB by moving some of the openstack compute GET dirs into my homedir22:34
rm_worklifeless: https://pypi.org/project/python-subunit/22:34
clarkbthat seems to be sufficient for resizing22:35
clarkband free disk is climbing now22:35
clarkbI can mv those dirs back into place and rerun over them when the current find is done22:35
corvusit is 92f (33 1/3c) outside my house22:35
rm_worklifeless: the last release was over a year ago... there was a fix made last december to fix a bug in py3, but no release has happened since then22:35
*** tosky_ is now known as tosky22:36
clarkbcorvus: it is a beautiful day here about 65F and sunny22:36
clarkbcorvus: I hope pg&e doesn't cut your power :/22:36
*** tosky has quit IRC22:36
corvusclarkb: okay if i come over? :)22:36
clarkbcorvus: sure22:36
*** weifan has quit IRC22:36
clarkbinfra-root I have a root screen running the find and whisper resize22:36
paladoxthere still cutting power in CA?22:37
corvuspaladox: yeah, they did it again up north, but there's a fire anyway (cause unknown at this point)22:37
paladox:(22:37
*** weifan has joined #openstack-infra22:38
ianwcorvus: any thoughts on why https://review.opendev.org/690798 keeps returning NODE_FAILURE?  thinking about it, might be the first time i've tried to have multiple architectures in nodes, i wonder if somehow the way arm64 is only on nl03 has some effect?22:38
corvusianw: all the nodes have to be from the same provider22:39
corvusi'm guessing that's not going to be able to happen here22:39
ianwahhhh, of course ...22:39
corvusianw: i guess we'll need a duplicate job?22:39
*** armax has joined #openstack-infra22:39
ianwyeah, that's a bummer because i don't want port bridge.o.o to be arm64 compatible22:40
corvusianw: or if this is just for ... oh...22:40
fungiattempting to simulate our multi-architecture environment in a multi-node job22:40
ianwi really want the standard x86 bridge node to talk to the arm64 mirror (the bit i'm interested in making sure works)22:40
fungibut yeah, the fact that we assume all nodes in a nodeset are in the same provider makes that impossible with single-architecture providers for each architecture22:41
corvuswhat's different about the arm64 mirror that makes it worth testing in a job like this?  i guess the different arch is like a different distro?22:41
corvusso we want to test that all the packages are there, behave in the same way, etc?22:41
ianwyep, and openafs builds etc22:42
fungiwe have some special builds of openafs, yeah22:42
corvusgotcha22:42
mordredyeah, openafs builds yay22:42
mordredthis would be a great thing for if we coudl run the playbooks directly on the executor22:42
corvuswelp.  i acklowdge this use case with these conditions is poorly served by the current design.  :|22:42
mordredlike if the executor filled the role of bridge22:42
ianwyeah it just didn't click that linaro-london doesn't have any x86 nodes till you said it :)22:43
mordredbut we'd need to drop the localhost execution restriction for that22:43
corvusianw: what kind of effort do you think might be needed for bridge.yaml to be arm64 compat?22:43
fungithere's always the counter-argument that ideally we'd have multi-architecture providers22:43
ianwcorvus: i dunno, i can try it ...22:44
corvusmordred: agreed22:44
mordredbridge.yaml shouldn't do *too* many things that are fancy22:44
mordredcorvus: did we ever get any consensus around that?22:44
corvusianw: i think that may be the shortest and best path to happiness22:44
corvusmordred: i think i need to resurrect a ml thread, but i also think we need to get zk/gearman auth/ssl completely sorted first22:44
ianwi'll give it a go.  otherwise, i can probably test roles one-by-one22:44
mordredcorvus: oh right. zk/gearman auth was the thing22:45
paladoxcorvus i learned over the week that ofgem has a safty net so if a engery supplier goes bust, the customer doin't need to do much.22:45
ianwor, indeed, just bring up a production host and basically test it live22:45
corvuslike, probably to the point where we disallow unauth/untls zk/gear22:45
paladoxThey'll find a new engery supplier for you :)22:45
ianwalthough that's not very gitops-y22:45
mordredcorvus: yeah22:45
paladoxa energy supplier went bust here (small one as i've never herd of it)22:45
ianwi think a limiting factor for linaro-london might be we don't have space to run two mirrors at the same time in our control-plane tenant22:46
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Test buildset registry with k8s and docker  https://review.opendev.org/68928022:47
openstackgerritIan Wienand proposed opendev/system-config master: [wip] test arm64 mirror  https://review.opendev.org/69079822:47
*** igordc has joined #openstack-infra22:47
*** rlandy has quit IRC22:52
*** mriedem has quit IRC22:54
clarkbnow over a gigabyte is free22:54
*** weifan has quit IRC22:56
*** weifan has joined #openstack-infra22:58
*** slaweq has joined #openstack-infra23:00
openstackgerritClark Boylan proposed zuul/zuul-website master: Specify open source projects using Zuul  https://review.opendev.org/68011023:02
openstackgerritClark Boylan proposed zuul/zuul-website master: Web improvement ideas  https://review.opendev.org/68010923:02
openstackgerritMonty Taylor proposed opendev/system-config master: Update gitea to v1.10  https://review.opendev.org/69087323:04
*** tkajinam has joined #openstack-infra23:04
*** slaweq has quit IRC23:05
mordredclarkb, corvus: I *think* with that version ^^ we should be ready to go when 1.10 is released23:09
openstackgerritJeremy Stanley proposed zuul/zuul-website master: Update case studies on users page  https://review.opendev.org/69113023:12
*** rcernin has joined #openstack-infra23:13
*** zbr has quit IRC23:13
*** otsukahy has quit IRC23:15
ianwA start job is running for Glean for interface enp1s0 (29s / no limit) ... hrm not looking good for arm64 nodes atm :/23:16
ianwthe images are a bit out of date; i'm building new ones and monitoring upload23:16
ianwbut i don't know why glean service would be stuck23:16
*** mattw4 has quit IRC23:17
funginb03 ran out of disk according to clarkb earlier today23:18
clarkbya pretty sure that is why it hadn't built images in a while23:18
ianwit had also seemed to have a terminal zookeeper dropout23:18
fungii noticed that the debian arm64 images were about a week stale, likely the rest are as well23:18
ianwyeah bionic is ~10th23:18
openstackgerritMerged zuul/zuul-website master: Reduce padding to remove extraneous whitespace  https://review.opendev.org/61768023:21
ianwit eventually sort of came up, but it seem no networking http://paste.openstack.org/show/785566/23:22
mordredianw: well that looks like it's going to be fun to figure out23:25
ianwas a start, we should probably add the "systemd journal to serial console" element to see what it thinks is happening23:26
openstackgerritJeremy Stanley proposed zuul/zuul-website master: Update case studies on users page  https://review.opendev.org/69113023:26
*** mattw4 has joined #openstack-infra23:28
*** yamamoto has joined #openstack-infra23:30
openstackgerritIan Wienand proposed openstack/project-config master: arm64 images: add journal-to-console  https://review.opendev.org/69113223:30
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Test buildset registry with k8s and docker  https://review.opendev.org/68928023:31
clarkbwe have a dir of whisper files for every image we've ever created23:33
clarkbI'm betting we have a whisper file for every instance we've ever created too23:33
clarkbmordred: ^23:33
corvusthat sounds unuseful23:34
clarkb./api/object-store/PUT/MossoCloudFS_28a933f9-900d-40b8-9de3-deb24d213aa2/gentoo-17-0-systemd-1560155509/count.wsp for example23:34
clarkband then we do that for every image that nodepool has created looks like23:34
clarkband ya I'm not sure that is the correct axis to report on23:35
clarkbI'm going to bet it is a similar story for instances given the size of the compute dir23:35
*** yamamoto has quit IRC23:35
*** xek_ has quit IRC23:41
*** slaweq has joined #openstack-infra23:43
openstackgerritMerged openstack/project-config master: arm64 images: add journal-to-console  https://review.opendev.org/69113223:44
*** dchen has joined #openstack-infra23:45
*** slaweq has quit IRC23:48
clarkbmordred: corvus I think https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/proxy.py#L197-L199 is the source of the problem23:49
openstackgerritMonty Taylor proposed opendev/system-config master: Apply review container to review-dev  https://review.opendev.org/69048323:49
openstackgerritMonty Taylor proposed opendev/system-config master: Use explicit image paths  https://review.opendev.org/69051123:49
openstackgerritMonty Taylor proposed opendev/system-config master: Update pip3 role to install from get-pip.py  https://review.opendev.org/69076623:49
clarkbwhat that extract function does is get bits out of the url that constitute names23:50
mordredclarkb: oh yeah. that's going to be super un-useful for objet-store23:50
clarkbI'm not positive yet but it may actually do the correct thing for compute23:50
clarkbbut object-store because it doens't use id's but uses well names causes problems23:50
mordredit should be doing the correct thing for compute - making sure we didn't report on every instance was part of the original design there23:51
mordredyup23:51
clarkbmordred: ya my skim read I think that object-store is the odd one out here23:51
mordredI think what we need is to be able to override _report_stats_statsd in the object-store proxy23:51
mordredbecause its url pattern is just completely different23:52
mordredreporting stats like "container_create" or "object_create" is much more interesting than "object/x/y/z create"23:52
clarkb++++++23:53
clarkbmelwitt: ^ I know we don't have a good answer for you yet but we are making changes that should be improving things re graphite23:54
*** weifan has quit IRC23:55
clarkbmelwitt: basically openstacksdk is doing some weird things and we are trying to retain data for 5 years. So we're reducing retention and hopefully sdk will update23:55
clarkband things will be happy in a little while23:55
melwittclarkb: sounds cool, thanks and looking forward to seeing the improvements :)23:56

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