| ianw | does look blank | 00:00 |
|---|---|---|
| clarkb | ya I think I may have gotten the provider name wrong. double checking against graphite.o.o now | 00:02 |
| *** lennyb has quit IRC | 00:02 | |
| *** lennyb has joined #openstack-infra | 00:04 | |
| openstackgerrit | Clark Boylan proposed openstack/project-config master: Fix airship citycloud grafana query https://review.opendev.org/708278 | 00:04 |
| clarkb | that should fix it I think | 00:04 |
| clarkb | now to debug why that fix to gear is unhappy | 00:07 |
| *** dychen has quit IRC | 00:07 | |
| *** dychen has joined #openstack-infra | 00:08 | |
| *** ahosam has quit IRC | 00:11 | |
| *** slaweq has joined #openstack-infra | 00:11 | |
| ianw | the tension between grafyaml and nice code config and grafana's very polished UI for generating dashboards is quite large | 00:11 |
| ianw | https://github.com/weaveworks/grafanalib | 00:13 |
| ianw | interesting now i look | 00:13 |
| *** dingyichen has joined #openstack-infra | 00:13 | |
| *** slaweq has quit IRC | 00:16 | |
| *** dychen has quit IRC | 00:17 | |
| ianw | oh, i've forgotten the acme-challenge entry in openstack.org for tarballs, updating | 00:37 |
| *** tetsuro has joined #openstack-infra | 00:49 | |
| ianw | i have rsynced the openstack tarballs volume to afs. since the jobs should be publishing in both places, they should now remain in sync | 00:50 |
| clarkb | exciting | 00:51 |
| openstackgerrit | Clark Boylan proposed opendev/gear master: Revert "Revert "Add BSD/Darwin support."" https://review.opendev.org/708267 | 00:53 |
| clarkb | I think ^ should result in happy gear with epoll or poll | 00:53 |
| *** tetsuro has quit IRC | 00:57 | |
| *** dchen has quit IRC | 01:06 | |
| *** rlandy has quit IRC | 01:10 | |
| *** slaweq has joined #openstack-infra | 01:11 | |
| *** slaweq has quit IRC | 01:17 | |
| *** Goneri has quit IRC | 01:25 | |
| *** dave-mccowan has quit IRC | 01:32 | |
| *** rkukura has quit IRC | 01:33 | |
| *** rkukura has joined #openstack-infra | 01:35 | |
| ianw | ok, if anyone wants to override /etc/hosts with "23.253.245.150 tarballs.openstack.org tarballs.opendev.org" that should be working | 01:39 |
| *** rkukura has quit IRC | 01:41 | |
| clarkb | I can check after dinner | 01:42 |
| *** armax has quit IRC | 01:43 | |
| *** tetsuro has joined #openstack-infra | 01:44 | |
| *** larainema has joined #openstack-infra | 01:44 | |
| *** tetsuro has quit IRC | 01:45 | |
| *** tetsuro has joined #openstack-infra | 01:48 | |
| *** rkukura has joined #openstack-infra | 01:51 | |
| *** tetsuro has quit IRC | 01:52 | |
| *** tetsuro has joined #openstack-infra | 01:54 | |
| kevinz | ianw: Morning! | 01:58 |
| *** Lucas_Gray has joined #openstack-infra | 01:59 | |
| kevinz | ianw: I wonder how could I trigger a os-jobs in linaro-us? I submitted a patch here: https://review.opendev.org/#/c/708102/8,but it looks that it comes to linaro-london | 02:01 |
| *** smarcet has joined #openstack-infra | 02:03 | |
| *** tetsuro has quit IRC | 02:03 | |
| ianw | kevinz: hrm, we can't really pick a region, although if required we could turn off linaro-london and that would force it to use US | 02:04 |
| openstackgerrit | Merged opendev/system-config master: Add project.tarballs to periodic AFS release https://review.opendev.org/706732 | 02:05 |
| kevinz | ianw: could we reduce the number to 4 at linaro-london, to force the newly creation jobs to linaro-us? | 02:06 |
| ianw | kevinz: the switch to toggle would be @ https://opendev.org/openstack/project-config/src/branch/master/nodepool/nl03.openstack.org.yaml#L391 | 02:07 |
| kevinz | sure, I will submit a change | 02:08 |
| *** hongbin has joined #openstack-infra | 02:09 | |
| clarkb | ianw: that /etc/hosts entry seems to work. tarballs.openstack.org redirects to tarballs.opendev.org/openstack and tarballs.opendev.org/ root is browseable and LE cert is in place | 02:10 |
| clarkb | and the redirect seems to work over https | 02:10 |
| ianw | clarkb: thanks for confirming | 02:10 |
| *** slaweq has joined #openstack-infra | 02:11 | |
| *** ociuhandu has joined #openstack-infra | 02:12 | |
| *** ociuhandu has quit IRC | 02:16 | |
| *** slaweq has quit IRC | 02:16 | |
| *** cdearborn has quit IRC | 02:17 | |
| *** igordc has quit IRC | 02:17 | |
| *** Lucas_Gray has quit IRC | 02:25 | |
| openstackgerrit | Kevin Zhao proposed openstack/project-config master: reduce machine number from Linaro London https://review.opendev.org/708288 | 02:30 |
| *** rfolco has quit IRC | 02:33 | |
| *** rfolco has joined #openstack-infra | 02:33 | |
| *** rfolco has quit IRC | 02:38 | |
| openstackgerrit | Merged openstack/project-config master: reduce machine number from Linaro London https://review.opendev.org/708288 | 02:47 |
| *** jamesmcarthur has joined #openstack-infra | 02:55 | |
| *** ramishra has joined #openstack-infra | 03:08 | |
| *** slaweq has joined #openstack-infra | 03:11 | |
| *** slaweq has quit IRC | 03:16 | |
| *** ijw_ has quit IRC | 03:22 | |
| kevinz | ianw: it looks that the test nodes are not scheduled to linaro-us http://zuul.openstack.org/status | 03:22 |
| *** smarcet has quit IRC | 03:22 | |
| *** yamamoto has joined #openstack-infra | 03:23 | |
| kevinz | they just waiting in queue.. | 03:23 |
| ianw | kevinz: hrm, ok let me check the launcher | 03:24 |
| kevinz | ianw: thx | 03:25 |
| *** ijw has joined #openstack-infra | 03:25 | |
| *** ijw has quit IRC | 03:29 | |
| *** psachin has joined #openstack-infra | 03:41 | |
| ianw | kevinz: Exception: Unable to find flavor with min ram: 8000 ... maybe | 03:47 |
| kevinz | ianw: aha,,,I will manually create it. thx | 03:47 |
| ianw | there is m1.xlarge? | 03:48 |
| ianw | what are we telling it to use? | 03:48 |
| ianw | ahhh ... os.large | 03:48 |
| ianw | replicate that flavor in the us ci region? | 03:49 |
| ianw | didn't think of that | 03:49 |
| kevinz | 7ce03675-5d14-4b1b-85a4-e641d21320db | m1.xlarge | 8192 | 80 | 0 | 8 | True | 03:50 |
| kevinz | yes, in us cloud, we have m1.xlarge, suffcient | 03:50 |
| kevinz | so, I will submit a change to replace it. | 03:51 |
| ianw | we do set the flavor name to "os.large" https://opendev.org/openstack/project-config/src/branch/master/nodepool/nl03.openstack.org.yaml#L395 | 03:51 |
| ianw | istr something to do with this, is os.large 16 cpus maybe? | 03:51 |
| kevinz | OK, I create a same flavor os.large in linaro-us, that don't need any change in code | 03:52 |
| ianw | | 69af1b47-5557-42fc-bdcb-9cf48dd5e607 | os.large | 8200 | 80 | 80 | 8 | False | | 03:52 |
| kevinz | yes, 8C8G 80GB | 03:52 |
| ianw | m1.xlarge looks similar, execpt for 0 ephemeral storage as opposed to 80g | 03:53 |
| kevinz | yes | 03:53 |
| ianw | i think for general sanity copying os.large is best idea so it's 100% consistent | 03:53 |
| kevinz | 312093a9-e19b-45a3-b194-11decb37f05d | os.large | 8200 | 80 | 80 | 8 | True | | 03:56 |
| kevinz | new one created in linaro-us | 03:56 |
| ianw | cool ... *possibly* nodepool-launcher needs a restart or we need to wait a bit for cache to time out and it to notice it | 03:58 |
| ianw | kevinz: maybe try submitting a new job? | 04:05 |
| kevinz | ianw: I will start a new one :D | 04:05 |
| ianw | Exception deleting instance f15b6a42-ac33-462c-9f6e-56dba3724792 from vexxhost-sjc1 ... this seem to be coming up a lot too | 04:06 |
| *** udesale has joined #openstack-infra | 04:10 | |
| *** slaweq has joined #openstack-infra | 04:11 | |
| *** jamesmcarthur has quit IRC | 04:14 | |
| *** slaweq has quit IRC | 04:16 | |
| *** jamesmcarthur has joined #openstack-infra | 04:19 | |
| kevinz | ianw: submit a new one, now 6 jobs in queue | 04:20 |
| *** yamamoto has quit IRC | 04:29 | |
| *** tetsuro has joined #openstack-infra | 04:32 | |
| *** yamamoto has joined #openstack-infra | 04:35 | |
| *** redrobot has quit IRC | 04:43 | |
| *** imacdonn has quit IRC | 04:43 | |
| *** imacdonn has joined #openstack-infra | 04:44 | |
| ianw | hrm | 04:53 |
| ianw | 2020-02-18 04:52:05,424 ERROR nodepool.driver.NodeRequestHandler[nl03-11177-PoolWorker.linaro-us-main]: Declining node request 100-0007475839 due to exception in NodeRequestHandler: | 04:55 |
| ianw | Tue Feb 18 04:55:42 | 04:55 |
| ianw | i think maybe a restart before we start investigating other things | 04:56 |
| ianw | yeah, it seems like it's building like 5+ nodes now | 04:58 |
| ianw | kevinz: also; check http://grafana.openstack.org/d/vcvSN5Siz/nodepool-linaro?orgId=1 if you didn't know about it :) | 05:00 |
| *** ramishra has quit IRC | 05:01 | |
| kevinz | Great! Thanks! I saw them running now | 05:02 |
| kevinz | ianw: restarting is always a good way :D | 05:02 |
| ianw | yep, turn it on and off again :) | 05:03 |
| *** ramishra has joined #openstack-infra | 05:08 | |
| *** hongbin has quit IRC | 05:10 | |
| *** slaweq has joined #openstack-infra | 05:11 | |
| *** slaweq has quit IRC | 05:16 | |
| *** ykarel|away is now known as ykarel | 05:16 | |
| *** dave-mccowan has joined #openstack-infra | 05:17 | |
| *** yamamoto has quit IRC | 05:20 | |
| *** jamesmcarthur has quit IRC | 05:21 | |
| *** yamamoto has joined #openstack-infra | 05:22 | |
| *** evrardjp has quit IRC | 05:34 | |
| *** evrardjp has joined #openstack-infra | 05:34 | |
| *** dancek has quit IRC | 05:37 | |
| ianw | #status log added 100gb to yum-puppetlabs mirror to stop it hitting afs quota | 05:37 |
| openstackstatus | ianw: finished logging | 05:37 |
| ianw | although, i'd say nobody is using it because it's been broken for quite a while | 05:37 |
| *** dancek has joined #openstack-infra | 05:38 | |
| *** dancek has quit IRC | 05:46 | |
| *** ykarel is now known as ykarel|mtg | 05:47 | |
| *** dancek has joined #openstack-infra | 05:49 | |
| *** udesale_ has joined #openstack-infra | 05:51 | |
| *** udesale has quit IRC | 05:52 | |
| openstackgerrit | Ian Wienand proposed openstack/project-config master: Update AFS granfana page https://review.opendev.org/708302 | 05:52 |
| openstackgerrit | Merged openstack/project-config master: Fix airship citycloud grafana query https://review.opendev.org/708278 | 05:59 |
| openstackgerrit | Merged zuul/zuul master: Cleanup handlers defined in OS_LOG_DEFAULTS https://review.opendev.org/707942 | 06:05 |
| *** lmiccini has joined #openstack-infra | 06:06 | |
| *** slaweq has joined #openstack-infra | 06:11 | |
| *** slaweq has quit IRC | 06:16 | |
| openstackgerrit | Merged zuul/zuul master: Don't set OS_LOG_DEFAULTS if unset https://review.opendev.org/707783 | 06:21 |
| *** redrobot has joined #openstack-infra | 06:27 | |
| *** dayou has quit IRC | 06:34 | |
| *** dayou has joined #openstack-infra | 06:34 | |
| ianw | kevinz: do you want to revert the london change back? | 06:49 |
| *** yamamoto has quit IRC | 06:51 | |
| openstackgerrit | Merged openstack/project-config master: Update AFS granfana page https://review.opendev.org/708302 | 07:00 |
| *** xek has joined #openstack-infra | 07:08 | |
| *** ykarel|mtg is now known as ykarel | 07:10 | |
| *** slaweq has joined #openstack-infra | 07:11 | |
| *** dklyle has quit IRC | 07:15 | |
| *** dklyle has joined #openstack-infra | 07:16 | |
| *** slaweq has quit IRC | 07:16 | |
| *** yamamoto has joined #openstack-infra | 07:16 | |
| *** lpetrut has joined #openstack-infra | 07:20 | |
| *** pgaxatte has joined #openstack-infra | 07:23 | |
| openstackgerrit | Merged zuul/zuul master: cleanup: install docker-compose via role https://review.opendev.org/707903 | 07:24 |
| kevinz | ianw: I think we can temperily keep this change, since linaro London don't have good performance | 07:32 |
| *** ccamacho has quit IRC | 07:34 | |
| *** ccamacho has joined #openstack-infra | 07:34 | |
| *** yamamoto has quit IRC | 07:34 | |
| ianw | sounds good | 07:34 |
| *** yamamoto has joined #openstack-infra | 07:34 | |
| *** dpawlik has joined #openstack-infra | 07:38 | |
| *** Lucas_Gray has joined #openstack-infra | 07:40 | |
| *** rpittau|afk is now known as rpittau | 07:58 | |
| *** slaweq has joined #openstack-infra | 07:59 | |
| *** tesseract has joined #openstack-infra | 08:02 | |
| *** arif-ali has quit IRC | 08:06 | |
| *** arif-ali has joined #openstack-infra | 08:08 | |
| *** dingyichen has quit IRC | 08:15 | |
| *** roman_g has quit IRC | 08:18 | |
| *** tkajinam has quit IRC | 08:31 | |
| *** tosky has joined #openstack-infra | 08:32 | |
| *** amoralej|off is now known as amoralej | 08:32 | |
| *** ralonsoh has joined #openstack-infra | 08:35 | |
| *** gfidente has joined #openstack-infra | 08:36 | |
| *** pkopec has joined #openstack-infra | 08:45 | |
| *** jpena|off is now known as jpena | 08:52 | |
| *** priteau has joined #openstack-infra | 08:58 | |
| *** lucasagomes has joined #openstack-infra | 09:08 | |
| *** rcernin has quit IRC | 09:12 | |
| *** ramishra has quit IRC | 09:14 | |
| *** ramishra has joined #openstack-infra | 09:16 | |
| *** derekh has joined #openstack-infra | 09:25 | |
| openstackgerrit | Sorin Sbarnea proposed opendev/system-config master: Improve 3rd-party logging guidelines https://review.opendev.org/708323 | 09:28 |
| openstackgerrit | Sorin Sbarnea proposed opendev/system-config master: Improve 3rd-party logging guidelines https://review.opendev.org/708323 | 09:29 |
| *** ccamacho has quit IRC | 09:34 | |
| *** Lucas_Gray has quit IRC | 09:35 | |
| *** Lucas_Gray has joined #openstack-infra | 09:41 | |
| *** sshnaidm|afk is now known as sshnaidm | 09:59 | |
| *** Wryhder has joined #openstack-infra | 10:01 | |
| *** Lucas_Gray has quit IRC | 10:01 | |
| *** Wryhder is now known as Lucas_Gray | 10:01 | |
| *** migawa is now known as migawa|AFK | 10:04 | |
| *** dtantsur|afk is now known as dtantsur | 10:05 | |
| openstackgerrit | Mattias Jernberg proposed opendev/git-review master: Allow the default of notopic to be configurable https://review.opendev.org/697448 | 10:10 |
| openstackgerrit | Ilya Etingof proposed openstack/project-config master: Add publish-to-pypi template for sushy-cli https://review.opendev.org/708329 | 10:19 |
| openstackgerrit | Merged zuul/zuul master: Fix github app authentication to work with checks API endpoints (2) https://review.opendev.org/708109 | 10:20 |
| dtantsur | hi folks! could you check https://review.opendev.org/#/c/708329/ please? | 10:20 |
| dtantsur | we missed it when creating sushy-cli and now cannot release | 10:20 |
| *** fdegir has quit IRC | 10:34 | |
| *** fdegir has joined #openstack-infra | 10:35 | |
| *** Lucas_Gray has quit IRC | 10:51 | |
| *** Lucas_Gray has joined #openstack-infra | 10:56 | |
| *** udesale_ has quit IRC | 10:57 | |
| *** Lucas_Gray has quit IRC | 10:58 | |
| *** petevg has quit IRC | 11:08 | |
| *** ykarel is now known as ykarel|afk | 11:09 | |
| *** Lucas_Gray has joined #openstack-infra | 11:20 | |
| *** Lucas_Gray has quit IRC | 11:20 | |
| *** adriancz has joined #openstack-infra | 11:22 | |
| *** Lucas_Gray has joined #openstack-infra | 11:24 | |
| *** ykarel|afk is now known as ykarel | 11:26 | |
| *** Lucas_Gray has quit IRC | 11:33 | |
| *** Lucas_Gray has joined #openstack-infra | 11:34 | |
| *** Lucas_Gray has quit IRC | 11:41 | |
| *** andrewbonney has joined #openstack-infra | 11:45 | |
| *** Lucas_Gray has joined #openstack-infra | 11:51 | |
| *** migawa|AFK is now known as migawa | 11:54 | |
| *** rfolco has joined #openstack-infra | 11:55 | |
| openstackgerrit | Merged openstack/project-config master: Add publish-to-pypi template for sushy-cli https://review.opendev.org/708329 | 11:57 |
| *** amoralej is now known as amoralej|lunch | 11:59 | |
| *** gfidente has quit IRC | 12:02 | |
| *** rpittau is now known as rpittau|bbl | 12:07 | |
| *** ykarel is now known as ykarel|afk | 12:09 | |
| *** gfidente has joined #openstack-infra | 12:14 | |
| *** tkajinam has joined #openstack-infra | 12:23 | |
| *** yamamoto has quit IRC | 12:28 | |
| *** fdegir has quit IRC | 12:29 | |
| *** yamamoto has joined #openstack-infra | 12:31 | |
| *** fdegir has joined #openstack-infra | 12:31 | |
| *** adriant has quit IRC | 12:33 | |
| *** adriant4 has joined #openstack-infra | 12:33 | |
| *** iokiwi has quit IRC | 12:38 | |
| *** iokiwi has joined #openstack-infra | 12:38 | |
| *** Wryhder has joined #openstack-infra | 12:39 | |
| *** jpena is now known as jpena|lunch | 12:39 | |
| *** Lucas_Gray has quit IRC | 12:41 | |
| *** Wryhder is now known as Lucas_Gray | 12:41 | |
| *** smarcet has joined #openstack-infra | 12:42 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Increase cherrypy thread pool https://review.opendev.org/708377 | 12:43 |
| *** ykarel|afk is now known as ykarel | 12:46 | |
| *** adriant4 has quit IRC | 12:48 | |
| *** adriant has joined #openstack-infra | 12:49 | |
| *** lucasagomes has quit IRC | 12:51 | |
| *** smarcet has quit IRC | 12:52 | |
| *** lucasagomes has joined #openstack-infra | 12:55 | |
| *** rlandy has joined #openstack-infra | 12:56 | |
| *** smarcet has joined #openstack-infra | 12:57 | |
| *** beagles is now known as beagles_mtg | 13:01 | |
| *** rh-jelabarre has joined #openstack-infra | 13:02 | |
| *** rpittau|bbl is now known as rpittau | 13:06 | |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: Add reference pipelines file for Github driver https://review.opendev.org/672712 | 13:08 |
| *** amoralej|lunch is now known as amoralej | 13:08 | |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: A Zuul reporter for Elasticsearch https://review.opendev.org/644927 | 13:12 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: URLTrigger driver time based https://review.opendev.org/635567 | 13:14 |
| *** ramishra has quit IRC | 13:15 | |
| openstackgerrit | Rico Lin proposed opendev/irc-meetings master: Rename meeting name for Automation SIG https://review.opendev.org/708384 | 13:20 |
| *** ramishra has joined #openstack-infra | 13:25 | |
| *** smarcet has quit IRC | 13:31 | |
| *** ccamacho has joined #openstack-infra | 13:36 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: WIP: Store unparsed branch config in Zookeeper https://review.opendev.org/705716 | 13:40 |
| *** jpena|lunch is now known as jpena | 13:42 | |
| *** ykarel is now known as ykarel|afk | 13:44 | |
| *** Goneri has joined #openstack-infra | 13:46 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Increase cherrypy thread pool size https://review.opendev.org/708377 | 13:51 |
| *** yamamoto has quit IRC | 13:52 | |
| *** udesale has joined #openstack-infra | 14:02 | |
| *** jamesmcarthur has joined #openstack-infra | 14:03 | |
| *** igordc has joined #openstack-infra | 14:04 | |
| *** aaronsheffield has joined #openstack-infra | 14:07 | |
| *** yamamoto has joined #openstack-infra | 14:10 | |
| *** smarcet has joined #openstack-infra | 14:14 | |
| *** owalsh_ is now known as owalsh | 14:26 | |
| openstackgerrit | Thierry Carrez proposed openstack/project-config master: check-release-approval: retry once when Gerrit fails https://review.opendev.org/708397 | 14:27 |
| *** Tengu has quit IRC | 14:30 | |
| *** Tengu has joined #openstack-infra | 14:31 | |
| openstackgerrit | Alfredo Moralejo proposed zuul/zuul-jobs master: Use CentOS8 dependencies repo to install openvswitch https://review.opendev.org/708402 | 14:36 |
| *** cdearborn has joined #openstack-infra | 14:36 | |
| cdearborn | morning folks - i really messed up a couple of backport patches. i've tried everything i can think of to fix the mess with no luck. can someone help? | 14:39 |
| cdearborn | the scenario is: i cherry picked a patch to stable/stein and stable/queens | 14:39 |
| cdearborn | the patch was reviewed, and WF+1 | 14:39 |
| cdearborn | it merged | 14:40 |
| cdearborn | i accidentally did a git review on the patch again | 14:40 |
| cdearborn | that appeared to unmerge it | 14:40 |
| cdearborn | now the CI won't run on the patch, even when WF+1 again | 14:40 |
| cdearborn | and recheck does nothing | 14:41 |
| cdearborn | any idea how i can get these 2 patches back into the merged state? | 14:41 |
| *** udesale has quit IRC | 14:41 | |
| clarkb | cdearborn: can you link to the changes in question? fwiw git review cannot (un)merge things asit only pushes to gerrits magic proposal refs | 14:42 |
| cdearborn | k | 14:42 |
| *** zigo has quit IRC | 14:42 | |
| cdearborn | original patch: https://review.opendev.org/#/c/629613/ | 14:42 |
| cdearborn | cherry pick 1: https://review.opendev.org/#/c/660215/ | 14:42 |
| cdearborn | cherry pick 2: https://review.opendev.org/#/c/706383/ | 14:43 |
| cdearborn | i just abandoned the 2 cherry picks | 14:43 |
| cdearborn | but can undo that | 14:43 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Log duration of tenant reconfigurations https://review.opendev.org/708403 | 14:43 |
| clarkb | only the first change merged. neither of the cherry picks appear to have merged | 14:45 |
| cdearborn | ok | 14:45 |
| clarkb | I also dont see any job results on any patchset of the cherry picks | 14:46 |
| cdearborn | k | 14:46 |
| clarkb | I think that you probably dont have jobs configured there and so nothing runs (and that prevents merging) | 14:46 |
| clarkb | the two cherry picks should be managed independent of each other though. They areon different branches and are distinct changes | 14:47 |
| cdearborn | yes, completely agree | 14:47 |
| cdearborn | so we relatively recently made zuulv3 changes - maybe something is messed up there? | 14:48 |
| cdearborn | https://github.com/openstack/python-dracclient/blob/master/zuul.d/project.yaml | 14:48 |
| clarkb | cdearborn: because python-dracclient is branched zuul applies that config only to the branvh that config lives in | 14:49 |
| clarkb | you need zuul configs on the stable branches too | 14:50 |
| *** Lucas_Gray has quit IRC | 14:51 | |
| cdearborn | ohhhhhhhh - that makes complete sense. will get that addressed. thank you very much for your help clarkb! | 14:51 |
| clarkb | cdearborn: if the zuul configs are updated on the stable branches you should be able to recheck andmerge those backports | 14:52 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: wip: Proof of concept for matrix jobs https://review.opendev.org/708406 | 14:53 |
| cdearborn | clarkb: perfect - obviously a lot of fog of war going on here. thx again for your help! | 14:53 |
| *** Lucas_Gray has joined #openstack-infra | 14:55 | |
| *** jamesmcarthur has quit IRC | 14:56 | |
| *** jamesmcarthur has joined #openstack-infra | 14:57 | |
| *** michael-beaver has joined #openstack-infra | 14:59 | |
| *** igordc has quit IRC | 15:00 | |
| *** roman_g has joined #openstack-infra | 15:02 | |
| *** Goneri has quit IRC | 15:03 | |
| smcginnis | Howdy infra folks (cc ttx) - we've had a few release job failures this morning. They all look like they just need to be rerun. Four puppet forge server errors and one where we got a username prompt on "git review -s". | 15:03 |
| smcginnis | I've added them all starting on line 365 here: https://etherpad.openstack.org/p/ussuri-relmgt-tracking | 15:04 |
| *** jamesmcarthur has quit IRC | 15:04 | |
| smcginnis | Does anyone have a few minutes to get those reenqueued? | 15:04 |
| openstackgerrit | Alfredo Moralejo proposed zuul/zuul-jobs master: Use CentOS8 dependencies repo to install openvswitch https://review.opendev.org/708402 | 15:05 |
| clarkb | smcginnis: I can do it in after meetings this morning if no one beats me to it | 15:07 |
| smcginnis | Thanks! | 15:07 |
| *** Goneri has joined #openstack-infra | 15:11 | |
| *** lpetrut has quit IRC | 15:11 | |
| smcginnis | fungi: Related to that ML thread. ^^ | 15:21 |
| *** jamesmcarthur has joined #openstack-infra | 15:24 | |
| openstackgerrit | Lee Yarwood proposed openstack/diskimage-builder master: WIP Add Fedora 31 support and test job https://review.opendev.org/708416 | 15:27 |
| *** smarcet has quit IRC | 15:30 | |
| *** jaosorior has joined #openstack-infra | 15:34 | |
| fungi | smcginnis: thanks! clarkb and i can arm wrestle for it | 15:34 |
| *** petevg has joined #openstack-infra | 15:35 | |
| smcginnis | :) | 15:36 |
| *** pgaxatte has quit IRC | 15:36 | |
| *** eharney has quit IRC | 15:38 | |
| *** Lucas_Gray has quit IRC | 15:41 | |
| *** takamatsu has joined #openstack-infra | 15:41 | |
| *** Lucas_Gray has joined #openstack-infra | 15:43 | |
| *** udesale has joined #openstack-infra | 15:51 | |
| *** armax has joined #openstack-infra | 15:53 | |
| openstackgerrit | Alfredo Moralejo proposed zuul/zuul-jobs master: [DNM] Use centos8 deps repo for RHEL8 nodes https://review.opendev.org/708430 | 15:53 |
| *** tesseract has quit IRC | 15:55 | |
| *** smarcet has joined #openstack-infra | 16:01 | |
| *** beagles_mtg is now known as beagles | 16:02 | |
| *** rpittau is now known as rpittau|afk | 16:06 | |
| *** udesale_ has joined #openstack-infra | 16:06 | |
| *** udesale has quit IRC | 16:07 | |
| *** zigo has joined #openstack-infra | 16:08 | |
| *** rosmaita has quit IRC | 16:09 | |
| *** gshippey has joined #openstack-infra | 16:12 | |
| *** ccamacho has quit IRC | 16:13 | |
| *** rosmaita has joined #openstack-infra | 16:14 | |
| *** udesale_ has quit IRC | 16:17 | |
| fungi | gonna nab early lunch and can then reenqueue releases in an hour-ish | 16:21 |
| fungi | back soon | 16:21 |
| clarkb | smcginnis: was sushi-cli the git review -s prompt? | 16:22 |
| clarkb | thats odd, if it doesn't happen consistently. but I suppose could be a side effect of a network issue | 16:22 |
| smcginnis | clarkb: Yep, that was the one. | 16:23 |
| *** udesale has joined #openstack-infra | 16:23 | |
| smcginnis | Seemed really odd to me too, considering how often we do that operation in our jobs. | 16:23 |
| clarkb | roman_g: not sure if you saw my questions yesterday, but wanted to followup on your ci status when you had a cahnce (just to make sure things are working now or if sitll having trouble we can address that) | 16:24 |
| smcginnis | My assumption was a network glitch too. | 16:24 |
| clarkb | smcginnis: git review assumes a failure may be due to wrong username because its such a common error due to mismatch between local username and remote | 16:24 |
| *** psachin has quit IRC | 16:27 | |
| *** Lucas_Gray has quit IRC | 16:29 | |
| clarkb | smcginnis: ok I've enqueued the sushi-cli one | 16:29 |
| clarkb | looking at puppet now | 16:30 |
| *** jamesmcarthur_ has joined #openstack-infra | 16:32 | |
| *** jamesmcarthur has quit IRC | 16:36 | |
| clarkb | smcginnis: fungi ok I think I've taken care of it | 16:37 |
| smcginnis | clarkb: Hmm, something wrong with that I guess. Same failure with sushy-cli - https://581819ea67919485b97e-6002fae613cad806f99007086c39ea60.ssl.cf2.rackcdn.com/81e6ec96d584c89df4cdf94fa8c530d0ca2c8e0c/release-post/tag-releases/8fdc2ae/job-output.txt | 16:44 |
| *** jackedin has joined #openstack-infra | 16:45 | |
| amoralej | AJaeger, wrt https://review.opendev.org/#/c/708402/ apparently openvswitch installation is taking 20 mins in gentoo | 16:45 |
| *** jackedin has quit IRC | 16:45 | |
| amoralej | any idea? | 16:46 |
| clarkb | smcginnis: ssh://release@review.openstack.org:29418/openstack/sushy-cli.git did not work. Description: Host key verification failed. | 16:46 |
| clarkb | smcginnis: is it possible the host key being loaded is for review.opendev.org? | 16:46 |
| *** jpena is now known as jpena|brb | 16:46 | |
| clarkb | amoralej: is it compiling ovs? | 16:46 |
| clarkb | amoralej: if so we can probably set that job to non voting, but also probably better question for #zuul though prometheanfire might not see it there | 16:47 |
| smcginnis | clarkb: Maybe? How would we check that? And why would this one be different than others? | 16:47 |
| amoralej | ack | 16:47 |
| prometheanfire | sup? | 16:47 |
| amoralej | clarkb, not idea how gentoo installs that, it may be compiling it | 16:47 |
| clarkb | prometheanfire: https://zuul.opendev.org/t/zuul/build/4bac5bb45c924547855c33861f921253 job on gentoo testing multinode networking setup seems to take 20 minutes to install ovs (possibly compiling it?) | 16:47 |
| clarkb | prometheanfire: that causes the job to timeout. Maybe we can install precompiled package instead somehow? | 16:48 |
| smcginnis | clarkb: Ah, found it I think. | 16:48 |
| prometheanfire | probably | 16:48 |
| clarkb | smcginnis: I think that may be part of a secret? | 16:48 |
| clarkb | smcginnis: but I don't know | 16:48 |
| prometheanfire | sys-kernel/vanilla-kernel-bin is a precompiled kernel package | 16:48 |
| smcginnis | sushy-cli has host=review.openstack.org | 16:48 |
| clarkb | smcginnis: oh in .gitreview | 16:48 |
| smcginnis | I think everywhere else we have host=review.opendev.org | 16:48 |
| clarkb | that would do it | 16:48 |
| prometheanfire | not sure exactly what it's installing (could be a dep taking a long time to build, ovs doesn't take that long iirc | 16:48 |
| smcginnis | dtantsur: ^^ | 16:48 |
| dtantsur | ugh | 16:49 |
| dtantsur | lemme fix it | 16:49 |
| smcginnis | So I think that means we will not be able to release that one. | 16:49 |
| smcginnis | dtantsur: Just put up a patch if you can approve it. | 16:49 |
| smcginnis | We will need to do another release and just increment the bugfix rev. | 16:50 |
| dtantsur | smcginnis: done. I guess we cannot re-release even if nothing really has been done? | 16:50 |
| smcginnis | Well, actually... | 16:50 |
| clarkb | you have to change the target revision but the tag name can stay 0.1.0 I think | 16:50 |
| smcginnis | It never got tagged and released, so I guess we could revert the sushy-cli release patch, then resubmit with the new hash. | 16:51 |
| smcginnis | Moving back over to #openstack-release since this is no longer an infra thing. | 16:51 |
| prometheanfire | it doesn't look like we can see the primary/secondary logs? | 16:51 |
| smcginnis | Thanks clarkb | 16:51 |
| clarkb | prometheanfire: probably a side effect of timing out? | 16:52 |
| prometheanfire | ya | 16:52 |
| prometheanfire | guess I need to try a manual build, or something | 16:52 |
| mordred | smcginnis, clarkb: http://codesearch.openstack.org/?q=host%3Dreview.openstack.org&i=nope&files=&repos= - we might want to follow up on those | 16:53 |
| * prometheanfire will ask mgorny if the binary kernel supports building modules against it | 16:53 | |
| clarkb | prometheanfire: oh it could be that we have to build a kernel in order to build the ovs module? | 16:53 |
| clarkb | and that would take time for sure if so | 16:53 |
| AJaeger | prometheanfire: increase the timeout temporarily for the job and see what takes so long? | 16:53 |
| mordred | looks like sushi-cli is the only "real" instance | 16:54 |
| prometheanfire | clarkb: typically, no | 16:54 |
| prometheanfire | AJaeger: ya, not sure what it's building and that's the quick way to know | 16:54 |
| prometheanfire | the image should have sources installed already (and used to build it's running kernel) | 16:55 |
| smcginnis | mordred: Great idea to check. That's more than I would have expected at this point. | 16:55 |
| mordred | smcginnis: seems mostly to be docs - sushy-cli and then the cookiecutter repos | 16:55 |
| smcginnis | Oh right. That's good. | 16:56 |
| openstackgerrit | Sean McGinnis proposed opendev/puppet-infra-cookiecutter master: Fix gitreview host name https://review.opendev.org/708452 | 16:58 |
| *** lmiccini has quit IRC | 16:58 | |
| *** udesale_ has joined #openstack-infra | 16:58 | |
| dtantsur | mordred: I guess we used the cookiecutter | 16:59 |
| *** udesale has quit IRC | 16:59 | |
| *** gyee has joined #openstack-infra | 17:00 | |
| openstackgerrit | Lee Yarwood proposed openstack/diskimage-builder master: WIP Add Fedora 31 support and test job https://review.opendev.org/708416 | 17:01 |
| *** eernst has joined #openstack-infra | 17:02 | |
| openstackgerrit | Sean McGinnis proposed openstack/cookiecutter master: Fix gitreview host name https://review.opendev.org/708456 | 17:02 |
| openstackgerrit | Sean McGinnis proposed openstack/specs-cookiecutter master: Fix gitreview host name https://review.opendev.org/708457 | 17:03 |
| *** dtantsur is now known as dtantsur|afk | 17:06 | |
| *** eernst has quit IRC | 17:07 | |
| *** igordc has joined #openstack-infra | 17:09 | |
| clarkb | with that sorted out I'm going to find food so that I' not starving during the infra meeting :) | 17:10 |
| *** udesale_ has quit IRC | 17:12 | |
| *** lucasagomes has quit IRC | 17:13 | |
| *** udesale has joined #openstack-infra | 17:14 | |
| *** jpena|brb is now known as jpena | 17:23 | |
| *** udesale has quit IRC | 17:27 | |
| *** jaosorior has quit IRC | 17:28 | |
| *** yamamoto has quit IRC | 17:33 | |
| *** evrardjp has quit IRC | 17:34 | |
| *** evrardjp has joined #openstack-infra | 17:34 | |
| *** armax has quit IRC | 17:39 | |
| openstackgerrit | Merged zuul/zuul-operator master: Update sdk to v0.13.0 and install dhall-to-json https://review.opendev.org/702103 | 17:39 |
| openstackgerrit | Kendall Nelson proposed openstack/cookiecutter master: Add include for CONTRIBUTING.rst https://review.opendev.org/707735 | 17:42 |
| openstackgerrit | Lee Yarwood proposed openstack/diskimage-builder master: WIP Add Fedora 31 support and test job https://review.opendev.org/708416 | 17:49 |
| *** gfidente has quit IRC | 17:54 | |
| *** weifan has joined #openstack-infra | 17:55 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-operator master: Add zuul-operator-promote-image job https://review.opendev.org/704187 | 17:56 |
| *** derekh has quit IRC | 18:04 | |
| *** weifan has quit IRC | 18:05 | |
| zbr | clarkb: fungi: maybe you can send some feedback on https://review.opendev.org/#/c/708323/ on 3rd party logging guidelines. | 18:06 |
| clarkb | zbr: +2 thanks. That seems like a good update for practicality and useability | 18:08 |
| *** weifan has joined #openstack-infra | 18:08 | |
| *** smarcet has quit IRC | 18:09 | |
| *** weifan has quit IRC | 18:13 | |
| *** weifan has joined #openstack-infra | 18:14 | |
| *** weifan has quit IRC | 18:18 | |
| *** jpena is now known as jpena|off | 18:33 | |
| *** tosky has quit IRC | 18:35 | |
| *** ralonsoh has quit IRC | 18:36 | |
| *** priteau has quit IRC | 18:39 | |
| *** amoralej is now known as amoralej|off | 18:41 | |
| openstackgerrit | Merged opendev/system-config master: Improve 3rd-party logging guidelines https://review.opendev.org/708323 | 18:42 |
| *** eernst has joined #openstack-infra | 18:44 | |
| *** larainema has quit IRC | 18:53 | |
| openstackgerrit | Merged openstack/specs-cookiecutter master: Fix gitreview host name https://review.opendev.org/708457 | 18:55 |
| openstackgerrit | Merged openstack/cookiecutter master: Fix gitreview host name https://review.opendev.org/708456 | 18:55 |
| mordred | diablo_rojo: your cookiecutter patch above - that works in GH? like - the include gets processed right? | 18:56 |
| *** matt_kosut has joined #openstack-infra | 18:57 | |
| diablo_rojo | mordred, I think so? I will go check that thing. I think I know of a repo where we have an example of this in practice.. | 18:57 |
| mordred | diablo_rojo: cool. thanks! (neat concept) | 18:57 |
| diablo_rojo | I hope it works lol. | 18:58 |
| mordred | diablo_rojo: I have complete and utter faith that it does. except for the skepticism | 18:58 |
| diablo_rojo | mordred, It seems like it should work.. in theory..then again Github is silly. | 18:58 |
| weshay|ruck | ianw, qq.. any plans for devstack centos-8, and do you need help? | 19:03 |
| weshay|ruck | zbr, ^ | 19:04 |
| zbr | o/ | 19:04 |
| *** matt_kosut has quit IRC | 19:14 | |
| *** priteau has joined #openstack-infra | 19:19 | |
| mordred | ianw: so based on that - I'm not sure why the .cer isn't updated | 19:21 |
| *** priteau has quit IRC | 19:23 | |
| ianw | mordred: i wonder if *maybe* ansible runs were broken for some other reason at that point, so maybe it got that far, but then never issued the cert, but now doesn't notice? | 19:24 |
| ianw | mordred: the quickest thing might be to just move the certs out of the way so that it think it needs to regenerate them | 19:24 |
| mordred | ianw: hrm | 19:26 |
| fungi | mordred: diablo_rojo: my original recommendation was the exact opposite of https://review.opendev.org/707735 | 19:26 |
| fungi | maybe github knows how to do that, but someone cloning the repo and opening CONTRIBUTING.rst with their text editor or cat doesn't automatically provide useful info | 19:27 |
| mordred | fungi: were you suggesting putting the data in CONTRIBUTING then linking it in the docs? | 19:27 |
| fungi | sphinx on the other hand is great at using the include | 19:27 |
| openstackgerrit | Ian Wienand proposed openstack/project-config master: nodepool-elements: Use venv for utilities https://review.opendev.org/707750 | 19:27 |
| mordred | yeah | 19:27 |
| fungi | so putting the include directive in the doc source is far more correct, while leaving the content in the more discoverable file | 19:27 |
| fungi | i left a review comment to that effect | 19:30 |
| *** ijw has joined #openstack-infra | 19:37 | |
| *** ijw has quit IRC | 19:42 | |
| ianw | 2020-02-12 07:49:22,716 p=root u=4548 | RUNNING HANDLER [letsencrypt-create-certs : Reload apache] ********************* | 19:44 |
| ianw | 2020-02-12 07:49:22,717 p=root u=4548 | Wednesday 12 February 2020 07:49:22 +0000 (0:00:02.619) 0:01:15.213 **** | 19:44 |
| ianw | 2020-02-12 07:49:23,854 p=root u=4548 | fatal: [static01.opendev.org]: FAILED! => {"changed": false, "msg": "Unable to reload service apache2: Job for apache2.service failed because the control process exited with error code.\nSee \"systemctl status apache2.service\" and \"journalctl -xe\" for details.\n"} | 19:44 |
| ianw | mordred: ^ i think that's a smoking gun | 19:44 |
| ianw | that was my screw up ... i didn't have the handlers in place for tarballs.openstack.org so it wasn't deploying the cert properly | 19:44 |
| ianw | i think that your change got messed up in there and due to that never got reloaded | 19:45 |
| *** ijw has joined #openstack-infra | 19:45 | |
| *** auristor has quit IRC | 19:46 | |
| mordred | ianw: AH weird | 19:47 |
| mordred | ianw: so we shoudl just delete the .cer and re-run ansible probably? | 19:47 |
| ianw | mordred: possibly just restart it? | 19:48 |
| ianw | that explains to me why it never restarted at least | 19:48 |
| *** eernst has quit IRC | 19:48 | |
| mordred | ianw: well - but it also has an old cert file | 19:49 |
| *** auristor has joined #openstack-infra | 19:49 | |
| ianw | the log period is /var/log/ansible/ansible.log.6.gz | 19:50 |
| ianw | mordred: 2020-02-12 19:20:23,776 p=root u=3050 | fatal: [review01.openstack.org]: FAILED! => {"msg": "An unhandled exception occurred while running the lookup plugin 'dict'. Error was a <class 'ansible.errors.AnsibleError'>, original message: with_dict expects a dict"} | 19:54 |
| ianw | ahhh so something went wrong | 19:54 |
| mordred | hrm. | 19:56 |
| mordred | ianw: I don't see anywhere that we're using with_dict :( | 19:57 |
| mordred | oh - lookup plugin | 19:57 |
| ianw | that's also review not review-dev ... still looking through the logs at that period | 19:57 |
| ianw | http://paste.openstack.org/show/789728/ -- it noticed it needed updating and got the txt token | 19:58 |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul master: Flesh out the glossary significantly https://review.opendev.org/704391 | 19:58 |
| *** Goneri has quit IRC | 19:58 | |
| ianw | mordred: i think it's static that gummed up the works. i had not put in the _acme-challenge redirect for static.openstack.org, so that certificate issue was constantly failing | 20:00 |
| ianw | we see | 20:01 |
| ianw | 2020-02-12 19:21:02,393 p=root u=3050 | TASK [letsencrypt-create-certs : Run acme.sh driver for static01-tarballs-opendev-org certificate issue] *** | 20:01 |
| ianw | but never the same for review-dev | 20:01 |
| mordred | ianw: nod. so it's not likely to be an actual issue in our ansible as much as a point in time issue | 20:01 |
| ianw | well, i think that we could definitely handle errors better | 20:02 |
| mordred | yah | 20:02 |
| ianw | i have had https://github.com/acmesh-official/acme.sh/pull/2606 out for a long time with no response | 20:02 |
| ianw | that would help | 20:03 |
| ianw | at this point, i would remove the certs and let them get regenerated on the next run | 20:03 |
| mordred | ianw: kk. I'm moving it out of the way | 20:03 |
| mordred | and we'll see how it goes | 20:04 |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Switch specs publishing to AFS https://review.opendev.org/708500 | 20:04 |
| AJaeger | ianw: what do you think ^ | 20:04 |
| mordred | ianw: now - I believe you're supposed to be on PTO? So thanks - and go away! | 20:04 |
| ianw | heh yes but nobody else is awake yet :) | 20:04 |
| AJaeger | ianw: sorry for pinging, enjoy your vacation! | 20:05 |
| mordred | ianw: hehe :) | 20:05 |
| *** eernst has joined #openstack-infra | 20:05 | |
| mordred | ianw: I do the same thing ... | 20:05 |
| AJaeger | infra-root, could we merge https://review.opendev.org/#/c/708452/1 to fix the puppet-infra-cookiecutter .gitreview, please? | 20:05 |
| diablo_rojo | fungi, so on that cookiecutter patch..you think I should keep it how it was originally? With the contributing.rst including the CONTRIBUTING.rst | 20:06 |
| diablo_rojo | ? | 20:06 |
| fungi | that was what i had originally suggested, yes | 20:07 |
| diablo_rojo | Okay so I did have that right.. I read the conversation like ten times to make sure I had it like that.. | 20:07 |
| diablo_rojo | But at the same time I do think it would be helpful to have the stuff show up when people look at the github repo | 20:07 |
| ianw | AJaeger: lgtm, the volume is there ... not sure if i created apache configs yet? | 20:07 |
| fungi | CONTRIBUTING.rst is the canonical location people expect that information at in the repository. the include directive is for gluing it into the sphinx sources so that sphinx integrates the content when building documentation in other formats (html, et cetera) | 20:07 |
| diablo_rojo | (assuming the include works how I think it does) | 20:07 |
| openstackgerrit | Ian Wienand proposed opendev/system-config master: Make haproxy role more generic https://review.opendev.org/677903 | 20:08 |
| openstackgerrit | Ian Wienand proposed opendev/system-config master: Add a service load balancer https://review.opendev.org/678159 | 20:08 |
| diablo_rojo | fungi, so maybe make mention of the contributing.rst doc in CONTRIBUTING.rst and put the include in contributing.rst? | 20:08 |
| AJaeger | ianw: before we merge, we need to rsync all content over... Note that I have no parallel publishing. | 20:09 |
| *** eernst has quit IRC | 20:09 | |
| AJaeger | ianw: if you want parallel publishing, we need to rework... | 20:10 |
| diablo_rojo | fungi, like 'For actual details on contributing to this repo go here' in CONTRIBUTING.rst and link to contributing.rst | 20:10 |
| diablo_rojo | and then have the include in contributing.rst to CONTRIBUTING.rst? | 20:10 |
| fungi | diablo_rojo: if there's something worth mentioning about doc/source/contributor/contributing.rst which isn't already conveyed in the CONTRIBUTING.rst file then i guess you could mention it, though i imagined it would just be an empty shell with a .. include:: ../../../CONTRIBUTING.rst | 20:10 |
| AJaeger | ianw: so, want to be safe and complex - or go for it? | 20:10 |
| fungi | diablo_rojo: is there something projects are going to want in doc/source/contributor/contributing.rst which doesn't belong in CONTRIBUTING.rst? | 20:11 |
| ianw | AJaeger: no, i think we can just do it quickly. first though we need to be serving that directory from a site in https://opendev.org/opendev/system-config/src/branch/master/playbooks/roles/static/files | 20:11 |
| diablo_rojo | fungi, but then the template should be CONTRIBUTING.rst and currently its not.. its contributing.rst. Unless my brain is broken or I don't understand the cookiecutter repo (which I feel is likely at this point) | 20:12 |
| ianw | AJaeger: so we should depend-on a change to do that. i can do an initial rsync for testing, then we can quickly merge the publishing update and move the dns close together | 20:12 |
| AJaeger | ianw: ok, let me update sotryboard | 20:12 |
| * diablo_rojo needs more caffeine | 20:12 | |
| AJaeger | ianw: can you do the sstem-config one, please? | 20:13 |
| fungi | diablo_rojo: i think that may be because someone recently added a template there? | 20:13 |
| * mordred goes to find lunch | 20:13 | |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Switch specs publishing to AFS https://review.opendev.org/708500 | 20:14 |
| fungi | diablo_rojo: looking back through the repo history, i think i'm actually arguing for a revert of https://review.opendev.org/691737 | 20:17 |
| fungi | we used to have {{cookiecutter.repo_name}}/doc/source/contributor/contributing.rst as an empty shell including the ../../../CONTRIBUTING.rst file to get its content | 20:17 |
| fungi | er, i mean revert of https://review.opendev.org/696001 | 20:18 |
| openstackgerrit | Ian Wienand proposed opendev/system-config master: [wip] add specs.openstack.org https://review.opendev.org/708507 | 20:19 |
| fungi | it looks like zaneb argued in that change that some contributor documentation doesn't belong in the CONTRIBUTING.rst file. i disagree but missed the discussion in the midst of holiday travel | 20:19 |
| diablo_rojo | Okay so.. the template shouldnt have been in {{cookiecutter.repo_name}}/doc/source/contributor/contributing.rst there.. like I had done.. it should be in {{cookiecutter.repo_name}}/CONTRIBUTING.rst ? | 20:20 |
| ianw | AJaeger: ^^ that should be the basics | 20:20 |
| diablo_rojo | well I guess my original approach was the top level CONTRIBUTING.rst but thats for the actual cookie cutter repo so it should have been the file one level lower. | 20:21 |
| fungi | diablo_rojo: in my opinion, yes. contributor documentation (whatever contributor documentation a project thinks is relevant) should be in their CONTRIBUTING.rst file and then their doc/source/contributor/contributing.rst is ideally just glue to make it so sphinx can find CONTRIBUTING.rst and incorporate it into documentation bulds | 20:21 |
| fungi | builds | 20:21 |
| diablo_rojo | fungi, that makes sense to me and I am happy to go and fix that. | 20:21 |
| fungi | well, there are people who argued in 696001 that CONTRIBUTING.rst and doc/source/contributor/contributing.rst should contain different information, so that probably needs discussion | 20:22 |
| fungi | i think projects ought to be able to do either way and still meet the cycle goal, but i'd rather the cookiecutter example didn't recommend folks split contributor information between those files | 20:23 |
| diablo_rojo | fungi, I think the way you are proposing makes the most sense. I can put up a patch at least to rehash that argument. I think I would rather not revert the template patch, but start a new one with putting the template in the right place? | 20:25 |
| AJaeger | ianw: I'll add a depends-on, thanks | 20:25 |
| fungi | diablo_rojo: yeah, i'm adding another comment | 20:25 |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Switch specs publishing to AFS https://review.opendev.org/708500 | 20:26 |
| *** smarcet has joined #openstack-infra | 20:26 | |
| openstackgerrit | Kendall Nelson proposed openstack/cookiecutter master: Fix Contributing template location https://review.opendev.org/708511 | 20:40 |
| diablo_rojo | fungi, ^^ should be what you were talking about? | 20:40 |
| *** beagles has quit IRC | 20:44 | |
| fungi | diablo_rojo: almost, {{cookiecutter.repo_name}}/CONTRIBUTING.rst instead i think? | 20:45 |
| *** andrewbonney has quit IRC | 20:45 | |
| diablo_rojo | le sigh | 20:45 |
| diablo_rojo | Yes that would be correct. | 20:45 |
| diablo_rojo | that top level CONTRIBUTING keeps tripping me up | 20:45 |
| fungi | working on the cookiecutter templates is a little mind-bending at times, yeah | 20:45 |
| *** michael-beaver has quit IRC | 20:49 | |
| AJaeger | ianw, mordred, mnaser, https://review.opendev.org/708514 is step one for service-type publishing to use AFS promote job. | 20:49 |
| *** jamesmcarthur_ has quit IRC | 20:52 | |
| *** jamesmcarthur has joined #openstack-infra | 20:53 | |
| openstackgerrit | Kendall Nelson proposed openstack/cookiecutter master: Fix Contributing template location https://review.opendev.org/708511 | 20:53 |
| diablo_rojo | fungi, ^ finally | 20:53 |
| * diablo_rojo goes to get lunch and caffeine | 20:53 | |
| fungi | mmm, caffeine lunch | 20:57 |
| *** jamesmcarthur has quit IRC | 20:58 | |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Add promote job for service-types-authority https://review.opendev.org/708517 | 21:06 |
| AJaeger | ianw: and service-types-authority as well ^ | 21:07 |
| AJaeger | mnaser: I'll take this over ^ | 21:08 |
| *** Goneri has joined #openstack-infra | 21:08 | |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Remove publish-service-types-authority https://review.opendev.org/708518 | 21:10 |
| AJaeger | config-core, please review https://review.opendev.org/708517 | 21:10 |
| *** jamesmcarthur has joined #openstack-infra | 21:12 | |
| *** jamesmcarthur has quit IRC | 21:13 | |
| *** jamesmcarthur has joined #openstack-infra | 21:13 | |
| openstackgerrit | Andreas Jaeger proposed openstack/project-config master: Switch specs publishing to AFS https://review.opendev.org/708500 | 21:16 |
| AJaeger | ianw: so, taken care of specs and service-types - could you update system-config for service-types-authority as well, please? | 21:18 |
| * AJaeger calls it a day and waves good night | 21:18 | |
| clarkb | AJaeger: good night, thank you for the help | 21:19 |
| clarkb | ianw: https://review.opendev.org/#/c/708507/1 lgtm, I know its your day off so no rush but wanted to let you know I think you are good to rsync that content (if not done already) | 21:25 |
| *** takamatsu has quit IRC | 21:32 | |
| openstackgerrit | Merged zuul/nodepool master: Dockerfile: create a nodepool user with uid 10001 https://review.opendev.org/705497 | 21:32 |
| openstackgerrit | wes hayutin proposed openstack/diskimage-builder master: WIP: Validate virtualenv and pip https://review.opendev.org/707104 | 21:45 |
| *** aaronsheffield has quit IRC | 22:03 | |
| *** matt_kosut has joined #openstack-infra | 22:12 | |
| *** rh-jelabarre has quit IRC | 22:14 | |
| *** rcernin has joined #openstack-infra | 22:15 | |
| openstackgerrit | Merged zuul/zuul master: Handle draft pull requests in canMerge https://review.opendev.org/704150 | 22:24 |
| openstackgerrit | Merged zuul/zuul master: Disable pip cache when managing ansible https://review.opendev.org/704184 | 22:25 |
| *** rfolco has quit IRC | 22:26 | |
| clarkb | I've submitted the ptg form for infra/opendev | 22:37 |
| clarkb | requested space for between half a day and 2 days and noted we can be flexible. I think what we found in shanghai was that our small group could manage adhoc meetups pretty well, but having an official presence of some sort was helpful for people that were looking for us | 22:37 |
| *** slaweq has quit IRC | 22:38 | |
| *** migawa is now known as migawa|AFK|AFK | 22:39 | |
| fungi | sgtm | 22:44 |
| mordred | clarkb, fungi: got a sec for an easy one: https://review.opendev.org/#/c/707668/ ? | 22:46 |
| mordred | clarkb: ++ | 22:46 |
| *** haleyb is now known as haleyb|away | 22:46 | |
| fungi | approvidado | 22:47 |
| mordred | fungi: tyty | 22:47 |
| *** jamesmcarthur has quit IRC | 22:49 | |
| *** rcernin has quit IRC | 22:49 | |
| *** rcernin has joined #openstack-infra | 22:50 | |
| openstackgerrit | Merged openstack/project-config master: Add promote job for service-types-authority https://review.opendev.org/708517 | 22:53 |
| openstackgerrit | Merged zuul/zuul master: Flesh out the glossary significantly https://review.opendev.org/704391 | 22:53 |
| *** Goneri has quit IRC | 22:54 | |
| openstackgerrit | Merged openstack/diskimage-builder master: Fix cache-url -f https://review.opendev.org/705902 | 22:55 |
| *** dave-mccowan has quit IRC | 22:55 | |
| *** dave-mccowan has joined #openstack-infra | 22:59 | |
| *** migawa|AFK|AFK is now known as migawa | 22:59 | |
| *** pkopec has quit IRC | 22:59 | |
| *** jamesmcarthur has joined #openstack-infra | 23:01 | |
| *** slaweq has joined #openstack-infra | 23:11 | |
| fungi | clarkb: smcginnis: just to be clear, the previous puppet module release failures got rerun and are all fine now, but the sushy-cli failure needed a new release change... there are no outstanding openstack release failures which need resolving now right? | 23:15 |
| clarkb | fungi: that is my understanding | 23:15 |
| fungi | okay, cool, removing from my todo list | 23:15 |
| *** slaweq has quit IRC | 23:16 | |
| openstackgerrit | Merged zuul/zuul master: Log duration of AnsibleJob.execute() https://review.opendev.org/704589 | 23:19 |
| *** jamesmcarthur has quit IRC | 23:19 | |
| openstackgerrit | Merged opendev/system-config master: Add curl to python-builder image https://review.opendev.org/707668 | 23:20 |
| *** iurygregory has quit IRC | 23:21 | |
| clarkb | mordred: looks like we have the votes to land the gitea upgrade now | 23:31 |
| clarkb | mordred: I need to mow the yard before it gets dark but can be help watch around that | 23:31 |
| clarkb | or we can do it tomrrow (but I'll be distracted by meetings much of the day) | 23:31 |
| mordred | clarkb: let's do it tomorrow | 23:34 |
| mordred | clarkb: I'm pretty much at EOD and have a parade and a ball tonight | 23:34 |
| clarkb | oh its that time of year in your new part of the world isn't it | 23:34 |
| mordred | that's right! | 23:34 |
| mordred | oh - speaking of - I meant to mention in the meeting | 23:34 |
| mordred | I will not be here next Tue | 23:34 |
| *** dchen has joined #openstack-infra | 23:36 | |
| *** rlandy has quit IRC | 23:41 | |
| openstackgerrit | Merged zuul/zuul master: ansible manager: only failed if last ansible failed https://review.opendev.org/704699 | 23:44 |
| *** smarcet has quit IRC | 23:45 | |
| mordred | clarkb: also - I will likely get a late start tomorrow - but I will be here | 23:47 |
| clarkb | mordred: k, ping me when you want to get started and we'll see where I'm at in meetings | 23:48 |
| mordred | coolio | 23:48 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!