pabelanger | prometheanfire: ianw: we should see about adding a non-voting jobs to both glean and diskimage-builder too, and depends-on the nodepool change | 00:00 |
---|---|---|
pabelanger | mordred: wfm | 00:01 |
mordred | corvus: that's kinda my feeling ... I could switch it - but I think I mostly just don't care :) | 00:02 |
openstackgerrit | Monty Taylor proposed openstack-infra/puppet-zuul master: Support updated REST and HTML layout https://review.openstack.org/556973 | 00:02 |
mordred | btw - the last 2 puppet spec test failures on puppet-zuul were actual real bugs | 00:03 |
mordred | so - yay for testing | 00:03 |
*** germs has quit IRC | 00:04 | |
openstackgerrit | Monty Taylor proposed openstack-infra/puppet-zuul master: Split v2 and v3 web config https://review.openstack.org/556869 | 00:04 |
openstackgerrit | Monty Taylor proposed openstack-infra/puppet-zuul master: Support updated REST and HTML layout https://review.openstack.org/556973 | 00:04 |
*** s-shiono has joined #openstack-infra | 00:04 | |
*** salv-orlando has joined #openstack-infra | 00:05 | |
corvus | okay, the cgit update is in place on all backends | 00:05 |
mordred | corvus: \o/ | 00:06 |
*** Swami has quit IRC | 00:06 | |
*** felipemonteiro has joined #openstack-infra | 00:07 | |
corvus | pabelanger, mordred, fungi: i haven't gotton a misfire yet, and my reload finger is getting sore | 00:08 |
*** salv-orlando has quit IRC | 00:10 | |
*** rlandy is now known as rlandy|bbl | 00:11 | |
*** rkukura_ has joined #openstack-infra | 00:11 | |
*** mrhillsman is now known as mrhillsman_in_mm | 00:11 | |
*** mythicalphoenix has quit IRC | 00:12 | |
*** yamamoto has joined #openstack-infra | 00:12 | |
*** technotony has joined #openstack-infra | 00:12 | |
*** mrhillsman_in_mm is now known as mrhillsman | 00:13 | |
*** openstack has joined #openstack-infra | 00:15 | |
*** ChanServ sets mode: +o openstack | 00:15 | |
prometheanfire | pabelanger: :D thanks | 00:15 |
corvus | i'll mark that spec complete after https://review.openstack.org/555104 lands -- there's a merge conflict and i don't want to rebase it | 00:15 |
* prometheanfire sees if clarkb is around (and doubts it) | 00:15 | |
*** technotony has joined #openstack-infra | 00:17 | |
*** yamamoto has quit IRC | 00:17 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul-website master: Update links to git.zuul-ci.org https://review.openstack.org/557099 | 00:18 |
openstackgerrit | Merged openstack-infra/puppet-zuul master: Add new parameters to zuul::web https://review.openstack.org/557084 | 00:20 |
openstackgerrit | Merged openstack-infra/system-config master: Pass web settings to zuul::web https://review.openstack.org/557085 | 00:20 |
*** stakeda has joined #openstack-infra | 00:20 | |
*** aeng has joined #openstack-infra | 00:20 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Use git.zuul-ci.org https://review.openstack.org/557100 | 00:21 |
*** dave-mccowan has joined #openstack-infra | 00:21 | |
ianw | pabelanger: ok ... did we verify things on limestone first? | 00:27 |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: Do not merge: Test limestone nodes https://review.openstack.org/556849 | 00:27 |
ianw | pabelanger / dmsimard: ^ running things on limestone to see what happens | 00:28 |
mordred | corvus, pabelanger, ianw: zomg. https://review.openstack.org/#/c/556869/ is actually green | 00:28 |
pabelanger | ianw: ah, yah. I knew there was a test | 00:28 |
pabelanger | mordred: woot | 00:29 |
*** VW has joined #openstack-infra | 00:30 | |
*** VW_ has quit IRC | 00:30 | |
mordred | so - it should be safe to land | 00:31 |
ianw | mordred: did you change the zuul dots? they're a little flat? https://imgur.com/a/LuGE8 | 00:31 |
mordred | ianw: I did not - that bug has actually been around for a bit -Ive got a fix for it in the javascript stack though | 00:31 |
corvus | oh, is it the *dots* that are wrong, i assumed it was the lines | 00:32 |
ianw | ok, cool, yeah i thought i'd seen something about it | 00:32 |
mordred | infra-root: I procedural -2'd the main zuul api change patch https://review.openstack.org/#/c/556967 - tomorrow we can get it landed and rolled out when I'm around to watch/fix | 00:32 |
corvus | mordred: 869 has 2x+2. i'm about to afk so don't want to +W | 00:32 |
mordred | corvus: ++ - same here | 00:33 |
mordred | I think we're set to go for tomorrow | 00:33 |
pabelanger | sounds like a plan | 00:33 |
ianw | pabelanger: openstack-infra-base-integration-opensuse423 -- node_failure ... that's a bummer | 00:33 |
*** gyee has quit IRC | 00:33 | |
pabelanger | ianw: we can only launch 10 nodes, wonder if that is the issue | 00:34 |
pabelanger | we are also hold 2 nodes | 00:34 |
pabelanger | let me see if we can delete them | 00:34 |
pabelanger | yah | 00:34 |
pabelanger | they are from me | 00:34 |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: DNM: debug some ipv6 variables https://review.openstack.org/557022 | 00:34 |
corvus | pabelanger: being at quota should not produce a node_failure | 00:35 |
pabelanger | corvus: good point, I'll check logs | 00:35 |
clarkb | ya I fixedthat bug | 00:35 |
clarkb | its tested now too | 00:35 |
corvus | clarkb: oh there was a bug? were launchers restarted with it? | 00:35 |
pabelanger | Feb13 is when it was last started | 00:36 |
pabelanger | on nl02 at least | 00:36 |
clarkb | corvus: ya this was forever ago | 00:36 |
clarkb | like december or january | 00:36 |
*** yamahata has joined #openstack-infra | 00:38 | |
*** yamamoto has joined #openstack-infra | 00:38 | |
openstackgerrit | zhongshengping proposed openstack-infra/project-config master: Retire puppet-ganesha(1/2) https://review.openstack.org/556696 | 00:39 |
*** felipemonteiro has quit IRC | 00:42 | |
pabelanger | clarkb: corvus: ianw: the nodeset in https://review.openstack.org/556849 for opensuse has a typo | 00:43 |
pabelanger | so the nodes aren't valid | 00:43 |
Jeffrey4l | ifra-root, since kolla create a new group "kolla-core-cli" https://review.openstack.org/#/admin/groups/1889,members , could you add me as the initial core member? thanks. | 00:43 |
Jeffrey4l | infra-root ^^ | 00:43 |
ianw | Jefrey4l: done | 00:44 |
ianw | pabelanger: heh, ok that would do it | 00:45 |
Jeffrey4l | thanks | 00:45 |
*** yamamoto has quit IRC | 00:45 | |
pabelanger | ianw: given that multinodes jobs passed, I'm good assuming single node does too | 00:46 |
*** efoley has quit IRC | 00:47 | |
openstackgerrit | zhongshengping proposed openstack-infra/project-config master: Retire puppet-ganesha(1/2) https://review.openstack.org/556696 | 00:47 |
ianw | pabelanger: it's so *weird* ... i've reverted back to "bare" ansible_default_ipv6 is defined in https://review.openstack.org/#/c/557022 and not seeing any failures to pick up ipv6 | 00:48 |
*** yamahata has quit IRC | 00:49 | |
pabelanger | ianw: have no thoughts | 00:50 |
ianw | heisenbug | 00:50 |
ianw | once it finishes, i'll try a run without the debug statements, if that passes ... ¯\_(ツ)_/¯ | 00:52 |
*** odyssey4me has quit IRC | 00:53 | |
*** odyssey4me has joined #openstack-infra | 00:53 | |
ianw | pabelanger: i also remember why we left the glean nodepool-src tests in project-config now too, because they were in queue: glean | 00:53 |
openstackgerrit | zhongshengping proposed openstack-infra/project-config master: Retire puppet-ganesha(2/2) https://review.openstack.org/556789 | 00:54 |
*** yamamoto has joined #openstack-infra | 00:56 | |
*** rkukura has quit IRC | 00:58 | |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: Do not merge: Test limestone nodes https://review.openstack.org/556849 | 01:04 |
openstackgerrit | Ian Wienand proposed openstack-infra/openstack-zuul-jobs master: DNM: debug some ipv6 variables https://review.openstack.org/557022 | 01:05 |
*** bobh has joined #openstack-infra | 01:05 | |
*** pahuang_ has quit IRC | 01:07 | |
Jeffrey4l | infra-root, recently, i tried to add a serial jobs in release pipeline in openstack/kolla project. yesterday, a new release 5.0.2 is release, but the jobs i added seems not loaded. is anyone could check this why? | 01:12 |
Jeffrey4l | here is the related patch https://review.openstack.org/556701 | 01:12 |
Jeffrey4l | here is the related release piple result http://logs.openstack.org/92/921bb0dc8b559e9cf3f6b5c17d0c6597c4debc44/release/ | 01:12 |
*** gyankum has joined #openstack-infra | 01:12 | |
*** yamamoto has quit IRC | 01:14 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: autohold: prevent using ',' in change argument https://review.openstack.org/556329 | 01:15 |
*** prometheanfire has quit IRC | 01:15 | |
*** gongysh has joined #openstack-infra | 01:16 | |
clarkb | Jeffrey4l: do those jobs exist on pike? | 01:16 |
Jeffrey4l | clarkb, yes. it is backported by https://review.openstack.org/#/c/556701/ | 01:17 |
*** bobh has quit IRC | 01:17 | |
Jeffrey4l | and 5.0.2 tag includes that patch. | 01:17 |
clarkb | looks like its in the same file ya | 01:17 |
*** notmyname has quit IRC | 01:17 | |
clarkb | I'm wondering if it is because tags are not directly associated to branches if this is why its unhappy | 01:18 |
Jeffrey4l | clarkb, maybe. is there any way i can add jobs in release pipeline? | 01:18 |
clarkb | you can add them in project config but I think we need to understand this better | 01:19 |
Jeffrey4l | i cloud. But i think this should be fixed in zuul, too. it is a noraml feature. | 01:20 |
clarkb | http://logs.openstack.org/92/921bb0dc8b559e9cf3f6b5c17d0c6597c4debc44/release/ is what it did run | 01:21 |
clarkb | can you find where those are defined for kolla? maybe in project config (hoping it gives us more info and I'm on a phone) | 01:21 |
*** cshastri has joined #openstack-infra | 01:22 | |
Jeffrey4l | these are not configure in kolla project. them should be configure in project-config and inherit from a jobs template. | 01:22 |
*** hongbin has joined #openstack-infra | 01:22 | |
clarkb | ya thats my guess | 01:22 |
*** prometheanfire has joined #openstack-infra | 01:22 | |
*** notmyname has joined #openstack-infra | 01:22 | |
clarkb | ya its project config | 01:23 |
clarkb | I think this may be one for corvus to understamd how tags map to bramches | 01:23 |
Jeffrey4l | yeah, i found it too. | 01:23 |
Jeffrey4l | cool, hope this featur could land :D | 01:24 |
*** pahuang_ has joined #openstack-infra | 01:25 | |
*** technotony has quit IRC | 01:25 | |
*** prometheanfire has quit IRC | 01:29 | |
*** armaan has quit IRC | 01:30 | |
*** armaan has joined #openstack-infra | 01:31 | |
dmsimard | ianw: re: logstash and ansible | 01:33 |
dmsimard | I want to wave my hands in several directions and mention ara | 01:33 |
openstackgerrit | zhongshengping proposed openstack-infra/project-config master: Retire puppet-ganesha(1/2) https://review.openstack.org/556696 | 01:34 |
dmsimard | But I'm going to bed early, but I wanted you to know that I have an opinion on the thing, especially with the eventual new version of ARA | 01:34 |
dmsimard | So I'll try to remember to tell you about my opinion :D | 01:35 |
dmsimard | Also, for those interested, my OpenStack meetup talk and openstack-infra went well and I'll share a YouTube link when it's up :D | 01:35 |
*** sree has quit IRC | 01:36 | |
*** sree has joined #openstack-infra | 01:37 | |
openstackgerrit | zhongshengping proposed openstack-infra/project-config master: Retire puppet-ganesha(2/2) https://review.openstack.org/556789 | 01:38 |
*** gongysh has quit IRC | 01:38 | |
*** vivsoni has quit IRC | 01:39 | |
*** gouthamr has joined #openstack-infra | 01:40 | |
*** vivsoni has joined #openstack-infra | 01:41 | |
*** sree has quit IRC | 01:41 | |
*** bobh has joined #openstack-infra | 01:45 | |
*** rosmaita has quit IRC | 01:50 | |
*** liusheng has quit IRC | 01:51 | |
*** liusheng has joined #openstack-infra | 01:52 | |
*** bobh has quit IRC | 01:53 | |
*** prometheanfire has joined #openstack-infra | 01:55 | |
*** germs has joined #openstack-infra | 02:00 | |
*** germs has quit IRC | 02:00 | |
*** germs has joined #openstack-infra | 02:00 | |
*** ykarel|away has joined #openstack-infra | 02:02 | |
*** germs has quit IRC | 02:04 | |
*** liusheng has quit IRC | 02:06 | |
*** salv-orlando has joined #openstack-infra | 02:07 | |
*** liusheng has joined #openstack-infra | 02:07 | |
*** cshastri has quit IRC | 02:08 | |
*** psachin has joined #openstack-infra | 02:11 | |
*** ramishra has joined #openstack-infra | 02:11 | |
*** salv-orlando has quit IRC | 02:12 | |
*** ykarel|away has quit IRC | 02:14 | |
*** jamesmcarthur has joined #openstack-infra | 02:15 | |
*** yamamoto has joined #openstack-infra | 02:15 | |
*** ykarel|away has joined #openstack-infra | 02:16 | |
*** cshastri has joined #openstack-infra | 02:18 | |
*** yamamoto has quit IRC | 02:21 | |
*** pahuang_ has quit IRC | 02:21 | |
*** VW has quit IRC | 02:25 | |
*** VW has joined #openstack-infra | 02:26 | |
*** zhurong has joined #openstack-infra | 02:33 | |
*** pahuang_ has joined #openstack-infra | 02:34 | |
*** ykarel|away has quit IRC | 02:34 | |
*** gongysh has joined #openstack-infra | 02:35 | |
*** zhangfei has joined #openstack-infra | 02:35 | |
*** yamamoto has joined #openstack-infra | 02:37 | |
*** sree has joined #openstack-infra | 02:39 | |
*** sree has quit IRC | 02:39 | |
*** sree has joined #openstack-infra | 02:39 | |
*** zhangfei has quit IRC | 02:40 | |
*** zhangfei has joined #openstack-infra | 02:41 | |
corvus | dmsimard: well, there's a mailing list thread, so don't fret, it'll still be there tomorrow. but i think the thing to keep in mind here that this is searching across all builds. | 02:41 |
corvus | Jeffrey4l, clarkb: when the jobs are added to kolla on a branch, they get an implied branch specifier, so https://review.openstack.org/556701 really means "run these jobs on changes to the stable/pike branch" because that's the branch it modified. we might be able to make this work the way you expect in the future, however, at the moment, i think the best thing to do is to remove those job | 02:46 |
corvus | invocations from the project definition in the kolla repo, and instead add them in project-config. | 02:46 |
*** armaan has quit IRC | 02:59 | |
*** yamamoto has quit IRC | 03:18 | |
*** jamesmcarthur has quit IRC | 03:21 | |
*** dhajare_ has joined #openstack-infra | 03:29 | |
*** janki has joined #openstack-infra | 03:32 | |
openstackgerrit | Tony Breeds proposed openstack-infra/irc-meetings master: Make Lujin Luo the main chair for Neutron upgrades meeting https://review.openstack.org/556990 | 03:39 |
*** anticw has quit IRC | 03:42 | |
*** anticw has joined #openstack-infra | 03:43 | |
*** hongbin has quit IRC | 03:43 | |
*** cshastri has quit IRC | 03:45 | |
*** cshastri has joined #openstack-infra | 03:45 | |
*** andreas_s has joined #openstack-infra | 03:46 | |
openstackgerrit | Merged openstack-infra/irc-meetings master: Make Lujin Luo the main chair for Neutron upgrades meeting https://review.openstack.org/556990 | 03:48 |
*** andreas_s has quit IRC | 03:51 | |
*** jamesmcarthur has joined #openstack-infra | 03:54 | |
*** dave-mccowan has quit IRC | 03:55 | |
*** VW has quit IRC | 03:55 | |
*** VW has joined #openstack-infra | 03:55 | |
*** jaosorior has joined #openstack-infra | 03:56 | |
*** udesale has joined #openstack-infra | 03:57 | |
*** VW_ has joined #openstack-infra | 04:00 | |
*** VW has quit IRC | 04:00 | |
*** germs has joined #openstack-infra | 04:01 | |
*** germs has quit IRC | 04:01 | |
*** germs has joined #openstack-infra | 04:01 | |
*** germs has quit IRC | 04:06 | |
*** stakeda has quit IRC | 04:07 | |
*** sshnaidm_ has joined #openstack-infra | 04:08 | |
*** sshnaidm has quit IRC | 04:08 | |
*** salv-orlando has joined #openstack-infra | 04:08 | |
*** salv-orlando has quit IRC | 04:13 | |
*** ykarel|away has joined #openstack-infra | 04:15 | |
*** yamamoto has joined #openstack-infra | 04:19 | |
*** gongysh has quit IRC | 04:19 | |
*** gouthamr has quit IRC | 04:22 | |
*** VW_ has quit IRC | 04:23 | |
*** VW has joined #openstack-infra | 04:24 | |
*** yamamoto has quit IRC | 04:25 | |
*** psachin has quit IRC | 04:25 | |
*** namnh has joined #openstack-infra | 04:27 | |
*** VW has quit IRC | 04:29 | |
*** sshnaidm_ has quit IRC | 04:32 | |
*** cshastri has quit IRC | 04:39 | |
*** psachin has joined #openstack-infra | 04:43 | |
*** links has joined #openstack-infra | 04:46 | |
*** sshnaidm_ has joined #openstack-infra | 04:48 | |
*** cshastri has joined #openstack-infra | 04:54 | |
*** cshastri has quit IRC | 04:54 | |
*** cshastri has joined #openstack-infra | 04:55 | |
*** lpetrut has joined #openstack-infra | 04:55 | |
*** udesale has quit IRC | 04:58 | |
*** udesale has joined #openstack-infra | 04:59 | |
*** armaan has joined #openstack-infra | 05:04 | |
*** sshnaidm_ has quit IRC | 05:09 | |
*** salv-orlando has joined #openstack-infra | 05:09 | |
*** salv-orlando has quit IRC | 05:09 | |
*** salv-orlando has joined #openstack-infra | 05:09 | |
AJaeger | dhellmann, prometheanfire, I commented on https://review.openstack.org/#/c/555426/ - if prometheanfire gives a +1, I'm fine to proceed. | 05:14 |
*** pgadiya has joined #openstack-infra | 05:19 | |
*** yamamoto has joined #openstack-infra | 05:21 | |
*** dhajare_ is now known as dhajare | 05:23 | |
*** sshnaidm has joined #openstack-infra | 05:24 | |
*** lpetrut has quit IRC | 05:26 | |
*** e0ne has joined #openstack-infra | 05:26 | |
*** lpetrut has joined #openstack-infra | 05:27 | |
*** yamamoto has quit IRC | 05:27 | |
openstackgerrit | zhongshengping proposed openstack-infra/project-config master: Retire puppet-ganesha(2/2) https://review.openstack.org/556789 | 05:27 |
*** e0ne has quit IRC | 05:33 | |
*** aeng has quit IRC | 05:34 | |
*** liusheng has quit IRC | 05:35 | |
*** liusheng has joined #openstack-infra | 05:36 | |
*** lpetrut has quit IRC | 05:39 | |
*** jtomasek has joined #openstack-infra | 05:43 | |
*** rlandy|bbl has quit IRC | 05:43 | |
*** armaan has quit IRC | 05:49 | |
*** e0ne has joined #openstack-infra | 05:50 | |
*** aeng has joined #openstack-infra | 05:51 | |
*** lpetrut has joined #openstack-infra | 05:52 | |
*** gfidente has joined #openstack-infra | 05:54 | |
*** gfidente has joined #openstack-infra | 05:54 | |
*** cshastri has quit IRC | 05:57 | |
*** claudiub|2 has joined #openstack-infra | 05:59 | |
*** greghaynes has quit IRC | 06:01 | |
*** aeng has quit IRC | 06:02 | |
*** germs has joined #openstack-infra | 06:02 | |
*** germs has quit IRC | 06:02 | |
*** germs has joined #openstack-infra | 06:02 | |
*** gongysh has joined #openstack-infra | 06:02 | |
*** germs has quit IRC | 06:06 | |
*** cshastri has joined #openstack-infra | 06:10 | |
*** e0ne has quit IRC | 06:10 | |
*** greghaynes has joined #openstack-infra | 06:10 | |
*** garyk has quit IRC | 06:11 | |
*** dhajare has quit IRC | 06:11 | |
*** vaidy has quit IRC | 06:13 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack-infra/project-config master: Normalize projects.yaml https://review.openstack.org/557157 | 06:14 |
*** Gorian has quit IRC | 06:14 | |
*** garyk has joined #openstack-infra | 06:14 | |
*** garyk has quit IRC | 06:15 | |
*** armaan has joined #openstack-infra | 06:15 | |
Jeffrey4l | infra-root it is possible that re-use a playbooks from kolla project in openstack-zuul-jobs' job definition? | 06:18 |
*** isviridov_away has quit IRC | 06:18 | |
*** Gorian has joined #openstack-infra | 06:21 | |
Jeffrey4l | now i have migrate some jobs from kolla to openstack-zuul-jobs(another story), is there any eailer way rather than copy all playbooks to openstack-zuul-jobs? | 06:23 |
*** yamamoto has joined #openstack-infra | 06:23 | |
*** gfidente has quit IRC | 06:24 | |
*** florianf has joined #openstack-infra | 06:24 | |
*** claudiub has joined #openstack-infra | 06:26 | |
*** garyk has joined #openstack-infra | 06:26 | |
*** chenying_ has joined #openstack-infra | 06:27 | |
*** chenying has quit IRC | 06:27 | |
*** udesale_ has joined #openstack-infra | 06:28 | |
*** auristor has joined #openstack-infra | 06:28 | |
*** garyk has quit IRC | 06:28 | |
*** yamamoto has quit IRC | 06:28 | |
*** auristor1 has quit IRC | 06:28 | |
*** claudiub|2 has quit IRC | 06:29 | |
*** Hal has quit IRC | 06:29 | |
*** jaosorior has quit IRC | 06:29 | |
*** ykarel_ has joined #openstack-infra | 06:30 | |
*** udesale has quit IRC | 06:30 | |
*** udesale__ has joined #openstack-infra | 06:30 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Retire puppet-ganesha(2/2) https://review.openstack.org/556789 | 06:31 |
*** dbecker has quit IRC | 06:32 | |
*** jtomasek has quit IRC | 06:32 | |
*** ykarel|away has quit IRC | 06:32 | |
*** udesale_ has quit IRC | 06:33 | |
AJaeger | Jeffrey4l: you cannot reuse playbooks, only roles. What are you trying to achieve? | 06:33 |
*** kiennt26 has joined #openstack-infra | 06:34 | |
AJaeger | Jeffrey4l: migration of jobs from kolla to openstack-zuul-jobs looks strange | 06:34 |
Jeffrey4l | AJaeger, i added some release jobs in kolla project. but it won't work as expected. corvus told me better migrate the jobs to infra project. | 06:34 |
Jeffrey4l | check http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2018-03-28.log.html#t2018-03-28T01:12:04 | 06:35 |
Jeffrey4l | and http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2018-03-28.log.html#t2018-03-28T02:41:58 | 06:35 |
AJaeger | Jeffrey4l: he said "invocations of the jobs", so just move the job invocation to zuul.d/projects.yaml in project-config. | 06:37 |
*** garyk has joined #openstack-infra | 06:38 | |
AJaeger | Jeffrey4l: leave the job definition where it is for now - but please double check with corvus | 06:38 |
*** pcaruana has joined #openstack-infra | 06:38 | |
Jeffrey4l | ok. i will check with corvus, thanks. | 06:39 |
*** alexchadin has joined #openstack-infra | 06:39 | |
prometheanfire | AJaeger: we are going to try and get the pbr bump in first, (want emonty/nemec's ack), then we move ahead with the per-project reqs | 06:39 |
*** garyk has quit IRC | 06:40 | |
AJaeger | prometheanfire: no pressure from my side ;) | 06:40 |
prometheanfire | about to go to bed, but I'll ask this anyway, just ping me with an answer if you have it. Can storyboard do cross project bugs (one bug, many projects) like launchpad can? | 06:40 |
AJaeger | prometheanfire: no idea ;( | 06:41 |
AJaeger | prometheanfire: good night! | 06:41 |
prometheanfire | nn | 06:41 |
AJaeger | infra-root, http://git.openstack.org/cgit/openstack/kolla-cli/tree/ was created and imported - but https://github.com/openstack/kolla-cli is not setup. Hope that heals magically... | 06:42 |
*** ihrachys has quit IRC | 06:43 | |
*** dbecker has joined #openstack-infra | 06:45 | |
*** dhajare has joined #openstack-infra | 06:47 | |
openstackgerrit | Merged openstack-infra/project-config master: Normalize projects.yaml https://review.openstack.org/557157 | 06:51 |
*** xinliang has quit IRC | 06:52 | |
*** andreas_s has joined #openstack-infra | 06:55 | |
*** lpetrut has quit IRC | 06:58 | |
*** ykarel__ has joined #openstack-infra | 07:03 | |
*** gongysh has quit IRC | 07:04 | |
*** xinliang has joined #openstack-infra | 07:05 | |
*** zhipeng has joined #openstack-infra | 07:06 | |
*** ykarel_ has quit IRC | 07:06 | |
*** zhurong has quit IRC | 07:08 | |
*** jaosorior has joined #openstack-infra | 07:10 | |
*** gongysh has joined #openstack-infra | 07:10 | |
*** ianychoi has quit IRC | 07:10 | |
openstackgerrit | zhongshengping proposed openstack-infra/project-config master: Prefix the project description with RETIRED for retired projects https://review.openstack.org/557166 | 07:11 |
*** ykarel__ is now known as ykarel | 07:13 | |
*** HeOS has joined #openstack-infra | 07:14 | |
*** hashar has joined #openstack-infra | 07:14 | |
*** rcernin has quit IRC | 07:15 | |
*** aviau has quit IRC | 07:19 | |
*** aviau has joined #openstack-infra | 07:19 | |
*** salv-orlando has quit IRC | 07:22 | |
*** salv-orlando has joined #openstack-infra | 07:22 | |
*** tosky has joined #openstack-infra | 07:23 | |
*** yamamoto has joined #openstack-infra | 07:25 | |
*** caphrim007_ has quit IRC | 07:25 | |
*** tesseract has joined #openstack-infra | 07:25 | |
*** caphrim007 has joined #openstack-infra | 07:26 | |
*** alexchadin has quit IRC | 07:26 | |
*** salv-orlando has quit IRC | 07:27 | |
*** electrofelix has joined #openstack-infra | 07:27 | |
*** garyk has joined #openstack-infra | 07:27 | |
*** garyk has quit IRC | 07:28 | |
*** yamamoto has quit IRC | 07:30 | |
*** pcichy has joined #openstack-infra | 07:30 | |
*** kiennt26 has quit IRC | 07:30 | |
*** alexchadin has joined #openstack-infra | 07:31 | |
frickler | prometheanfire: iiuc the idea is that you can have multiple tasks assigned to a story, those tasks may be attached to different projects, then | 07:33 |
*** gfidente has joined #openstack-infra | 07:36 | |
*** gfidente has joined #openstack-infra | 07:36 | |
frickler | prometheanfire: stories (the generic term for what lp calls bugs) aren't directly associated to projects at all, only via their tasks | 07:37 |
*** shardy has joined #openstack-infra | 07:38 | |
*** udesale__ is now known as udesale | 07:43 | |
*** salv-orlando has joined #openstack-infra | 07:45 | |
*** HeOS has quit IRC | 07:46 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack-infra/project-config master: proposal: OSA: Add support for removing files https://review.openstack.org/557176 | 07:49 |
*** zhangfei has quit IRC | 07:51 | |
*** bmace has quit IRC | 07:51 | |
*** bmace has joined #openstack-infra | 07:52 | |
*** salv-orlando has quit IRC | 07:54 | |
*** ralonsoh has joined #openstack-infra | 07:54 | |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack-infra/project-config master: proposal: OSA: Do not sync tests/tests-repo-clone.sh anymore https://review.openstack.org/557176 | 07:54 |
*** salv-orlando has joined #openstack-infra | 07:54 | |
*** lucas-afk is now known as lucasagomes | 07:56 | |
*** salv-orlando has quit IRC | 07:59 | |
*** germs has joined #openstack-infra | 08:03 | |
*** germs has quit IRC | 08:03 | |
*** germs has joined #openstack-infra | 08:03 | |
*** arxcruz|rover is now known as arxcruz|off | 08:03 | |
*** jpich has joined #openstack-infra | 08:04 | |
*** zhangfei has joined #openstack-infra | 08:07 | |
*** germs has quit IRC | 08:08 | |
*** derekh has joined #openstack-infra | 08:08 | |
*** Hal has joined #openstack-infra | 08:11 | |
kashyap | fungi: Excellent (on the wiki expiration). Thanks! | 08:11 |
*** zhipeng has quit IRC | 08:16 | |
*** zhipeng has joined #openstack-infra | 08:16 | |
*** Hal1 has joined #openstack-infra | 08:17 | |
*** Hal has quit IRC | 08:19 | |
*** priteau has joined #openstack-infra | 08:20 | |
*** pcichy has quit IRC | 08:22 | |
*** gongysh has quit IRC | 08:23 | |
*** rcernin has joined #openstack-infra | 08:24 | |
*** iranzo has joined #openstack-infra | 08:24 | |
*** s-shiono has quit IRC | 08:26 | |
*** yamamoto has joined #openstack-infra | 08:26 | |
*** e0ne has joined #openstack-infra | 08:27 | |
*** yamamoto has quit IRC | 08:30 | |
*** lihi has quit IRC | 08:32 | |
*** oanson has quit IRC | 08:34 | |
*** oanson has joined #openstack-infra | 08:36 | |
*** lihi has joined #openstack-infra | 08:39 | |
*** lpetrut has joined #openstack-infra | 08:40 | |
*** rcernin has quit IRC | 08:42 | |
*** lpetrut has quit IRC | 08:47 | |
*** lpetrut has joined #openstack-infra | 08:47 | |
*** HeOS has joined #openstack-infra | 08:52 | |
*** alexchadin has quit IRC | 08:57 | |
*** udesale has quit IRC | 08:58 | |
*** alexchadin has joined #openstack-infra | 08:58 | |
*** chenying_ has quit IRC | 08:59 | |
*** udesale has joined #openstack-infra | 09:05 | |
*** bhujay has joined #openstack-infra | 09:10 | |
*** shardy has quit IRC | 09:10 | |
*** shardy has joined #openstack-infra | 09:12 | |
*** zhipeng has quit IRC | 09:22 | |
openstackgerrit | Slawek Kaplonski proposed openstack-infra/project-config master: Add Fullstack gate failure rate to neutron dashboard https://review.openstack.org/557266 | 09:23 |
*** yamamoto has joined #openstack-infra | 09:27 | |
*** yamamoto has quit IRC | 09:32 | |
*** alexchadin has quit IRC | 09:34 | |
*** lpetrut_ has joined #openstack-infra | 09:41 | |
*** vaidy has joined #openstack-infra | 09:44 | |
*** dtantsur|afk is now known as dtantsur | 09:46 | |
*** ykarel is now known as ykarel|afk | 09:53 | |
*** sree has quit IRC | 09:56 | |
*** sree has joined #openstack-infra | 09:57 | |
*** salv-orlando has joined #openstack-infra | 09:58 | |
*** verdurin has joined #openstack-infra | 10:01 | |
*** sree has quit IRC | 10:02 | |
*** adarazs_lunch is now known as adarazs | 10:03 | |
*** germs has joined #openstack-infra | 10:03 | |
*** namnh has quit IRC | 10:03 | |
*** panda|off is now known as panda | 10:04 | |
*** germs has quit IRC | 10:08 | |
*** ykarel|afk is now known as ykarel | 10:09 | |
*** ianychoi has joined #openstack-infra | 10:15 | |
*** sambetts|afk is now known as sambetts | 10:18 | |
*** nhandler has quit IRC | 10:21 | |
*** nhandler has joined #openstack-infra | 10:23 | |
*** armaan has quit IRC | 10:28 | |
*** cshastri has quit IRC | 10:28 | |
*** nhandler has quit IRC | 10:28 | |
*** sshnaidm has quit IRC | 10:29 | |
*** yamamoto has joined #openstack-infra | 10:29 | |
*** nhandler has joined #openstack-infra | 10:30 | |
*** nhandler has quit IRC | 10:30 | |
*** nhandler has joined #openstack-infra | 10:32 | |
*** nhandler has quit IRC | 10:33 | |
*** nhandler has joined #openstack-infra | 10:33 | |
*** yamamoto has quit IRC | 10:34 | |
*** alexchadin has joined #openstack-infra | 10:35 | |
*** zhangfei has quit IRC | 10:36 | |
*** sshnaidm has joined #openstack-infra | 10:38 | |
*** pbourke has quit IRC | 10:44 | |
*** pbourke has joined #openstack-infra | 10:45 | |
*** udesale_ has joined #openstack-infra | 10:46 | |
slaweq | andreaf: hi, can You once again take a look at https://review.openstack.org/#/c/552846/? I addresses Your comments there and I want to be sure that it's fine now :) | 10:48 |
*** Hal1 has quit IRC | 10:48 | |
*** udesale has quit IRC | 10:49 | |
*** udesale_ has quit IRC | 10:52 | |
*** salv-orl_ has joined #openstack-infra | 10:53 | |
*** zhurong has joined #openstack-infra | 10:55 | |
*** zhipeng has joined #openstack-infra | 10:55 | |
*** sshnaidm has quit IRC | 10:55 | |
*** salv-orlando has quit IRC | 10:56 | |
*** gyankum has quit IRC | 10:56 | |
*** vaidy has quit IRC | 10:57 | |
*** salv-orlando has joined #openstack-infra | 10:58 | |
*** salv-orl_ has quit IRC | 10:58 | |
*** nhandler has quit IRC | 10:58 | |
*** nhandler has joined #openstack-infra | 10:59 | |
*** isviridov_away has joined #openstack-infra | 11:02 | |
*** nhandler has joined #openstack-infra | 11:03 | |
*** nhandler has quit IRC | 11:03 | |
*** ykarel_ has joined #openstack-infra | 11:03 | |
*** dhajare_ has joined #openstack-infra | 11:03 | |
*** chkumar246 has joined #openstack-infra | 11:03 | |
*** pgadiya_ has joined #openstack-infra | 11:03 | |
*** ykarel has quit IRC | 11:04 | |
*** links has quit IRC | 11:04 | |
*** chandankumar has quit IRC | 11:04 | |
*** pgadiya has quit IRC | 11:04 | |
*** salv-orlando has quit IRC | 11:05 | |
*** links has joined #openstack-infra | 11:05 | |
*** ykarel__ has joined #openstack-infra | 11:05 | |
*** armaan has joined #openstack-infra | 11:06 | |
*** salv-orlando has joined #openstack-infra | 11:06 | |
*** dhajare has quit IRC | 11:07 | |
*** sshnaidm has joined #openstack-infra | 11:07 | |
*** dhajare_ has quit IRC | 11:08 | |
*** chkumar246 has quit IRC | 11:08 | |
*** ykarel_ has quit IRC | 11:08 | |
*** pgadiya_ has quit IRC | 11:08 | |
*** nhandler has joined #openstack-infra | 11:09 | |
*** dave-mccowan has joined #openstack-infra | 11:09 | |
*** nhandler has quit IRC | 11:09 | |
*** nhandler has joined #openstack-infra | 11:10 | |
*** nhandler has quit IRC | 11:10 | |
*** jtomasek has joined #openstack-infra | 11:10 | |
*** auristor has quit IRC | 11:13 | |
*** sree has joined #openstack-infra | 11:14 | |
*** auristor has joined #openstack-infra | 11:15 | |
*** nhandler has joined #openstack-infra | 11:17 | |
*** panda is now known as panda|lunch | 11:18 | |
*** sree has quit IRC | 11:19 | |
*** nhandler has joined #openstack-infra | 11:19 | |
*** nhandler has quit IRC | 11:19 | |
*** chkumar246 has joined #openstack-infra | 11:20 | |
*** dhajare_ has joined #openstack-infra | 11:20 | |
*** nhandler has joined #openstack-infra | 11:22 | |
*** nhandler has quit IRC | 11:22 | |
*** nhandler has joined #openstack-infra | 11:24 | |
*** shardy is now known as shardy_lunch | 11:24 | |
*** nhandler has joined #openstack-infra | 11:26 | |
*** nhandler has quit IRC | 11:26 | |
*** jtomasek has quit IRC | 11:28 | |
*** sree has joined #openstack-infra | 11:29 | |
*** yamamoto has joined #openstack-infra | 11:31 | |
*** dtantsur is now known as dtantsur|brb | 11:31 | |
*** sree_ has joined #openstack-infra | 11:32 | |
*** sree_ is now known as Guest40822 | 11:33 | |
*** nhandler has joined #openstack-infra | 11:33 | |
*** rfolco|off is now known as rfolco|ruck | 11:34 | |
*** nhandler has quit IRC | 11:34 | |
*** katkapilatova has joined #openstack-infra | 11:34 | |
*** katkapilatova has left #openstack-infra | 11:34 | |
*** yamamoto has quit IRC | 11:35 | |
*** ldnunes has joined #openstack-infra | 11:35 | |
*** sree has quit IRC | 11:35 | |
*** nhandler has joined #openstack-infra | 11:36 | |
*** nhandler has quit IRC | 11:36 | |
*** salv-orl_ has joined #openstack-infra | 11:36 | |
*** nhandler has joined #openstack-infra | 11:37 | |
*** nhandler has joined #openstack-infra | 11:40 | |
*** aojea has joined #openstack-infra | 11:40 | |
*** salv-orlando has quit IRC | 11:41 | |
*** nhandler has joined #openstack-infra | 11:41 | |
AJaeger | infra-root, https://docs.openstack.org/infra/publications/ is broken - I don't know what broke the publishing job ;( Any help appreciated. | 11:42 |
*** nhandler has joined #openstack-infra | 11:43 | |
*** nhandler has quit IRC | 11:43 | |
*** dsariel has quit IRC | 11:43 | |
*** nhandler has joined #openstack-infra | 11:45 | |
*** nhandler has quit IRC | 11:45 | |
*** nhandler has joined #openstack-infra | 11:46 | |
*** nhandler has quit IRC | 11:46 | |
*** efried has quit IRC | 11:47 | |
*** efried has joined #openstack-infra | 11:48 | |
*** ldnunes has quit IRC | 11:49 | |
*** ykarel__ is now known as ykarel | 11:49 | |
*** ldnunes has joined #openstack-infra | 11:50 | |
*** nhandler has joined #openstack-infra | 11:53 | |
*** nhandler has quit IRC | 11:53 | |
*** nhandler has joined #openstack-infra | 11:54 | |
*** nhandler has quit IRC | 11:55 | |
*** lucasagomes is now known as lucas-hungry | 11:55 | |
*** jcoufal has joined #openstack-infra | 11:56 | |
*** jlabarre has joined #openstack-infra | 11:58 | |
*** nhandler has joined #openstack-infra | 11:58 | |
*** nhandler has quit IRC | 11:58 | |
*** nhandler has joined #openstack-infra | 12:00 | |
frickler | AJaeger: I'm not sure I understand how all this is going to work. is publish-infra-publications missing the publications-index step? seems the main issue is that the root directory there is missing some index.html, right? | 12:00 |
*** nhandler has quit IRC | 12:00 | |
*** andreas_s has quit IRC | 12:01 | |
*** udesale has joined #openstack-infra | 12:02 | |
*** andreas_s_ has joined #openstack-infra | 12:02 | |
AJaeger | frickler: publish-infra-publications-index should publish the index | 12:03 |
*** andreas__ has joined #openstack-infra | 12:03 | |
*** nhandler has joined #openstack-infra | 12:04 | |
*** germs has joined #openstack-infra | 12:04 | |
*** germs has quit IRC | 12:04 | |
*** germs has joined #openstack-infra | 12:04 | |
*** rosmaita has joined #openstack-infra | 12:04 | |
frickler | AJaeger: ah, yes, found that now. that job is failing here http://logs.openstack.org/04/0429cea062756448405f89a12a30a672dfc5f36d/post/publish-infra-publications-index/de10a61/job-output.txt.gz#_2018-03-26_17_46_08_167976 | 12:04 |
*** rosmaita has quit IRC | 12:04 | |
*** tpsilva has joined #openstack-infra | 12:05 | |
AJaeger | but apparently did not - or did wrongly, see the published bindep.txt in that directory | 12:05 |
AJaeger | frickler: argh ;( I tested that locally when I pushed the last change. | 12:06 |
*** bobh has joined #openstack-infra | 12:06 | |
AJaeger | But won't have time to look into this since I'm out the next days | 12:06 |
*** edmondsw has joined #openstack-infra | 12:06 | |
*** andreas_s_ has quit IRC | 12:07 | |
frickler | AJaeger: oh, I think the issue is that ./make-index only exists in the master branch, but that job has the branch from the change checked out, in that case "branch: help-wanted-howto" | 12:07 |
openstackgerrit | Andreas Jaeger proposed openstack-dev/hacking master: Import integration test https://review.openstack.org/547765 | 12:08 |
frickler | AJaeger: so we might just need a branch override | 12:08 |
AJaeger | frickler: ok | 12:08 |
*** nhandler has joined #openstack-infra | 12:08 | |
*** germs has quit IRC | 12:08 | |
AJaeger | frickler: cool - but do you know where https://docs.openstack.org/infra/publications/tox.ini comes from? Which job is doing that? | 12:09 |
*** dprince has joined #openstack-infra | 12:09 | |
*** armaan has quit IRC | 12:09 | |
frickler | AJaeger: http://logs.openstack.org/04/0429cea062756448405f89a12a30a672dfc5f36d/post/publish-infra-publications/5c7abca/job-output.txt.gz#_2018-03-26_17_49_07_727904 | 12:09 |
AJaeger | argh ;( | 12:10 |
AJaeger | so, http://logs.openstack.org/04/0429cea062756448405f89a12a30a672dfc5f36d/post/publish-infra-publications/5c7abca/job-output.txt.gz#_2018-03-26_17_49_03_987369 moves to one place and we publish from another ;( | 12:11 |
*** nhandler has joined #openstack-infra | 12:12 | |
frickler | AJaeger: no, we do publish exactly the files that got moved I'd say | 12:12 |
AJaeger | frickler: indeed, that logic above is broken ;( | 12:12 |
frickler | AJaeger: so the issue might rather be that we are moving the wrong files | 12:12 |
*** rosmaita has joined #openstack-infra | 12:12 | |
AJaeger | yeah ;( | 12:12 |
*** nhandler has joined #openstack-infra | 12:14 | |
AJaeger | frickler: or a wrong branch published - that might have been publish from master which should not publish anything... | 12:14 |
* AJaeger is confused - but has no time to look at it | 12:14 | |
*** lpetrut_ has quit IRC | 12:15 | |
frickler | AJaeger: seems to be multiple issues all coming together here. I don't have too much time currently, either, maybe someone else can dig into this later. like dmsimard who has another presentation up for publishing maybe ;) | 12:17 |
*** nhandler has joined #openstack-infra | 12:17 | |
*** nhandler has quit IRC | 12:17 | |
AJaeger | Let's hope ;) | 12:18 |
AJaeger | frickler: and I think his presentation is exactly the problem, we published those files verbatim which is wrong. | 12:19 |
AJaeger | dmsimard: ^ | 12:20 |
*** odyssey4me has quit IRC | 12:22 | |
*** odyssey4me has joined #openstack-infra | 12:22 | |
dmsimard | Huh ? | 12:22 |
*** nhandler has joined #openstack-infra | 12:22 | |
*** yamamoto has joined #openstack-infra | 12:22 | |
*** ramishra has quit IRC | 12:22 | |
*** zhipeng has quit IRC | 12:23 | |
frickler | AJaeger: put that one isn't merged yet. https://review.openstack.org/515853 does contain tox.ini and similar files, too, though. that's the last one that got published | 12:23 |
frickler | s/put/but | 12:23 |
*** nhandler has joined #openstack-infra | 12:23 | |
*** nhandler has quit IRC | 12:23 | |
dmsimard | I'll be able to look in ~15 minutes | 12:23 |
AJaeger | frickler: ah - yes. Sorry, dmsimard. | 12:23 |
frickler | dmsimard: not urgent, I'd say, seems broken for some time already | 12:24 |
*** xarses_ has joined #openstack-infra | 12:24 | |
*** nhandler_ has joined #openstack-infra | 12:25 | |
*** ramishra has joined #openstack-infra | 12:25 | |
*** andreww has quit IRC | 12:28 | |
*** nhandler_ has quit IRC | 12:29 | |
frickler | dmsimard: AJaeger: so to aggregate what I see, we have two issues: 1. publish-infra-publications-index needs a branch override "master" 2. ZUUL_REFNAME is not set here https://git.openstack.org/cgit/openstack-infra/project-config/tree/playbooks/publications/run.yaml#n12 , leading to the published docs ending one dir level too high | 12:30 |
*** zhurong has quit IRC | 12:34 | |
*** bobh has quit IRC | 12:34 | |
*** kgiusti has joined #openstack-infra | 12:34 | |
AJaeger | frickler: Ah, a legacy variable - we need to pass that in as zuul v3 variable | 12:34 |
*** andreas__ has quit IRC | 12:35 | |
*** andreas_s has joined #openstack-infra | 12:35 | |
openstackgerrit | Merged openstack-infra/zuul master: Use requests instead of urllib.request in tests https://review.openstack.org/554057 | 12:36 |
*** rlandy has joined #openstack-infra | 12:37 | |
*** andreas_s has quit IRC | 12:37 | |
*** andreas_s has joined #openstack-infra | 12:39 | |
*** nhandler has joined #openstack-infra | 12:40 | |
*** nhandler_ has joined #openstack-infra | 12:42 | |
*** chkumar246 is now known as chandankumar | 12:43 | |
*** nhandler_ has quit IRC | 12:44 | |
*** dsariel has joined #openstack-infra | 12:44 | |
*** sshnaidm is now known as sshnaidm|afk | 12:46 | |
*** olaph has joined #openstack-infra | 12:52 | |
*** jamesmcarthur has quit IRC | 12:52 | |
*** jamesmcarthur has joined #openstack-infra | 12:52 | |
dmsimard | maybe it was the first time we merged something since the v3 migration ? | 12:52 |
*** lucas-hungry is now known as lucasagomes | 12:55 | |
*** panda|lunch is now known as panda | 12:56 | |
*** salv-orlando has joined #openstack-infra | 12:56 | |
*** salv-or__ has joined #openstack-infra | 12:57 | |
*** salv-orl_ has quit IRC | 12:58 | |
*** trown|brb is now known as trown | 13:00 | |
*** salv-orlando has quit IRC | 13:02 | |
*** salv-or__ has quit IRC | 13:02 | |
*** salv-orlando has joined #openstack-infra | 13:03 | |
*** lbragstad has joined #openstack-infra | 13:04 | |
*** mriedem has joined #openstack-infra | 13:06 | |
*** Goneri has joined #openstack-infra | 13:06 | |
dhellmann | AJaeger : I'll split the patch into 2 steps; one to turn off the job and one to clean up the pieces. We can discuss the 2nd piece in more detail. | 13:06 |
*** jamesmcarthur has quit IRC | 13:07 | |
*** jamesmcarthur has joined #openstack-infra | 13:08 | |
*** salv-orlando has quit IRC | 13:08 | |
*** armaan has joined #openstack-infra | 13:11 | |
openstackgerrit | Thierry Carrez proposed openstack-infra/project-config master: Catch errors in release processing script https://review.openstack.org/557325 | 13:12 |
*** ykarel_ has joined #openstack-infra | 13:12 | |
*** jamesmcarthur has quit IRC | 13:12 | |
*** ykarel has quit IRC | 13:12 | |
*** ykarel__ has joined #openstack-infra | 13:14 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove the propose-update-requirements job https://review.openstack.org/555426 | 13:14 |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove "requirements" mode from propose_update.sh https://review.openstack.org/557326 | 13:14 |
*** eharney has joined #openstack-infra | 13:14 | |
dhellmann | AJaeger , prometheanfire : ^^ | 13:15 |
*** dtantsur|brb is now known as dtantsur | 13:15 | |
*** ykarel_ has quit IRC | 13:17 | |
*** gouthamr has joined #openstack-infra | 13:19 | |
yamamoto | AJaeger: https://review.openstack.org/#/c/555788/ is only about master? or stable branches as well? | 13:21 |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: check out the series branch when tagging a release https://review.openstack.org/557332 | 13:26 |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: improve error handling of clone_repo https://review.openstack.org/557333 | 13:26 |
dhellmann | smcginnis , ttx: ^^ | 13:26 |
*** Guest40822 has quit IRC | 13:27 | |
*** sree has joined #openstack-infra | 13:28 | |
*** jamesmcarthur has joined #openstack-infra | 13:28 | |
smcginnis | TIL git -C | 13:28 |
*** electrofelix has quit IRC | 13:29 | |
*** zhipeng has joined #openstack-infra | 13:29 | |
dhellmann | conceptually I like the subshell approach better, but *shrug* | 13:30 |
*** psachin has quit IRC | 13:30 | |
ttx | dhellmann: are you sure that changes anything ? | 13:31 |
*** ykarel__ is now known as ykarel | 13:31 | |
ttx | set -e should catch (true && false) alright | 13:32 |
dhellmann | ttx: the first patch is the one we care about. that should cause it to check out the stable branch for the project | 13:32 |
dhellmann | the other one was just a bit of cleanup | 13:32 |
dhellmann | I can abandon that one if you think it doesn't make any difference | 13:32 |
*** udesale has quit IRC | 13:32 | |
ttx | Well it's clearer, so I'll take it | 13:33 |
*** sree has quit IRC | 13:33 | |
ttx | but yeah, that was what I thought was misbehaving until I realized it was working as expected and the fail was elsewhere | 13:33 |
*** zhipeng has quit IRC | 13:33 | |
dhellmann | you had a head start on me :-) | 13:33 |
*** alexchadin has quit IRC | 13:37 | |
*** hashar is now known as hasharAway | 13:39 | |
*** alexchadin has joined #openstack-infra | 13:39 | |
*** aojea has quit IRC | 13:40 | |
*** haleyb has quit IRC | 13:41 | |
*** nhandler has joined #openstack-infra | 13:42 | |
*** garyk has joined #openstack-infra | 13:43 | |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources master: Added get summit location banners endpoint https://review.openstack.org/557339 | 13:44 |
openstackgerrit | Merged openstack-infra/openstackid-resources master: Added get summit location banners endpoint https://review.openstack.org/557339 | 13:45 |
*** nhandler has quit IRC | 13:47 | |
openstackgerrit | Monty Taylor proposed openstack-infra/puppet-zuul master: Fix rewrite rule for multi-tenant deploy https://review.openstack.org/557342 | 13:51 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Reorganize REST API and dashboard urls https://review.openstack.org/556967 | 13:51 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade to webpack 4 https://review.openstack.org/551987 | 13:51 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use typescript for webpack config https://review.openstack.org/555948 | 13:51 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade from angularjs (v1) to angular (v5) https://review.openstack.org/551989 | 13:51 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use glyphicons for status balls https://review.openstack.org/551992 | 13:51 |
*** kiennt26 has joined #openstack-infra | 13:52 | |
*** salv-orlando has joined #openstack-infra | 13:53 | |
*** jamesmcarthur has quit IRC | 13:54 | |
*** jamesmcarthur has joined #openstack-infra | 13:55 | |
openstackgerrit | Graham Hayes proposed openstack-infra/project-config master: Create 'Review-Priority' for designate repos https://review.openstack.org/295253 | 13:55 |
*** rosmaita has quit IRC | 13:55 | |
*** efried has quit IRC | 13:56 | |
*** efried has joined #openstack-infra | 13:56 | |
*** jamesmcarthur has quit IRC | 13:59 | |
*** armaan has quit IRC | 13:59 | |
*** armaan has joined #openstack-infra | 14:05 | |
*** zhipeng has joined #openstack-infra | 14:05 | |
*** udesale has joined #openstack-infra | 14:05 | |
*** germs has joined #openstack-infra | 14:05 | |
*** germs has quit IRC | 14:05 | |
*** germs has joined #openstack-infra | 14:05 | |
*** udesale has quit IRC | 14:05 | |
*** dklyle has joined #openstack-infra | 14:05 | |
*** david-lyle has quit IRC | 14:05 | |
*** udesale has joined #openstack-infra | 14:05 | |
*** esberglu has joined #openstack-infra | 14:05 | |
*** hongbin has joined #openstack-infra | 14:06 | |
*** jrist has quit IRC | 14:07 | |
corvus | AJaeger, Jeffrey4l: AJaeger's interpretation was correct -- only the invocation needs to move, the job definition can stay in kolla | 14:07 |
*** dklyle has quit IRC | 14:07 | |
*** dklyle has joined #openstack-infra | 14:07 | |
*** dangers_away is now known as dangers | 14:08 | |
*** hemna_ has joined #openstack-infra | 14:08 | |
*** dsariel has quit IRC | 14:09 | |
*** dsariel has joined #openstack-infra | 14:09 | |
*** germs has quit IRC | 14:09 | |
*** dangers is now known as dangers_away | 14:10 | |
*** dangers_away has quit IRC | 14:11 | |
openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: Handle ZK session loss during node launch https://review.openstack.org/552538 | 14:16 |
*** VW has joined #openstack-infra | 14:17 | |
*** florianf has quit IRC | 14:17 | |
*** VW has quit IRC | 14:17 | |
*** VW has joined #openstack-infra | 14:18 | |
*** ykarel has quit IRC | 14:20 | |
*** _alastor_ has quit IRC | 14:20 | |
*** alexchadin has quit IRC | 14:24 | |
*** eernst has joined #openstack-infra | 14:24 | |
*** florianf has joined #openstack-infra | 14:24 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Remove nullwrap config option https://review.openstack.org/557350 | 14:25 |
openstackgerrit | Graham Hayes proposed openstack-infra/project-config master: Create 'Review-Priority' for designate repos https://review.openstack.org/295253 | 14:25 |
*** jogo has quit IRC | 14:26 | |
*** zhipeng has quit IRC | 14:27 | |
*** r-daneel has joined #openstack-infra | 14:28 | |
*** zhipeng has joined #openstack-infra | 14:28 | |
*** ralonsoh has quit IRC | 14:29 | |
*** jamesmcarthur has joined #openstack-infra | 14:31 | |
AJaeger | infra-root, config-core, I'll be on FTO for a long weekend. I'll be back online end of next week. | 14:31 |
corvus | AJaeger: is that like pto? | 14:32 |
*** r-daneel has quit IRC | 14:32 | |
frickler | google finds https://en.wikipedia.org/wiki/Flextime for me | 14:32 |
*** r-daneel has joined #openstack-infra | 14:33 | |
*** felipemonteiro has joined #openstack-infra | 14:33 | |
*** jogo has joined #openstack-infra | 14:33 | |
*** jrist has joined #openstack-infra | 14:33 | |
*** jrist has quit IRC | 14:33 | |
*** jrist has joined #openstack-infra | 14:33 | |
*** ianychoi_ has joined #openstack-infra | 14:33 | |
AJaeger | corvus: yes | 14:34 |
*** ianychoi has quit IRC | 14:36 | |
*** ramishra has quit IRC | 14:38 | |
pabelanger | morning | 14:41 |
*** caphrim007 has quit IRC | 14:42 | |
*** pcichy has joined #openstack-infra | 14:43 | |
pabelanger | config-core: clarkb: I think we are ready to add limestone back to nodepool: https://review.openstack.org/557092/ | 14:45 |
AJaeger | pabelanger: +2A | 14:46 |
dmsimard | pabelanger: did we confirm the ipv4 and dns issues are resolved ? (sorry I haven't been able to focus on it as much as I wanted to) | 14:46 |
*** ykarel has joined #openstack-infra | 14:46 | |
pabelanger | dmsimard: yah, ianw did that last night | 14:46 |
dmsimard | awesome | 14:46 |
AJaeger | pabelanger: please self-approve second change once ready, just gave +2 | 14:47 |
pabelanger | AJaeger: wfm | 14:47 |
*** udesale has quit IRC | 14:50 | |
*** efried has quit IRC | 14:51 | |
*** efried has joined #openstack-infra | 14:52 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Reorganize REST API and dashboard urls https://review.openstack.org/556967 | 14:53 |
*** shardy_lunch is now known as shardy | 14:53 | |
*** zoli is now known as zoli|gone | 14:54 | |
*** zoli|gone is now known as zoli | 14:54 | |
corvus | mordred: should we approve the puppet patches now? | 14:55 |
*** sshnaidm|afk is now known as sshnaidm | 14:57 | |
mordred | corvus: yah - although maybe first let's put zuul.o.o in the emergency file so we can control the rollout | 14:57 |
openstackgerrit | Merged openstack-infra/project-config master: Remove limestone specific labels from nl02.o.o https://review.openstack.org/557092 | 14:57 |
mordred | corvus: k. I've put zuul01.openstack.org in the emergency file | 14:58 |
*** kevinbenton has quit IRC | 14:58 | |
*** udesale has joined #openstack-infra | 14:59 | |
*** gfidente has quit IRC | 14:59 | |
*** r-daneel has quit IRC | 15:00 | |
*** gfidente has joined #openstack-infra | 15:01 | |
*** gfidente has joined #openstack-infra | 15:01 | |
*** links has quit IRC | 15:03 | |
*** udesale has quit IRC | 15:03 | |
*** felipemonteiro_ has joined #openstack-infra | 15:04 | |
*** kevinbenton has joined #openstack-infra | 15:06 | |
*** r-daneel has joined #openstack-infra | 15:07 | |
*** felipemonteiro has quit IRC | 15:07 | |
*** kiennt26 has quit IRC | 15:08 | |
*** AJaeger has quit IRC | 15:08 | |
*** pcaruana has quit IRC | 15:09 | |
*** gyankum has joined #openstack-infra | 15:11 | |
*** prometheanfire has quit IRC | 15:14 | |
*** bhujay has quit IRC | 15:18 | |
*** agopi|out has joined #openstack-infra | 15:21 | |
*** pcichy has quit IRC | 15:22 | |
*** _alastor_ has joined #openstack-infra | 15:23 | |
*** imacdonn has quit IRC | 15:26 | |
*** imacdonn has joined #openstack-infra | 15:27 | |
*** prometheanfire has joined #openstack-infra | 15:28 | |
*** bobh has joined #openstack-infra | 15:31 | |
*** zhipeng has quit IRC | 15:32 | |
*** bobh has quit IRC | 15:36 | |
openstackgerrit | Monty Taylor proposed openstack-infra/puppet-zuul master: Apply caching to entire REST API https://review.openstack.org/556981 | 15:41 |
mordred | corvus: https://review.openstack.org/#/q/topic:javascript+project:openstack-infra/puppet-zuul wanna check the rest of the puppet patches? | 15:42 |
*** bobh has joined #openstack-infra | 15:43 | |
corvus | mordred: on it | 15:43 |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources master: Added endpoint to update category group https://review.openstack.org/557378 | 15:44 |
openstackgerrit | Michel Peterson proposed openstack-infra/project-config master: Add lower-constraints job to networking-odl https://review.openstack.org/557379 | 15:44 |
*** sree has joined #openstack-infra | 15:44 | |
openstackgerrit | Merged openstack-infra/openstackid-resources master: Added endpoint to update category group https://review.openstack.org/557378 | 15:45 |
pabelanger | okay, just +3 limestone patch to bring it back online | 15:47 |
corvus | mordred: i'm confused by 557342 -- what's the rule on line 51 for? | 15:48 |
*** bobh has quit IRC | 15:48 | |
*** sree has quit IRC | 15:49 | |
*** jamesmcarthur has quit IRC | 15:50 | |
*** jamesmcarthur has joined #openstack-infra | 15:50 | |
openstackgerrit | Michel Peterson proposed openstack-infra/project-config master: Add lower-constraints job to networking-odl https://review.openstack.org/557379 | 15:50 |
*** andreas_s has quit IRC | 15:51 | |
*** andreas_s has joined #openstack-infra | 15:51 | |
*** armaan has quit IRC | 15:52 | |
*** rosmaita has joined #openstack-infra | 15:52 | |
*** bobh has joined #openstack-infra | 15:53 | |
*** caphrim007 has joined #openstack-infra | 15:54 | |
*** caphrim007 has quit IRC | 15:54 | |
*** caphrim007 has joined #openstack-infra | 15:54 | |
openstackgerrit | Merged openstack-infra/project-config master: Enable limestone-regione for nodepool.o.o https://review.openstack.org/557093 | 15:54 |
*** caphrim007 has quit IRC | 15:54 | |
*** andreas_s_ has joined #openstack-infra | 15:55 | |
*** caphrim007 has joined #openstack-infra | 15:55 | |
Jeffrey4l | corvus, could u define what "invocation" is? how could i do this? | 15:56 |
corvus | Jeffrey4l: i just mean when you add it to the project stanza | 15:56 |
*** andreas_s has quit IRC | 15:56 | |
Jeffrey4l | corvus, is there any example for this? | 15:57 |
corvus | Jeffrey4l: actually -- don't do that yet, i'm not sure that will solve the problem. | 15:58 |
Jeffrey4l | hm. so no solution right now? | 15:58 |
corvus | Jeffrey4l, clarkb: i think the problem might be that the jobs themselves are branched. let me think about this a bit more. | 15:58 |
*** bobh has quit IRC | 15:58 | |
Jeffrey4l | yeah, thanks. | 15:59 |
*** rpioso has joined #openstack-infra | 15:59 | |
rpioso | Is there a way to specify a range of eavesdrop lines so that they are highlighted in the browser? | 15:59 |
*** felipemonteiro_ has quit IRC | 16:00 | |
*** bobh has joined #openstack-infra | 16:00 | |
*** andreas_s_ has quit IRC | 16:00 | |
rpioso | I'm thinking of something along the lines of what GitHub offers, e.g., https://github.com/openstack/ironic/blob/f85eb9ab05abdbfda9659a33117ae27546f3f3ce/ironic/drivers/agent.py#L3-L8 | 16:00 |
pabelanger | http://grafana.openstack.org/dashboard/db/nodepool-limestone | 16:01 |
pabelanger | back online | 16:01 |
mordred | corvus: it's for serving out the static files for each tenant in the multi-tenant static-offload case | 16:02 |
fungi | rpioso: nope, but if someone wanted to put together a spec for an alternative irc log viewer to host on that server i'd certainly give it consideration | 16:03 |
mordred | corvus: to make sure that a request for /t/openstack/status.html serves $docroot/status.html ... the previous version of the line was missing the path segment to handle the tenant name :) | 16:03 |
corvus | mordred: got it! | 16:03 |
*** florianf has quit IRC | 16:03 | |
rpioso | fungi: Thank you! | 16:03 |
mordred | fungi, rpioso: _should_ be fairly easy to update the existing log writing code to add anchors | 16:04 |
*** hasharAway is now known as hashar | 16:04 | |
fungi | mordred: anchors which hilighted arbitrary ranges of lines though? | 16:04 |
mordred | oh - actually - the anchors are already there | 16:04 |
*** camunoz has joined #openstack-infra | 16:04 | |
mordred | so really one should be able to do the range highlight by adding a bit of javascript - which would be nice, since we could potentially then backport such a feature to the existing logs | 16:05 |
*** bobh has quit IRC | 16:05 | |
corvus | Jeffrey4l: are the kolla-publish-* jobs the same on all branches of kolla? | 16:05 |
*** gyee has joined #openstack-infra | 16:06 | |
*** andreas_s has joined #openstack-infra | 16:06 | |
fungi | yep, though there may also be client-side log viewers which could operate on the raw logs we publish and do something nicer with them than the html files generated by meetbot | 16:06 |
corvus | mordred: i'm +2 on all puppet patches | 16:07 |
*** bobh has joined #openstack-infra | 16:08 | |
*** florianf has joined #openstack-infra | 16:08 | |
pabelanger | logan-: clarkb: dmsimard: limestone-region is back online for nodepool. Keeping an eye on jobs, but think we are in good shape to increase max-servers later today too | 16:08 |
logan- | awesome, keeping an eye on logstash.. the first job was a success at least :P | 16:09 |
clarkb | ianw corvus just dropping a note here in case I dont have opportunity to write proper email for a bit. But this is maybe an opportunity to leverage structured logging output? we already emit a json file for each job | 16:09 |
*** camunoz has quit IRC | 16:09 | |
clarkb | re logstash and ansible | 16:09 |
pabelanger | logan-: nice, have log? | 16:10 |
corvus | clarkb: yeah, could do | 16:10 |
logan- | http://logs.openstack.org/41/538341/51/check/openstack-helm-linter/c12201a/job-output.txt | 16:10 |
logan- | just using logstash query: node_provider:limestone-regionone AND message:"LOOP [upload-logs : gzip console log and json output]" | 16:10 |
logan- | (i just pick a message that every job generally emits) | 16:10 |
*** andreas_s has quit IRC | 16:10 | |
pabelanger | yah | 16:11 |
mordred | corvus: sweet | 16:11 |
pabelanger | looking at http://logs.openstack.org/41/538341/51/check/openstack-helm-linter/c12201a/zuul-info/inventory.yaml | 16:11 |
pabelanger | I wonder if we should maybe update nodepool.public_ip to be null, over empty string | 16:11 |
pabelanger | erro | 16:12 |
pabelanger | nodepool.public_ipv4 | 16:12 |
corvus | according to the graph, limestone occasionally goes to 11. very spinal tap. http://grafana.openstack.org/dashboard/db/nodepool-limestone | 16:12 |
pabelanger | corvus: I think there is a bug in statsd reporting, we only have 10 nodes online | 16:13 |
*** lucasagomes is now known as lucas-afk | 16:13 | |
pabelanger | 0 ready according to CLI | 16:13 |
fungi | comes with an 18-inch stonehenge replica | 16:14 |
corvus | pabelanger: yeah, i suspect it's a sequencing issue | 16:14 |
dmsimard | thanks everyone for helping get limestone online :) | 16:14 |
mordred | infra-root: I'm going to land the puppet-zuul changes- zuul01 is in the emergency file, so doing so should not have any impact | 16:14 |
pabelanger | mordred: ack | 16:14 |
fungi | thanks | 16:15 |
mordred | pabelanger: if you havea sec, https://review.openstack.org/#/c/557342 is not urgent (it's not relevant for us) but might as well go ahead and land it | 16:15 |
mordred | fungi: you were looking at apache and caching recently, have a sec to check out https://review.openstack.org/#/c/556981 ? | 16:16 |
fungi | sure | 16:16 |
dmsimard | #thanks logan- and Limestone Networks for donating cloud resources to the OpenStack CI infrastructure! | 16:16 |
openstackstatus | dmsimard: Added your thanks to Thanks page (https://wiki.openstack.org/wiki/Thanks) | 16:16 |
logan- | thank you dmsimard :) | 16:17 |
rpioso | mordred, fungi: It would be cool if that highlighting could be added. The use case I have in mind is a reference to an IRC discussion in a commit message. Highlighting the range would make it easier for folks to find the relevant content. | 16:18 |
*** efoley has joined #openstack-infra | 16:21 | |
*** isviridov_away has quit IRC | 16:23 | |
mordred | rpioso: ++ | 16:24 |
*** camunoz has joined #openstack-infra | 16:24 | |
*** ihrachys has joined #openstack-infra | 16:25 | |
*** florianf has quit IRC | 16:26 | |
*** haleyb has joined #openstack-infra | 16:27 | |
*** efoley has quit IRC | 16:28 | |
*** efoley has joined #openstack-infra | 16:28 | |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources master: Added endpoint to get track group by id https://review.openstack.org/557427 | 16:28 |
*** jpich has quit IRC | 16:29 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Remove double api prefix from tenants call https://review.openstack.org/557429 | 16:30 |
openstackgerrit | Merged openstack-infra/openstackid-resources master: Added endpoint to get track group by id https://review.openstack.org/557427 | 16:31 |
*** olaph has quit IRC | 16:31 | |
*** olaph has joined #openstack-infra | 16:31 | |
*** VW_ has joined #openstack-infra | 16:33 | |
openstackgerrit | Merged openstack-infra/puppet-zuul master: Split v2 and v3 web config https://review.openstack.org/556869 | 16:34 |
*** jamesmcarthur has quit IRC | 16:35 | |
*** rossella_s has joined #openstack-infra | 16:36 | |
*** VW__ has joined #openstack-infra | 16:36 | |
*** VW has quit IRC | 16:36 | |
*** VW_ has quit IRC | 16:36 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade to webpack 4 https://review.openstack.org/551987 | 16:36 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use typescript for webpack config https://review.openstack.org/555948 | 16:36 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade from angularjs (v1) to angular (v5) https://review.openstack.org/551989 | 16:36 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use glyphicons for status balls https://review.openstack.org/551992 | 16:36 |
*** ihrachys_ has joined #openstack-infra | 16:38 | |
*** dsariel has quit IRC | 16:39 | |
*** yamahata has joined #openstack-infra | 16:40 | |
*** armaan has joined #openstack-infra | 16:40 | |
*** ihrachys has quit IRC | 16:41 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Use git.zuul-ci.org https://review.openstack.org/557100 | 16:41 |
openstackgerrit | Merged openstack-infra/zuul master: Remove nullwrap config option https://review.openstack.org/557350 | 16:42 |
*** germs has joined #openstack-infra | 16:43 | |
*** germs has quit IRC | 16:43 | |
*** germs has joined #openstack-infra | 16:43 | |
*** germs has quit IRC | 16:43 | |
*** germs has joined #openstack-infra | 16:44 | |
*** germs has quit IRC | 16:44 | |
*** germs has joined #openstack-infra | 16:44 | |
*** lpetrut has quit IRC | 16:45 | |
*** trown is now known as trown|lunch | 16:46 | |
mordred | infra-root: ok. if nobody has any objections, I'm going to land the zuul change to update the rest api - and the correspnding on puppet-zuul change | 16:48 |
mordred | zuul01 is still in emergency, so we're not quite to rolling it live yet | 16:48 |
corvus | ++ | 16:48 |
Jeffrey4l | re are the kolla-publish-* jobs the same on all branches of kolla? , these exists on all branches except newton branch. | 16:49 |
mordred | corvus: got a sec to hit https://review.openstack.org/#/c/557429 ? | 16:49 |
corvus | Jeffrey4l: right, but are they identical on all the branches? | 16:49 |
corvus | mordred: done | 16:50 |
Jeffrey4l | corvus, hrm ocata branch patch is not backported. so master / queens and pike are the same. | 16:50 |
*** kgiusti has left #openstack-infra | 16:50 | |
*** dsariel has joined #openstack-infra | 16:51 | |
openstackgerrit | Raoul Scarazzini proposed openstack-infra/project-config master: Create tripleo-ha-utils project (zuul) https://review.openstack.org/556598 | 16:51 |
openstackgerrit | Raoul Scarazzini proposed openstack-infra/project-config master: Create tripleo-ha-utils project (zuul) https://review.openstack.org/556598 | 16:53 |
corvus | Jeffrey4l: should they be the same on all branches? | 16:54 |
Jeffrey4l | corvus, no. newton branch is out of maintain. it should be eol in fact. other branches should be the same. | 16:55 |
*** gouthamr has quit IRC | 16:55 | |
*** VW__ has quit IRC | 16:56 | |
corvus | Jeffrey4l: so for all branches+tags where these jobs *should* run, they should always do the same thing. in other words, you don't want to do publish something on the queens branch differently than master. is that correct? | 16:56 |
mgagne | mordred: this change could interest you as it suggests changing the network_data.json in a non-compatible way and I see that glance consumes the "latest" version: https://review.openstack.org/#/c/312626/ | 16:57 |
mordred | mgagne: looking - and thank you | 16:57 |
mgagne | mordred: and if you could provide input to the pending questions in the commit message, that would be appreciated (as a contributor to glean and consumer of network_data.json) | 16:57 |
mgagne | mordred: thanks | 16:57 |
Jeffrey4l | corvus, i think what i want is just like "push master&tag tarball file into tarballs.o.o site", i am trying to implement push docker images built by kolla on branches and tags to hub.docker.com site. | 16:58 |
corvus | Jeffrey4l: the tarball jobs are branchless, so i think that means the answer to my previous question is 'yes'. | 16:59 |
*** tosky has quit IRC | 16:59 | |
*** r-daneel_ has joined #openstack-infra | 16:59 | |
Jeffrey4l | how the branch tallball is pushed? doesn't it mean branch? like http://tarballs.openstack.org/nova/nova-stable-queens.tar.gz | 17:00 |
corvus | Jeffrey4l: next question: does kolla have a repo without any branches? for example, the tripleo project has a tripleo-ci repo which only has a master branch. they keep ci tooling in there that is used by all the tripleo projects. does kolla have anything like that? | 17:01 |
Jeffrey4l | corvus, no | 17:01 |
*** e0ne has quit IRC | 17:01 | |
corvus | Jeffrey4l: yes, the tarball jobs push a tarball based on their branch, but they always do the same thing regardless of what branch is involved. | 17:01 |
*** r-daneel has quit IRC | 17:01 | |
*** r-daneel_ is now known as r-daneel | 17:01 | |
corvus | Jeffrey4l: what we need to do is to make your publishing jobs branchless. there are 3 ways to do this: | 17:02 |
Jeffrey4l | hm, i get you meaning. in kolla, different branch is different. | 17:02 |
mordred | frickler, pabelanger, dmsimard: you may also want to look at https://review.openstack.org/#/c/312626 (although I see frickler already has) - from a glean pov perhaps we should update to using 2017-02-22 instead of latest? it probably wouldnt' be TOO hard to add support for both 2017-02-22 and mgagne's new proposed format | 17:02 |
*** VW has joined #openstack-infra | 17:03 | |
dmsimard | wow I remember when mgagne submitted that, it's been a long time :) | 17:03 |
mgagne | dmsimard: ¯\_(ツ)_/¯ | 17:03 |
corvus | Jeffrey4l: 1) move the job definitions into openstack-zuul-jobs. this is probably the easiest thing to do, but it means you won't have approval rights over the jobs, so changes might be slower. | 17:03 |
corvus | Jeffrey4l: 2) create a new repo, maybe call it kolla-ci to hold the jobs. You would have approval rights for them, but it's extra work to create a new repo. | 17:04 |
*** dsariel has quit IRC | 17:04 | |
Jeffrey4l | when migrate the job definitions, i have to move the playbooks(run/post/pre) to openstack-zuul-jobs, right? | 17:04 |
*** rkukura has joined #openstack-infra | 17:04 | |
pabelanger | yes, playbooks need to live next to zuul.yaml | 17:05 |
corvus | Jeffrey4l: 3) leave the jobs where they are, but remove them from all branches of kolla except the master branch. on the definitions in the master branch, set "branches: .*" so those versions run everywhere. but each time you branch master (ie, stable/s) you'll need to delete the jobs from the new branch. | 17:05 |
corvus | Jeffrey4l: i think any of those options will work. | 17:05 |
*** derekh has quit IRC | 17:07 | |
Jeffrey4l | cool. i am thinking the solution 1), may be i can split current jobs, and add a base job in openstack-zuul-jobs used by kolla, which just push the docker image, then this base job are branchless. | 17:07 |
openstackgerrit | Merged openstack-infra/zuul-website master: Update links to git.zuul-ci.org https://review.openstack.org/557099 | 17:07 |
*** hashar is now known as hasharDinner | 17:07 | |
openstackgerrit | Merged openstack-infra/zuul master: Reorganize REST API and dashboard urls https://review.openstack.org/556967 | 17:07 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade from angularjs (v1) to angular (v5) https://review.openstack.org/551989 | 17:08 |
*** Swami has joined #openstack-infra | 17:08 | |
corvus | Jeffrey4l: ok. you won't be able to inherit from that job in the kolla repo though (it would have the same problem you're seeing now). but you can run any of those jobs in the kolla repo (ie, you can add them to the release pipeline there) | 17:08 |
*** janki has quit IRC | 17:09 | |
Jeffrey4l | but my curiosity is: why peroidic jobs in peroidic-release pipeline works. But tag pipeline doesn't. | 17:09 |
prometheanfire | mordred: mind taking a look at glean? https://review.openstack.org/548604 | 17:09 |
Jeffrey4l | maybe i should read the zuul code first :) | 17:09 |
prometheanfire | frickler: or you, we do seem to be running into eachother in reviews recently | 17:10 |
*** dtantsur is now known as dtantsur|afk | 17:10 | |
corvus | Jeffrey4l: periodic jobs run with a branch ref (ie, "refs/stable/queens") so they match the branch restrictions in the job definition. but the release pipeline runs with a tag reference (ie "refs/tags/1.2.3") which won't match that restriction | 17:11 |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul-website master: Fix doc links in menu https://review.openstack.org/557442 | 17:11 |
mordred | prometheanfire: sure - also - you may want to look at https://review.openstack.org/#/c/312626 as well | 17:11 |
corvus | Jeffrey4l: jobs defined on a repo with branches have an implied branch restriction; option 3 disables that to make them run everywhere. options 1 and 2 don't have the restriction because they don't define the jobs in a repo with branches. | 17:12 |
corvus | Jeffrey4l: all the details are here: https://zuul-ci.org/docs/zuul/user/config.html#attr-job.branches | 17:12 |
Jeffrey4l | i see. i will try your solutions. thanks a lot corvus | 17:12 |
corvus | Jeffrey4l: you're welcome. let me know when they're moved | 17:13 |
Jeffrey4l | yeah | 17:13 |
*** sambetts is now known as sambetts|afk | 17:13 | |
dmsimard | Just finished booking Vancouver travel things \o/ | 17:14 |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Add non-voting job nodepool-functional-py35-gentoo-src https://review.openstack.org/557443 | 17:14 |
prometheanfire | mordred: nova spec? | 17:14 |
prometheanfire | oh, network info | 17:14 |
prometheanfire | sure | 17:15 |
pabelanger | prometheanfire: ianw: ^steps needed for non-voting gentoo jobs for glean / diskimage-builder | 17:15 |
prometheanfire | pabelanger: <3 | 17:15 |
openstackgerrit | Merged openstack-infra/zuul master: Use git.zuul-ci.org https://review.openstack.org/557100 | 17:16 |
prometheanfire | pabelanger: does that need this? https://review.openstack.org/556652 | 17:16 |
prometheanfire | mordred: pabelanger one of you mind workflowing? | 17:16 |
*** isviridov_away has joined #openstack-infra | 17:17 | |
*** efoley has quit IRC | 17:17 | |
pabelanger | prometheanfire: mordred: lets see if clarkb wants to review too | 17:18 |
prometheanfire | wfm | 17:18 |
openstackgerrit | James E. Blair proposed openstack-infra/nodepool master: Switch docs to alabaster theme https://review.openstack.org/557445 | 17:18 |
pabelanger | but we'd need a glean release to pick it up on nodepool.o.o too | 17:18 |
prometheanfire | pabelanger: ya, was wondering about that | 17:19 |
*** dsariel has joined #openstack-infra | 17:19 | |
openstackgerrit | Merged openstack-infra/zuul-website master: Fix doc links in menu https://review.openstack.org/557442 | 17:19 |
openstackgerrit | Matthew Thode proposed openstack-infra/project-config master: make Gentoo use the systemd profile https://review.openstack.org/556652 | 17:20 |
*** rkukura has quit IRC | 17:20 | |
* fungi disappears to run some errands but plans to be back in time for the storyboard meeting at 1900z | 17:22 | |
*** iyamahat has joined #openstack-infra | 17:22 | |
logan- | pabelanger: so far 1 fail out of 9 finished tests (did not check why), but in the job logs it looks like unbound is still using v4 dns. see http://logs.openstack.org/ea/eab921548303d4ecb4d4508bb572316136b06671/post/publish-openstack-puppet-branch-tarball/5b2ddc0/job-output.txt#_2018-03-28_16_55_50_457355, http://logs.openstack.org/42/554142/5/check/openstack-tox-pep8/997b6e1/job-output.txt#_2018-03-28_17_10_18_908823, etc | 17:22 |
*** dhajare_ has quit IRC | 17:23 | |
*** isviridov_away has quit IRC | 17:23 | |
pabelanger | looking | 17:25 |
*** Goneri has quit IRC | 17:25 | |
pabelanger | Oh | 17:26 |
pabelanger | haha | 17:26 |
pabelanger | I think I see the issue | 17:26 |
*** bhujay has joined #openstack-infra | 17:27 | |
pabelanger | I'm thinking we might want to move validate-host role before configuure-unbound, since validate-host is where we are actually dumping the facts to disk | 17:28 |
pabelanger | we can actually test that with base-test | 17:28 |
*** yamamoto has quit IRC | 17:29 | |
*** gouthamr has joined #openstack-infra | 17:29 | |
*** lpetrut has joined #openstack-infra | 17:30 | |
prometheanfire | dims_: did you mean to +W the req job removal? | 17:32 |
*** gouthamr has quit IRC | 17:32 | |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: base-test: Move validate-host before configure-unbound https://review.openstack.org/557449 | 17:33 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Add build-time configurable status refresh variable https://review.openstack.org/557450 | 17:33 |
pabelanger | infra-root: I'd like to try the following change to base-test to see if that helps with our ipv6 detection issues on limestone | 17:34 |
dims_ | prometheanfire : did not do any reviews today, which one? | 17:35 |
*** shardy has quit IRC | 17:35 | |
*** bhujay has quit IRC | 17:35 | |
*** tesseract has quit IRC | 17:35 | |
*** caphrim007_ has joined #openstack-infra | 17:36 | |
openstackgerrit | Graham Hayes proposed openstack-infra/project-config master: Create 'Review-Priority' for designate repos https://review.openstack.org/295253 | 17:37 |
openstackgerrit | Matthew Thode proposed openstack-infra/irc-meetings master: Update requirements meeting time https://review.openstack.org/557453 | 17:38 |
*** iyamahat has quit IRC | 17:38 | |
*** yamahata has quit IRC | 17:38 | |
openstackgerrit | Merged openstack-infra/project-config master: check out the series branch when tagging a release https://review.openstack.org/557332 | 17:39 |
corvus | pabelanger: are you proposing we also do this for the base job, or is this just a temporary change to base-test just to debug this one issue? | 17:39 |
*** armaan has quit IRC | 17:39 | |
openstackgerrit | Merged openstack-infra/project-config master: improve error handling of clone_repo https://review.openstack.org/557333 | 17:39 |
openstackgerrit | Merged openstack-infra/project-config master: Catch errors in release processing script https://review.openstack.org/557325 | 17:39 |
openstackgerrit | Merged openstack-infra/project-config master: Copy initial jobs for fixtures-git from git-upstream https://review.openstack.org/556212 | 17:39 |
prometheanfire | dims_: meant for dmsimard | 17:39 |
*** jamesmcarthur has joined #openstack-infra | 17:39 | |
dims_ | :) prometheanfire | 17:39 |
prometheanfire | dmsimard: did you mean to +W the req job removal? | 17:39 |
prometheanfire | :D | 17:39 |
pabelanger | corvus: yah, a temporary change to aid in debugging first | 17:39 |
*** Goneri has joined #openstack-infra | 17:40 | |
dmsimard | prometheanfire: which one ? I've been reviewing a lot of changes | 17:40 |
*** caphrim007 has quit IRC | 17:40 | |
dmsimard | corvus: asked a quick question in https://review.openstack.org/#/c/555053/ | 17:40 |
corvus | pabelanger: first? | 17:41 |
corvus | pabelanger: i'm asking because i don't think we can do that long-term. validate-host performs dns lookups, so i think we want to keep it where it is normally. but i'm fine moving it just to help debug this issue. | 17:42 |
*** felipemonteiro has joined #openstack-infra | 17:43 | |
prometheanfire | dmsimard: https://review.openstack.org/555426 | 17:43 |
openstackgerrit | Merged openstack-infra/project-config master: proposal: OSA: Do not sync tests/tests-repo-clone.sh anymore https://review.openstack.org/557176 | 17:43 |
openstackgerrit | Merged openstack-infra/project-config master: Retire puppet-ganesha(1/2) https://review.openstack.org/556696 | 17:43 |
openstackgerrit | Merged openstack-infra/project-config master: Add Zuul project for kolla-cli https://review.openstack.org/555898 | 17:43 |
*** felipemonteiro_ has joined #openstack-infra | 17:43 | |
openstackgerrit | Merged openstack-infra/project-config master: Add ansible-role-k8s-rabbitmq to zuul.d https://review.openstack.org/552681 | 17:43 |
prometheanfire | dmsimard: it's blocked by the depends-on, so is mostly fine | 17:43 |
pabelanger | corvus: oh, right. I forgot about that. In that case, if debugging session shows ipv6 works when we swap the roles, maybe we need to split out the dns check / facts to disk logic | 17:43 |
dmsimard | prometheanfire: yeah I always check depends-on, I saw the +2 but when I looked I don't think the -W was there | 17:44 |
corvus | pabelanger: wait, you think this is going to *fix* the problem? i thought you were just trying to record the values. | 17:45 |
*** dizquierdo has joined #openstack-infra | 17:45 | |
dmsimard | prometheanfire: Ah, yup. I reviewed at 13:29 and the -W was at 13:33 | 17:45 |
*** yamamoto has joined #openstack-infra | 17:46 | |
*** VW has quit IRC | 17:46 | |
prometheanfire | dmsimard: ya, I added it just in case someone else came along | 17:47 |
logan- | pabelanger: would it maybe make sense to run the setup module in the unbound config role with gather_subset '!all,!min,network' to make sure the network facts are fresh | 17:47 |
prometheanfire | dmsimard: I thought the project-config review was waiting on my +1 :P | 17:47 |
*** felipemonteiro has quit IRC | 17:47 | |
pabelanger | corvus: yah, I think it might fix but wanted to test. Our IPV6 logic is trying to use ansible_default_ipv6 varaible, and thinking it might not be loaded into memory at that point. But does when we write out the host-info file. Perhaps something around the usage of smart facts? | 17:48 |
dmsimard | prometheanfire: okay np, I removed the +W for the time being to be extra safe :) | 17:48 |
openstackgerrit | Matthew Thode proposed openstack-infra/irc-meetings master: Update requirements meeting time https://review.openstack.org/557453 | 17:49 |
corvus | pabelanger: fascinating | 17:49 |
pabelanger | logan-: http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/executor/server.py#n1531 is what we do on all hosts at the start, then we dump the variables into fact cache | 17:49 |
pabelanger | logan-: we default to smart facts, so I wonder if it is wonky or something | 17:49 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove legacy-tempest-dsvm-cells job definition https://review.openstack.org/549786 | 17:50 |
logan- | '!all' implies only 'min' subset is collected, which does not include network facts iirc | 17:50 |
pabelanger | Oh | 17:50 |
pabelanger | let me check something | 17:51 |
logan- | http://paste.openstack.org/raw/716327/ vs http://paste.openstack.org/raw/716328/ | 17:51 |
*** VW has joined #openstack-infra | 17:51 | |
*** r-daneel_ has joined #openstack-infra | 17:51 | |
pabelanger | logan-: Ah, so in that case. it think we need to set gather_facts on our pre-run playbook | 17:52 |
*** psachin has joined #openstack-infra | 17:52 | |
pabelanger | we can also test that | 17:52 |
*** r-daneel has quit IRC | 17:53 | |
*** r-daneel_ is now known as r-daneel | 17:53 | |
*** VW has quit IRC | 17:53 | |
*** VW has joined #openstack-infra | 17:54 | |
*** gfidente has quit IRC | 17:55 | |
pabelanger | logan-: http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/validate-host/tasks/main.yaml#n12 | 17:55 |
pabelanger | that is why the variables show up after we check for ipv6 | 17:55 |
logan- | ahh | 17:55 |
openstackgerrit | Merged openstack-infra/puppet-zuul master: Support updated REST and HTML layout https://review.openstack.org/556973 | 17:55 |
*** trown|lunch is now known as trown | 17:56 | |
pabelanger | corvus: okay, thanks to logan-, think we have a good theory now. We need to move http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/validate-host/tasks/main.yaml#n12 before our ipv6 checks | 17:57 |
pabelanger | which means https://review.openstack.org/557449/ should work, but isn't long term | 17:57 |
*** rossella_s has quit IRC | 17:57 | |
pabelanger | I think we could set gather_facts: true for pre-run playbook, then when it ran again in validate-host, there should be no impact due to fact cache | 17:58 |
*** rkukura has joined #openstack-infra | 17:58 | |
*** vaidy has joined #openstack-infra | 17:58 | |
*** jmccrory has quit IRC | 17:59 | |
*** panda is now known as panda|off | 17:59 | |
*** isviridov_away has joined #openstack-infra | 17:59 | |
bmace | hey folks. i'm hoping someone can give me a hand. we recently got the kolla-cli repository set up, based on an existing repo. there was an issue where the head of master wasn't pointing to the most recent code. the most recent code is tagged as o3l_4.0.1. i would like to get master pointing to that same commit but i can't seem to do it through gerrit and don't have permission to do direct git pushes, for understandable reasons. | 17:59 |
corvus | bmace: what about merging o3l_4.0.1 into master? | 18:01 |
bmace | i expect i could make a project-config change that would actually give myself push permission, but my expectation is that would be frowned upon. | 18:01 |
*** jamesmcarthur has quit IRC | 18:01 | |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: base-test: Gather facts for all hosts in pre-run https://review.openstack.org/557458 | 18:02 |
bmace | corvus: i tried that first, but i wasn't the only committer, and so when i try to post the review gerrit pukes saying i can't post reviews for other people.. and i would rather not sort of just post the code changes, all as me, and lose all that history. | 18:02 |
corvus | bmace: you should be able to do it as a merge commit | 18:02 |
*** lpetrut has quit IRC | 18:02 | |
pabelanger | corvus: logan-: mordred: dmsimard: thoughts on trying^ first to address ipv6 variable issue | 18:03 |
corvus | bmace: what happens when you 'git merge o3l_4.0.1'; git commit --amend; git review ? | 18:03 |
dmsimard | pabelanger: why is the setup task that the Zuul executor run not sufficient ? | 18:03 |
logan- | dmsimard: gather_subset: '!all' doesnt collect network facts | 18:04 |
pabelanger | dmsimard: see logan-, it doesn't get network variables | 18:04 |
bmace | corvus: it gives me a list about 50 commits that would be posted, complains that i am not the only committer related to those changes, and says it won't post the review. | 18:04 |
pabelanger | and we only use it to validate SSH is working not for getting all facts | 18:04 |
dmsimard | logan-, pabelanger: and is there a reason why we don't want to change that ? | 18:04 |
pabelanger | dmsimard: I don't think we want to use that code to gather facts | 18:05 |
dmsimard | All I'm saying is that fact gathering has a cost so we should probably not do that twice if we can help it | 18:06 |
corvus | bmace: ah, i understand. best thing to do might be to send an email to openstack-infra@lists.openstack.org with links to the import history and a request that we force-push the tag as master. | 18:06 |
bmace | corvus: https://pastebin.com/BXsRUJdP | 18:06 |
*** jmccrory has joined #openstack-infra | 18:06 | |
pabelanger | dmsimard: we have a fact cache setup, so won't be an impact | 18:06 |
bmace | corvus: sounds good, thanks much! | 18:06 |
*** dsariel has quit IRC | 18:06 | |
dmsimard | pabelanger: I'm not sure the cache expiry is granular to the individual facts | 18:06 |
dmsimard | (but I'd be happy to be pleasantly surprised) | 18:07 |
pabelanger | dmsimard: we don't expire facts by default | 18:07 |
pabelanger | dmsimard: would need to be forced, IIRC | 18:07 |
dmsimard | pabelanger: would need to test it, I'm not sure what's the behavior... :/ | 18:08 |
pabelanger | dmsimard: right, base-test above should get us some information | 18:08 |
dmsimard | pabelanger: another thing I wonder is that this is /our/ base job -- if we're assuming facts to be available in zuul-jobs, we should maybe run a setup task with the network subset we're interested in ? | 18:10 |
pabelanger | dmsimard: https://review.openstack.org/#/c/507933/1/zuul/executor/server.py@1667 is some history on network check for zuul | 18:10 |
pabelanger | dmsimard: well, we do today, just the order if wrong. https://review.openstack.org/557449/ should fix that, but isn't a long term solution as corvus points out | 18:11 |
*** yamamoto has quit IRC | 18:12 | |
*** pblaho has quit IRC | 18:12 | |
pabelanger | so, either gather_facts: yes in playbook or move http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/validate-host/tasks/main.yaml#n12 into another role | 18:12 |
pabelanger | give that we have fact cache by default, I think validate-host will just noop | 18:13 |
dmsimard | pabelanger: gather_facts is just an implicit setup task, these two plays are equivalent: https://gist.github.com/dmsimard/f6e754baee79f537147ed22adb2e2217 | 18:13 |
dmsimard | pabelanger: what I'm saying is that we could probably run that setup task explicitely where we expect network facts to be available | 18:14 |
dmsimard | so that way, we're not relying on zuul operators to have a "gather_facts: yes" in their base job pre playbook. | 18:15 |
*** lpetrut has joined #openstack-infra | 18:16 | |
*** psachin has quit IRC | 18:16 | |
logan- | ya thats something i suggested earlier, maybe consider running setup with gather_subset: '!all,!min,network' in the setup unbound role to assert that the network facts the role depends on are always present. i dont know enough about the whole zuul layout to say whether thats the best fix but its certainly one fix :) | 18:16 |
pabelanger | dmsimard: right, I understand that, and we do today. We just need to do a refactor and move that setup task before configure-unbound. I'm not sure add it into configure-unbound and validate-host is the best | 18:16 |
*** yamahata has joined #openstack-infra | 18:16 | |
*** caphrim007_ has quit IRC | 18:18 | |
*** caphrim007 has joined #openstack-infra | 18:18 | |
dmsimard | it turns out that gather_facts and setup might behave differently actually.. (discussing in #ansible-devel) | 18:18 |
*** e0ne has joined #openstack-infra | 18:19 | |
*** yamamoto has joined #openstack-infra | 18:21 | |
*** caphrim007_ has joined #openstack-infra | 18:24 | |
*** yamamoto has quit IRC | 18:26 | |
pabelanger | Hmm, this more I think about it, gather_facts: yes is the default, so setting that in the playbook might not get us the info, we might also need gather_facts: !all,network | 18:27 |
pabelanger | gather_subset* | 18:27 |
pabelanger | dmsimard: I think you might be right about setup task | 18:28 |
*** caphrim007 has quit IRC | 18:28 | |
*** iranzo has quit IRC | 18:29 | |
*** caphrim007_ has quit IRC | 18:30 | |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: base-test: Gather network facts for all hosts in pre-run https://review.openstack.org/557458 | 18:30 |
pabelanger | logan-: dmsimard: ^ | 18:30 |
*** caphrim007 has joined #openstack-infra | 18:30 | |
pabelanger | maybe you are right, that it should be just done in configue-unbound | 18:30 |
pabelanger | I'm just concerned about impact to hosts if we run multiple setup tasks | 18:31 |
dmsimard | I'm still confused and still asking in #ansible-devel :) | 18:31 |
pabelanger | k | 18:32 |
*** rosmaita has quit IRC | 18:33 | |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: base-test: Gather network facts for all hosts in pre-run https://review.openstack.org/557458 | 18:34 |
dhellmann | dmsimard , AJaeger, mnaser , pabelanger , prometheanfire : please, someone approve https://review.openstack.org/555426 before we end up with a bunch of projects with their lower constraints out of sync | 18:34 |
dhellmann | waiting for the patches in the requirements repo to land won't help | 18:34 |
dhellmann | we need to turn the job off asap | 18:34 |
dmsimard | dhellmann: the depended-on patch is still -W by prometheanfire | 18:35 |
dhellmann | gah | 18:35 |
mnaser | yeah that won't merge :\ | 18:35 |
dhellmann | ok, that depends-on is wrong | 18:35 |
dhellmann | I'm going to remove it | 18:36 |
dhellmann | we need the job turned off | 18:36 |
dhellmann | otherwise we're going to have a bunch of repos that start failing their checks as soon as we do land the requirements repo changes | 18:36 |
*** gyankum has quit IRC | 18:36 | |
logan- | lol pabelanger you beat me to it | 18:36 |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove the propose-update-requirements job https://review.openstack.org/555426 | 18:36 |
pabelanger | logan-: yah, I should have just had you propose the patch :) | 18:36 |
*** yamamoto has joined #openstack-infra | 18:36 | |
*** dhajare_ has joined #openstack-infra | 18:37 | |
dhellmann | dmsimard , mnaser : the older version of that change, which removed some code, did make sense to block on the requirements patch. That is now split out and I will put the depends-on back on that patch | 18:38 |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: remove "requirements" mode from propose_update.sh https://review.openstack.org/557326 | 18:39 |
*** jlk has left #openstack-infra | 18:39 | |
dhellmann | dmsimard, mnaser : does what I'm doing make sense? I'm afraid if we keep dragging this out we're going to end up in a bad state | 18:39 |
mnaser | dhellmann: it does make sense to me, this thing has to be done quickly or otherwise we'll end up with a mess we don't want to deal with | 18:40 |
*** gyankum has joined #openstack-infra | 18:40 | |
dhellmann | yes | 18:40 |
jokke_ | o/ | 18:40 |
jokke_ | Anyone here who could help me to get a feature branch for glance repo? | 18:41 |
dhellmann | jokke_ : you can do that via the releases repo | 18:41 |
jokke_ | oh I can? :o | 18:41 |
dhellmann | there should be instructions in the readme there | 18:41 |
*** yamamoto has quit IRC | 18:41 | |
dhellmann | all branches can be managed through the deliverable files | 18:41 |
jokke_ | nice, is there documentation for that somewhere I have totally missed? | 18:41 |
dhellmann | readme.rst | 18:41 |
jokke_ | uuh, nice | 18:41 |
dhellmann | in the releases repo | 18:41 |
jokke_ | in that case, sorry and thanks | 18:42 |
dhellmann | if that's not sufficient, let us know so we can improve it, please | 18:42 |
*** corvus is now known as kermitf | 18:42 | |
*** kermitf is now known as corvus | 18:42 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade to webpack 4 https://review.openstack.org/551987 | 18:43 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use typescript for webpack config https://review.openstack.org/555948 | 18:43 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade from angularjs (v1) to angular (v5) https://review.openstack.org/551989 | 18:43 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Remove need for ZUUL_BASE_HREF build time option https://review.openstack.org/557472 | 18:43 |
openstackgerrit | Kendall Nelson proposed openstack-infra/storyboard master: Document some usage instructions for a freshly deployed dev instance https://review.openstack.org/556018 | 18:45 |
openstackgerrit | Merged openstack-infra/project-config master: remove the propose-update-requirements job https://review.openstack.org/555426 | 18:46 |
fungi | dhellmann: ^ it should be gone now | 18:46 |
dhellmann | fungi : thank you | 18:46 |
jokke_ | dhellmann: looking for example from the old deliverables, the branches location section, can that be HEAD, hash or does it need to be tag? | 18:47 |
dhellmann | jokke_ : feature branches can be a sha; stable branches must be a version number | 18:47 |
dhellmann | but you have to be specific about the sha (you can't use HEAD) | 18:47 |
jokke_ | dhellmann: cheers, that info would be nice addition to the readme | 18:47 |
*** gyankum has quit IRC | 18:48 | |
dhellmann | jokke_ : lines 514-526 | 18:48 |
dmsimard | pabelanger, logan-: so even #ansible-devel is confused a little bit but my understanding is that the setup task will *always* run regardless of cache and will *always* update the cache | 18:49 |
dmsimard | and also the caching is not granular, it's an all or nothing deal | 18:49 |
dmsimard | and that confuses me a little bit because it seems like they just check if the ansible_facts key exists | 18:49 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Use github3.py >= 1.0.2 https://review.openstack.org/557475 | 18:49 |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: base-test: Gather network facts for all hosts in pre-run https://review.openstack.org/557458 | 18:50 |
logan- | dmsimard: yes thats my understanding as well. running the setup module will always refresh the cache for any facts in the gather_subset scope | 18:50 |
pabelanger | logan-: dmsimard: so followed along in #ansible-devel, ^ shoudl fix our ipv6 issue, however http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/validate-host/tasks/main.yaml#n12 is just going to collect them again, missing the fact cache | 18:50 |
pabelanger | dmsimard: yes, that is how I followed too | 18:51 |
openstackgerrit | Jesse Keating proposed openstack-infra/zuul master: Use latest release of github3.py https://review.openstack.org/557478 | 18:51 |
*** yamamoto has joined #openstack-infra | 18:52 | |
pabelanger | dmsimard: so, we likely want to remove http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/validate-host/tasks/main.yaml#n12 make it top-level and default some sane default, then you can use gather_subset: !all,!min,foo to get what has been missed, updating the fact cache for next plays | 18:52 |
openstackgerrit | Jesse Keating proposed openstack-infra/zuul master: Use latest release of github3.py https://review.openstack.org/557478 | 18:52 |
dmsimard | pabelanger: but it's the subset thing that confuses me | 18:53 |
dmsimard | oh actually nevermind, my brain just figured it out | 18:54 |
*** lpetrut has quit IRC | 18:54 | |
jokke_ | dhellmann: ahh ;) | 18:56 |
*** yamamoto has quit IRC | 18:56 | |
openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources master: Added endpoint to delete track group https://review.openstack.org/557485 | 19:02 |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: base-test: Gather network facts for all hosts in pre-run https://review.openstack.org/557458 | 19:03 |
openstackgerrit | Merged openstack-infra/openstackid-resources master: Added endpoint to delete track group https://review.openstack.org/557485 | 19:03 |
pabelanger | dmsimard: so, are you okay with https://review.openstack.org/557458 | 19:04 |
pabelanger | actually | 19:05 |
pabelanger | logan-: min doesn't look to work with ansible 2.3 | 19:05 |
dmsimard | pabelanger: if we're going to run a setup task in a pre base playbook, we shouldn't run just a subset | 19:05 |
dmsimard | because like I mentioned earlier, it's a full cache overwrite -- it's not granular | 19:05 |
dmsimard | so if we run just the network facts, we're "deleting" all the other facts | 19:06 |
dmsimard | at least from my understanding | 19:06 |
logan- | dmsimard: it only overwrites vars that are collected, existing vars remain set | 19:06 |
pabelanger | yes | 19:06 |
pabelanger | that is how I understood | 19:06 |
logan- | pabelanger: ah yeah i dont think you can fence out min on old ansibles | 19:06 |
dmsimard | also, it's a nit but I'd put pre_tasks before the roles declaration | 19:06 |
dmsimard | logan-: are you positive about that ? It seems like it only checks the ansible_facts key, not what's underneath it | 19:07 |
dmsimard | at least from what I can parse from #ansible-devel | 19:07 |
logan- | i'm not sure about ansible_facts, only the effects on hostvars | 19:08 |
dmsimard | discussion here: http://paste.openstack.org/show/716372/ | 19:08 |
logan- | gotcha so you're saying the new cached result will only contain the subset | 19:08 |
logan- | that makes sense i guess | 19:08 |
dmsimard | I don't know if it makes sense but that seems to be how it works | 19:10 |
logan- | ++ | 19:10 |
dmsimard | I'll ask another question just to make extra sure | 19:10 |
*** yamahata has quit IRC | 19:11 | |
*** yamahata has joined #openstack-infra | 19:12 | |
pabelanger | dmsimard: logan-: no, I don't think it will just save the new fact, is will merge them with facts in memory and write that into cache. | 19:14 |
pabelanger | atleast that is what I see locally | 19:14 |
logan- | cool, i wasnt sure on the caching behavior, thats good to know | 19:15 |
dhellmann | jokke_ : if you join #openstack-release we can talk about the validation error for your branch patch | 19:16 |
*** yamamoto has joined #openstack-infra | 19:17 | |
*** yamamoto has quit IRC | 19:17 | |
*** dhajare_ has quit IRC | 19:17 | |
*** sree has joined #openstack-infra | 19:19 | |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: base-test: Gather network facts for all hosts in pre-run https://review.openstack.org/557458 | 19:21 |
rm_work | anyone know if WSME has a channel? | 19:21 |
pabelanger | config-core: could I get a review on ^ as possible solution to our ipv6 dns issue for limestone. I'd like to test this now if possible, other wise I fear we'll need to diable limestone again until we resolve the ipv4 dns issues | 19:22 |
*** olaph1 has joined #openstack-infra | 19:22 | |
*** sree has quit IRC | 19:23 | |
*** olaph has quit IRC | 19:24 | |
*** pblaho has joined #openstack-infra | 19:25 | |
*** esberglu has quit IRC | 19:25 | |
*** esberglu has joined #openstack-infra | 19:26 | |
pabelanger | looking into a possible issue with bandersnatch | 19:27 |
*** dprince has quit IRC | 19:27 | |
pabelanger | don't think we are releasing afs volumes properly | 19:27 |
*** yamamoto has joined #openstack-infra | 19:28 | |
tobiash | pabelanger: maybe just the index is not synced yet? | 19:28 |
*** mat128 has quit IRC | 19:29 | |
pabelanger | let me check | 19:29 |
*** jlk has joined #openstack-infra | 19:30 | |
jlk | pabelanger: tobiash: I wasn't in here, what was the result of discussion over here about github3.py on the mirrors? | 19:30 |
*** esberglu has quit IRC | 19:31 | |
pabelanger | jlk: looking into it now | 19:31 |
pabelanger | grabbing lock on bandersnatch to see what is happening | 19:31 |
jlk | okay. The recheck hasn't resulted in jobs attempting yet if I'm reading this right. | 19:31 |
fungi | rm_work: their pypi page says they have a mailing list: http://groups.google.com/group/python-wsme | 19:31 |
rm_work | yes. I hate mailing lists. >_< | 19:31 |
jlk | Zuul tried things at 12:12, but the recheck hit at 12:19 | 19:32 |
fungi | rm_work: if there's a wsme-related irc channel, dhellmann will probably know what/where it is | 19:32 |
rm_work | that's why I am always in IRC :P | 19:32 |
dhellmann | there used to be #wsme | 19:32 |
dhellmann | it may be empty due to inactivity | 19:32 |
jlk | oh but you're saying the package should have been there by 12:12 | 19:32 |
rm_work | seems dead, yeah | 19:32 |
*** yamamoto has quit IRC | 19:32 | |
dhellmann | rm_work : why don't we move to #openstack-dev to avoid a long unrelated conversation in this channel. That's at least a "general purpose" channel. | 19:33 |
rm_work | k | 19:34 |
pabelanger | jlk: tobiash: oh, it doesn't look like we release pypi packages every 5 mins, there is a matrix to decided when to release the volume | 19:36 |
pabelanger | I didn't know that | 19:36 |
tobiash | pabelanger: ah ok, so just recheck in a few hours? | 19:36 |
jlk | TIL | 19:37 |
pabelanger | tobiash: yah, I'll see if I can find out when it should release | 19:37 |
*** bobh has quit IRC | 19:37 | |
pabelanger | http://git.openstack.org/cgit/openstack-infra/system-config/tree/modules/openstack_project/files/bandersnatch-mirror-update.sh | 19:38 |
pabelanger | is the code | 19:38 |
*** esberglu has joined #openstack-infra | 19:38 | |
pabelanger | https://review.openstack.org/472842/ | 19:38 |
pabelanger | was the code clarkb added | 19:38 |
pabelanger | jlk: tobiash: up to 4 hours | 19:39 |
jlk | :cool: | 19:39 |
*** felipemonteiro_ has quit IRC | 19:40 | |
*** felipemonteiro_ has joined #openstack-infra | 19:40 | |
pabelanger | 2018-03-28T17:36:04,479757010+00:00 was the last time | 19:40 |
pabelanger | we did a release of the volume | 19:40 |
tobiash | so the next will be in about two hours | 19:41 |
pabelanger | is that the last thing for a release? | 19:41 |
tobiash | no | 19:41 |
tobiash | it's not time critical | 19:41 |
pabelanger | okay, cool | 19:41 |
pabelanger | ++ | 19:41 |
tobiash | just did the math if I'll watch this change merging or going to bed ;) | 19:42 |
*** yamamoto has joined #openstack-infra | 19:43 | |
jlk | tobiash: I'll try to remember to recheck in a couple hours. | 19:45 |
jlk | (dropping the channel again) | 19:45 |
*** jlk has left #openstack-infra | 19:45 | |
*** yamamoto has quit IRC | 19:47 | |
*** yamahata has quit IRC | 19:48 | |
*** HeOS has quit IRC | 19:50 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Improve root api endpoint finding https://review.openstack.org/557429 | 19:50 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Add build-time configurable status refresh variable https://review.openstack.org/557450 | 19:50 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade to webpack 4 https://review.openstack.org/551987 | 19:50 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use typescript for webpack config https://review.openstack.org/555948 | 19:51 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade from angularjs (v1) to angular (v5) https://review.openstack.org/551989 | 19:51 |
openstackgerrit | Merged openstack-infra/project-config master: base-test: Gather network facts for all hosts in pre-run https://review.openstack.org/557458 | 19:52 |
*** felipemonteiro__ has joined #openstack-infra | 19:53 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Improve root api endpoint finding https://review.openstack.org/557429 | 19:54 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Add build-time configurable status refresh variable https://review.openstack.org/557450 | 19:54 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade to webpack 4 https://review.openstack.org/551987 | 19:54 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use typescript for webpack config https://review.openstack.org/555948 | 19:54 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade from angularjs (v1) to angular (v5) https://review.openstack.org/551989 | 19:54 |
*** olaph has joined #openstack-infra | 19:55 | |
*** caphrim007_ has joined #openstack-infra | 19:56 | |
*** olaph1 has quit IRC | 19:57 | |
*** yamamoto has joined #openstack-infra | 19:57 | |
*** yamamoto has quit IRC | 19:57 | |
*** ykarel is now known as ykarel|away | 19:57 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/storyboard master: Use charset=utf8mb4 in pymysql DBURIs https://review.openstack.org/557498 | 19:58 |
*** felipemonteiro_ has quit IRC | 19:58 | |
*** tosky has joined #openstack-infra | 19:58 | |
*** caphrim007 has quit IRC | 19:59 | |
*** jamesmcarthur has joined #openstack-infra | 20:00 | |
*** bobh has joined #openstack-infra | 20:01 | |
prometheanfire | dhellmann: do we have a lower-constraints job defined in project-config? (asking for new tag) | 20:01 |
dhellmann | prometheanfire : it's in openstack-zuul-jobs and it's called openstack-tox-lower-constraints | 20:02 |
prometheanfire | ah, k | 20:02 |
*** caphrim007_ has quit IRC | 20:03 | |
*** caphrim007 has joined #openstack-infra | 20:03 | |
*** dizquierdo has quit IRC | 20:05 | |
openstackgerrit | Merged openstack-infra/nodepool master: Switch docs to alabaster theme https://review.openstack.org/557445 | 20:05 |
*** tosky has quit IRC | 20:05 | |
*** ykarel|away has quit IRC | 20:06 | |
*** tosky has joined #openstack-infra | 20:09 | |
ianw | pabelanger: so ... we weren't crazy and actually weren't getting the net info? | 20:11 |
pabelanger | ianw: ya, our base-minimal job doesn't have same path as base | 20:12 |
pabelanger | so, our testing would likely always work | 20:12 |
pabelanger | because of validate-host | 20:12 |
pabelanger | I think I have a fix, just testing locally | 20:13 |
*** eharney has quit IRC | 20:13 | |
openstackgerrit | Doug Hellmann proposed openstack-infra/infra-manual master: update retirement instructions to keep the .gitreview file https://review.openstack.org/557502 | 20:17 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Improve root api endpoint finding https://review.openstack.org/557429 | 20:18 |
pabelanger | ianw: Ha, I just reproduced it | 20:19 |
pabelanger | and I think our fix, actually breaks it more :) | 20:19 |
pabelanger | 1 min | 20:19 |
*** HeOS has joined #openstack-infra | 20:21 | |
pabelanger | okay, fix doesn't break it more | 20:21 |
pabelanger | let me test in gate now | 20:22 |
openstackgerrit | Merged openstack-infra/zuul master: Clarify location of depends-on footer https://review.openstack.org/555185 | 20:23 |
ianw | so it's just that the network isn't up at the time we gather facts? a priori i wonder how we're getting facts without the network up, but i'm sure there's an explaination :) | 20:25 |
*** caphrim007 has quit IRC | 20:26 | |
pabelanger | ianw: no, it is this setup command, http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/executor/server.py#n1531 | 20:26 |
*** caphrim007 has joined #openstack-infra | 20:26 | |
pabelanger | by default we don't get the network facts, only after validate-host, will we get them because we do another setup | 20:26 |
pabelanger | which right now, happened after configure-unbound in base / base-test | 20:27 |
pabelanger | but base-minimal, validate-host is first, then configure-unbound | 20:27 |
corvus | dhellmann: are we releasing retired projects? | 20:30 |
dhellmann | corvus : we're trying to release a version of django_openstack_auth from a stable branch, yes | 20:31 |
dhellmann | there is an ongoing discussion in #openstack-release | 20:32 |
*** r-daneel_ has joined #openstack-infra | 20:32 | |
corvus | dhellmann: of all the things that we might have foreseen, i'm not going to fault us for not having foreseen that one. :) | 20:32 |
* corvus catches up | 20:32 | |
*** r-daneel has quit IRC | 20:33 | |
*** r-daneel_ is now known as r-daneel | 20:33 | |
dhellmann | prometheanfire : one more fix | 20:34 |
dhellmann | oops, wrong window | 20:35 |
fungi | corvus: there's some detailed discussion in #openstack-release about this topic | 20:36 |
fungi | weighing the pros and cons, it seems like the least icky solution for now. otherwise it involves figuring out a different place to be able to grab sufficient metadata to create a gerrit remote | 20:37 |
prometheanfire | lol | 20:39 |
*** nicolasbock has joined #openstack-infra | 20:39 | |
pabelanger | dmsimard: ianw: logan-: gather_subset: !all,network seems broken in 2.3.3.0. It doesn't work as expected locally | 20:42 |
pabelanger | but gather_subnet: network does | 20:43 |
pabelanger | using ansible 2.4, it works as expected | 20:43 |
pabelanger | so, I think we are hitting a bug there | 20:43 |
corvus | pabelanger: what about min,network? | 20:43 |
pabelanger | corvus: min isn't in 2.3 | 20:44 |
pabelanger | gather_subset options allowed: all, hardware, ohai, network, virtual, facter | 20:45 |
pabelanger | is the error I get | 20:45 |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: base-test: use gather_subset: network for ansible 2.3 https://review.openstack.org/557505 | 20:45 |
pabelanger | ^ is what works locally | 20:45 |
*** andreas_s has joined #openstack-infra | 20:45 | |
pabelanger | with our configure-mirror role | 20:45 |
pabelanger | and updates fact-cache properly | 20:46 |
corvus | neat -- what does !all do in 2.3? | 20:46 |
pabelanger | that works by itself, but nothing after it seems to work | 20:46 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use reno for release notes https://review.openstack.org/262597 | 20:47 |
*** e0ne has quit IRC | 20:48 | |
*** andreas_s has quit IRC | 20:49 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use reno for release notes https://review.openstack.org/262597 | 20:50 |
*** tosky has quit IRC | 20:50 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use reno for release notes https://review.openstack.org/262597 | 20:52 |
*** iyamahat has joined #openstack-infra | 20:52 | |
*** dsariel has joined #openstack-infra | 20:52 | |
openstackgerrit | Colleen Murphy proposed openstack-infra/zuul-jobs master: Fix ansible-lint job https://review.openstack.org/557507 | 20:53 |
pabelanger | corvus: http://paste.openstack.org/show/716518/ is the extra data we get into fact cache when using 557505, the diff is what zuul-executor would do by default with setup playbook it first runs | 20:53 |
openstackgerrit | Monty Taylor proposed openstack-infra/nodepool master: Add reno releasenotes support https://review.openstack.org/536861 | 20:53 |
*** tosky has joined #openstack-infra | 20:54 | |
openstackgerrit | Colleen Murphy proposed openstack-infra/openstack-zuul-jobs master: Convert infra-puppet-apply jobs to non-legacy https://review.openstack.org/557011 | 20:54 |
cmurphy | corvus: I think https://review.openstack.org/557507 fixes my problem | 20:55 |
*** yamamoto has joined #openstack-infra | 20:57 | |
corvus | cmurphy: lgtm; pabelanger ^ | 20:58 |
*** yamahata has joined #openstack-infra | 20:58 | |
*** beisner is now known as beisner-afk | 20:59 | |
mordred | corvus, cmurphy: lgtm - but I left it open in case pabelanger wanted to lok | 20:59 |
pabelanger | +3 | 21:00 |
cmurphy | thanks guys | 21:00 |
*** r-daneel_ has joined #openstack-infra | 21:01 | |
corvus | cmurphy: you should be able to go ahead and depends-on that in your other change; i think. | 21:02 |
cmurphy | corvus: i did :) | 21:02 |
*** r-daneel has quit IRC | 21:03 | |
*** r-daneel_ is now known as r-daneel | 21:03 | |
*** yamahata has quit IRC | 21:03 | |
*** yamamoto has quit IRC | 21:03 | |
*** jamesmcarthur has quit IRC | 21:03 | |
cmurphy | though the depends-on didn't seem to work | 21:04 |
*** trown is now known as trown|outtypewww | 21:08 | |
*** camunoz has quit IRC | 21:09 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade to webpack 4 https://review.openstack.org/551987 | 21:10 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use typescript for webpack config https://review.openstack.org/555948 | 21:10 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade from angularjs (v1) to angular (v5) https://review.openstack.org/551989 | 21:10 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use glyphicons for status balls https://review.openstack.org/551992 | 21:10 |
*** jamesmcarthur has joined #openstack-infra | 21:10 | |
openstackgerrit | Colleen Murphy proposed openstack-infra/zuul-jobs master: Fix ansible-lint job https://review.openstack.org/557507 | 21:11 |
*** jamesmcarthur has quit IRC | 21:12 | |
*** jamesmcarthur has joined #openstack-infra | 21:12 | |
cmurphy | fixed now ^ | 21:12 |
*** olaph1 has joined #openstack-infra | 21:16 | |
*** olaph has quit IRC | 21:16 | |
openstackgerrit | Merged openstack-infra/zuul master: Improve root api endpoint finding https://review.openstack.org/557429 | 21:22 |
*** rlandy is now known as rlandy|brb | 21:22 | |
*** iyamahat has quit IRC | 21:22 | |
*** HeOS has quit IRC | 21:23 | |
openstackgerrit | Colleen Murphy proposed openstack-infra/openstack-zuul-jobs master: Fix ansible-lint job https://review.openstack.org/557514 | 21:24 |
*** yamahata has joined #openstack-infra | 21:24 | |
pabelanger | corvus: dmsimard: mordred: mind a +3 on https://review.openstack.org/557505/ to continue testing ipv6 dns issues | 21:25 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/storyboard master: Use charset=utf8mb4 in pymysql DBURIs https://review.openstack.org/557498 | 21:25 |
openstackgerrit | Colleen Murphy proposed openstack-infra/puppet-gerrit master: [DNM] Test unlegacied puppet-apply jobs https://review.openstack.org/557515 | 21:26 |
*** rfolco|ruck is now known as rfolco|off | 21:26 | |
*** e0ne has joined #openstack-infra | 21:29 | |
*** jamesmcarthur has quit IRC | 21:30 | |
*** HeOS has joined #openstack-infra | 21:30 | |
*** Goneri has quit IRC | 21:32 | |
*** jamesmcarthur has joined #openstack-infra | 21:33 | |
*** jlabarre has quit IRC | 21:33 | |
openstackgerrit | Merged openstack-infra/project-config master: base-test: use gather_subset: network for ansible 2.3 https://review.openstack.org/557505 | 21:35 |
openstackgerrit | Merged openstack-infra/zuul master: Use latest release of github3.py https://review.openstack.org/557478 | 21:38 |
mordred | infra-root: so - who's ready for me to roll out the dashboard patches? | 21:41 |
pabelanger | mordred: I'm able to suport | 21:42 |
pabelanger | support* | 21:42 |
mordred | I'll take either type :) | 21:42 |
*** hasharDinner has quit IRC | 21:42 | |
*** edmondsw has quit IRC | 21:43 | |
fungi | mordred: i'm about to replace my monitors, so my availability may be spotty for a bit but i'll try to check in periodically in case something's gone sideways | 21:43 |
corvus | fungi: if you have a rotating mount, should be easy to fix if they do go sideways | 21:44 |
mordred | ++ | 21:44 |
*** sree has joined #openstack-infra | 21:45 | |
fungi | hah. in fact, i'm mounting them sideways | 21:45 |
pabelanger | YAY, finally: http://logs.openstack.org/67/549867/1/check/openstack-zuul-jobs-linters/60cb81c/job-output.txt.gz#_2018-03-28_21_43_26_651991 | 21:46 |
pabelanger | ipv6 dns setup | 21:46 |
fungi | same size and resolution as my current panels, just newer, lighter, less power-hungry, a little crisper and less flaky | 21:46 |
mordred | how about: #status notice the zuul web dashboard will experience a short downtime as we roll out some changes - no job execution should be affected | 21:46 |
mordred | pabelanger: yay! | 21:46 |
*** pickle has quit IRC | 21:48 | |
*** rlandy|brb is now known as rlandy | 21:48 | |
mordred | corvus, pabelanger, fungi: that notice look ok? | 21:49 |
corvus | mordred: ++ | 21:49 |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Promote base-test to bast for ipv6 DNS fixes https://review.openstack.org/557522 | 21:49 |
pabelanger | mordred: ++ | 21:50 |
mordred | #status notice the zuul web dashboard will experience a short downtime as we roll out some changes - no job execution should be affected | 21:50 |
openstackstatus | mordred: sending notice | 21:50 |
fungi | mordred: lgtm | 21:50 |
*** sree has quit IRC | 21:51 | |
pabelanger | corvus: mordred: dmsimard: ianw: clarkb: ^that is our fix to get dns working on properly for IPV6. Thoughts on making that change to base job? | 21:51 |
openstackgerrit | Colleen Murphy proposed openstack-infra/openstack-zuul-jobs master: Convert infra-puppet-apply jobs to non-legacy https://review.openstack.org/557011 | 21:51 |
-openstackstatus- NOTICE: the zuul web dashboard will experience a short downtime as we roll out some changes - no job execution should be affected | 21:52 | |
*** pblaho has quit IRC | 21:53 | |
*** harlowja has joined #openstack-infra | 21:53 | |
openstackstatus | mordred: finished sending notice | 21:53 |
mordred | ok. here we go | 21:55 |
*** e0ne has quit IRC | 21:55 | |
openstackgerrit | Colleen Murphy proposed openstack-infra/openstack-zuul-jobs master: Convert infra-puppet-apply jobs to non-legacy https://review.openstack.org/557011 | 21:59 |
*** yamamoto has joined #openstack-infra | 21:59 | |
mordred | infra-root: can someone help me figure out what's wrong with the apache rewrite rules? | 21:59 |
mordred | the zuul-web service has been restarted, the new api routes show up via curls to localhost:9000 ... | 22:00 |
mordred | but apache is 404ing everything | 22:00 |
fungi | looking | 22:00 |
*** VW has quit IRC | 22:01 | |
pabelanger | same | 22:01 |
pabelanger | curl http://127.0.0.1:9000/openstack/ doesn't work for me | 22:02 |
mordred | OH | 22:02 |
mordred | it's totally ... | 22:02 |
mordred | I'm dumb | 22:02 |
fungi | out of curiosity, why do we still aliasmatch git patterns to /var/lib/zuul/git? i guess that's something we could wrap in a conditional for v2-only in the puppet module | 22:02 |
logan- | awesome pabelanger :) | 22:02 |
clarkb | fungi: likely | 22:03 |
mordred | fungi: we don't - that's why I'm dumb | 22:03 |
fungi | ohhh | 22:03 |
mordred | I was doing a local puppet apply - and didn't update the puppet-zuul module | 22:03 |
fungi | makes sense! | 22:04 |
*** yamamoto has quit IRC | 22:04 | |
pabelanger | zuul.o.o seems better now | 22:04 |
mordred | ok. backup and working | 22:04 |
*** yamahata has quit IRC | 22:04 | |
mordred | and with the new reorganized api | 22:04 |
fungi | much better | 22:05 |
openstackgerrit | Merged openstack-infra/project-config master: Add Fullstack gate failure rate to neutron dashboard https://review.openstack.org/557266 | 22:06 |
mordred | woot. I think that looks good and we can take zuul out of the emergency file - any dissent? | 22:06 |
pabelanger | wfm | 22:10 |
corvus | me too | 22:11 |
*** Goneri has joined #openstack-infra | 22:12 | |
mordred | ok. zuul01 has been removed from the emergency file | 22:14 |
fungi | go for it | 22:14 |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Promote base-test to bast for ipv6 DNS fixes https://review.openstack.org/557522 | 22:15 |
pabelanger | ianw: hows^ | 22:15 |
fungi | so... storyboard is running close to (and frequently over) the 30-minute timeout for tox-py27 and tox-py35... should i set variants on those in project-config or put that in a .zuul.yaml in the sb repo? | 22:16 |
ianw | pabelanger: thanks, i think it's just non-obvious enough that we collect things twice to put that in :) | 22:17 |
pabelanger | ianw: yah | 22:17 |
mordred | fungi: I tihnk in the storyboard repo | 22:17 |
*** bobh has quit IRC | 22:17 | |
*** eernst has quit IRC | 22:17 | |
fungi | cool, i'll give that a shot | 22:18 |
mordred | fungi: I mean - honestly, I'd argue that for storyboard we could shift all the jobs form project-config into zuul - it's not a project that follows the PTI or anything | 22:18 |
mordred | gah | 22:18 |
mordred | s/zuul/storyboard repo/ | 22:18 |
fungi | yup, that's what i'm gonna try | 22:18 |
mordred | swet | 22:18 |
mordred | sweet | 22:19 |
*** HeOS has quit IRC | 22:21 | |
pabelanger | mordred: got time for a quick review on https://review.openstack.org/557522/ comments in line should hopefully explain the fix | 22:22 |
*** eernst has joined #openstack-infra | 22:24 | |
openstackgerrit | Colleen Murphy proposed openstack-infra/openstack-zuul-jobs master: Convert infra-puppet-apply jobs to non-legacy https://review.openstack.org/557011 | 22:25 |
tonyb | pabelanger, fungi: Can we take care of the OpenStackAnsible newton EOL today? | 22:28 |
*** rcernin has joined #openstack-infra | 22:28 | |
mordred | pabelanger: lgtm - I left of +A in case you wanted to manage how it lands | 22:28 |
*** eernst has quit IRC | 22:29 | |
pabelanger | tonyb: yah, I think so, given that we added you before to project bootstrap. Are you ready now? | 22:31 |
pabelanger | mordred: thanks | 22:31 |
tonyb | pabelanger: gimme 5, so I can do one more review ... I like to stay off review.o.o while I'm in bootstrappers ;P | 22:31 |
pabelanger | ianw: do you want to +A https://review.openstack.org/557522/ now and support? otherwise, we can do tomorrow | 22:32 |
pabelanger | tonyb: yah, seems safe | 22:32 |
*** esberglu has quit IRC | 22:32 | |
pabelanger | infra-root: any objections adding tonyb to gerrit project bootstrappers to eol some neuton branches? | 22:32 |
clarkb | nope | 22:32 |
*** priteau has quit IRC | 22:33 | |
fungi | please do | 22:33 |
ianw | pabelanger: yeah, i'm happy to watch it | 22:33 |
ianw | i only have to disappear for a little in a few hours to watch an easter hat parade | 22:33 |
*** ldnunes has quit IRC | 22:33 | |
tonyb | pabelanger: okay I've +2+W'd that thing I'm good to go when you are | 22:33 |
pabelanger | tonyb: kk | 22:34 |
ianw | i'm not sure if that is a thing that translates to USA people :) | 22:34 |
pabelanger | tonyb: okay, added tony@bakeyournoodle.com | 22:34 |
*** mriedem has quit IRC | 22:34 | |
pabelanger | #status log added tonyb to Group Project Bootstrappers on review.o.o | 22:34 |
openstackstatus | pabelanger: finished logging | 22:34 |
pabelanger | ianw: thanks | 22:34 |
*** priteau has joined #openstack-infra | 22:35 | |
pabelanger | ianw: yah, had to google easter hat parade :) | 22:35 |
*** mriedem has joined #openstack-infra | 22:36 | |
*** iyamahat has joined #openstack-infra | 22:38 | |
*** HeOS has joined #openstack-infra | 22:38 | |
openstackgerrit | Merged openstack-infra/project-config master: Promote base-test to bast for ipv6 DNS fixes https://review.openstack.org/557522 | 22:39 |
*** priteau has quit IRC | 22:40 | |
*** iyamahat has quit IRC | 22:40 | |
pabelanger | logan-: ^ | 22:41 |
*** iyamahat has joined #openstack-infra | 22:41 | |
*** felipemonteiro__ has quit IRC | 22:43 | |
*** tpsilva has quit IRC | 22:44 | |
tonyb | On a slightly related note .... we should do a keysigning (formal ir otherwise) at some point. It'd be good if the key I'm sigined these EOL tags with had some sigs from OpenStack folks | 22:45 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Upgrade from angularjs (v1) to angular (v5) https://review.openstack.org/551989 | 22:45 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Use glyphicons for status balls https://review.openstack.org/551992 | 22:45 |
*** mriedem has quit IRC | 22:47 | |
*** ansiwen[q] has joined #openstack-infra | 22:47 | |
tonyb | Hmm ok I just got: | 22:48 |
tonyb | + git push gerrit newton-eol | 22:48 |
tonyb | Counting objects: 1, done. | 22:48 |
tonyb | Writing objects: 100% (1/1), 830 bytes | 830.00 KiB/s, done. | 22:48 |
tonyb | Total 1 (delta 0), reused 0 (delta 0) | 22:48 |
tonyb | remote: Processing changes: refs: 1, done | 22:48 |
tonyb | To ssh://review.openstack.org:29418/openstack/openstack-ansible-security.git | 22:48 |
tonyb | ! [remote rejected] newton-eol -> newton-eol (prohibited by Gerrit) | 22:48 |
tonyb | error: failed to push some refs to 'ssh://tonyb@review.openstack.org:29418/openstack/openstack-ansible-security.git' | 22:48 |
openstackgerrit | megan guiney proposed openstack-infra/puppet-refstack master: :Upgrade NPM to latest version https://review.openstack.org/557539 | 22:48 |
*** jamesmcarthur has quit IRC | 22:48 | |
*** pblaho has joined #openstack-infra | 22:48 | |
tonyb | I'll check gerrit perms but if there's somethign y'all can check on the backend that'd be cool | 22:49 |
pabelanger | tonyb: did I use the right email? | 22:49 |
tonyb | Yeah it seems that repo is read-only | 22:49 |
ianw | i feel like that's the same email i put in last time we did this, and it worked | 22:50 |
*** andreww has joined #openstack-infra | 22:50 | |
tonyb | pabelanger: I pushed $lots of tags | 22:50 |
tonyb | I think it's specific to that repo being RO ? | 22:50 |
pabelanger | did we retire that repo? | 22:50 |
openstackgerrit | Colleen Murphy proposed openstack-infra/openstack-zuul-jobs master: Convert infra-puppet-apply jobs to non-legacy https://review.openstack.org/557011 | 22:50 |
tonyb | pabelanger: checking | 22:51 |
pabelanger | tonyb: yah, looks retired | 22:51 |
tonyb | pabelanger: Yeah. | 22:51 |
tonyb | I guess I'll just ignore that as it's explained and move on? | 22:52 |
pabelanger | wfm | 22:52 |
tonyb | it's RO so we can't actuially land anything on stable/newton anyway | 22:52 |
pabelanger | yah | 22:53 |
fungi | sure, likely wouldn't bother eol'ing branches on retired repos | 22:53 |
*** xarses_ has quit IRC | 22:53 | |
tonyb | Oh and that was the last one in the list anyway ;P | 22:53 |
tonyb | fungi: cool. | 22:53 |
fungi | doesn't seem like there's a lot of point to trying | 22:53 |
*** hongbin has quit IRC | 22:54 | |
tonyb | I have removed myself from bootstrappers: https://review.openstack.org/#/admin/groups/26,members | 22:54 |
tonyb | pabelanger: Thanks. | 22:54 |
fungi | though it came up in the release channel a few hours back that the new em plan may actually make it harder to ever fully retire a project if we continue to follow phased retirement where we only retire the repo once all its branches have reached eol | 22:54 |
pabelanger | tonyb: thanks! | 22:55 |
pabelanger | #status log removed tonyb to Group Project Bootstrappers on review.o.o | 22:55 |
openstackstatus | pabelanger: finished logging | 22:55 |
fungi | i suppose if a project is retiring maybe that's an opportunity to selectively declare its stable branches eol after some transitional period | 22:55 |
tonyb | fungi: Yeah I guess we 'retire' master (and perhaps other selected branches) but leave the repo Read-Write until the last branch goe -eol? | 22:57 |
tonyb | fungi: which as you say makes it much harder to fully retire a project | 22:57 |
fungi | that's what we do for things right now which are dependencies of other things (or maybe test dependencies) | 22:57 |
fungi | for leaf projects we can still just retire them in one shot | 22:58 |
tonyb | fungi: ok, I guess that thing can be a little hard to track as the openstack-ansible-security thing showed | 22:59 |
*** pblaho has quit IRC | 23:00 | |
*** yamamoto has joined #openstack-infra | 23:00 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Remove rest_api_url from info payload https://review.openstack.org/557544 | 23:02 |
*** yamamoto has quit IRC | 23:06 | |
openstackgerrit | Ian Wienand proposed openstack-infra/system-config master: Fix mariadb sync https://review.openstack.org/553085 | 23:07 |
ianw | pabelanger: i found fqdn_rand() which i think does what we want for ^ | 23:08 |
ianw | i could also use that to offset all the jobs i guess, rather than https://review.openstack.org/#/c/553894/ | 23:08 |
*** rlandy has quit IRC | 23:10 | |
*** beisner-afk is now known as beisner | 23:14 | |
*** bobh has joined #openstack-infra | 23:16 | |
*** tosky has quit IRC | 23:16 | |
*** jamesmcarthur has joined #openstack-infra | 23:18 | |
ianw | clarkb: pre comment many hours ago ... yeah after looking I feel like the deprecation of the multiline filter is probably telling us that we want to consider ingestion of logs longer term, probably to collate things more before it's fed to logstash | 23:18 |
clarkb | ianw: ya and thinking even more about it maybe we ingest all the service logs via systemd files | 23:19 |
clarkb | since those are structured | 23:19 |
*** iyamahat has quit IRC | 23:20 | |
*** salv-orlando has quit IRC | 23:21 | |
ianw | clarkb: so basically just run the devstack.journal.xz file in at the end of the test run? | 23:22 |
*** Goneri has quit IRC | 23:22 | |
*** salv-orlando has joined #openstack-infra | 23:22 | |
*** jamesmcarthur has quit IRC | 23:22 | |
clarkb | ya. Woukd need to be tested for feasability but then that should map reasonably well to how people injest logs if using systemd | 23:23 |
clarkb | at least format wise | 23:23 |
clarkb | maybe not line protocol because of our weird back and forth | 23:23 |
ianw | if you had that, and the job-output.json for each test ... that would cover almost anything? | 23:23 |
*** olaph has joined #openstack-infra | 23:23 | |
clarkb | ya I think so | 23:23 |
ianw | it might be possible to have an input on a different port, and feed that the job-output.json? | 23:25 |
*** olaph1 has quit IRC | 23:25 | |
clarkb | or just select on some metadata like we already do | 23:25 |
clarkb | but ya | 23:25 |
ianw | yeah, the thing is you can't really select things in an input {} ... afaict | 23:25 |
*** caphrim007_ has joined #openstack-infra | 23:26 | |
*** salv-orlando has quit IRC | 23:27 | |
*** caphrim007_ has quit IRC | 23:28 | |
*** caphrim007 has quit IRC | 23:30 | |
*** gyee has quit IRC | 23:32 | |
openstackgerrit | Merged openstack-infra/zuul master: Remove rest_api_url from info payload https://review.openstack.org/557544 | 23:35 |
corvus | i'm going to kick puppet on zuul01 and restart the scheduler with the tip of master. | 23:36 |
corvus | i'm also going to restart all the executors and zuul-web | 23:36 |
corvus | and mergers | 23:36 |
corvus | this is a final sanity check before tagging zuul 3.0.0 | 23:36 |
corvus | should we also restart nodepool, or is it up to date? | 23:36 |
*** HeOS has quit IRC | 23:37 | |
clarkb | Nodepool likely needs a restart too. Did the statsd thing get sorted there? | 23:39 |
corvus | clarkb: what's the statsd thing? | 23:39 |
clarkb | corvus: the same thing we talked about for zuul. Iirc pabelanger got the change in to set the statsd ipv6 config option we were going to revert? | 23:39 |
corvus | clarkb: that change was reverted | 23:40 |
clarkb | ya asking if we need restart ti check that | 23:40 |
corvus | ah. looks like the last nl restarts were feb13 | 23:41 |
corvus | so there have been changes since then, but that post-dates those | 23:41 |
mordred | corvus: I have to afk - the dinner is ready ... but feel free to go ahead from my end | 23:44 |
corvus | i will stop nodepool-builders, stop nodepool-launchers, stop zuul-executors, stop zuul-mergers, save queues, stop zuul-scheduler, stop zuul-web. then start zuul-scheduler, start zuul-mergers, start zuul-executors, start nodepool-launchers, start nodepool-builders, start zuul-web, re-enqueue | 23:44 |
*** claudiub has quit IRC | 23:52 | |
*** rkukura has quit IRC | 23:53 | |
pabelanger | yah, we also need to restart nodepool | 23:53 |
pabelanger | ++ | 23:53 |
corvus | everything is restarted now; zuul is loading config | 23:54 |
*** iyamahat has joined #openstack-infra | 23:55 | |
corvus | #status notice Zuul has been restarted to update to the latest code; existing changes have been re-enqueued, you may need to recheck changes uploaded in the past 10 minutes | 23:57 |
openstackstatus | corvus: sending notice | 23:57 |
*** rpioso is now known as rpioso|afk | 23:57 | |
-openstackstatus- NOTICE: Zuul has been restarted to update to the latest code; existing changes have been re-enqueued, you may need to recheck changes uploaded in the past 10 minutes | 23:58 | |
*** rossella_s has joined #openstack-infra | 23:58 | |
corvus | jobs seem to be running | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!