Monday, 2018-09-24

*** bobh has quit IRC00:04
*** bobh has joined #openstack-infra00:06
*** bobh has quit IRC00:11
*** bobh has joined #openstack-infra00:22
*** longkb has joined #openstack-infra00:23
*** jamesmcarthur has quit IRC00:24
*** bobh has quit IRC00:26
*** bobh_ has joined #openstack-infra00:26
*** bobh_ has quit IRC00:31
*** bobh has joined #openstack-infra00:36
*** rkukura has quit IRC00:43
*** fried_rolls1 has joined #openstack-infra00:48
*** bobh has quit IRC00:49
*** fried_rolls has quit IRC00:50
*** fried_rolls1 is now known as fried_rolls00:50
pabelangerdhellmann: looks like testr is missing: http://logs.openstack.org/53/604653/4/check/tox-cover/fb44cc8/job-output.txt.gz#_2018-09-24_00_01_51_32361600:55
pabelangerlooks like runtests command wasn't updated properly00:56
*** fried_rolls is now known as efried01:07
openstackgerritAndreas Jaeger proposed openstack-infra/storyboard master: Update coverage environment  https://review.openstack.org/60466701:09
AJaegerdhellmann: let's try this ^01:09
*** imacdonn has quit IRC01:17
*** imacdonn has joined #openstack-infra01:17
openstackgerritMerged openstack-infra/zuul-jobs master: lay down gentoo use file for openvswitch  https://review.openstack.org/60465501:18
AJaegerconfig-core, karbor and cinder are ready with python3-first - please review https://review.openstack.org/#/c/594834/  and https://review.openstack.org/#/c/60112701:20
*** mrsoul has joined #openstack-infra01:22
*** rcernin has quit IRC01:40
*** rcernin has joined #openstack-infra01:40
prometheanfireAJaeger: we need a new gentoo image first don't we?01:47
prometheanfiremaybe not, since it's ansible01:47
*** weshay is now known as weshay_afk01:49
*** hongbin has joined #openstack-infra01:49
*** Bhujay has joined #openstack-infra02:04
*** Bhujay has quit IRC02:12
prometheanfireAJaeger: I'm guessing lineinfile doesn't create the file if it doesn't already exist?02:18
prometheanfirelesigh02:18
prometheanfireDestination /etc/portage/package.use/ovs does not exist !02:18
openstackgerritMatthew Thode proposed openstack-infra/zuul-jobs master: create the pacage.use file...  https://review.openstack.org/60467102:20
prometheanfireyet another round...02:20
*** jamesmcarthur has joined #openstack-infra02:24
*** pcrews has joined #openstack-infra02:26
*** jamesmcarthur has quit IRC02:28
*** imacdonn has quit IRC02:35
*** psachin has joined #openstack-infra02:40
*** yamamoto has quit IRC02:42
*** jamesmcarthur has joined #openstack-infra02:43
*** bobh has joined #openstack-infra02:43
*** imacdonn has joined #openstack-infra02:50
*** jamesmcarthur has quit IRC02:54
*** annp has joined #openstack-infra03:07
*** bobh has quit IRC03:15
*** vivsoni has joined #openstack-infra03:25
*** ijw has joined #openstack-infra03:28
*** hongbin has quit IRC03:39
*** psachin has quit IRC03:48
*** udesale has joined #openstack-infra03:50
*** icey has quit IRC03:56
*** icey has joined #openstack-infra03:57
*** psachin has joined #openstack-infra04:00
openstackgerritAndreas Jaeger proposed openstack-infra/storyboard master: Update coverage environment  https://review.openstack.org/60466704:05
AJaegerprometheanfire: you can test with depends-on...04:06
*** yamamoto has joined #openstack-infra04:08
*** vivsoni has quit IRC04:10
*** _ari_ has quit IRC04:11
prometheanfireok04:16
openstackgerritMatthew Thode proposed openstack-infra/openstack-zuul-jobs master: add Gentoo jobs and fix install test  https://review.openstack.org/60243904:18
*** ykarel has joined #openstack-infra04:22
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: executor: add support for generic build resource  https://review.openstack.org/57066804:29
*** vivsoni has joined #openstack-infra04:30
*** vivsoni has quit IRC04:37
*** vivsoni has joined #openstack-infra04:37
*** vivsoni has quit IRC04:43
*** vivsoni has joined #openstack-infra04:46
*** jamesmcarthur has joined #openstack-infra04:50
*** jamesmcarthur has quit IRC04:54
*** Bhujay has joined #openstack-infra04:58
*** ramishra has joined #openstack-infra04:59
*** rkukura has joined #openstack-infra05:00
*** spsurya has joined #openstack-infra05:02
openstackgerritTristan Cacqueray proposed openstack-infra/nodepool master: Implement a Kubernetes driver  https://review.openstack.org/53555705:09
prometheanfirelol, neutron type errror05:14
openstackgerritMatthew Thode proposed openstack-infra/zuul-jobs master: use find instead of ls to list interfaces  https://review.openstack.org/60467705:22
openstackgerritMatthew Thode proposed openstack-infra/openstack-zuul-jobs master: add Gentoo jobs and fix install test  https://review.openstack.org/60243905:23
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Support node caching in the nodeIterator  https://review.openstack.org/60464805:25
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Use node cache in quota calculations  https://review.openstack.org/60464905:25
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Cache iterations over ready nodes  https://review.openstack.org/60465005:25
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Use cache when counting poolnodes  https://review.openstack.org/60465105:25
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Use cache when deleting oldest unused nodes  https://review.openstack.org/60465205:25
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Fix race in test_launchNode_delete_error  https://review.openstack.org/60467805:25
*** yamamoto has quit IRC05:30
*** dave-mccowan has joined #openstack-infra05:36
*** jtomasek has joined #openstack-infra05:37
*** yamamoto has joined #openstack-infra05:39
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Support node caching in the nodeIterator  https://review.openstack.org/60464805:40
*** dayou has quit IRC05:43
*** dayou has joined #openstack-infra05:44
*** quique|rover|off is now known as quiquell|rover05:45
*** jiapei has joined #openstack-infra05:46
*** Bhujay has quit IRC05:53
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Move storyboard cover job in-repo  https://review.openstack.org/60467905:57
*** Bhujay has joined #openstack-infra05:59
*** dpawlik has joined #openstack-infra06:03
*** ajo has joined #openstack-infra06:03
*** pcaruana has joined #openstack-infra06:05
*** apetrich has joined #openstack-infra06:05
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add labels page  https://review.openstack.org/60468206:13
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add nodes page  https://review.openstack.org/60468306:13
AJaegerfrickler, ianw, karbor and cinder are ready with python3-first - could either of you review https://review.openstack.org/#/c/594834/  and https://review.openstack.org/#/c/601127 , please?06:18
AJaegerand also https://review.openstack.org/#/c/604529/ and https://review.openstack.org/604608 and https://review.openstack.org/601946, please06:19
prometheanfireAJaeger: looks like 'Validate ipv4 private firewall configuration' is failing, have any insight? http://logs.openstack.org/39/602439/7/check/openstack-infra-multinode-integration-gentoo-17-0-systemd/eeaf2c8/ara-report/reports/3eb0c0c2-a3ed-4787-8fd4-de5d388fe78f.html06:21
* prometheanfire is running a recheck on it just in case06:22
AJaegerprometheanfire: sorry, no ;(06:23
*** e0ne has joined #openstack-infra06:23
prometheanfirehopefully recheck fixes it, otherwise I'll have to do debug statements I think06:23
prometheanfirethis looks wrong though http://logs.openstack.org/39/602439/7/check/openstack-infra-multinode-integration-gentoo-17-0-systemd/eeaf2c8/job-output.txt.gz#_2018-09-24_05_59_48_77005606:24
*** longkb has quit IRC06:25
fricklerprometheanfire: seems the firewall rules generation is still skipped completely http://logs.openstack.org/39/602439/7/check/openstack-infra-multinode-integration-gentoo-17-0-systemd/eeaf2c8/job-output.txt.gz#_2018-09-24_05_59_46_18133506:25
*** longkb has joined #openstack-infra06:25
prometheanfirefrickler: I think that's a limitation of ansible itself...06:28
fricklerprometheanfire: not sure what you mean by that. to me it looks like the rules aren't getting installed which then causes the checks whether they have been installed to fail06:30
prometheanfire2018-09-24 05:59:46.707012 | WARNING: Gentoo is not supported by this role yet. The execution of the job will continue without persisting iptables rules.06:30
prometheanfirewhere does that role exist?06:30
fricklerlet me check06:31
prometheanfireoh.. found it lol06:31
prometheanfirewas looking in files, not for files06:31
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Support node caching in the nodeIterator  https://review.openstack.org/60464806:31
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Use node cache in quota calculations  https://review.openstack.org/60464906:31
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Cache iterations over ready nodes  https://review.openstack.org/60465006:31
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Use cache when counting poolnodes  https://review.openstack.org/60465106:31
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Use cache when deleting oldest unused nodes  https://review.openstack.org/60465206:31
openstackgerritIan Wienand proposed openstack-infra/nodepool master: Use zuul-sphinx for configuration layout  https://review.openstack.org/60427406:33
*** chandankumar has joined #openstack-infra06:33
*** chandankumar is now known as chkumar|ruck06:34
openstackgerritMerged openstack-infra/zuul-jobs master: log-inventory: remove inventory_file usage  https://review.openstack.org/57823806:34
prometheanfirefrickler: should I be saving rules before shutdown or letting ansible handle all rule generation?06:36
*** ijw has quit IRC06:36
fricklerprometheanfire: iiuc what the test does is flush all the iptables rules and then check that they get created again after the iptables service is restarted06:38
fricklersee http://logs.openstack.org/39/602439/7/check/openstack-infra-multinode-integration-gentoo-17-0-systemd/eeaf2c8/job-output.txt.gz#_2018-09-24_05_59_47_159963 ff.06:38
openstackgerritMatthew Thode proposed openstack-infra/zuul-jobs master: Add Gentoo iptables handling  https://review.openstack.org/60468806:38
fricklernot sure how to best achieve that in gentoo, but I think it needs to happen on the os side and not in ansible06:38
prometheanfirefrickler: yep, I'm not saving it on shutdown ^06:38
AJaegerprometheanfire: add a depends-on again to test it, please06:39
prometheanfirefrickler: as long as nothing overwrites the file it'll restore to it06:39
prometheanfireyep06:39
openstackgerritMerged openstack-infra/project-config master: remove job settings for cinder repositories  https://review.openstack.org/60112706:39
openstackgerritMerged openstack-infra/project-config master: remove job settings for karbor repositories  https://review.openstack.org/59483406:39
fricklerprometheanfire: o.k., that looks reasonable to me, let's see how the test goes06:39
prometheanfirethis'll be my last one before slee06:39
prometheanfirethis'll be my last one before sleep06:39
openstackgerritMatthew Thode proposed openstack-infra/openstack-zuul-jobs master: add Gentoo jobs and fix install test  https://review.openstack.org/60243906:40
*** e0ne_ has joined #openstack-infra06:41
prometheanfireok, nn06:42
AJaegerprometheanfire: good night!06:43
*** e0ne has quit IRC06:44
openstackgerritMerged openstack-infra/project-config master: Avoid double root-marker  https://review.openstack.org/60194606:54
*** adrianreza has quit IRC06:56
*** quiquell|rover is now known as quique|rover|brb06:58
openstackgerritMatthew Thode proposed openstack-infra/zuul-jobs master: use find instead of ls to list interfaces  https://review.openstack.org/60467706:58
prometheanfireok, now I sleep07:00
*** ginopc has joined #openstack-infra07:03
*** ijw has joined #openstack-infra07:06
*** rcernin has quit IRC07:06
*** ijw has quit IRC07:10
*** gfidente has joined #openstack-infra07:13
*** aojea has joined #openstack-infra07:13
openstackgerritMerged openstack-infra/zuul-jobs master: create the pacage.use file...  https://review.openstack.org/60467107:17
*** holser_ has joined #openstack-infra07:23
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Cache node iterations of DeletedNodeWorker  https://review.openstack.org/60469107:23
*** shardy has joined #openstack-infra07:27
*** ykarel is now known as ykarel|lunch07:27
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: Cache node iterations in cleanup workers  https://review.openstack.org/60469107:29
*** quique|rover|brb is now known as quiquell|rover07:30
*** gfidente has quit IRC07:31
*** gfidente has joined #openstack-infra07:36
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Allow debootstrap to clenaup without a kernel  https://review.openstack.org/60469207:37
*** roman_g has joined #openstack-infra07:43
*** dtantsur|afk is now known as dtantsur07:46
AJaegerconfig-core, https://review.openstack.org/604526 is ready (retire fuxi) and https://review.openstack.org/604529 (retire charm-ceph) needs second +2 - please review07:47
*** jpena|off is now known as jpena07:49
*** alexchadin has joined #openstack-infra07:51
*** tosky has joined #openstack-infra07:51
ianwAJaeger: first one is WIP but i assume it's ready as all deps are07:52
*** jpich has joined #openstack-infra07:56
openstackgerritMerged openstack-infra/project-config master: Retire charm-ceph (step 4)  https://review.openstack.org/60452907:57
*** longkb has quit IRC08:03
*** hashar has joined #openstack-infra08:06
*** jcoufal has joined #openstack-infra08:07
*** ginopc has quit IRC08:09
*** alexchadin has quit IRC08:14
AJaegerianw: Oh, yes - will remove now. Thanks08:17
openstackgerritFabien Boucher proposed openstack-infra/zuul master: Doc: executor operations - explain jobs will be restarted at restart  https://review.openstack.org/60313608:18
openstackgerritFabien Boucher proposed openstack-infra/zuul master: Doc: executor operations - explain jobs will be restarted at restart  https://review.openstack.org/60313608:21
AJaegerthanks frickler and ianw for reviewing!08:22
*** openstackgerrit has quit IRC08:22
*** openstackgerrit has joined #openstack-infra08:30
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove migrated keystone swift jobs  https://review.openstack.org/60319908:30
openstackgerritMerged openstack-infra/project-config master: Retire fuxi projects (step 4)  https://review.openstack.org/60452608:30
AJaegerdmellado: ^08:31
dmelladoAJaeger: awesome, I was just checking that on the gerrit as well ;)08:31
AJaegerdmellado: https://review.openstack.org/#/c/604527/ (the governance change) will be merged in a week if there are no objections. So, our work should be done here ;)08:32
dmellado\o/08:33
*** janki has joined #openstack-infra08:33
*** janki has quit IRC08:34
*** janki has joined #openstack-infra08:34
*** holser_ has quit IRC08:35
*** holser_ has joined #openstack-infra08:36
*** holser_ has left #openstack-infra08:36
openstackgerritMerged openstack-infra/python-storyboardclient master: Update the URL in HACKING.rst  https://review.openstack.org/60463908:37
*** ykarel|lunch is now known as ykarel08:38
*** Bhujay has quit IRC08:38
*** finucannot is now known as stephenfin08:38
*** rossella_s has quit IRC08:41
*** alexchadin has joined #openstack-infra08:42
*** derekh has joined #openstack-infra08:44
*** yamamoto has quit IRC08:45
*** jamesmcarthur has joined #openstack-infra08:50
*** jamesmcarthur has quit IRC08:55
openstackgerritAntonio Ojea proposed openstack-infra/openstack-zuul-jobs master: Add openSUSE Leap 15.0 job for platform testing  https://review.openstack.org/60456309:00
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Allow debootstrap to clenaup without a kernel  https://review.openstack.org/60469209:03
*** openstackgerrit has quit IRC09:05
*** gfidente is now known as gfidente|afk09:05
*** jcoufal has quit IRC09:07
*** kukacz has quit IRC09:10
*** jaosorior has quit IRC09:11
*** kukacz has joined #openstack-infra09:12
*** openstackgerrit has joined #openstack-infra09:13
openstackgerritMerged openstack-infra/project-config master: Remove most job entries for microversion-parse  https://review.openstack.org/59945309:13
*** rpittau has joined #openstack-infra09:20
*** pbourke has joined #openstack-infra09:21
*** ianychoi_ has joined #openstack-infra09:30
*** Bhujay has joined #openstack-infra09:32
*** ianychoi has quit IRC09:34
*** adrianreza has joined #openstack-infra09:36
*** Bhujay has quit IRC09:37
*** e0ne_ has quit IRC09:38
*** e0ne has joined #openstack-infra09:38
*** yamamoto has joined #openstack-infra09:40
*** hrw has quit IRC09:44
openstackgerritMerged openstack-infra/storyboard master: Update coverage environment  https://review.openstack.org/60466709:56
dtantsurhi all! does anyone know why https://docs.openstack.org/metalsmith/ does not redirect to https://docs.openstack.org/metalsmith/latest/ ?09:57
openstackgerritAndreas Jaeger proposed openstack-infra/storyboard master: skip some alembic migrations for sqlite  https://review.openstack.org/60465309:57
AJaegerdtantsur: is metalsmith listed in openstack-manuals?10:00
dtantsurAJaeger: unlikely, but nor are many other small projects, I suspect10:01
AJaegerdtantsur: best to follow https://docs.openstack.org/doc-contrib-guide/doc-index.html10:01
*** Emine has joined #openstack-infra10:01
dtantsurAJaeger: oh, are all projects added there manually?10:02
AJaegerdtantsur: the latest is generated automatically from governance repo, and metalsmith was only recently added to governance. Might be we did not publish from openstack-manuals since then...10:03
AJaegerdtantsur: so, two things: To add to an index page, send change for openstack-manuals10:03
AJaegerdtantsur: and to create latest, wait until next merge of teh repo, we go over governance list of projects automatically10:04
dtantsurah, okay. I think the latter is enough10:04
AJaegerdtantsur: you can add metalsmith to the "Bare Metal subprojects" list on https://docs.openstack.org/rocky/projects.html as well10:05
AJaeger(for master -> stein release)10:05
AJaegerbbl10:06
dtantsurstein is not a thing yet10:06
AJaegerdtantsur: /latest will be stein10:07
dtantsurAJaeger: yeah, but you said it's auto-generated, no?10:07
*** e0ne has quit IRC10:08
*** janki is now known as janki|lunch10:14
*** jaosorior has joined #openstack-infra10:22
*** ssbarnea|bkp has joined #openstack-infra10:23
*** jiapei has quit IRC10:23
*** e0ne has joined #openstack-infra10:27
*** e0ne has quit IRC10:40
*** Bhujay has joined #openstack-infra10:40
AJaegerdtantsur: I mean  https://docs.openstack.org/latest/projects.html -> in prepartion for stein10:46
*** e0ne has joined #openstack-infra10:49
slaweqAJaeger: hi10:49
AJaegerhi slaweq...10:49
slaweqAJaeger: can You maybe put some job for autohold for me?10:49
slaweqAJaeger: I need to debug once again what I still miss in job neutron-tempest-plugin-dvr-multinode-scenario-zuulv3 that it is still not working properly10:50
slaweqit's in neutron-tempest-plugin repo10:50
AJaegerslaweq: no, cannot - you need an adin for that. Please ask "infra-root" - and feel free to leave a message here in general10:50
slaweqpatch https://review.openstack.org/#/c/578796/10:50
AJaegerinfra-root, can you help slaweq, please? ^10:50
slaweqok, sorry AJaeger, I though that maybe You can also do that :)10:51
AJaegerslaweq: no worries10:52
*** jcoufal has joined #openstack-infra10:56
*** e0ne has quit IRC10:57
fricklerslaweq: autohold set up, which ssh key should I use once the job finishes?11:00
*** Bhujay has quit IRC11:02
*** Bhujay has joined #openstack-infra11:05
*** Bhujay has quit IRC11:06
*** Bhujay has joined #openstack-infra11:07
*** Bhujay has quit IRC11:08
*** Bhujay has joined #openstack-infra11:08
*** Bhujay has quit IRC11:09
*** janki|lunch is now known as janki11:10
*** Bhujay has joined #openstack-infra11:10
*** jpena is now known as jpena|lunch11:10
*** Bhujay has quit IRC11:11
*** Bhujay has joined #openstack-infra11:11
openstackgerritMerged openstack-dev/pbr master: Fix typo in contribution instructions.  https://review.openstack.org/60128111:14
slaweqfrickler: thx, I just sent You my pub key in priv11:18
*** alexchadin has quit IRC11:20
*** udesale has quit IRC11:25
*** aidin has joined #openstack-infra11:36
*** quiquell|rover is now known as quique|rover|brb11:40
*** ansmith has quit IRC11:43
*** quique|rover|brb is now known as quiquell|rover11:45
*** rh-jelabarre has joined #openstack-infra11:46
*** _ari_ has joined #openstack-infra11:56
*** alexchadin has joined #openstack-infra11:59
*** alexchadin has quit IRC12:05
*** gfidente|afk is now known as gfidente12:06
*** alexchadin has joined #openstack-infra12:06
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Add an option to skip generating RELEASENOTES.rst  https://review.openstack.org/58720412:06
openstackgerritStephen Finucane proposed openstack-dev/pbr master: docs: Add docs for reno integration  https://review.openstack.org/60475412:06
openstackgerritStephen Finucane proposed openstack-dev/pbr master: Remove my_ip from generated wsgi script  https://review.openstack.org/60132512:07
*** e0ne has joined #openstack-infra12:10
*** boden has joined #openstack-infra12:11
*** trown|outtypewww is now known as trown12:11
*** jamesmcarthur has joined #openstack-infra12:11
*** psachin has quit IRC12:13
*** jpena|lunch is now known as jpena12:14
*** graphene has joined #openstack-infra12:15
*** Bhujay has quit IRC12:15
*** tpsilva has joined #openstack-infra12:16
*** Bhujay has joined #openstack-infra12:16
*** rfolco has joined #openstack-infra12:17
*** aidin has quit IRC12:18
mordredAJaeger: if you have a second, I got a failure on an sdk patch that's similar to ones I've been getting a bunch recently: https://review.openstack.org/#/c/60462812:21
mordredAJaeger: specifically you can see openstack.tests.functional.network.v2.test_trunk.TestTrunk.test_subports failed with "StringException" - which usually means it was timed out12:22
AJaeger;(12:22
mordredI have timeout set to 9000 http://git.openstack.org/cgit/openstack/openstacksdk/tree/.zuul.yaml#n5912:23
mordredand the indiviaul test timeout set to 90: http://git.openstack.org/cgit/openstack/openstacksdk/tree/tox.ini#n3412:24
AJaegerlooking at http://logs.openstack.org/28/604628/2/check/openstacksdk-functional-devstack-tips/bfabb8e/job-output.txt.gz#_2018-09-23_18_24_01_66669812:24
AJaegerand the killed 2 lines below, I see 109 seconds12:25
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: rewrite interface in react  https://review.openstack.org/59160412:25
AJaegerso, that looks like your individual test timeout of 90 - doesn't it?12:25
mordredAJaeger: oh! so it does12:25
AJaegermordred: sorry, no expert in those...12:25
AJaegermordred: please double check!12:25
mordredyeah - I think you're right ... thanks, that's a good place to look12:26
mordredalso - I have a lot of those "resetting dropped connection" - I might also need to look in to what I need to do to not have keepalives going12:26
AJaegerconfig-core, could you review https://review.openstack.org/604679 and https://review.openstack.org/604608, please?12:26
AJaegermordred: yes, worth doing12:27
*** jamesmcarthur has quit IRC12:27
*** jamesmcarthur has joined #openstack-infra12:28
*** aidin has joined #openstack-infra12:30
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: rewrite interface in react  https://review.openstack.org/59160412:33
*** jamesmcarthur has quit IRC12:33
dhellmannAJaeger , pabelanger : thanks!12:33
*** sambetts has joined #openstack-infra12:36
*** kgiusti has joined #openstack-infra12:38
*** Bhujay has quit IRC12:39
*** Bhujay has joined #openstack-infra12:40
*** yumiriam has joined #openstack-infra12:40
*** agopi has quit IRC12:41
*** Bhujay has quit IRC12:41
*** Bhujay has joined #openstack-infra12:41
*** jaosorior has quit IRC12:42
*** jcoufal has quit IRC12:43
*** alexchadin has quit IRC12:43
*** jaosorior has joined #openstack-infra12:44
*** udesale has joined #openstack-infra12:46
*** jiapei has joined #openstack-infra12:47
*** jamesmcarthur has joined #openstack-infra12:51
*** yumiriam has quit IRC12:52
*** alexchadin has joined #openstack-infra12:53
*** icey3 has joined #openstack-infra12:58
*** jrist has joined #openstack-infra12:59
*** icey3 has quit IRC13:01
*** ansmith has joined #openstack-infra13:02
*** bobh has joined #openstack-infra13:02
*** agopi has joined #openstack-infra13:04
*** yamamoto has quit IRC13:07
*** yamamoto has joined #openstack-infra13:08
*** mriedem has joined #openstack-infra13:09
mriedemdoes putting a project in required-projects in a job definition make zuul-cloner clone that repo before devstack/grenade ever run?13:13
mriedemlike http://logs.openstack.org/54/604454/4/check/neutron-grenade/9883195/job-output.txt.gz#_2018-09-22_19_12_52_39170413:13
*** smarcet has joined #openstack-infra13:13
AJaegermriedem: zuul-cloner is Zuul v2 and needs a list of repos to clone. It can only clone repos in required-projects.13:14
mriedemb/c i'm getting this when trying to git clone openstack/placement in a grenade run http://logs.openstack.org/54/604454/4/check/neutron-grenade/9883195/logs/grenade.sh.txt.gz#_2018-09-22_20_08_04_79913:14
mriedemAJaeger: yeah if it clones the repo that's fine,13:15
mriedembut i'm not sure where it is cloning it to13:15
mriedemb/c for grenade we need it to be cloned to /opt/stack/new13:15
AJaegerwith zuul legacy jobs ,you need to add to required-projects *and* update the zuul-cloner invocation.13:15
mriedemi've got the former,13:17
*** agopi is now known as agopi|ansible|tr13:17
mriedemnot sure what the latter is,13:17
mriedemis it this? http://git.openstack.org/cgit/openstack/neutron/tree/playbooks/legacy/neutron-grenade/run.yaml#n1013:17
*** lbragstad has joined #openstack-infra13:18
*** aidin has quit IRC13:19
*** sthussey has joined #openstack-infra13:21
mriedemi think i see what was missing,13:29
mriedemstill need to list openstack/placement in the d-g PROJECTS variable to get it cloned into /opt/stack/new/13:30
AJaegermriedem: I think so - but those legacy jobs are old,  I would rather touch the Zuul v3 ones. Unfortunately grenade is not ready yet.13:33
fungiinfra-root: we may want to keep an eye on these and prod for a sru importing the fixes from debian if they drag on: https://people.canonical.com/~ubuntu-security/cve/2018/CVE-2018-16947.html https://people.canonical.com/~ubuntu-security/cve/2018/CVE-2018-16948.html https://people.canonical.com/~ubuntu-security/cve/2018/CVE-2018-16949.html13:33
fungiand if they don't patch the versions in trusty, we might want to consider accelerating the replacement of those servers with xenial sooner than some others13:34
*** icey has quit IRC13:34
*** SteelyDan is now known as dansmith13:34
*** e0ne has quit IRC13:38
*** jamesmcarthur has quit IRC13:40
fungimmm, another wave of wikispam today. just mass-deleted several hundred pages created by half a dozen new accounts13:45
slaweqfrickler: this job which You hold for me failed already: http://logs.openstack.org/96/578796/23/check/neutron-tempest-plugin-dvr-multinode-scenario-zuulv3/f51be2a/13:46
slaweqfrickler: can You add there my ssh key?13:46
fungithe usual scam phone numbers in search engine results tactic, this time advertising prescription drugs13:46
slaweqmordred: hi, or maybe You can add it if frickler is not around? ^^ :)13:46
fungioh, also a smattering of help desk scam phone numbers13:46
*** e0ne has joined #openstack-infra13:47
*** joabdearaujo has quit IRC13:48
*** edmondsw has joined #openstack-infra13:49
mordredslaweq: ips are 38.108.68.22 and 38.108.68.4113:52
mordredadding keys13:52
slaweqmordred: thx a lot :)13:52
mordredslaweq: k - you should be good on both13:53
mordredand you're welcome!13:53
slaweqmordred: thx13:53
openstackgerritMatthew Thode proposed openstack-infra/openstack-zuul-jobs master: add Gentoo jobs and vars and also fix install test  https://review.openstack.org/60243913:54
*** yamamoto has quit IRC13:54
*** yamamoto has joined #openstack-infra13:55
*** yamamoto has quit IRC13:55
*** yamamoto has joined #openstack-infra13:55
*** jaosorior has quit IRC13:55
*** yamamoto has quit IRC13:55
*** bdodd has joined #openstack-infra13:55
*** beekneemech is now known as bnemec13:57
openstackgerritMatthew Thode proposed openstack-infra/openstack-zuul-jobs master: add Gentoo jobs and vars and also fix install test  https://review.openstack.org/60243913:58
*** agopi|ansible|tr is now known as agopi|ansible13:58
openstackgerritMerged openstack-dev/cookiecutter master: Update the outdated URL in README.rst  https://review.openstack.org/60459813:59
*** ykarel is now known as ykarel|away14:03
*** dhill_ has quit IRC14:06
*** dhill_ has joined #openstack-infra14:06
*** chkumar|ruck has quit IRC14:08
*** icey has joined #openstack-infra14:09
*** jamesmcarthur has joined #openstack-infra14:12
*** chandankumar has joined #openstack-infra14:14
*** ramishra has quit IRC14:14
*** chandankumar is now known as chkumar|ruck14:15
*** quiquell|rover is now known as quique|rover|off14:15
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: added public endpoint to get summit by id  https://review.openstack.org/60480114:30
*** roman_g has quit IRC14:32
*** roman_g has joined #openstack-infra14:32
openstackgerritMerged openstack-infra/openstackid-resources master: added public endpoint to get summit by id  https://review.openstack.org/60480114:33
corvusfungi: happily we don't run the butc openafs process14:36
fungiawesome, that was the one i was most worried about out of the three, but wasn't 100% sure how to confirm14:36
*** dpawlik has quit IRC14:37
fungithe second one is mildly worrying if it can leak process memory which contains some credential leading to compromise of the service, but reading through the implication is that's nontrivial to exploit usefully14:37
*** dulek has quit IRC14:38
fungithe third one is really just a risk that some bored script kiddie decides it would be fun to knock the service offline14:38
*** dulek has joined #openstack-infra14:39
*** dpawlik has joined #openstack-infra14:40
*** dpawlik has quit IRC14:40
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: remove job settings for ironic repositories  https://review.openstack.org/59247214:43
*** armax has joined #openstack-infra14:45
*** eernst has joined #openstack-infra14:46
*** dave-mccowan has quit IRC14:50
*** aojea has quit IRC14:50
ssbarneadid anyone over here ever created openstack.atlassian.net instance? I wanted to register it for some testing but it seems to be already taken.14:58
*** chkumar|ruck is now known as chkumar|off15:02
funginot to my knowledge15:03
fungialso, heading out to run some lunch errands, but should be back in not too long15:03
*** Bhujay has quit IRC15:07
dhellmannfungi : it looks like our pypi upload job is reporting failures even when it can upload the wheel - http://logs.openstack.org/6e/6e3778dea94f5b3b215706042c32c2125d0a2ce9/release/release-openstack-python/69d77c6/ara-report/result/68f058f3-f727-40f9-bf7e-d3ba16ddbc1f/15:08
dhellmannI wonder if the legacy API is reporting errors to try to move folks to a newer one?15:11
dhellmannI have at least 3 examples of that failure and it looks like several more15:12
mordredssbarnea: I certainly have not. I feel like back in the day there was a foundation-sponsored metrics project (bitergia) that for reasons I don't understand used confluence for something15:12
*** dave-mccowan has joined #openstack-infra15:12
mordredssbarnea: so it's concievable that someone at the foundation registered that15:12
mordredjbryce: ^^ maybe you know?15:12
*** yamamoto has joined #openstack-infra15:13
ssbarneafungi: yep, I had two remove the old legacy pypi upload address to make it upload packages again (no openstack infra). If I remember well solution was to remove the URL completly to make it work, from the config.15:14
*** gsanso_ has joined #openstack-infra15:16
*** dave-mccowan has quit IRC15:17
mordredso maybe we should prioritize getting to the new api?15:17
dhellmanndoes twine not use that automatically?15:19
dhellmannssbarnea : when you say "remove the old legacy pypi upload address" where did you remove it from?15:20
dhellmannand what value did you replace it with?15:21
ssbarnea|bkpdhellmann: nothing, twine default behavior is to use correct url, but you need to avoid creating a config file. put credentials in env vars and it will work. i may be able to find the bug that caused this.15:22
dhellmannssbarnea : that would be great, thanks15:22
mordreddhellmann: we're setting15:22
mordredit15:22
mordredrepository_url: https://test.pypi.org/legacy/15:22
*** weshay_afk is now known as weshay15:23
dhellmannwhy are we using the test server?15:23
mordredoh - wait - nevermind - that's in our test pypi secret15:23
mordredwe do not set that in production15:23
*** dave-mccowan has joined #openstack-infra15:23
ssbarnea|bkpsounds very similar to my issues, had the same both production and testing, i had to drop testing and started to publish only on production, both releases and prereleases.15:23
mordredok, I'm satisfied that we should not be writing a repository url into our pypirc files15:24
mordredwe are writing index-servers = pypi and then [pypi]\nusername:{{ username }}\npassword:{{ password }}15:25
mordredoh - wait15:25
mordredssbarnea|bkp: you're saying that we can't put credentials into file and have this work?15:26
*** jesusaur has quit IRC15:26
mordredthat's ... exciting15:26
ssbarnea|bkpmordred not sure about that, you need to test. removing url from config may be enough.15:26
*** smarcet has quit IRC15:28
*** tbachman has joined #openstack-infra15:29
*** gagehugo has quit IRC15:29
tbachmanquestion for process/procedure to the channel — if I need to delete a tag (carelessly applied against the wrong branch) in an upstream repo, is there a process for doing this?15:30
*** jesusaur has joined #openstack-infra15:31
mordredssbarnea|bkp: ok. we shouldn't be setting it - but obviously something is unhappy somewhere :)15:32
mordredssbarnea|bkp, dhellmann: we could also just put https://upload.pypi.org/ in as our repository_url15:32
ssbarneamordred: i found https://github.com/pypa/twine/issues/289 but not much else. I didn't had a rc file on CI (travis) but had one locally. Just remove "repository=" and it will start working.15:32
*** gagehugo has joined #openstack-infra15:33
ssbarneai checked my local file and is commented too15:33
mordredssbarnea: ok - so like this? http://paste.openstack.org/show/730640/  (also, is username: really a thing?)15:34
dhellmannmordred : this feels like something we should not set, if the PyPA folks are updating twine in advance of breaking changes to the API15:34
*** gsanso_ has quit IRC15:34
mordreddhellmann: yes. I agree- I would definitely like to not set it15:35
ssbarnea|bkpmordred: yep your file looks correct, almost identical with my local copy.15:36
mordredcool. this: https://github.com/pypa/twine/blob/cdd5b2d602d8966fac5ecc127e26c5095360b001/tests/test_utils.py#L28-L47 seems to be their test that such a setup should result in utils.DEFAULT_REPOSITORY in repository15:36
ssbarnea|bkptwine issues are related to pypi CDN migration that happened few months bach, not sure if is their fault.15:37
*** efried has quit IRC15:37
*** dpawlik has joined #openstack-infra15:38
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Use = not : for ini settings in pypirc  https://review.openstack.org/60482015:39
*** efried has joined #openstack-infra15:39
mordredno clue if that has any impact at all15:39
*** dpawlik has quit IRC15:42
dhellmannI have a long list of the releases that seem to have failed15:44
dhellmannI'll put those into an etherpad so we can redo them15:45
dhellmannmordred : did you have a patch to not set the URL? or do we not need that?15:45
mordreddhellmann: we shouldn't need it - we shoudln't be setting the url currently15:45
dhellmannoh, so then we don't know why the job is failing?15:46
mordrednope - nor why it's trying legacy.pypi.org15:46
mordredhttp://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/secrets.yaml#n60 is the secret definition15:46
dhellmannhmm15:46
mordredcompared to http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/secrets.yaml#n42 which is the test_pypi_secret which shows setting a url15:47
mordredand then the defaults for the role: http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/upload-pypi/defaults/main.yaml15:47
*** alexchadin has quit IRC15:47
mordredand the pypirc jinja template: http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/upload-pypi/templates/.pypirc.j215:47
openstackgerritClark Boylan proposed openstack-infra/glean master: Add option to ignore config drive interfaces info  https://review.openstack.org/60419315:53
clarkbprometheanfire: mordred ianw ^ now with testing15:53
clarkbI'm going to see if I can get the integration tests to do a one off test of that for us too15:53
prometheanfirecool15:54
mordred++15:54
mordredclarkb: the patch seems sensible15:54
dhellmannthere's an unfortunately long list of releases to re-enqueue https://etherpad.openstack.org/p/DCFL3JyTQH15:55
prometheanfireya, oslo.messaging-8.1.1 wasn't in pypi15:55
dhellmannoh, so some of them aren't being uploaded?15:56
dhellmannsome were15:56
dhellmannat least I thought so15:56
openstackgerritClark Boylan proposed openstack-infra/nodepool master: Do Not Merge Testing glean + nodepool integration  https://review.openstack.org/60482615:56
clarkbmordred: ^ I think that will test the nodepool piece with glean that ianw has been working on (we have to be able to set the configuration properties on the nodepool side)15:57
clarkbmordred: fwiw I don't know if the instance metadata keys go in the meta dict or if they are top level15:57
clarkbI guess I should sort that out next15:57
clarkbtbachman: not sure if you got an answer yet, but there isn't really a great answer for that as tags don't update on pull15:58
clarkbtbachman: so you can't expect the tag to actually move for all of your downstream consumers15:58
tbachmanclarkb: thx for the response! I acknowledge this is a mess for downstream consumers.16:00
dhellmanntbachman : we usually just tag again in this case (using a new version number)16:01
tbachmanI’m left with the worse of two evils: keep the tag, which is incorrectly applied against a branch, or delete it and deal with anything downstream16:01
*** e0ne has quit IRC16:01
tbachmandhellmann: thx for the reply! Let me see if there’s a way I can make creating a new tag work. The problem is that the tag has a higher semantic version than what’s usually used for this branch. That said, let me try some things to see if I can make it work.16:02
AJaegerconfig-core, could you review https://review.openstack.org/604679 and https://review.openstack.org/604608, please?16:03
clarkbAJaeger: looking16:03
mordredclarkb: I think they go into the meta dict16:04
AJaegerthanks, clarkb16:04
mordredoh - wait - of what?16:05
AJaegerclarkb: could you also review corvus's change to add cherrypy to 3rdparty-ci, please? https://review.openstack.org/#/c/600846/16:06
dhellmannmordred : this looks suspicious, is this the template? http://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/templates/pypirc.erb#n616:06
AJaegerbbl16:07
clarkbAJaeger: done. That should be safe because it isn't in any pipelines yet so we won't spam them16:07
clarkb(also I think they may need to set up the application on the github side to get the real spam going16:07
mordreddhellmann: hrm. it shouldn't be - that should only have impact if puppet ran somewhere, and we don't run puppet on our nodepool nodes anymore16:07
mordredwe should go ahead and remove that line though16:07
clarkbmordred: ya I'm booting a new test instance in ovh to double check16:07
*** agopi|ansible is now known as agopi|training16:08
mordredclarkb: sorry - if we were talking about the server dict from openstacksdk, the metadata should be found in the metadata key16:09
clarkbmordred: this is the meta_data.json file in the config drive16:09
mordredyeah - there it looks like it's in the meta key16:10
*** lbragstad has quit IRC16:10
mordredlooking at a node nodepool booted16:10
clarkband I forgot to boot it with a config drive16:10
mordred\o/16:12
openstackgerritMerged openstack-infra/project-config master: Move storyboard cover job in-repo  https://review.openstack.org/60467916:12
clarkbmordred: yup it goes in the 'meta' dict, that means my change to glean should now have valid testing16:14
slaweqmordred: You can destroy "my" nodes now16:15
slaweqmordred: thx a lot once again16:15
openstackgerritMerged openstack-infra/project-config master: Remove extra branches attributes  https://review.openstack.org/60460816:15
openstackgerritMerged openstack-infra/project-config master: Add cherrypy to third-party ci  https://review.openstack.org/60084616:15
*** lbragstad has joined #openstack-infra16:18
dhellmannmordred : so, how do i debug this? it looks like we're using a role defined way down in zuul-jobs, so any debugging would have to land and then I would have to run the job, right?16:19
*** janki has quit IRC16:20
dhellmannhttps://github.com/pypa/twine/issues/403 makes me think it's a twine bug?16:21
dhellmannah, and twine 1.12.0 was release today16:23
dhellmannyay16:23
clarkbdhellmann: coffee time?16:25
clarkbwe can probably pin twine if we have to16:25
*** spotz is now known as spotz_16:25
clarkbor just wait for a fix and release16:25
*** spotz_ is now known as spotz16:25
dhellmannit appears that the pypi uploads actually worked, but our tarball uploads failed16:25
dhellmannI don't remember why those are happening after the pypi uploads16:26
openstackgerritMatt Riedemann proposed openstack-infra/elastic-recheck master: Add query for cinder test_retype_multiattach bug 1794143  https://review.openstack.org/60483616:26
openstackbug 1794143 in Cinder "cinder.tests.unit.volume.test_volume_retype.VolumeRetypeTestCase.test_retype_multiattach intermittently fails" [Medium,Confirmed] https://launchpad.net/bugs/179414316:26
dhellmannclarkb : so if there's some way to run the tarball uploads from your side, I think I can do the announce and constraint updates16:27
dhellmannand then we can just not release anything else today and see if twine is fixed16:27
clarkbwe probably download the tarball from pypi, check the hash then put that on tarballs.o.o?16:27
dhellmannI don't know how that step works16:27
dhellmannI would expect us to sign and upload the thing we just uploaded to pypi but maybe you're right16:28
clarkbya I mean for after the fact fixing now16:28
dhellmannoh, yeah16:28
clarkbI want to say those artifacts are ephemeral until we upload them to pypi and tarballs so if we want a tarball that matches the one on pypi we need to publish the one that is there16:28
*** icey has quit IRC16:29
dhellmannyes, you're right about that16:29
dhellmannsorry, I thought you were asking about the job16:29
dhellmannnear the top of https://etherpad.openstack.org/p/DCFL3JyTQH there are some lists of the things that exist on pypi but not tarballs.o.o16:29
bnemecHey, can someone add me to https://review.openstack.org/#/admin/groups/1946,members ?16:30
clarkbbnemec: done16:30
bnemecclarkb: Thanks!16:30
*** aidin has joined #openstack-infra16:30
clarkbhrm we don't upload the signatures to pypi do we. Preserving those may make this more complicated16:32
*** icey has joined #openstack-infra16:33
pabelangerno, we opted not to becase they don't use them any more16:33
*** aidin has quit IRC16:35
clarkbfungi: ^ are we going to have to download from pypi, sign with release key, then push to tarballs in this case?16:37
*** dave-mccowan has quit IRC16:38
clarkbdhellmann: in the list of missing files karbor is only missing a signature file and not the tarball?16:43
clarkbthinking about this more I wonder if we want to sign these files with the CI system if we are pulling from pypi. Harder to verify the CI system actually generated them16:43
*** jiapei has quit IRC16:43
fungiclarkb: dhellmann: in the past we've just lived with some of the release artifacts lacking signatures. this was intentionally designed around previously when we used multiple jobs since we could have a dependent job retrieve the artifacts from tarballs.o.o, check their signatures, then upload the retrieved artifacts to pypi. with the push to combine all this into fewer jobs we lost those safety16:44
fungichecks16:44
fungiclarkb: dhellmann: also looks like they started work on releasing twine 1.12.1 around 16:30z16:48
fungihttps://github.com/pypa/twine/pull/40616:49
*** jpich has quit IRC16:50
*** trown is now known as trown|lunch16:50
*** diablo_rojo has joined #openstack-infra16:51
clarkbfungi: in that case just upload the tarballs on pypi to tarballs.o.o without signatures?16:54
clarkbI also wonder if we should flip the order of operations here so that things we control happen before things we don't control16:54
*** hashar is now known as hasharDinner16:55
openstackgerritFabien Boucher proposed openstack-infra/zuul master: WIP - Pagure driver  https://review.openstack.org/60440416:55
*** rkukura has quit IRC16:56
fungiyes, i think we should16:57
fungi(on both)16:57
*** hamzy_ has joined #openstack-infra16:58
fungiusing the old order it was simpler since we could take the artifacts we already published to tarballs.o.o and manually upload them to pypi if necessary16:58
*** derekh has quit IRC16:59
*** hamzy has quit IRC16:59
*** dpawlik has joined #openstack-infra17:00
clarkbya and we could rerun the job if not already on pypi17:01
clarkbbut pypi will error if there is already a package there17:01
fungirerunning individual jobs is also nontrivial17:02
fungiwe can reenqueue tag refs and rerun _all_ the jobs for them17:03
clarkbreading the job defs we use a list of post run playbooks the first of which is the tarballs upload playbook then the pypi upload playbook. I thought that we ran those in order17:04
clarkbnow I'm extra confused over why we don't have things on tarballs.o.o17:05
clarkboh wait we don't actually upload to tarballs in this jobs?17:07
fungiare we still talking about http://logs.openstack.org/6e/6e3778dea94f5b3b215706042c32c2125d0a2ce9/release/release-openstack-python/69d77c6/ or some other build?17:09
clarkbfungi: that is what I am looking at17:09
clarkbbasically where do we attempt to publish to tarballs.o.o? that can't be part of the twine playbook can it? (maybe it is17:09
*** udesale has quit IRC17:10
fungilooks like it would have happened in openstack-artifacts.yaml but was scheduled after pypi.yaml and so its tasks were skipped when the predecessor failed17:10
*** sshnaidm has joined #openstack-infra17:11
fungithe publish-artifacts-to-fileserver role17:12
clarkbah it is the behavior of post run ordering of parents17:12
clarkbthe job order is like a matryoshka doll17:13
clarkbs/job/play/17:13
*** sshnaidm is now known as sshnaidm|pto17:13
*** bobh has quit IRC17:14
clarkbnow to understand why we skip the subsequent playbook there instead of running it17:14
clarkb(Ithought post always ran?)17:14
clarkbwhen: zuul_success | bool17:15
dhellmannthose steps check if the job is failing17:15
*** bobh has joined #openstack-infra17:15
dhellmannyeah17:15
clarkbthe mismatch here then is we don't want to publish tarballs if pypi fails to publish17:16
clarkbbut twine is confusing things17:16
dhellmannwhen I'm done with lunch I will work out how to run just the announce and constraint jobs for those releases17:16
dhellmannyeah17:16
dhellmannis there any way to change the order of those steps without rebuilding the whole job?17:16
clarkbdhellmann: no I think zuul hardcodes that matryoshka behavior of the inherited pre and post plays17:18
clarkbmaybe this is a reason to go back to the upload to tarballs job then download and push to pypi job split out that we had before?17:18
*** jamesmcarthur has quit IRC17:19
*** sshnaidm|pto has quit IRC17:20
*** rkukura has joined #openstack-infra17:20
*** bobh has quit IRC17:20
*** jpena is now known as jpena|off17:21
dhellmannhmm17:21
openstackgerritMerged openstack-infra/nodepool master: Add instance boot properties  https://review.openstack.org/60424817:23
*** anteaya has joined #openstack-infra17:25
*** smarcet has joined #openstack-infra17:25
*** tosky has quit IRC17:30
*** sshnaidm|pto has joined #openstack-infra17:33
*** dave-mccowan has joined #openstack-infra17:35
*** jamesmcarthur has joined #openstack-infra17:35
*** harlowja has joined #openstack-infra17:36
*** sambetts is now known as sambetts|afk17:39
*** jamesmcarthur has quit IRC17:40
clarkbdhellmann: smcginnis: should I go ahead and start copying tarballs from pypi to tarballs and not bother with the signature files?17:45
dhellmannclarkb : yes, please17:45
dhellmannI'm working on the script to run the announce and constraint updates17:45
dhellmannbut I have a meeting starting in a bit so it may take me a while17:46
clarkbdhellmann: do we also copy the wheels to tarballs?17:46
clarkbfor example os-net-config only has a .whl file17:47
clarkbwe must upload .whl before the .tar.gz?17:47
clarkbnow the question becomes do we want to generate sdists by hand where they are missing?17:48
dhellmannoh, I didn't check that17:48
dhellmannugh17:48
* dhellmann makes a note to always run a test release before doing any real releases17:49
dhellmannI wonder if it would be easier to just re-tag all of these things17:50
dhellmannis there some way to pin the version of twine we use?17:50
mordreddhellmann: yah - we install it in the ensure-twine role17:50
*** jamesmcarthur has joined #openstack-infra17:51
clarkbthat is in zuul-jobs17:51
mordreddhellmann: it might, in fact, be easier to just do a patch-level version bump - but I don't know if that's the rightchoice17:51
*** dpawlik has quit IRC17:51
*** trown|lunch is now known as trown17:52
fungithe fix was merged to twine a couple hours ago. i was hoping they'd have merged the 1.12.1 release pr shortly thereafter17:52
*** jamesmcarthur has quit IRC17:53
*** bobh has joined #openstack-infra17:53
dhellmannmordred : I think it probably will be. I'll start working on that patch now and then we can figure out how to do the right thing with twine17:54
mordredfungi: so we want either 1.11 or 1.12.1 but not 1.12.0 right?17:55
clarkbmordred: yes17:55
fungicorrect17:55
mordredfungi: we could just directly put "twine!=1.12.0" in the role17:55
fungiyes17:55
mordredand that should pin us to 1.11 until 1.12.1 is out17:55
fungiprecisely17:55
*** dtantsur is now known as dtantsur|afk17:56
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Block twine 1.12.0 when we install it  https://review.openstack.org/60486217:57
*** harlowja has quit IRC17:58
fungii don't suppose you need to quote the parameter when passing it in ansible like that?17:58
fungithe ! isn't going to get interpreted by the shell it's passed off to, or it will get quoted by ansible itself?17:58
*** akhila has joined #openstack-infra17:58
clarkbfungi: I typed != into bash and and it say != command not found17:59
clarkbso I think that is ok17:59
fungicool17:59
clarkband yaml should be fine with the ! as long as it isn't the first character in the string?17:59
fungii know > and < in pip install versions tend to make it ornery17:59
mordredyah- I tried the pip install command locally in the shell and it worked17:59
fungiawesome18:00
clarkbyou need a space after ! for it to be parsed as negation by bash looks like18:00
clarkb!false doesn't work but ! false does18:00
openstackclarkb: Error: "false" is not a valid command.18:00
fungimeetbot agrees18:02
AJaegerI have a docs update for bindep - anybody wants to review it, please? https://review.openstack.org/59975518:04
*** e0ne has joined #openstack-infra18:06
*** ediardo has joined #openstack-infra18:07
openstackgerritClark Boylan proposed openstack-infra/nodepool master: Add release note for instance-properties config  https://review.openstack.org/60486318:10
*** coreycb has joined #openstack-infra18:14
openstackgerritAndreas Jaeger proposed openstack-infra/bindep master: Update docs building  https://review.openstack.org/59975518:16
AJaegerfungi: addressed your comment ^18:16
*** dpawlik has joined #openstack-infra18:17
*** dpawlik_ has joined #openstack-infra18:17
prometheanfireAJaeger: I think the gentoo stuff 'works' now, just waiting on multinode fedora18:18
fungithanks AJaeger!18:18
*** dpawlik has quit IRC18:18
clarkbcorvus: when you have a second we should do a quick chat about the state of zuul triggering ansible on bridge.o.o18:19
*** bobh has quit IRC18:20
*** bobh has joined #openstack-infra18:20
prometheanfireone thing that may want to be looked at is adding a binhost/mirror for binkpkgs to save on gate time18:20
clarkbprometheanfire: that is precompiled packages for gentoo?18:21
prometheanfireya18:21
prometheanfireI do that for my home infra18:21
prometheanfireand https://review.openstack.org/604268 does it for DIB18:21
prometheanfirewould just have to have a job that compiled some of the bigger programs and stored it in a mirror location18:22
AJaegerfungi, thanks!18:22
prometheanfirehttps://wiki.gentoo.org/wiki/Binary_package_guide18:22
fungiAJaeger: thanks for fixing it up!18:22
corvusclarkb: how about after lunch?18:25
clarkbcorvus: sure18:26
prometheanfireclarkb: in order to do it though we'd need a list of packages and a periodic job type setup18:26
clarkbprometheanfire: left some notes, not sure I understand enough of gentoo packaging to actually know why you've made those choices18:26
*** jamesmcarthur has joined #openstack-infra18:32
prometheanfireianw: clarkb: would you rather me clean it up now or later?18:32
*** sshnaidm|pto has quit IRC18:36
*** ykarel|away has quit IRC18:39
prometheanfiretoo late18:39
openstackgerritMatthew Thode proposed openstack/diskimage-builder master: clean up gentoo overlay provisioning  https://review.openstack.org/60487318:39
dhellmannmordred, fungi, clarkb : it looks like twine 1.12.1 is out now. I have a patch up to run a release test. Should I wait for any other changes to land before approving that?18:48
mordreddhellmann: I don't think you should need to - each job installs twine from scratch18:49
dhellmannmordred : ok, thanks18:50
*** ianychoi_ is now known as ianychoi18:54
*** panda has quit IRC18:56
*** yumiriam has joined #openstack-infra18:58
*** jmorgan1 has quit IRC19:06
mwhahahaclarkb: did we ever get caching stats from the mirrors recorded somewhere? we're trying to track down why container fetching sometimes takes longer than other runs19:06
clarkbmwhahaha: no, I pushed my scrupt up for basic numbers but ut isnt publishing anywhere19:08
*** anteaya has quit IRC19:08
fungii guess i or someone can run that over current logs and put up a paste of the results?19:10
*** jmorgan1 has joined #openstack-infra19:11
*** tosky has joined #openstack-infra19:11
*** shardy has quit IRC19:12
*** pcaruana has quit IRC19:17
*** jamesmcarthur has quit IRC19:27
slaweqclarkb: hi, can You help me maybe with network config for dvr multinode job?19:30
slaweqclarkb: I was trying to do what we discussed few days ago19:30
clarkbslaweq: sure I'm eating lunch right now thoufh19:30
slaweqclarkb: ok, please ping me when You will have some time later :)19:30
slaweqclarkb: bon appetit :)19:31
*** jamesmcarthur has joined #openstack-infra19:32
openstackgerritMichael Johnson proposed openstack-infra/project-config master: Adding openstack/octavia-lib project  https://review.openstack.org/60488919:33
*** jamesmcarthur_ has joined #openstack-infra19:33
openstackgerritMichael Johnson proposed openstack-infra/project-config master: Adding openstack/octavia-lib project  https://review.openstack.org/60488919:35
*** jamesmcarthur has quit IRC19:37
*** panda has joined #openstack-infra19:37
corvusclarkb: o/19:37
clarkbcorvus: Im finishing uo lunch about 10-15 minutes away19:38
corvusk19:38
*** graphene has quit IRC19:49
*** graphene has joined #openstack-infra19:50
*** harlowja has joined #openstack-infra19:50
*** dpawlik_ has quit IRC19:51
clarkbcorvus: ok the first thing is that nil nodeset jobs are weird. We don't capture the inventory (useful for the zuul vars and related data) and we don't remove the master key from the ssh agent because the ssh setup role runs against all.19:51
clarkbcorvus: the other thing is that bridge.o.o doesn't allow root login over ssh generally. You have to ssh from one of the two bastions for that to work19:52
clarkbfor the first thing we should probably sort out how we want to deal with nil nodeset jobs (maybe a new base job?) for the bridge ssh problem maybe we should add a zuul user to bridge.o.o?19:52
clarkbfungi: if you get a chance to run that it would probably be helpful19:53
fungiclarkb: where was the script? is there a review containing it or something?19:54
*** dpawlik has joined #openstack-infra19:55
*** brwyatt_ has joined #openstack-infra19:56
*** dpawlik has quit IRC19:56
clarkbfungi: system-config/modules/openstack_project/files/mirror/cache-stats.sh19:57
clarkbit merged looks like19:57
fungioh, excellent. thanks!19:58
*** njohnston has joined #openstack-infra19:59
*** brwyatt_ is now known as brwyatt19:59
clarkbmordred: what is the current state of group membership configuration in ansible? I need to rebase https://review.openstack.org/#/c/591017/1 to modify groups for etherpad servers and don't want to end up conflicting again19:59
fungii have noted the upgraded configdrive/metadata problem with recent servers i've booted in my personal ovh account too. cloud-init similarly acts on them, but if i go in and replace the cloud-init recorded static addressing and routing with auto dhcp then everything just works20:00
fungii'll probably add some ovh-specific cloud-init overrides in userdata to work around it at first boot20:00
njohnstonHi!  Quick question - I am working on setting up a job that will run some neutron tests in zuul using a Fedora 28 image, and it looks like something (probably devstack-gate) assumed that `yum` will be assumed on RPM-based systems.  But the dnf-yum package is not installed so it looks like there is no yum executable.20:04
njohnstonIs this something where it would be better to add to the image, or add something to install yum in devstack-gate or whatever is the right place in the toolchain?20:05
njohnstonerror in question: http://logs.openstack.org/30/600530/8/check/neutron-tempest-iptables_hybrid-fedora/f21b11a/job-output.txt.gz#_2018-09-24_17_27_49_17013220:06
clarkbslaweq: I'm around the computer now to talk networking things20:06
slaweqclarkb: thx20:07
clarkbnjohnston: new jobs shouldn't use devstack-gate, it is basically on life support to support the existing legacy jobs that we had pre zuulv320:07
clarkbnjohnston: devstack itself should use dnf iirc20:07
slaweqclarkb: basically I did something related to what we discussed few days ago in this dvr multinode job20:07
slaweqclarkb: I had to propose additional patch https://review.openstack.org/#/c/604285/ to create patch ports between br-ex and br-infra20:08
slaweqclarkb: I also changed devstack config that it will configure public_gateway_ip on br-infra bridge instead of br-ex directly, it's in https://review.openstack.org/#/c/578796/20:09
slaweqclarkb: but even with that this job failed again20:09
slaweqclarkb: but now I just found that maybe BUILD_TIMEOUT was too short there so I'm trying again20:09
slaweqclarkb: so for now only thing I would like from You is to take a look at https://review.openstack.org/#/c/604285/ and tell me if such change makes any sense for You :)20:10
clarkbslaweq: ok, I'm not sure why the public_gateway_ip would need to be on br-infra instread of br-ex if the two are bridged together but otherwise that seems right20:10
clarkbslaweq: I think you should do that on the devstack side of things and not in the generic bridge role. The zuul-jobs roles are itnended to be generally useful and not encode openstack specific things20:10
slaweqclarkb: ok, so are You saying that it should be done directly in my job's definition somehow or in some devstack role (where I can find it?)20:12
clarkbslaweq: yes, because people using the zuul-jobs roles shouldn't be faced with neutron options. All of the devstack setup should be in devstack itself20:13
pabelangerclarkb: ianw: so, if we +3 https://review.openstack.org/#/c/585574/ for glean, do you think we can cut a release today? I am at a loss why nodepool booting fedora fails on glean, but manually it works20:13
clarkbslaweq: you probably want to add a play or role to devstack to configure the additional networking if dvr is enabled20:13
pabelangerunless, I am some how using different shade flags manually, then nodepool is20:13
*** dpawlik has joined #openstack-infra20:13
clarkbpabelanger: I need more context, but I would be very surprized if that fixed anything for you20:14
slaweqclarkb: ok, I will take a look where to add it then :)20:14
slaweqthx for explanation20:14
fungiclarkb: mwhahaha: mnaser: looks like for mirror.ca-ymq-1.vexxhost (the one in question in the slow job linked from #tripleo earlier) this is the cache breakdown over the past few days: http://paste.openstack.org/show/73066920:14
pabelangerclarkb: yah, I don't think it is the fix either. But happy to explain more.20:15
fungifairly high cache miss percentage on the port 8080 proxy20:15
*** agopi|training has quit IRC20:15
clarkbfungi: we should update the script to look at port 80 now since pypi is a proxy there too20:15
pabelangerclarkb: basically, trying to get fedora-minimal DIB to boot in vexxhost from nodepool, using glean, but for some reason doesn't properly setup the network. The bad part, if I manually boot a VM using shade, I don't have a failure with glean.20:16
fungii'll try that20:16
mwhahahawhich port is for which service?20:16
mnaserfungi: clarkb mwhahaha could we just be unlucky and not have a lot of hits lately..?20:16
clarkbmnaser: it could be due to new data (so not cached), expired data needing updates, data too big to be cached (I think our limit is like 5gb though which is huge)20:17
mnaserclarkb: it *seems* like this issue seems to be limited to our mirror server though20:17
mnaserbut i'm assuming things, mwhahaha can comment20:17
*** dpawlik has quit IRC20:18
fungilooks like the script will need a bit of refactoring to accommodate port 80 since it's not going to appear in the log filenames20:18
clarkbfungi: it shouldn't20:20
pabelangerclarkb: also, if I could figure out how to redirect glean.sh to console log for openstack, that should explain what is failing20:20
fungimwhahaha: the configuration is here: https://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/templates/mirror.vhost.erb20:20
clarkbfungi: ya just need to make that loop a function and pass in the file path as an arg probably20:21
mwhahahaso 8181 is docker bits20:21
clarkbmwhahaha: it depends on which version of docker, because well docker20:21
*** harlowja has quit IRC20:22
mwhahahawell from those stats, clearly we're not using 8282 :D20:22
pabelangerthere is also the issue that docker invalids the cache every 24 hours, for some reason20:22
clarkbya I think that was set up for loci as they are bleeding edge docker and they don't run many jobs20:22
clarkbpabelanger: we do that in apache20:22
pabelangerclarkb: right, but if we refersh, it gets reset doesn't it?20:22
clarkbpabelanger: it shouldn't be an issue apache will check if the file has changed since it cached it and reset the counter20:22
clarkbpabelanger: what are the chacnes you forgot to turn on config drive with your fedora problem?20:23
clarkbsince that is what breaks glean for me 99% of the time20:23
mwhahahathe docker stuff seems to be 88% hits which is not too bad20:23
pabelangerclarkb: will confirm again, that nodepool.yaml had it enabled20:23
mwhahahaso that would seem to point to transit to the mirror from the vms20:24
clarkbpabelanger: if you openstack server show the instance it will tell you if config drive is enabled or not, probably good idea to double check if it is actually set on that side20:24
pabelangerclarkb: docker expires, this is outside of our apache refresh however. They do something to their headers, regardless of blob age, force expire after 24 hours20:24
*** harlowja has joined #openstack-infra20:25
clarkbpabelanger: yes thats fine, all that means is apache has to check the validity of the data on the next request, it only redownloads it if the content has changed20:25
clarkbit uses an HTTP HEAD for that iirc20:25
pabelangerclarkb: okay, I'd have to look again, but think it did results of us missing a cache and redownloading20:26
pabelangerbut been a few months since I last looked20:26
clarkbpabelanger: in fungi's paste, http://paste.openstack.org/show/730669/, the cache refresh is this situation20:26
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Update cache-stats.sh to include the port 80 proxy  https://review.openstack.org/60489720:27
clarkbif the remote is down or if the data did update then we have to redownload. But since docker images are sha256sum identified we should only ever reconfirm the data is still valid and use what is cached20:28
pabelangerclarkb: yah, I guess for 8081, we'd need to check the last time tripleo create a new image, because if they are rotating them a lot, then I'd expect misses to be high, if they don't rebuild often, misses should be lower20:28
fungihttp://paste.openstack.org/show/730670 to include the port 80 proxy (per patch in 604897)20:28
clarkbpabelanger: yup20:28
pabelangerhttp://paste.openstack.org/show/725348/20:29
fungioh, i have an expansion error in there. fixing20:29
pabelangerthat was from my last round of testing20:29
pabelangersha256 blobs expire every 4 hours, sorry20:29
*** kgiusti has left #openstack-infra20:29
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Update cache-stats.sh to include the port 80 proxy  https://review.openstack.org/60489720:31
fungihttp://paste.openstack.org/show/730671 is the output from the latest patchset20:31
pabelangerclarkb: config_drive: True20:32
*** akhila has quit IRC20:32
openstackgerritClark Boylan proposed openstack-infra/system-config master: Add digit support to etherpad servers  https://review.openstack.org/59101720:33
*** sshnaidm|pto has joined #openstack-infra20:34
clarkbpabelanger: ok so not config drive, is this fedora 28 like what we are booting for openstack?20:34
pabelangerclarkb: I am doing boot-from-volume, so I guess I could try to snapshot or copy volume and look at logs20:34
pabelangerclarkb: no, not the same. I am using some of the same elements, -minimal, growroot, etc, but different20:35
pabelangerclarkb: this does work with centos-7 as the same elements as fedora-2820:36
pabelangerso, something on fedora side is different20:36
clarkbpabelanger: and how are you observing the failure? is it that nodepool can't ssh in? any idea if you can ping the instances?20:36
clarkbI wonder if it is a glean issue vs a services on the host issue20:37
*** e0ne has quit IRC20:37
pabelangerclarkb: yah, nodepool return NODE_FAILURE, I manually grab the console log via openstack client and see: [[0;1;31mFAILED[0m] Failed to start Glean for interface ens3.20:38
pabelangerbut glean doesn't output debug to console20:39
clarkbpabelanger: it does if you pass --debug I think20:39
clarkbyou could modify your simple-init init scripts to add that flag20:39
pabelangeryah, did, nothing sadly20:39
pabelangerI think systemd blocks it20:40
pabelangerlet me confirm20:40
*** dpawlik has joined #openstack-infra20:40
pabelangeryah, working VM no info in console, but is in journald20:40
pabelangerStandardOutput=journal+console20:41
pabelangershould fix that however20:41
*** jamesmcarthur_ has quit IRC20:41
clarkbsystemd.journald.forward_to_console will set it globally too I think20:42
clarkbprobably too noisy to do that20:42
*** jamesmcarthur has joined #openstack-infra20:42
*** jamesmcarthur_ has joined #openstack-infra20:44
*** jamesmcarthur has quit IRC20:44
pabelangeryah, pushed up patch, rebuilding image20:44
*** jamesmcarthur has joined #openstack-infra20:44
clarkbpabelanger: also that comes from log.debug which may go to stderror20:45
pabelangerclarkb: ++, did both to be safe20:45
*** jamesmcarthur_ has quit IRC20:48
clarkbmwhahaha: fungi mnaser the high cache miss rate on port 80 appears to be due to pypi setting a short ttl (uncacheable basically) on pypi indexes20:49
clarkbI think the packages themselves are being cached20:49
*** jackatrack_ has joined #openstack-infra20:51
clarkbmwhahaha: fungi mnaser on the docker side of things grepping for images that were missed shows they were only ever used once20:51
clarkb(at least a small handful of semi randomly chosen examples)20:51
mwhahahaLikely less frequent jobs on older branches20:52
*** ansmith has quit IRC20:53
clarkbmwhahaha: tripleorocky/centos-binary-rsyslog-base seems to be one of the examples of missed and never used again20:54
mwhahahaThat's odd20:54
*** slaweq has quit IRC21:01
jackatrack_Hopefully I'm in the right place!  I wanted to bring up a possible bug that is noticeable while looking at the “Nodepool Images” panels on http://grafana.openstack.org/d/rZtIH5Imz/nodepool?orgId=1 , all of the graphs appear to report a minimum constant of one (1) build in progress.  I would expect the minimum to be zero.  thanks21:03
*** trown is now known as trown|outtypewww21:05
clarkbjackatrack_: I think we don't send any data to statsd when we are not doing work21:05
*** evrardjp has joined #openstack-infra21:06
clarkbjackatrack_: we send data when doing work so it gets set to 1 then never back to 021:06
jackatrack_I see, that makes sense.  Any thoughts as to how to fix it?  even if it's a work around/21:06
clarkbwe could probably send an event when we complete a task to decrement the counter? corvus may have ideas for that21:07
*** bobh has quit IRC21:08
jackatrack_out of curiosity, can you point me to the code that is responsible for updating statsd ?21:08
*** sshnaidm|pto has quit IRC21:10
*** diablo_rojo_phon has quit IRC21:10
clarkbjackatrack_: https://git.zuul-ci.org/cgit/nodepool/tree/nodepool/stats.py I think that is the library that nodepool calls into to do those updates21:10
jackatrack_that looks like it21:12
*** dpawlik has quit IRC21:12
*** rh-jelabarre has quit IRC21:13
pabelangerclarkb: bingo: http://paste.openstack.org/show/730674/21:13
pabelangerhostnname too long21:14
pabelanger:)21:14
fungidhellmann: if you need, we can bump your test release change into the gate so release requests don't get held up any longer than necessary21:14
clarkbpabelanger: the template for the names is configurable21:14
clarkbpabelanger: but ya thats a limit defined by dns21:14
pabelangerclarkb: ++21:14
pabelangeryah, ever hit that before21:14
pabelangeransible-fedora-28-vexxhost-ansible-networking-ca-ymq-1-000018881221:15
pabelangeris what it is trying to set21:15
clarkb64 per dot and 256 total?21:15
clarkbsomething like that21:15
corvusclarkb: ah, i see you're suggesting a new base job for zero-node jobs which runs add-build-sshkey on localhost?21:15
pabelangerclarkb: yah, need to read up and confirm21:16
clarkbcorvus: ya, though maybe we need a new role entirely? I pushed up a thing to make the existing job work but I'm not sure I care for it. Let me see if I can find the links21:16
pabelangermy hostname is 6521:16
clarkbcorvus: https://review.openstack.org/#/c/603240/ and https://review.openstack.org/#/c/603241/21:16
fungiietf rfc 1035: "labels 63 octets or less, names 255 octets or less"21:17
clarkbI was off by one!21:18
fungiyou were basically right21:18
clarkbpabelanger: maybe we should update the simple init systemd config to write to console then if someone adds debug it just works for them21:20
clarkb(worth noting the file close fix would not change this behavior for you)21:20
pabelangerclarkb: think that is a good idea, will propose a patch21:21
openstackgerritMerged openstack-infra/nodepool master: Add release note for instance-properties config  https://review.openstack.org/60486321:23
*** Tim_ok has joined #openstack-infra21:23
openstackgerritMerged openstack-infra/bindep master: Update docs building  https://review.openstack.org/59975521:24
corvusclarkb: yeah, that's a puzzle.  lemme cipher on that.  the second thing -- a zuul user on bridge vs logging in as root -- that seems okay to me, and it can 'become' root to run ansible-playbook on bridge.  let's run that by mordred if he hasn't seen that yet.21:27
clarkbcorvus: I mentioned it to him I think he was ok with that plan but lets double check21:27
mordredyah - I think it's a good plan21:28
dhellmannfungi : that may be a good idea. I'm going offline for a bit now, but I can come back and check on things later this evening21:31
pabelangerclarkb: and glean fixed. Thanks for being sounding board :)21:32
pabelangerclarkb: maybe we should  / could update nodepool to warn about hostname > 64 too21:32
*** jtomasek has quit IRC21:37
*** sshnaidm|pto has joined #openstack-infra21:40
*** agopi|training has joined #openstack-infra21:43
*** bobh has joined #openstack-infra21:45
*** gfidente has quit IRC21:47
*** ansmith has joined #openstack-infra21:49
clarkbI'm going to install https://review.openstack.org/#/c/604193/ on a test node in bhs1, remove the existing eth0 config and reboot and see if glean sets things up with dhcp as expected21:51
clarkbwoot it works21:55
*** jamesmcarthur has quit IRC21:56
*** bobh has quit IRC21:56
*** bobh has joined #openstack-infra21:57
clarkbinfra-root I think https://review.openstack.org/#/c/604193/4 is ready to go as a workaround for new bhs1 behavior. Reviews would be much apprecated as the loss of capacity is part of the reason for slow test processing last week and this week21:57
*** mriedem is now known as mriedem_away21:57
fungii wonder if ovh is going to find a more general solution to that, because the problem affects cloud-init in exactly the same way22:01
* fungi has instances in ovh to prove it22:02
*** Emine has quit IRC22:03
*** emerson has joined #openstack-infra22:04
*** elod has quit IRC22:04
*** graphene has quit IRC22:05
clarkbfungi: amorin mentioned on the change that they would like to find a way to fix it22:05
*** bobh has quit IRC22:06
*** boden has quit IRC22:06
*** graphene has joined #openstack-infra22:06
*** anteaya has joined #openstack-infra22:07
*** evrardjp has quit IRC22:07
ianwi guess nodepool will need a restart along with adding the flag to project-config22:09
clarkbianw: yup22:10
*** AJaeger_ has joined #openstack-infra22:10
*** elod has joined #openstack-infra22:11
*** AJaeger has quit IRC22:11
clarkbmordred: corvus for the zuul user being added to bridge.o.o do we want ot treat that like a normal system user via the extra_users mechanism? we can set the public key that way too22:14
clarkbor would you rather see it set up as a system user?22:14
clarkbI'm sort of waffling on this one22:14
*** emerson has quit IRC22:15
corvusextra_users sounds fine to me22:15
*** evrardjp has joined #openstack-infra22:15
*** dpawlik has joined #openstack-infra22:16
openstackgerritIan Wienand proposed openstack-infra/zuul-sphinx master: Add example and type options to attributes  https://review.openstack.org/60426722:17
mordredclarkb: +2 on the glean - left it un-+Ad in case you want to manage that22:17
mordredclarkb: and yeah - I think extra_users sounds good22:17
*** dpawlik has quit IRC22:20
openstackgerritClark Boylan proposed openstack-infra/system-config master: Add zuul user to bridge.openstack.org  https://review.openstack.org/60492522:22
clarkbmordred: corvus ^ I don't actually know if it will handle the mutliple keys there properly22:22
corvusclarkb: not using the url makes me very sad22:23
clarkbwe might be able to have a jinja template that pulls the multiple keys down from zuul somehow?22:24
clarkbI'm not really sure how that would work with the current user management22:24
corvusor adjust the role to handle key being a list of urls?22:24
corvus(or, well, list of keys for that matter, it doesn't have to care it's a url)22:24
openstackgerritClark Boylan proposed openstack-infra/system-config master: Add zuul user to bridge.openstack.org  https://review.openstack.org/60492522:25
clarkbupdated the job to use that new user22:25
*** tbachman has left #openstack-infra22:26
openstackgerritIan Wienand proposed openstack-infra/zuul-sphinx master: Add example and type options to attributes  https://review.openstack.org/60426722:26
corvusthough, if we did have the users role fetch the key(s), we could keep using the exclusive option which would be nice22:26
clarkblooks like it takes a url22:26
clarkbso ya maybe the role just needs to be updated to handle that as a list22:26
corvusauthorized_key takes a url, but it doesn't take a list of urls22:27
clarkbright we'd have to do the iteration a layer above (and then lose the exclusive flag on those)22:27
corvusbut if we did the fetch ourselves, we could combine them and keep using exclusive22:30
corvusbest of both worlds, but more work22:30
prometheanfireprobably the most reviewed glean change22:30
corvus"make cloud work" changes tend to get attention22:31
clarkbI've just melted my brain trying to understand nested loops where inner loop refers to outer loop in ansible22:31
prometheanfireyep22:32
*** rkukura has quit IRC22:32
clarkbis include_tasks really the easiest way to describe this?22:33
clarkbthats kind of amazing22:33
clarkbdoes ansible preserve python pass by reference behavior with dicts? eg if I have a piece of ansible update part of the host_vars will that be exposed to subsequent tasks?22:37
*** panda is now known as panda|off22:38
prometheanfireya, ansible changed inlcude to include_tasks/roles22:44
*** rcernin has joined #openstack-infra22:45
openstackgerritMerged openstack-infra/elastic-recheck master: Add query for cinder test_retype_multiattach bug 1794143  https://review.openstack.org/60483622:45
openstackbug 1794143 in Cinder "cinder.tests.unit.volume.test_volume_retype.VolumeRetypeTestCase.test_retype_multiattach intermittently fails" [Medium,Confirmed] https://launchpad.net/bugs/179414322:45
clarkbprometheanfire: mostly I'm amazed that this is the way to do a nested loop more so than that change22:45
clarkbI've decided that its better to write pythonfor this as a result22:46
prometheanfire:D22:47
*** andreww has joined #openstack-infra22:51
*** emerson has joined #openstack-infra22:51
*** lathiat_ has joined #openstack-infra22:53
*** owalsh_ has joined #openstack-infra22:53
*** kukacz_ has joined #openstack-infra22:55
*** emerson has quit IRC22:57
*** dims_ has joined #openstack-infra22:59
*** tomaw- has joined #openstack-infra22:59
*** _d34dh0r53_ has joined #openstack-infra23:00
*** stephenfin_ has joined #openstack-infra23:00
*** kukacz has quit IRC23:00
*** _ari_ has quit IRC23:00
*** owalsh has quit IRC23:00
*** tdasilva has quit IRC23:00
*** dims has quit IRC23:00
*** d34dh0r53 has quit IRC23:00
*** Diabelko has quit IRC23:00
*** jlvillal has quit IRC23:00
*** lathiat has quit IRC23:00
*** xarses_ has quit IRC23:00
*** cburgess has quit IRC23:00
*** strigazi has quit IRC23:00
*** xinliang has quit IRC23:00
*** weshay has quit IRC23:00
*** bandini has quit IRC23:00
*** wznoinsk has quit IRC23:00
*** dabukalam has quit IRC23:00
*** stephenfin has quit IRC23:00
*** tomaw has quit IRC23:00
*** eglute has quit IRC23:00
*** kambiz has quit IRC23:00
*** emerson has joined #openstack-infra23:02
*** andreykurilin has quit IRC23:03
*** ruhe has quit IRC23:03
*** tosky has quit IRC23:03
*** jamesmcarthur has joined #openstack-infra23:04
*** sshnaidm|pto has quit IRC23:04
*** jamesmcarthur has quit IRC23:05
*** jamesmcarthur has joined #openstack-infra23:05
*** hasharDinner has quit IRC23:05
*** andreykurilin has joined #openstack-infra23:05
*** ruhe has joined #openstack-infra23:06
*** evrardjp has quit IRC23:07
*** Diabelko has joined #openstack-infra23:08
*** evrardjp has joined #openstack-infra23:08
*** rkukura has joined #openstack-infra23:08
openstackgerritClark Boylan proposed openstack-infra/system-config master: Manage user ssh keys from urls  https://review.openstack.org/60493223:10
clarkbmordred: corvus ^ I really have no idea if I am doing that right and considering how important it is that we be able to ssh into the bastion...23:10
clarkbI'll WIP it for now but I think that is roughly what corvus would like23:11
clarkbI've approved the glean change. We won't actually use it until we make a glean release and rebuild images23:14
clarkbI likely won't get around to that today (and considering we have to restart nodepool and update nodepool config anyway that is probably fine)23:14
clarkbianw: ^ unsure if you are interested in doing follow the sun on that though23:14
ianwi'll take a look23:16
ianwis there anything else in glean that might cause bigger issues?23:17
clarkbianw: this would be the only change on top of the last release I think23:17
clarkbalso we should decide if we want to update nodepool side first or glean and images first23:18
ianwglean and images i think23:19
ianwdid we want to add that close() thing too?23:19
clarkbianw: I'm happy to add the close thing too fi we can get another infra root to review it23:20
clarkbinfra-root https://review.openstack.org/#/c/585574/ is the glean close() thing ianw is talking about23:20
*** anteaya has quit IRC23:21
pabelanger+2 since I was looking at it today23:22
pabelangerI won't get my23:23
pabelangererr23:23
pabelangerI won't get my stderr redirect to console patch for today, so will be the next glean tag23:23
ianwthere's https://review.openstack.org/#/c/493443/ which has been there for a year23:23
*** aidin has joined #openstack-infra23:25
clarkbianw: prometheanfire ^ comment on23:26
clarkbI think the 0 should be dropped23:26
*** jamesmcarthur has quit IRC23:28
clarkband now for something completely different23:31
clarkbthe ovh gra1 mirror seems to be down23:31
clarkbit pings but ssh isn't responding23:32
clarkbalso e-r indicates that the pypi proxy doesn't work23:32
clarkb[17984452.588512] blk_update_request: I/O error, dev sda, sector 92290184 says the console log23:33
clarkbbefore that [740409.091725] INFO: task apache2:8599 blocked for more than 120 seconds.23:33
clarkbI'm going to try a reboot23:33
ianw++23:34
clarkbseems like it is up now23:36
fungilive migration gone wrong during upgrade maintenance maybe?23:37
clarkbthis is gra1 where we aren't supposed to have done upgrades yet23:38
clarkbbut could be upgrade prep23:38
*** tpsilva has quit IRC23:43
clarkbhttp://status.openstack.org/elastic-recheck/gate.html#1449136 should hopefully fall back to zero now23:43
*** smarcet has quit IRC23:47
*** sthussey has quit IRC23:51
openstackgerritClark Boylan proposed openstack-infra/nodepool master: Do Not Merge Testing glean + nodepool integration  https://review.openstack.org/60482623:54
clarkbsilly yaml types23:54
clarkbianw: ^ thats sort of a hgih level sanity check23:54
clarkbit failed because Boolean types are not string in python23:54
*** eernst has quit IRC23:55

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