*** andreas_s has joined #openstack-infra | 00:00 | |
*** lindycoder has quit IRC | 00:00 | |
ianw | project-core: i wouldn't mind moving on https://review.openstack.org/514117 and https://review.openstack.org/513233 . it's moving zuul.projects to a dict ... we're halfway there with zuul exporting zuul._projects, but i'd like to get rid of that asap | 00:00 |
---|---|---|
*** hongbin has quit IRC | 00:01 | |
ianw | fungi: did you notice the puppet-agent errors when you were launching? https://review.openstack.org/#/c/515279/ worked for me, but also we determined i was on an older checkout | 00:03 |
ianw | i can abandon that if it's fixed elsewhere | 00:03 |
*** hughhalf has joined #openstack-infra | 00:07 | |
dhinesh | still hitting https://pastebin.com/dVXAH3ia on nodepool 0.4.1 , 0.4.0 works fine without issues any suggestions? | 00:09 |
clarkb | dhinesh: yes as explained earlier 0.4.1 isn't a "real" version because we have not tagged that version. This means you can't specify it as the version for pip to install | 00:10 |
clarkb | dhinesh: it is a logical version for all commits after 0.4.0 because the next version will have to be at least 0.4.1 | 00:10 |
*** catintheroof has joined #openstack-infra | 00:11 | |
*** burgerk has joined #openstack-infra | 00:13 | |
*** rcernin has joined #openstack-infra | 00:21 | |
jeblair | ianw: +2 from me on both of those. i'd also like to wrap that up. :) | 00:21 |
*** bobh has joined #openstack-infra | 00:22 | |
dhinesh | clarkb: ok, looks like 0.4.1 start the service as nodepool user, i was able to fix it after i installed it through nodepool user | 00:22 |
*** eroux has quit IRC | 00:25 | |
*** andreas_s has quit IRC | 00:26 | |
*** andreas_s has joined #openstack-infra | 00:28 | |
*** catintheroof has quit IRC | 00:32 | |
*** burgerk has quit IRC | 00:36 | |
*** catintheroof has joined #openstack-infra | 00:38 | |
*** gyee has quit IRC | 00:40 | |
*** sree has joined #openstack-infra | 00:40 | |
*** catintheroof has quit IRC | 00:42 | |
*** andreas_s has quit IRC | 00:43 | |
*** xarses has joined #openstack-infra | 00:43 | |
*** Apoorva_ has joined #openstack-infra | 00:43 | |
*** catintheroof has joined #openstack-infra | 00:44 | |
*** andreas_s has joined #openstack-infra | 00:44 | |
*** sree has quit IRC | 00:45 | |
*** Apoorva has quit IRC | 00:47 | |
*** Apoorva_ has quit IRC | 00:47 | |
*** andreas_s has quit IRC | 00:48 | |
*** gouthamr has joined #openstack-infra | 00:48 | |
dhinesh | after upgrading nodepool from 3.1 to 4.1 none of the commands seem to be working, https://pastebin.com/cDEQFrGU | 00:50 |
dhinesh | is the hostname parameter need to be set anywhere? | 00:50 |
*** kiennt26 has joined #openstack-infra | 00:55 | |
*** andreas_s has joined #openstack-infra | 00:55 | |
*** catintheroof has quit IRC | 00:55 | |
*** catintheroof has joined #openstack-infra | 00:56 | |
*** LindaWang has joined #openstack-infra | 00:56 | |
*** zzzeek has quit IRC | 00:58 | |
*** xarses has quit IRC | 01:00 | |
*** andreas_s has quit IRC | 01:00 | |
*** zzzeek has joined #openstack-infra | 01:00 | |
*** andreas_s has joined #openstack-infra | 01:01 | |
*** smatzek has joined #openstack-infra | 01:02 | |
fungi | ianw: yeah, for some reason the puppet packages on xenial now enable the puppet-agent daemon by default and it gets unhappy because we don't configure it to use a puppetmaster | 01:12 |
*** andreas_s has quit IRC | 01:12 | |
ianw | yep that was what i found, it was disabled, but not actually stopped | 01:13 |
fungi | dmsimard: i didn't get a quote for the shipping, we didn't get that far. i got a preliminary quote from the osl for what the setup and ongoing hosting costs would be, but that was contingent on us figuring out switches and mounting rails | 01:13 |
*** huanxie has joined #openstack-infra | 01:13 | |
dmsimard | fungi: Ok, thanks ! | 01:14 |
*** liujiong has joined #openstack-infra | 01:15 | |
*** mriedem has joined #openstack-infra | 01:15 | |
*** mriedem_away has quit IRC | 01:15 | |
*** thorst has joined #openstack-infra | 01:16 | |
*** thorst has quit IRC | 01:17 | |
*** andreas_s has joined #openstack-infra | 01:18 | |
*** catintheroof has quit IRC | 01:20 | |
*** namnh has joined #openstack-infra | 01:21 | |
*** catintheroof has joined #openstack-infra | 01:21 | |
*** Kevin_Zheng has joined #openstack-infra | 01:21 | |
*** fanzhang has quit IRC | 01:24 | |
*** fanzhang_ has joined #openstack-infra | 01:24 | |
*** catintheroof has quit IRC | 01:24 | |
*** gmann_afk is now known as gmann | 01:25 | |
*** dave-mccowan has quit IRC | 01:26 | |
*** andreas_s has quit IRC | 01:28 | |
*** dave-mccowan has joined #openstack-infra | 01:28 | |
*** psachin has joined #openstack-infra | 01:35 | |
*** andreas_s has joined #openstack-infra | 01:37 | |
*** csomerville has joined #openstack-infra | 01:42 | |
*** rcernin has quit IRC | 01:43 | |
*** yamamoto has joined #openstack-infra | 01:43 | |
*** rcernin has joined #openstack-infra | 01:44 | |
*** dhinesh has quit IRC | 01:44 | |
*** fanzhang_ is now known as fanzhang | 01:44 | |
*** cody-somerville has quit IRC | 01:46 | |
*** andreas_s has quit IRC | 01:47 | |
*** yamamoto has quit IRC | 01:49 | |
*** smatzek has quit IRC | 01:50 | |
*** catintheroof has joined #openstack-infra | 01:55 | |
*** salv-orlando has joined #openstack-infra | 01:55 | |
*** andreas_s has joined #openstack-infra | 01:56 | |
*** askb has quit IRC | 01:56 | |
*** salv-orlando has quit IRC | 02:00 | |
*** catintheroof has quit IRC | 02:00 | |
*** andreas_s has quit IRC | 02:02 | |
*** hongbin has joined #openstack-infra | 02:05 | |
*** stakeda has joined #openstack-infra | 02:09 | |
*** askb has joined #openstack-infra | 02:10 | |
*** bobh has quit IRC | 02:11 | |
*** xarses has joined #openstack-infra | 02:13 | |
*** rkukura has quit IRC | 02:13 | |
*** gcb has joined #openstack-infra | 02:16 | |
*** thorst has joined #openstack-infra | 02:18 | |
*** liujiong has quit IRC | 02:18 | |
*** liujiong has joined #openstack-infra | 02:18 | |
*** bh526r has quit IRC | 02:19 | |
*** huanxie has quit IRC | 02:22 | |
*** andreas_s has joined #openstack-infra | 02:22 | |
*** thorst has quit IRC | 02:23 | |
*** annp has joined #openstack-infra | 02:23 | |
*** andreas_s has quit IRC | 02:26 | |
*** catintheroof has joined #openstack-infra | 02:28 | |
*** lindycoder has joined #openstack-infra | 02:29 | |
*** yamahata has quit IRC | 02:29 | |
*** iyamahat has quit IRC | 02:30 | |
*** andreas_s has joined #openstack-infra | 02:33 | |
*** catintheroof has quit IRC | 02:33 | |
*** fanzhang has left #openstack-infra | 02:34 | |
*** cuongnv has joined #openstack-infra | 02:34 | |
*** andreas_s has quit IRC | 02:44 | |
*** yamamoto has joined #openstack-infra | 02:45 | |
*** lindycoder has quit IRC | 02:49 | |
*** spzala has quit IRC | 02:50 | |
*** yamamoto has quit IRC | 02:51 | |
Jeffrey4l | when one project depends on another project's jobs, the second project is not added into the first project's dependency. is this correct? and how can i fix this? | 02:53 |
Jeffrey4l | like this patch https://review.openstack.org/#/c/513971/ | 02:53 |
*** andreas_s has joined #openstack-infra | 02:57 | |
*** armaan has joined #openstack-infra | 02:59 | |
kazsh | Hi infra core team, could you someone review my change on project-config ? https://review.openstack.org/#/c/514895/ this is show-stopper for our project... | 03:01 |
*** spzala has joined #openstack-infra | 03:02 | |
openstackgerrit | Merged openstack-infra/project-config master: Remove Tacker legacy job https://review.openstack.org/515341 | 03:03 |
openstackgerrit | Merged openstack-infra/project-config master: Update neutron-failure-rate grafana dashboard https://review.openstack.org/515439 | 03:03 |
openstackgerrit | Merged openstack-infra/project-config master: Add nodejs4-publish jobs https://review.openstack.org/515332 | 03:03 |
openstackgerrit | Merged openstack-infra/project-config master: Switch kuryr back to publish-to-pypi https://review.openstack.org/515511 | 03:05 |
openstackgerrit | Merged openstack-infra/project-config master: Switch octavia back to publish-to-pypi https://review.openstack.org/515518 | 03:05 |
clarkb | Jeffrey4l: I'm not sure I understand the question. But my understanding is that both repos are pushed into the job when there is a depends on | 03:06 |
*** spzala has quit IRC | 03:06 | |
*** andreas_s has quit IRC | 03:07 | |
clarkb | as for using other projects jobs and what repos are available the devstack job may be a good example for that? I think shade is using jobs defined in devstack now | 03:07 |
*** andreas_s has joined #openstack-infra | 03:12 | |
openstackgerrit | Merged openstack-infra/project-config master: Add docs job for VPNaaS and VPNaaS dashboard https://review.openstack.org/514906 | 03:12 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 03:13 |
*** dave-mccowan has quit IRC | 03:14 | |
Jeffrey4l | check the newer patch in https://review.openstack.org/513971 , i think i found the solution. | 03:14 |
Jeffrey4l | clarkb, | 03:14 |
openstackgerrit | Merged openstack-infra/project-config master: Updated templates for heat-dashboard https://review.openstack.org/514895 | 03:15 |
*** bhavik1 has joined #openstack-infra | 03:15 | |
*** yamamoto has joined #openstack-infra | 03:16 | |
*** ihrachys has quit IRC | 03:17 | |
johnsom | Low priority, but stackalytics is throwing 502s (nginx) | 03:17 |
*** andreas_s has quit IRC | 03:18 | |
*** andreas_s has joined #openstack-infra | 03:19 | |
*** huanxie has joined #openstack-infra | 03:19 | |
clarkb | johnsom: which stackalytics? | 03:23 |
clarkb | johnsom: we don't run the production one | 03:23 |
johnsom | http://stackalytics.com | 03:23 |
clarkb | (there is a stackalytics.openstack.org though that we've fiddled with running) | 03:23 |
johnsom | Ah, didn't know | 03:23 |
johnsom | Just happened to notice so thought I would mention it. | 03:23 |
*** andreas_s has quit IRC | 03:24 | |
*** kiennt26 has quit IRC | 03:25 | |
*** kiennt26 has joined #openstack-infra | 03:26 | |
*** bhavik1 has quit IRC | 03:27 | |
clarkb | ya unfortunately I don't think there is much we can do | 03:29 |
*** nicolasbock has quit IRC | 03:30 | |
openstackgerrit | Tony Breeds proposed openstack-infra/system-config master: Remove unneeded encoding change. https://review.openstack.org/515580 | 03:33 |
*** andreas_s has joined #openstack-infra | 03:34 | |
*** hongbin has quit IRC | 03:35 | |
*** yamamoto has quit IRC | 03:36 | |
*** andreas_s has quit IRC | 03:38 | |
*** kiennt26 has quit IRC | 03:39 | |
*** yamahata has joined #openstack-infra | 03:40 | |
*** kiennt26 has joined #openstack-infra | 03:40 | |
*** markvoelker has quit IRC | 03:43 | |
*** andreas_s has joined #openstack-infra | 03:47 | |
*** andreas_s has quit IRC | 03:52 | |
*** gongysh has joined #openstack-infra | 03:53 | |
*** yamamoto has joined #openstack-infra | 03:54 | |
*** yamamoto has quit IRC | 03:54 | |
*** yamamoto has joined #openstack-infra | 03:55 | |
*** kiennt26 has quit IRC | 03:56 | |
*** salv-orlando has joined #openstack-infra | 03:56 | |
*** udesale has joined #openstack-infra | 03:58 | |
*** salv-orlando has quit IRC | 04:00 | |
*** huanxie has quit IRC | 04:01 | |
*** catintheroof has joined #openstack-infra | 04:01 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove non-voting from legacy-puppet-beaker-rspec https://review.openstack.org/515582 | 04:03 |
*** psachin has quit IRC | 04:03 | |
*** spzala has joined #openstack-infra | 04:03 | |
AJaeger | ianw: that's my proposed fix for https://review.openstack.org/514872 ^ | 04:04 |
*** catintheroof has quit IRC | 04:05 | |
*** spzala has quit IRC | 04:07 | |
ianw | AJaeger: ok, so the job def was wrong, that makes sense | 04:09 |
*** bstinson has quit IRC | 04:10 | |
*** ykarel has joined #openstack-infra | 04:10 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Silence ansible-lint https://review.openstack.org/515520 | 04:11 |
AJaeger | and here's one fix I like in as well ^ | 04:11 |
*** bstinson has joined #openstack-infra | 04:14 | |
*** mriedem has quit IRC | 04:15 | |
*** kiennt26 has joined #openstack-infra | 04:15 | |
*** Apoorva has joined #openstack-infra | 04:16 | |
*** rosmaita has quit IRC | 04:18 | |
*** thorst has joined #openstack-infra | 04:19 | |
*** zoli has quit IRC | 04:19 | |
*** dbecker has quit IRC | 04:20 | |
*** annp has quit IRC | 04:20 | |
*** claudiub|2 has joined #openstack-infra | 04:20 | |
*** psachin has joined #openstack-infra | 04:20 | |
*** pleia2 has quit IRC | 04:21 | |
*** zoli has joined #openstack-infra | 04:21 | |
*** pleia2 has joined #openstack-infra | 04:22 | |
*** thorst has quit IRC | 04:24 | |
openstackgerrit | Merged openstack-infra/project-config master: Remove legacy-osc-dsvm-functional* jobs https://review.openstack.org/515075 | 04:24 |
*** gildub has quit IRC | 04:24 | |
*** annp has joined #openstack-infra | 04:28 | |
*** janki has joined #openstack-infra | 04:36 | |
*** dbecker has joined #openstack-infra | 04:36 | |
*** sree has joined #openstack-infra | 04:37 | |
*** logan- has quit IRC | 04:38 | |
*** gildub has joined #openstack-infra | 04:40 | |
*** ramishra has quit IRC | 04:42 | |
*** logan- has joined #openstack-infra | 04:42 | |
*** gongysh has quit IRC | 04:42 | |
*** ramishra has joined #openstack-infra | 04:44 | |
*** csomerville has quit IRC | 04:44 | |
*** csomerville has joined #openstack-infra | 04:45 | |
*** Apoorva has quit IRC | 04:46 | |
*** xarses has quit IRC | 04:48 | |
*** lamt has quit IRC | 04:51 | |
*** _Cyclone_ has quit IRC | 04:51 | |
*** pleia2 has quit IRC | 04:51 | |
*** bstinson has quit IRC | 04:51 | |
*** kuromagi has quit IRC | 04:51 | |
*** cburgess has quit IRC | 04:51 | |
*** cody-somerville has joined #openstack-infra | 04:53 | |
*** cody-somerville has joined #openstack-infra | 04:53 | |
*** csomerville has quit IRC | 04:55 | |
*** shu-mutou has joined #openstack-infra | 05:02 | |
*** spzala has joined #openstack-infra | 05:02 | |
*** bhavik1 has joined #openstack-infra | 05:02 | |
*** _Cyclone_ has joined #openstack-infra | 05:03 | |
*** cburgess has joined #openstack-infra | 05:03 | |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: DNM: test ansible crash logging https://review.openstack.org/515470 | 05:04 |
*** lamt has joined #openstack-infra | 05:04 | |
*** bstinson has joined #openstack-infra | 05:04 | |
*** lamt is now known as Guest53850 | 05:04 | |
*** jaosorior has joined #openstack-infra | 05:05 | |
*** pleia2 has joined #openstack-infra | 05:05 | |
*** kuromagi has joined #openstack-infra | 05:05 | |
*** spzala has quit IRC | 05:06 | |
*** huanxie has joined #openstack-infra | 05:08 | |
openstackgerrit | Merged openstack-infra/zuul-jobs master: Silence ansible-lint https://review.openstack.org/515520 | 05:10 |
chandankumar | ianw: please have a look on this https://review.openstack.org/#/c/513698/ thanks :-) | 05:10 |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: [DNM] start at templating out the zuulv3 jobs https://review.openstack.org/512450 | 05:10 |
*** slaweq has joined #openstack-infra | 05:11 | |
*** psachin has quit IRC | 05:14 | |
*** psachin has joined #openstack-infra | 05:15 | |
*** slaweq has quit IRC | 05:16 | |
*** bstinson has quit IRC | 05:19 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: WIP: Run ansible-lint on repo https://review.openstack.org/515593 | 05:19 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: WIP: Run ansible-lint on repo https://review.openstack.org/515593 | 05:19 |
*** yamamoto has quit IRC | 05:21 | |
*** bstinson has joined #openstack-infra | 05:22 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove non-voting from legacy-puppet-beaker-rspec https://review.openstack.org/515582 | 05:24 |
AJaeger | ianw: missed one place ^ | 05:25 |
*** salv-orlando has joined #openstack-infra | 05:25 | |
*** gongysh has joined #openstack-infra | 05:25 | |
*** Guest87185 has quit IRC | 05:31 | |
*** dhajare|mtg has joined #openstack-infra | 05:36 | |
*** salv-orlando has quit IRC | 05:38 | |
*** salv-orlando has joined #openstack-infra | 05:38 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: zuul-jobs: Run ansible-lint https://review.openstack.org/515596 | 05:39 |
*** iyamahat has joined #openstack-infra | 05:39 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: WIP: Run ansible-lint on repo https://review.openstack.org/515593 | 05:39 |
AJaeger | pabelanger, ianw, frickler, tobiash, with these two changes above, we can run ansible-lint on zuul-jobs ^ | 05:39 |
AJaeger | Made it non-voting for now - like it's on openstack-zuul-jobs | 05:39 |
tobiash | yay | 05:40 |
tobiash | AJaeger: the first has a syntax error reported from zuul | 05:40 |
AJaeger | tobiash: coming... | 05:40 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: zuul-jobs: Run ansible-lint https://review.openstack.org/515596 | 05:40 |
AJaeger | once that is in, we can tackle project-config... My previous test run succeeded on zuul-jobs, so now doing it properly | 05:41 |
tobiash | AJaeger: shall this replace tox-linters? | 05:41 |
AJaeger | tobiash: yes | 05:41 |
AJaeger | tobiash: uses same tox environment | 05:42 |
AJaeger | tobiash: pabelanger plans to make an ansible job, so we might reintroduce tox-linters later ;) | 05:42 |
AJaeger | ansible job instead of calling tox | 05:42 |
tobiash | ah, ok | 05:42 |
*** yolanda has joined #openstack-infra | 05:42 | |
tobiash | then you have my +1 | 05:43 |
AJaeger | thanks | 05:43 |
AJaeger | do you want to review zuul-jobs as well? I'll remove WIP now ;( | 05:43 |
*** markvoelker has joined #openstack-infra | 05:43 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Run ansible-lint on repo https://review.openstack.org/515593 | 05:44 |
tobiash | ah, thought it is a not ready for review yet | 05:44 |
AJaeger | tobiash: recheck will not help, it depends on the project-config. I left the WIP in by accident ;( | 05:44 |
AJaeger | See the first run for success | 05:44 |
AJaeger | (first changeset) | 05:44 |
tobiash | AJaeger: is that 'voting: false' correct there: https://review.openstack.org/#/c/515593/4/tox.ini | 05:48 |
tobiash | ? | 05:48 |
tobiash | that's new to me | 05:48 |
AJaeger | tobiash: good catch, I screwed up ;( | 05:48 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Run ansible-lint on repo https://review.openstack.org/515593 | 05:49 |
AJaeger | trying again ... | 05:49 |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: DNM: test ansible crash logging https://review.openstack.org/515470 | 05:49 |
*** cshastri has joined #openstack-infra | 05:51 | |
*** iyamahat has quit IRC | 05:52 | |
tobiash | +2 | 05:53 |
*** dsariel has joined #openstack-infra | 05:56 | |
AJaeger | thanks | 05:56 |
*** aeng has quit IRC | 06:01 | |
*** yolanda__ has joined #openstack-infra | 06:02 | |
*** spzala has joined #openstack-infra | 06:03 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Ansible-lint project-config https://review.openstack.org/515603 | 06:04 |
AJaeger | pabelanger: and one job for project-config ^ | 06:04 |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: Do late decoding of log stream buffer https://review.openstack.org/515043 | 06:04 |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: Correctly stream the remaining buffer https://review.openstack.org/515470 | 06:04 |
*** spzala has quit IRC | 06:07 | |
*** jbadiapa has joined #openstack-infra | 06:10 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Add ansible-linter job for project-config https://review.openstack.org/515607 | 06:12 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Ansible-lint project-config https://review.openstack.org/515603 | 06:12 |
*** vsaienk0 has joined #openstack-infra | 06:17 | |
*** markvoelker has quit IRC | 06:18 | |
*** bhavik1 has quit IRC | 06:18 | |
*** gildub has quit IRC | 06:19 | |
*** thorst has joined #openstack-infra | 06:20 | |
*** gouthamr has quit IRC | 06:21 | |
*** spectr has joined #openstack-infra | 06:22 | |
*** andreas_s has joined #openstack-infra | 06:24 | |
*** thorst has quit IRC | 06:25 | |
*** gongysh has quit IRC | 06:36 | |
*** cody-somerville has quit IRC | 06:36 | |
*** kzaitsev1pi has joined #openstack-infra | 06:38 | |
*** kzaitsev_pi has quit IRC | 06:40 | |
openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: Correctly stream the remaining buffer https://review.openstack.org/515470 | 06:45 |
*** ccamacho has quit IRC | 06:47 | |
*** pgadiya has joined #openstack-infra | 06:48 | |
tobiash | infra-root: I noticed a strange failure in a pre playbook of a pep8 job on the zuul project: http://paste.openstack.org/show/624784/ | 06:51 |
tobiash | looks like something infrastructure (mirror?) related | 06:51 |
tobiash | the change is 515043,5 | 06:54 |
tobiash | zuul did correctly retry and the overall build is green | 06:54 |
*** cody-somerville has joined #openstack-infra | 06:56 | |
*** csomerville has joined #openstack-infra | 06:58 | |
*** pblaho has joined #openstack-infra | 07:00 | |
*** cody-somerville has quit IRC | 07:01 | |
*** armaan has quit IRC | 07:04 | |
*** armaan has joined #openstack-infra | 07:05 | |
*** rossella_s has joined #openstack-infra | 07:06 | |
ianw | tobiash: 37.153.172.245 ... hmm | 07:09 |
tobiash | ianw: so bindep timed out and collect tox logs threw this ssh error | 07:10 |
tobiash | just wanted to inform that there might be something wrong | 07:10 |
tobiash | but could also be some transient network issue | 07:11 |
*** florianf has joined #openstack-infra | 07:11 | |
*** slaweq has joined #openstack-infra | 07:12 | |
ianw | yeah, all the cityhost mirrors seem up | 07:14 |
*** markvoelker has joined #openstack-infra | 07:14 | |
*** armaan has quit IRC | 07:15 | |
*** armaan has joined #openstack-infra | 07:15 | |
*** slaweq has quit IRC | 07:16 | |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: [DNM] start at templating out the zuulv3 jobs https://review.openstack.org/512450 | 07:17 |
*** vsaienk0 has quit IRC | 07:20 | |
*** vsaienk0 has joined #openstack-infra | 07:20 | |
*** armax has quit IRC | 07:21 | |
*** tesseract has joined #openstack-infra | 07:22 | |
*** armax has joined #openstack-infra | 07:22 | |
*** armax has quit IRC | 07:22 | |
*** armax has joined #openstack-infra | 07:23 | |
*** armax has quit IRC | 07:23 | |
*** armax has joined #openstack-infra | 07:24 | |
*** armax has quit IRC | 07:24 | |
*** armax has joined #openstack-infra | 07:24 | |
*** armax has quit IRC | 07:25 | |
*** armax has joined #openstack-infra | 07:25 | |
*** armax has quit IRC | 07:25 | |
*** armax has joined #openstack-infra | 07:26 | |
*** armax has quit IRC | 07:26 | |
*** slaweq has joined #openstack-infra | 07:26 | |
*** armax has joined #openstack-infra | 07:27 | |
*** yolanda__ has quit IRC | 07:27 | |
*** armax has quit IRC | 07:27 | |
openstackgerrit | Vasyl Saienko proposed openstack-infra/project-config master: Remove legacy ironic-lib jobs https://review.openstack.org/515620 | 07:28 |
*** salv-orlando has quit IRC | 07:30 | |
openstackgerrit | Vasyl Saienko proposed openstack-infra/openstack-zuul-jobs master: Cleanup ironic-lib legacy jobs https://review.openstack.org/515623 | 07:31 |
*** salv-orlando has joined #openstack-infra | 07:31 | |
*** thorst has joined #openstack-infra | 07:33 | |
*** namnh has quit IRC | 07:34 | |
*** salv-orlando has quit IRC | 07:35 | |
*** pblaho has quit IRC | 07:37 | |
*** thorst has quit IRC | 07:38 | |
*** pblaho has joined #openstack-infra | 07:42 | |
*** CrayZee has quit IRC | 07:43 | |
vsaienk0 | AJaeger: yolanda please add to your review queue - cleanup ironic-lib legacy jobs https://review.openstack.org/515620 and https://review.openstack.org/515623 | 07:43 |
*** e0ne has joined #openstack-infra | 07:45 | |
*** kazsh has left #openstack-infra | 07:47 | |
*** markvoelker has quit IRC | 07:48 | |
*** jpena|off is now known as jpena | 07:49 | |
*** ralonsoh has joined #openstack-infra | 07:50 | |
*** amoralej|off is now known as amoralej | 07:51 | |
*** salv-orlando has joined #openstack-infra | 07:52 | |
*** makowals has joined #openstack-infra | 07:58 | |
*** rcernin has quit IRC | 07:58 | |
*** gcb has quit IRC | 08:00 | |
*** liujiong has quit IRC | 08:02 | |
*** liujiong has joined #openstack-infra | 08:02 | |
*** gcb has joined #openstack-infra | 08:03 | |
*** slaweq has quit IRC | 08:04 | |
*** Hal has joined #openstack-infra | 08:05 | |
*** slaweq has joined #openstack-infra | 08:05 | |
*** Hal is now known as Guest72852 | 08:05 | |
*** hashar has joined #openstack-infra | 08:09 | |
*** ccamacho has joined #openstack-infra | 08:10 | |
AJaeger | vsaienk0: commented. What about moving tox-cover into project-config, see my comments | 08:14 |
*** yamahata has quit IRC | 08:14 | |
openstackgerrit | Vasyl Saienko proposed openstack-infra/project-config master: Remove legacy ironic-lib jobs https://review.openstack.org/515620 | 08:17 |
vsaienk0 | AJaeger: updated | 08:18 |
AJaeger | vsaienk0: +2 | 08:18 |
*** armaan has quit IRC | 08:18 | |
*** armaan has joined #openstack-infra | 08:19 | |
vsaienk0 | AJaeger: thanks! | 08:22 |
*** jascott1 has quit IRC | 08:22 | |
*** jascott1 has joined #openstack-infra | 08:23 | |
*** martinkopec has joined #openstack-infra | 08:25 | |
*** lucas-afk is now known as lucasagomes | 08:26 | |
*** jascott1 has quit IRC | 08:27 | |
*** panda|ruck|off is now known as panda|ruck | 08:29 | |
openstackgerrit | Jose Luis Franco proposed openstack-infra/tripleo-ci master: WIP: Upgrade UC and OC using tripleo-upgrade role https://review.openstack.org/515643 | 08:29 |
*** mhickey has joined #openstack-infra | 08:30 | |
openstackgerrit | Merged openstack-infra/project-config master: Remove ansible-role-jobs template from openstack-ansible-ops https://review.openstack.org/515377 | 08:33 |
*** udesale has quit IRC | 08:35 | |
hwoarang | jeblair: mordred: we are facing an issue in openstack-ansible where our post-run playbook becomes available (cloned) during the run phase. However, zuul complains that 'ERROR Unable to find playbook ...' blah blah. Do you know if it's possible for zuulv3 to do the post-run evaluation after the 'run' phase? | 08:38 |
SamYaple | hwoarang: i wonder if you could drop a placeholder in there to trick zuul | 08:38 |
SamYaple | or if its rendering all of that ahead of time | 08:39 |
hwoarang | yeah we can't drop a dummy file easily there. if we go down that road we may as well copy the playbook | 08:40 |
hwoarang | just wanted to check if there is a magic flag to turn on like static/dynamic playbook inclusion in ansible | 08:40 |
SamYaple | hwoarang: not sure your exact use case, but why not clone the project initially? | 08:40 |
SamYaple | do you have a patchset you are workign with | 08:40 |
vsaienk0 | AJaeger: we found that after migration to zuulv3 synchronize task stop working for ironic-python-agent project. When we fixed https://review.openstack.org/#/c/515356/ we found that now it uploads UPLOAD_RAW/ and UPLOAD_TAR folders instead of copying content should be address by https://review.openstack.org/#/c/515484/ . Just curious if it is known issue? | 08:41 |
hwoarang | SamYaple: the 'run' playbook is responsible to clone the post-run one | 08:41 |
hwoarang | but it will not run | 08:41 |
hwoarang | wait | 08:41 |
SamYaple | hwoarang: right, but why do it that way? why not clone it ahead of time ala required-projects | 08:42 |
SamYaple | actually, now that im saying that that wont work | 08:42 |
hwoarang | hmm | 08:42 |
hwoarang | that sounds reasonable. Why would it not work | 08:43 |
SamYaple | hwoarang: you can try it, but i suspect you wont be able to since its only going to be looking in that repos relative directories | 08:44 |
SamYaple | the job would need to be declared in teh repo where the post-playbook lives and parented in the other repo you want to consume that playbook i believe | 08:44 |
AJaeger | vsaienk0: better ask rest of team, I'm not yet familiar with these ;( | 08:44 |
hwoarang | yeah that's true | 08:44 |
*** huanxie has quit IRC | 08:44 | |
*** markvoelker has joined #openstack-infra | 08:45 | |
*** derekh has joined #openstack-infra | 08:45 | |
*** sflanigan has joined #openstack-infra | 08:46 | |
*** sflanigan has joined #openstack-infra | 08:46 | |
SamYaple | hwoarang: i bet you could declare the base job in repo A, then parent it in repo B with your post-playbook, then parent that job in repo A again to consume it | 08:46 |
*** shu-mutou is now known as shu-mutou-AWAY | 08:46 | |
SamYaple | (depending on what youre trying to do) | 08:46 |
AJaeger | frickler: could you review https://review.openstack.org/#/q/status:open+topic:newton-eol and https://review.openstack.org/#/q/status:open+topic:ansible-lint , please? Neither contains urgent fixes, just allow me to move forward with testing and cleanup ;) | 08:46 |
vsaienk0 | infra team could you please check issue with synchronize task described above ^ | 08:47 |
tobiash | hwoarang: zuul prepares all playbooks it runs by itself before running each playbook, the job cannot clone further playbooks and expect zuul to run them | 08:48 |
*** electrofelix has joined #openstack-infra | 08:48 | |
tobiash | hwoarang: what are you trying to achieve? | 08:49 |
*** spiffxp_ has quit IRC | 08:50 | |
hwoarang | tobiash: we have a post-run playbook that we want to re-use across different repos. So we were hopping to clone the common playbook from that repo in our other repos and use it | 08:50 |
hwoarang | something like that | 08:50 |
tobiash | hwoarang: for this you can define a job in the repo with the common playbook only specifying the post-run | 08:51 |
*** ykarel is now known as ykarel|lunch | 08:51 | |
*** spiffxp has joined #openstack-infra | 08:51 | |
*** namnh has joined #openstack-infra | 08:51 | |
tobiash | hwoarang: your other jobs can inherit from this one and all will use this post playbook | 08:51 |
tobiash | hwoarang: regardless of the repo they are defined in | 08:51 |
*** pblaho has quit IRC | 08:51 | |
*** udesale has joined #openstack-infra | 08:53 | |
openstackgerrit | Merged openstack-infra/project-config master: Remove legacy ironic-lib jobs https://review.openstack.org/515620 | 08:53 |
hwoarang | tobiash: interesting. let me try | 08:54 |
*** dizquierdo has joined #openstack-infra | 08:54 | |
*** sdague has joined #openstack-infra | 08:54 | |
tobiash | hwoarang: the key for this is job inheritence: https://docs.openstack.org/infra/zuul/user/config.html#job | 08:56 |
AJaeger | hwoarang: that should work without any change, zuul should do the magic | 08:56 |
hwoarang | ok i will check thank you for all the pointers | 08:57 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Newton-EOL: Remove obsolete jobs https://review.openstack.org/515493 | 08:59 |
openstackgerrit | Jose Luis Franco proposed openstack-infra/tripleo-ci master: WIP: Upgrade UC and OC using tripleo-upgrade role https://review.openstack.org/515643 | 09:02 |
*** sflanigan has quit IRC | 09:02 | |
*** spzala has joined #openstack-infra | 09:04 | |
*** spzala has quit IRC | 09:09 | |
*** bmjen has quit IRC | 09:11 | |
SamYaple | tobiash: they are just reusing the job from repo A in repo B, without parenting it. *however* they were not explictly setting repo A in the required-projects list of the job | 09:13 |
tobiash | SamYaple: I think that should work automatically without having A in required-projects | 09:14 |
SamYaple | tobiash: it does not | 09:15 |
SamYaple | it will implicitly clone the current repo, but when you run in repo B, that means its only cloning repo B | 09:15 |
SamYaple | you have to explicity declare repo A to use it when running the job in repo B | 09:15 |
* SamYaple just went through this for cross-project loci jobs | 09:16 | |
*** slaweq has quit IRC | 09:16 | |
*** slaweq has joined #openstack-infra | 09:17 | |
tobiash | SamYaple: but that sounds like a bug (and might also be related to the branching issues jeblair is fixing currently) | 09:17 |
*** tosky has joined #openstack-infra | 09:17 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Fuel branch cleanup https://review.openstack.org/515501 | 09:17 |
AJaeger | frickler: thanks for reviewing, fixed ^ | 09:17 |
*** bmjen has joined #openstack-infra | 09:17 | |
*** markvoelker has quit IRC | 09:18 | |
tobiash | SamYaple: you won't have repo A in your workspace but the playbook must be available to zuul | 09:18 |
SamYaple | tobiash: there was some conversations about it. in the past. ill have to check my logs. im not sure if it was a bug, or making the clone automatically work was a new feature, but it is documented that you have to explictly declare it, so someone, at sometime knew about this | 09:18 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Fuel branch cleanup https://review.openstack.org/515501 | 09:18 |
tobiash | SamYaple: can you point me to where this is documented? | 09:19 |
*** ykarel|lunch is now known as ykarel | 09:20 | |
SamYaple | tobiash: yea im looking through my logs. someone in infra walked me through this when we first rolled to v3 (before the rollback) | 09:20 |
*** slaweq has quit IRC | 09:22 | |
*** slaweq has joined #openstack-infra | 09:22 | |
SamYaple | oops tell a lie. this was only a couple weeks ago. time flies right | 09:23 |
SamYaple | tobiash: http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2017-10-16.log.html#t2017-10-16T19:55:47 | 09:23 |
SamYaple | tobiash: i did absoluely require that to make this work, just fyi | 09:25 |
SamYaple | perhaps mordred can explain why its needed (and if its a bug)? he doesnt seem to be talking like its a bug | 09:25 |
tobiash | SamYaple: because you use the other repo in your job or because the jobs playbooks weren't there? | 09:26 |
tobiash | SamYaple: there is a difference in having the playbook available on the executor (as part of the job) | 09:26 |
SamYaple | tobiash: because i was putting a loci job (repo A) in the cinder pipelines (repo B) i needed to explictly declare repo A in my required projects list | 09:26 |
tobiash | SamYaple: and having the repo available in the node where the job is executed | 09:27 |
SamYaple | this is the same thing openstack-ansible is doing | 09:27 |
SamYaple | tobiash: https://review.openstack.org/#/c/512400/ was required for https://review.openstack.org/#/c/512398/ | 09:28 |
*** LindaWang has quit IRC | 09:29 | |
*** huanxie has joined #openstack-infra | 09:29 | |
*** LindaWang has joined #openstack-infra | 09:29 | |
SamYaple | tobiash: yea i get what youre saying. im not sure then. its like 530am right now. if you find out my understanding is incorrect, please correct me :D i will check scrollback when i wake up | 09:30 |
tobiash | SamYaple: does the loci-cinder job something with the loci repo? | 09:31 |
tobiash | that would be something what would require required-projects | 09:32 |
SamYaple | the run job uses files from the loci repo, yes. which i saw was your point earlier | 09:32 |
*** armaan has quit IRC | 09:32 | |
SamYaple | but thats what OSA is doign too | 09:32 |
*** armaan has joined #openstack-infra | 09:32 | |
tobiash | ah I see | 09:32 |
tobiash | then required-projects is needed | 09:32 |
SamYaple | yea i understand the distinction you are making too | 09:32 |
SamYaple | im still happy to be wrong here, but i cant process any more zuul stuff tonight, so im heading out | 09:33 |
SamYaple | thanks for explaining more to me | 09:33 |
tobiash | I have to check the doc, maybe it's not clear enough about that | 09:33 |
tobiash | np | 09:33 |
*** thorst has joined #openstack-infra | 09:34 | |
AJaeger | we might have some subtile differences here - http://git.openstack.org/cgit/openstack/openstack-manuals/tree/.zuul.yaml#n7 is used in http://git.openstack.org/cgit/openstack/api-site/tree/.zuul.yaml#n7 - without required-projects | 09:35 |
AJaeger | but since it's all self-testing, hwoarang should be be able to iterate on it ;) | 09:35 |
tobiash | yes, the difference is if the job needs the repo in the work dir on the node it needs required-projects | 09:36 |
*** gcb has quit IRC | 09:37 | |
*** LindaWang has quit IRC | 09:38 | |
*** stakeda has quit IRC | 09:38 | |
AJaeger | tobiash: yeah. for api-site, we need *only* the playbook | 09:38 |
*** liujiong has quit IRC | 09:39 | |
*** LindaWang has joined #openstack-infra | 09:39 | |
*** thorst has quit IRC | 09:39 | |
*** liujiong has joined #openstack-infra | 09:39 | |
*** links has quit IRC | 09:42 | |
*** liujiong has quit IRC | 09:46 | |
*** dsariel has quit IRC | 09:47 | |
*** LindaWang has quit IRC | 09:52 | |
*** maciejjozefczyk has joined #openstack-infra | 09:52 | |
*** slaweq_ has joined #openstack-infra | 09:52 | |
*** dpawlik has joined #openstack-infra | 09:52 | |
*** dizquierdo is now known as dizquierdo_afk | 09:54 | |
*** slaweq has quit IRC | 09:54 | |
*** danpawlik has quit IRC | 09:55 | |
*** maciejjo1 has quit IRC | 09:55 | |
*** dizquierdo_afk is now known as dizquierdo | 09:55 | |
*** armaan has quit IRC | 09:58 | |
*** gmann is now known as gmann_afk | 10:02 | |
openstackgerrit | Jose Luis Franco proposed openstack-infra/tripleo-ci master: WIP: Upgrade UC and OC using tripleo-upgrade role https://review.openstack.org/515643 | 10:04 |
*** kaisers1 has quit IRC | 10:05 | |
*** psachin has quit IRC | 10:05 | |
*** lindycoder has joined #openstack-infra | 10:10 | |
*** LindaWang has joined #openstack-infra | 10:10 | |
frickler | infra-root: sdague: mtreinish: https://review.openstack.org/509313 seems ready now, getting elastic-recheck working with Zuul v3 again | 10:10 |
*** dsariel has joined #openstack-infra | 10:11 | |
sdague | frickler: cool, though maybe just convert the console.html to tags:console? | 10:11 |
sdague | that simplifies things, right? | 10:12 |
*** daidv has quit IRC | 10:12 | |
sdague | frickler: I'm +A as is | 10:12 |
sdague | but that might be a nice cleanup afterwards | 10:12 |
*** hashar is now known as hasharLunch | 10:13 | |
*** kaisers has joined #openstack-infra | 10:14 | |
frickler | sdague: yeah, this is still trying to keep compatibility with v2 for now, just in case | 10:14 |
sdague | frickler: sure | 10:14 |
sdague | though tags:console should also match things that were tags:console.html | 10:14 |
sdague | it was mostly us not being consistent with bugs | 10:15 |
*** markvoelker has joined #openstack-infra | 10:15 | |
*** kiennt26 has quit IRC | 10:17 | |
*** mhickey has quit IRC | 10:17 | |
*** Goneri has joined #openstack-infra | 10:20 | |
openstackgerrit | Merged openstack-infra/elastic-recheck master: Add support for Zuulv3-specific parameters in elastic-recheck https://review.openstack.org/509313 | 10:22 |
*** yamamoto has joined #openstack-infra | 10:24 | |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Cleanup ironic-lib legacy jobs https://review.openstack.org/515623 | 10:26 |
*** andreas_s has quit IRC | 10:26 | |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove osc-dsvm-functional* jobs https://review.openstack.org/515088 | 10:26 |
*** andreas_s has joined #openstack-infra | 10:27 | |
*** ldnunes has joined #openstack-infra | 10:27 | |
*** armaan has joined #openstack-infra | 10:31 | |
*** cuongnv has quit IRC | 10:33 | |
openstackgerrit | Merged openstack-infra/project-config master: Enable documentation building for monascaclient https://review.openstack.org/514954 | 10:34 |
*** boden has joined #openstack-infra | 10:34 | |
*** namnh has quit IRC | 10:36 | |
*** vsaienk0 has quit IRC | 10:37 | |
*** andreas_s has quit IRC | 10:37 | |
*** andreas_s has joined #openstack-infra | 10:39 | |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove tacker legacy job https://review.openstack.org/515346 | 10:39 |
*** dizquierdo is now known as dizquierdo_afk | 10:41 | |
*** armaan has quit IRC | 10:43 | |
*** pblaho has joined #openstack-infra | 10:44 | |
*** pbourke has quit IRC | 10:44 | |
*** yamamoto has quit IRC | 10:45 | |
*** pbourke has joined #openstack-infra | 10:46 | |
*** markvoelker has quit IRC | 10:49 | |
*** andreas_s has quit IRC | 10:50 | |
*** rossella_s has quit IRC | 10:51 | |
openstackgerrit | Merged openstack-infra/project-config master: Upload to pypi if pre / run was successful https://review.openstack.org/514398 | 10:52 |
*** Goneri has quit IRC | 10:53 | |
*** andreas_s has joined #openstack-infra | 10:55 | |
*** Goneri has joined #openstack-infra | 10:55 | |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove legacy oslo.db job https://review.openstack.org/511412 | 10:55 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove migrated legacy puppet openstack check jobs https://review.openstack.org/515263 | 10:55 |
*** vsaienk0 has joined #openstack-infra | 11:00 | |
*** andreas_s has quit IRC | 11:01 | |
*** andreas_s has joined #openstack-infra | 11:02 | |
*** robcresswell has quit IRC | 11:03 | |
*** annp has quit IRC | 11:03 | |
*** yamamoto has joined #openstack-infra | 11:05 | |
*** spzala has joined #openstack-infra | 11:06 | |
*** lucasagomes is now known as lucas-hungry | 11:07 | |
*** dsariel has quit IRC | 11:09 | |
*** andreas_s has quit IRC | 11:12 | |
*** spzala has quit IRC | 11:12 | |
openstackgerrit | Witold Bedyk proposed openstack-infra/project-config master: Add openstack/monasca-specs repo https://review.openstack.org/515685 | 11:13 |
*** andreas_s has joined #openstack-infra | 11:13 | |
*** smatzek has joined #openstack-infra | 11:15 | |
*** andreas_s has quit IRC | 11:15 | |
*** andreas_s has joined #openstack-infra | 11:16 | |
*** mat128 has joined #openstack-infra | 11:17 | |
*** yamamoto has quit IRC | 11:21 | |
ykarel | dhellmann, hi | 11:22 |
ykarel | I was trying to understand why magnum was not included in: https://review.openstack.org/#/c/493239/ add stub deliverable files for queens | 11:22 |
*** dizquierdo_afk is now known as dizquierdo | 11:23 | |
ykarel | do you know why? | 11:23 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Add missing role for deploy-guide.yaml https://review.openstack.org/515687 | 11:26 |
AJaeger | ykarel: there's an #openstack-release channel where you'll find the release management team | 11:27 |
ykarel | AJaeger, Ok | 11:27 |
AJaeger | config-core, we missed one case with role, see 515687 | 11:27 |
*** Goneri has quit IRC | 11:30 | |
*** hamzy has joined #openstack-infra | 11:30 | |
*** sdague has quit IRC | 11:31 | |
*** sdague has joined #openstack-infra | 11:32 | |
*** baoli has joined #openstack-infra | 11:34 | |
dmsimard | AJaeger: +3 | 11:35 |
*** thorst has joined #openstack-infra | 11:35 | |
*** Goneri has joined #openstack-infra | 11:36 | |
*** sdague has quit IRC | 11:37 | |
AJaeger | thanks, dmsimard | 11:37 |
*** baoli has quit IRC | 11:37 | |
*** sdague has joined #openstack-infra | 11:38 | |
*** sdague has quit IRC | 11:38 | |
*** baoli has joined #openstack-infra | 11:38 | |
*** dizquierdo is now known as dizquierdo_afk | 11:38 | |
*** martinkopec has quit IRC | 11:38 | |
openstackgerrit | Merged openstack-infra/project-config master: Add missing role for deploy-guide.yaml https://review.openstack.org/515687 | 11:39 |
AJaeger | pabelanger: Some more work on ansible-lint, see https://review.openstack.org/#/q/status:open+topic:ansible-lint | 11:39 |
*** thorst has quit IRC | 11:40 | |
*** sambetts|afk is now known as sambetts | 11:41 | |
sambetts | AJaeger: hey I'm seeing a failure in one of my legacy jobs: http://logs.openstack.org/43/510243/7/check/legacy-networking-cisco-tox-ocata/162268f/job-output.txt.gz#_2017-10-27_11_25_28_109483 | 11:42 |
*** electrofelix has quit IRC | 11:43 | |
sambetts | AJaeger: it seems to be related to infra code but I'm not sure | 11:43 |
*** electrofelix has joined #openstack-infra | 11:43 | |
*** nicolasbock has joined #openstack-infra | 11:43 | |
dmsimard | sambetts: looking | 11:43 |
sambetts | dmsimard: thanks | 11:45 |
*** thorst has joined #openstack-infra | 11:46 | |
*** sdague has joined #openstack-infra | 11:46 | |
dmsimard | sambetts: has that worked before or did it just break ? | 11:46 |
*** markvoelker has joined #openstack-infra | 11:46 | |
sambetts | dmsimard: it passed on a different patch 3 minutes ago https://review.openstack.org/#/c/512309/ | 11:48 |
*** hasharLunch is now known as hashar | 11:48 | |
dmsimard | sambetts: okay, it's a bit weird -- I'm not familiar with those chunks of code but the error came from here: http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/scripts/run-tox.sh#n76 | 11:49 |
dmsimard | So this script returned 1 .. http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/scripts/jenkins-sudo-grep.sh | 11:50 |
*** yamamoto has joined #openstack-infra | 11:50 | |
*** rosmaita has joined #openstack-infra | 11:50 | |
dmsimard | because it found this in the logs "> Oct 27 11:03:16 ubuntu sudo: zuul : command not allowed ; TTY=unknown ; PWD=/home/zuul/workspace ; USER=root ; COMMAND=/bin/true" | 11:50 |
dmsimard | Something tried to run sudo /bin/true ? :/ | 11:51 |
dmsimard | Maybe someone else would know ^ | 11:51 |
tobiash | dmsimard: the revoke-sudo role runs sudo to test if it's not allowed: http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/revoke-sudo/tasks/main.yaml | 11:56 |
dmsimard | tobiash: that's not this role, though | 11:56 |
dmsimard | tobiash: oh, wait, yes it is | 11:56 |
dmsimard | tobiash: so we must fix the script to avoid catching /bin/true then, good catch ! | 11:57 |
* dmsimard sends patch | 11:57 | |
openstackgerrit | Thomas Herve proposed openstack-infra/tripleo-ci master: Fail on curl error when setting up repo https://review.openstack.org/515695 | 11:57 |
tobiash | :) | 11:57 |
dmsimard | tobiash++ | 11:57 |
*** dizquierdo_afk is now known as dizquierdo | 11:57 | |
*** jpena is now known as jpena|lunch | 11:58 | |
*** dtantsur|afk is now known as dtantsur | 11:58 | |
*** yamamoto has quit IRC | 11:59 | |
*** huanxie has quit IRC | 12:01 | |
evrardjp | I am clueless on why some job don't trigger, may I borrow a pair of eyes? | 12:01 |
evrardjp | dmsimard maybe? | 12:01 |
sambetts | tobiash, dmsimard: thanks guys! is quite frustrating that http://logs.openstack.org/43/510243/7/check/legacy-networking-cisco-tox-ocata/162268f/job-output.txt.gz#_2017-10-27_11_03_15_159733 doesn't log the role name or task name to make it easy to grep for | 12:02 |
dmsimard | sambetts: we'll eventually make things easier to find | 12:04 |
dmsimard | sambetts: once the big fires have been put off, there's a lot of great opportunities for improving people's lives :D | 12:05 |
dmsimard | evrardjp: what's up ? | 12:05 |
evrardjp | apparently not my eyelids! | 12:05 |
evrardjp | :p | 12:05 |
evrardjp | let me show you something | 12:05 |
sambetts | dmsimard: of course :) | 12:05 |
evrardjp | https://github.com/openstack/openstack-ansible/blob/master/zuul.d/jobs.yaml#L40-L45 | 12:05 |
*** dprince has joined #openstack-infra | 12:05 | |
evrardjp | as you can see, pike is defined there | 12:06 |
evrardjp | and the job is defined in the project | 12:06 |
evrardjp | https://github.com/openstack/openstack-ansible/blob/master/zuul.d/project.yaml#L23 | 12:06 |
evrardjp | so question | 12:06 |
evrardjp | why this: https://review.openstack.org/#/c/512046/ | 12:06 |
evrardjp | has not tested this job? | 12:07 |
*** askb has quit IRC | 12:08 | |
*** nsingh has joined #openstack-infra | 12:08 | |
*** abelur_ has quit IRC | 12:08 | |
*** spzala has joined #openstack-infra | 12:08 | |
tobiash | evrardjp: there are currently some branch related fixes in progress jeblair is working on | 12:09 |
*** niraj_singh has quit IRC | 12:09 | |
tobiash | evrardjp: it could be that you are affected by the current branch related bugs | 12:10 |
evrardjp | that must be a nightmare | 12:10 |
AJaeger | evrardjp: read also https://docs.openstack.org/infra/manual/zuulv3.html#stable-branches | 12:10 |
AJaeger | evrardjp: I don#t see a zuul.d directoy in stable/pike branch | 12:11 |
evrardjp | AJaeger: yeah, because this time I defined branches | 12:11 |
evrardjp | in master | 12:11 |
evrardjp | but ok I will backport those changes one per one | 12:11 |
AJaeger | evrardjp: that will not help as explained in the link I gave. | 12:11 |
evrardjp | yeah | 12:11 |
evrardjp | I've seen | 12:11 |
*** jaypipes has joined #openstack-infra | 12:12 | |
evrardjp | AJaeger: on your link there is something saying | 12:12 |
evrardjp | "unless branches are explicitly specified for the jobs in the project stanza," | 12:12 |
AJaeger | evrardjp: but as tobiash pointed out, there's bugs in branch handling as well | 12:12 |
evrardjp | AJaeger: I am merely following the docs there :p | 12:12 |
evrardjp | I am fine backporting those repo things, I just thought I was covered, due to the branches thing. | 12:13 |
AJaeger | evrardjp: indeed - consider me confused. Let's wait for the jeblair magic to happen ;) | 12:13 |
*** spzala has quit IRC | 12:13 | |
evrardjp | :D | 12:13 |
*** yamamoto has joined #openstack-infra | 12:14 | |
evrardjp | in the meantime I will propose the patches. It's maybe safer in the long run. | 12:14 |
AJaeger | dmsimard: care to review https://review.openstack.org/515501, please? | 12:14 |
*** lucas-hungry is now known as lucasagomes | 12:14 | |
*** trown|outtypewww is now known as trown | 12:18 | |
*** markvoelker has quit IRC | 12:20 | |
*** Guest72852 has quit IRC | 12:24 | |
*** camunoz has joined #openstack-infra | 12:24 | |
*** askb has joined #openstack-infra | 12:25 | |
*** rhallisey has quit IRC | 12:25 | |
*** abelur_ has joined #openstack-infra | 12:26 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Make prepare-docs-for-afs useable for different paths https://review.openstack.org/515697 | 12:26 |
dmsimard | evrardjp: sorry got distracted, did you get your answer ? | 12:27 |
dmsimard | AJaeger: ouch, is someone from fuel weighing in on that patch ? | 12:27 |
*** markvoelker has joined #openstack-infra | 12:27 | |
dmsimard | AJaeger: oh wait, nevermind | 12:27 |
*** pgadiya has quit IRC | 12:28 | |
AJaeger | dmsimard: no - this is result of branch retiring and careful review of our automatic conversion | 12:28 |
*** StevenK has quit IRC | 12:29 | |
*** rm_work has quit IRC | 12:29 | |
evrardjp | dmsimard: yes and now | 12:29 |
evrardjp | no* | 12:29 |
*** rm_work has joined #openstack-infra | 12:30 | |
*** rm_work has quit IRC | 12:30 | |
*** rm_work has joined #openstack-infra | 12:30 | |
*** jcoufal has joined #openstack-infra | 12:30 | |
*** sflanigan has joined #openstack-infra | 12:31 | |
*** sflanigan has joined #openstack-infra | 12:31 | |
*** rlandy has joined #openstack-infra | 12:32 | |
*** baoli has quit IRC | 12:32 | |
dmsimard | AJaeger: added a comment before +3 | 12:32 |
AJaeger | dmsimard: commented - that line was broken conversoin, carefull analysis of zuul/layout shows that we never run that job at all | 12:34 |
dmsimard | AJaeger: ack | 12:35 |
AJaeger | those fuel regexes were a mess ;( | 12:35 |
dmsimard | AJaeger: btw does the zuul layout compile in yaml, json or xml in your brain ? :p | 12:35 |
AJaeger | dmsimard: it's yaml ;) - and there once was a job that outputs the jobs with conditions | 12:36 |
*** baoli has joined #openstack-infra | 12:37 | |
*** armaan has joined #openstack-infra | 12:37 | |
openstackgerrit | David Moreau Simard proposed openstack-infra/project-config master: Exclude COMMAND=/bin/true from sudoers verification https://review.openstack.org/515700 | 12:38 |
openstackgerrit | Merged openstack-infra/project-config master: Fuel branch cleanup https://review.openstack.org/515501 | 12:38 |
dmsimard | sambetts, tobiash, AJaeger ^ I *think* that should be fine, tested it locally | 12:38 |
AJaeger | dmsimard: thanks - I wonder why this shows up now and not earlier... | 12:39 |
dmsimard | AJaeger: sambetts seemed to suggest that it flapped, it is related to revoke sudo role.. maybe there are tox jobs running revoke-sudo and some aren't. | 12:39 |
AJaeger | change looks fine but I let others look at it first... | 12:40 |
*** StevenK has joined #openstack-infra | 12:41 | |
dmsimard | AJaeger: only thing IMO is that I might wrongly assume egrep (grep -P) is available everywhere | 12:41 |
dmsimard | ¯\_(ツ)_/¯ | 12:41 |
dmsimard | pretty sure it is, but who knows | 12:41 |
*** dsariel has joined #openstack-infra | 12:45 | |
*** vsaienk0 has quit IRC | 12:46 | |
*** jpena|lunch is now known as jpena | 12:50 | |
*** spzala has joined #openstack-infra | 12:50 | |
*** dsariel has quit IRC | 12:51 | |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Move test_model.test_job_inheritance_configloader https://review.openstack.org/509495 | 12:51 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Remove test_job_inheritance https://review.openstack.org/509509 | 12:51 |
*** Hal has joined #openstack-infra | 12:52 | |
*** dhajare|mtg has quit IRC | 12:52 | |
*** wolverineav has joined #openstack-infra | 12:52 | |
*** Hal is now known as Guest43518 | 12:52 | |
*** dsariel has joined #openstack-infra | 12:55 | |
*** amoralej is now known as amoralej|lunch | 12:55 | |
*** mdbooth has quit IRC | 12:58 | |
*** ari[m] has quit IRC | 12:58 | |
evrardjp | egrep is not available everywhere dmsimard, but there is a change I might be trolling you for large distros. | 12:59 |
evrardjp | chance* | 12:59 |
evrardjp | but then you need gnu grep and all. | 13:00 |
evrardjp | jeblair: could you enlighten us with the branch issue you're working on right now? | 13:00 |
dmsimard | evrardjp: I mean I'm PRETTY confident it's there for fedora, centos, rhel, debian, ubuntu | 13:00 |
evrardjp | :D | 13:00 |
dmsimard | but I don't know for like... gentoo or opensuse | 13:00 |
*** marst has joined #openstack-infra | 13:01 | |
kklimonda | hmm, how are various mirrors updated for nodepool-provisioned machines? `ready-script: create_mirror.sh` is no longer there in nodepool v3 | 13:01 |
*** pblaho has quit IRC | 13:01 | |
*** pblaho has joined #openstack-infra | 13:02 | |
*** jaypipes is now known as leakypipes | 13:02 | |
tobiash | evrardjp: he wrote a lengthly mail about that on the *-infra mailing list: http://lists.openstack.org/pipermail/openstack-infra/2017-October/005634.html | 13:02 |
*** Mutter has joined #openstack-infra | 13:02 | |
tobiash | kklimonda: in zuulv3 the base job does this | 13:02 |
evrardjp | thanks will read. | 13:02 |
AJaeger | dmsimard: it's there fore openSUSE | 13:02 |
openstackgerrit | Merged openstack-infra/project-config master: zuul-jobs: Run ansible-lint https://review.openstack.org/515596 | 13:02 |
*** armaan has quit IRC | 13:02 | |
kklimonda | tobiash: oh right, I just had that idea - configure-mirrors, right? | 13:02 |
tobiash | yepp | 13:03 |
*** spzala has quit IRC | 13:04 | |
*** bauzas is now known as bauwser | 13:04 | |
*** Mutter has quit IRC | 13:04 | |
*** catintheroof has joined #openstack-infra | 13:04 | |
dmsimard | AJaeger: comment on https://review.openstack.org/#/c/515603/ | 13:05 |
*** shardy has quit IRC | 13:07 | |
AJaeger | dmsimard: thanks, fixing... | 13:07 |
AJaeger | good spot! | 13:07 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Ansible-lint project-config https://review.openstack.org/515603 | 13:07 |
*** udesale has quit IRC | 13:08 | |
*** baoli has quit IRC | 13:09 | |
dmsimard | AJaeger: +2 :) | 13:09 |
*** kgiusti has joined #openstack-infra | 13:13 | |
*** andreas_s has quit IRC | 13:15 | |
*** gibi is now known as pow_2_30 | 13:17 | |
*** mriedem has joined #openstack-infra | 13:18 | |
*** vsaienk0 has joined #openstack-infra | 13:18 | |
*** fdegir has quit IRC | 13:19 | |
*** pahuang has quit IRC | 13:19 | |
*** andreas_s has joined #openstack-infra | 13:19 | |
*** fdegir has joined #openstack-infra | 13:19 | |
evrardjp | tobiash: thanks for the link. It's a intense read. | 13:20 |
*** sflanigan has quit IRC | 13:20 | |
*** mdbooth has joined #openstack-infra | 13:21 | |
*** ari[m] has joined #openstack-infra | 13:21 | |
fungi | evrardjp: however, the changes to solve it are up for review as of ~12 hours ago | 13:22 |
*** dsariel has quit IRC | 13:23 | |
evrardjp | I am not sure I can technically have a look at those, because I am not inside the system enough to take a good decision or not. | 13:23 |
evrardjp | but I will have a look at them anyway :D | 13:23 |
*** pahuang has joined #openstack-infra | 13:24 | |
*** andreas_s has quit IRC | 13:25 | |
*** andreas_s has joined #openstack-infra | 13:25 | |
*** hemna_ has joined #openstack-infra | 13:25 | |
vsaienk0 | infra-root: ironic-python-agent post upload job was corrupted after switching to zuulv3, and on tarballs.openstack.org now we have corrupted directory structure, we need to remove the following folders and its content completely http://tarballs.openstack.org/ironic-python-agent/coreos/UPLOAD_TAR/ http://tarballs.openstack.org/ironic-python-agent/coreos/files/UPLOAD_RAW/ http://tarballs.openstack.org/ironic-python-agent/tinyipa/UPLOAD_TAR/ and | 13:26 |
vsaienk0 | dtantsur: ^ fyi | 13:26 |
AJaeger | config-core, please review https://review.openstack.org/515697 - fixes are needed for deploy-guide publishing | 13:27 |
dtantsur | vsaienk0: ugh :( thanks for tracking this | 13:27 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Add ansible-linter job for project-config https://review.openstack.org/515607 | 13:28 |
*** claudiub has joined #openstack-infra | 13:28 | |
*** dchen has joined #openstack-infra | 13:28 | |
*** dingyichen has quit IRC | 13:28 | |
fungi | vsaienk0: i can take care of that in just a moment | 13:29 |
*** dansmith is now known as superdan | 13:30 | |
*** claudiub|2 has quit IRC | 13:31 | |
*** dizquierdo has quit IRC | 13:31 | |
*** dizquierdo has joined #openstack-infra | 13:32 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Make linter job voting on zuul-jobs https://review.openstack.org/515708 | 13:32 |
fungi | vsaienk0: http://tarballs.openstack.org/ironic-python-agent/tinyipa/files/UPLOAD_RAW/ too? | 13:33 |
vsaienk0 | fungi: yes all UPLOAD_RAW/ and UPLOAD_TAR/ folders | 13:33 |
*** mhickey has joined #openstack-infra | 13:33 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Run linters as voting https://review.openstack.org/515710 | 13:34 |
*** catintheroof has quit IRC | 13:34 | |
fungi | vsaienk0: cool, done | 13:34 |
vsaienk0 | fungi: thanks for help! | 13:34 |
*** Goneri has quit IRC | 13:35 | |
fungi | vsaienk0: you bet. i've stashed them in my homedir temporarily for safekeeping in case you discover i removed something i shouldn't have (i go back through periodically and clean up my homedir) | 13:36 |
AJaeger | dmsimard, fungi, please +2a https://review.openstack.org/#/c/515593/ - we're ready for the linters on zuul-jobs | 13:36 |
*** janki has quit IRC | 13:37 | |
*** esberglu has joined #openstack-infra | 13:40 | |
*** Goneri has joined #openstack-infra | 13:41 | |
*** catintheroof has joined #openstack-infra | 13:42 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Make linter job voting on zuul-jobs https://review.openstack.org/515708 | 13:43 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Run linters as voting https://review.openstack.org/515710 | 13:43 |
AJaeger | config-core, added proper dependencies for the ansible-lint jobs, for full list of changes see https://review.openstack.org/#/q/status:open+topic:ansible-lint | 13:44 |
AJaeger | bbl | 13:44 |
fungi | looks like frickler beat me to approving | 13:47 |
fungi | i need to pop out to run an errand but hope to be back in an hour or so | 13:47 |
*** jascott1 has joined #openstack-infra | 13:47 | |
*** spectr has quit IRC | 13:49 | |
*** jascott1 has quit IRC | 13:52 | |
*** dave-mccowan has joined #openstack-infra | 13:53 | |
*** sree has quit IRC | 13:54 | |
*** dhajare|mtg has joined #openstack-infra | 13:55 | |
*** niraj_singh has joined #openstack-infra | 13:55 | |
*** sree has joined #openstack-infra | 13:55 | |
*** nsingh has quit IRC | 13:55 | |
*** owalsh has quit IRC | 13:56 | |
*** amoralej|lunch is now known as amoralej | 13:56 | |
*** arxcruz is now known as arxcruz|pto | 13:56 | |
*** owalsh has joined #openstack-infra | 13:59 | |
openstackgerrit | Merged openstack-infra/zuul-jobs master: Run ansible-lint on repo https://review.openstack.org/515593 | 13:59 |
*** sree has quit IRC | 14:00 | |
*** gouthamr has joined #openstack-infra | 14:01 | |
AJaeger | three more to go: https://review.openstack.org/#/q/status:open+topic:ansible-lint ;) | 14:01 |
*** baoli has joined #openstack-infra | 14:01 | |
*** Dinesh_Bhor has quit IRC | 14:04 | |
*** d0ugal_ has joined #openstack-infra | 14:04 | |
*** d0ugal has quit IRC | 14:05 | |
*** eumel8 has quit IRC | 14:06 | |
*** andreas_s has quit IRC | 14:06 | |
*** tosky has quit IRC | 14:06 | |
*** tosky has joined #openstack-infra | 14:08 | |
*** spectr has joined #openstack-infra | 14:09 | |
*** andreas_s has joined #openstack-infra | 14:10 | |
*** claudiub has quit IRC | 14:13 | |
*** andreas_s has quit IRC | 14:14 | |
*** andreas_s has joined #openstack-infra | 14:14 | |
*** andreas_s has quit IRC | 14:14 | |
openstackgerrit | Samuel Matzek proposed openstack-infra/project-config master: Remove tempest job https://review.openstack.org/514769 | 14:14 |
*** hongbin has joined #openstack-infra | 14:15 | |
*** andreas_s has joined #openstack-infra | 14:15 | |
*** armax has joined #openstack-infra | 14:17 | |
*** dsariel has joined #openstack-infra | 14:17 | |
*** andreas_s has quit IRC | 14:19 | |
*** dave-mccowan has quit IRC | 14:20 | |
pabelanger | AJaeger: dmsimard: frickler: 515607 isn't going to work, as we cannot parent to jobs in ozj from project-config. We can just use openstack-zuul-jobs-linters job directly however | 14:23 |
*** felipemonteiro_ has joined #openstack-infra | 14:26 | |
*** baoli has quit IRC | 14:29 | |
jeblair | evrardjp: you can use branch matchers in-repo, but my recommendation would instead be to copy the jobs + playbooks to the various branches. and hopefully that should work correctly some time today. | 14:31 |
*** vsaienk0 has quit IRC | 14:31 | |
*** dsariel has quit IRC | 14:31 | |
evrardjp | jeblair: yes, I've read the whole conversation and thought that would be the recommendation. I've started to port all these definitions back. I think a bug will emerge though.... | 14:31 |
*** felipemonteiro__ has joined #openstack-infra | 14:32 | |
evrardjp | give you more feedback soon, when your series of patches has landed, this way I don't disturb you! | 14:33 |
*** dsariel has joined #openstack-infra | 14:33 | |
jeblair | evrardjp: i'd love to hear about the bug you suspect! hopefully it's already addressed and i can just make sure it's clear in the docs, but if it will be a problem, before we roll it out would be a good time to know. :) | 14:34 |
*** rbrndt has joined #openstack-infra | 14:34 | |
evrardjp | ok I'll shoot then:p | 14:34 |
*** baoli has joined #openstack-infra | 14:35 | |
*** dbecker has quit IRC | 14:35 | |
*** dbecker has joined #openstack-infra | 14:35 | |
*** felipemonteiro_ has quit IRC | 14:36 | |
openstackgerrit | Sean McGinnis proposed openstack-infra/project-config master: Switch vitrage service to service release job https://review.openstack.org/515726 | 14:36 |
evrardjp | on this patchset, where I (now) define in-tree the jobs for this stable branch (newton), I see none of the new in-tree project jobs defined. | 14:36 |
evrardjp | https://review.openstack.org/#/c/515713/1/zuul.d/project.yaml | 14:37 |
evrardjp | anything that was defined in master is fine. | 14:37 |
evrardjp | which makes me think that the inheritence here: https://review.openstack.org/#/c/515713/1/zuul.d/jobs.yaml would be broken if not completely redefined. | 14:37 |
evrardjp | would that be a correct assumption? | 14:38 |
*** vsaienk0 has joined #openstack-infra | 14:38 | |
AJaeger | pabelanger: so, what do you mean - move the job to project-config? | 14:38 |
jeblair | evrardjp: where is openstack-ansible-deploy-aio-ubuntu-xenial defined? | 14:39 |
evrardjp | master | 14:39 |
*** spectr has quit IRC | 14:39 | |
*** dbecker has quit IRC | 14:40 | |
evrardjp | https://github.com/openstack/openstack-ansible/blob/master/zuul.d/jobs.yaml#L40 | 14:40 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Revert "Add ansible-linter job for project-config" https://review.openstack.org/515727 | 14:40 |
*** dbecker has joined #openstack-infra | 14:40 | |
evrardjp | Oh I may need to remove the branch matching on master. That may be the cause. | 14:40 |
evrardjp | jeblair: my bad! | 14:40 |
*** baoli has quit IRC | 14:42 | |
pabelanger | AJaeger: lets just use openstack-zuul-jobs-linters job for now, and I believe jeblair has patches up fix the late binding issue | 14:42 |
jeblair | evrardjp: yep that's it | 14:42 |
evrardjp | sorry :( | 14:43 |
evrardjp | overlook apparently. | 14:43 |
pabelanger | AJaeger: or rename openstack-zuul-jobs-linters to something else, that you are comfortable with | 14:43 |
openstackgerrit | Julia Kreger proposed openstack-infra/project-config master: Remove legacy bifrost jobs https://review.openstack.org/515728 | 14:45 |
jeblair | evrardjp: np. aside from that, the way you've constructed things should work. i'd recommend the other approach because it will be more natural as time passes and we make new branches off of master (all of which will contain all of this content). however, if you want to maintain most or all the logic in master, that should be possible, just with a bit more upkeep. and maybe you want to get the jobs working and stable in master with ... | 14:45 |
jeblair | ... branch matchers and then backport them when ready. that's fine too. :) | 14:45 |
openstackgerrit | Julia Kreger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy bifrost jobs https://review.openstack.org/515729 | 14:45 |
*** e0ne has quit IRC | 14:45 | |
*** andreas_s has joined #openstack-infra | 14:45 | |
AJaeger | pabelanger: I copy it over to project-config and rename on that way... | 14:45 |
*** baoli has joined #openstack-infra | 14:45 | |
*** HeOS has quit IRC | 14:45 | |
evrardjp | jeblair: I think it's more self contained that way, like you suggested. Thanks for the advice! | 14:46 |
jeblair | evrardjp: np | 14:46 |
*** baoli has quit IRC | 14:48 | |
AJaeger | pabelanger: https://review.openstack.org/515603 passed the tests - I'm not convinced why it would not work | 14:48 |
*** armaan has joined #openstack-infra | 14:48 | |
*** HeOS has joined #openstack-infra | 14:48 | |
AJaeger | pabelanger: we have other jobs in project-config running for project-config in similar way. | 14:49 |
AJaeger | Like project-config-gerrit in openstack-zuul-jobs | 14:49 |
pabelanger | AJaeger: do they parent to something outside project-config? | 14:49 |
AJaeger | http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/zuul.d/jobs.yaml#n667 | 14:50 |
AJaeger | parents to tox which is in zuul-jobs | 14:50 |
pabelanger | if you use openstack-zuul-jobs-linters directly, it should work. But IIRC, because project-config-zuul-jobs-linter parents to openstack-zuul-jobs-linters, that isn't going to work because ozj is loaded after project-config | 14:50 |
pabelanger | AJaeger: no, that is fine. ozj, is after zuul-jobs in main | 14:50 |
pabelanger | main.yaml | 14:51 |
*** andreas_s has quit IRC | 14:51 | |
*** LindaWang1 has joined #openstack-infra | 14:51 | |
openstackgerrit | James E. Blair proposed openstack-infra/infra-manual master: Zuul v3: clarify stable branches https://review.openstack.org/515732 | 14:52 |
AJaeger | pabelanger: I'm not getting it yet. we *use* project-config-gerrit in project-config the same way we use project-config-zuul-jobs-linters in it IMHO. | 14:52 |
openstackgerrit | Sean McGinnis proposed openstack-infra/project-config master: Remove publish to pypi for aodh and panko https://review.openstack.org/515733 | 14:52 |
AJaeger | Why would openstack-zuul-jobs-linters work but project-config-zuul-jobs-linters not? | 14:52 |
AJaeger | And why is zuul v3 check ok? | 14:52 |
*** andreas_s has joined #openstack-infra | 14:52 | |
*** LindaWang has quit IRC | 14:53 | |
*** LindaWang1 is now known as LindaWang | 14:53 | |
*** Guest43518 has quit IRC | 14:53 | |
*** e0ne has joined #openstack-infra | 14:53 | |
*** e0ne has quit IRC | 14:53 | |
pabelanger | AJaeger: okay, nevermind | 14:53 |
*** e0ne has joined #openstack-infra | 14:54 | |
AJaeger | pabelanger: you might be right, I might just not getting it ;) | 14:54 |
pabelanger | AJaeger: For some reason I tought project-config-zuul-jobs-linters was in project-config .zuul.yaml | 14:54 |
openstackgerrit | Rabi Mishra proposed openstack-infra/devstack-gate master: Set DEVSTACK_GATE_CEILOMETER_BACKEND to gnocchi https://review.openstack.org/515734 | 14:54 |
pabelanger | AJaeger: so, that will work correctly | 14:54 |
pabelanger | AJaeger: if it was in project-config .zuul.yaml, it would not work | 14:54 |
*** e0ne has quit IRC | 14:54 | |
pabelanger | AJaeger: +3 | 14:54 |
AJaeger | correct, that was my first try- cpoying it over. it didn't know parent tox ;( | 14:54 |
AJaeger | thanks, pabelanger | 14:54 |
*** bobh has joined #openstack-infra | 14:55 | |
AJaeger | pabelanger: thanks! I still learned something ;) | 14:55 |
*** e0ne has joined #openstack-infra | 14:55 | |
AJaeger | pabelanger: what do you think of my two other changes - see https://review.openstack.org/#/q/status:open+topic:ansible-lint | 14:55 |
*** e0ne has quit IRC | 14:56 | |
*** e0ne has joined #openstack-infra | 14:56 | |
*** e0ne has quit IRC | 14:57 | |
*** andreas_s has quit IRC | 14:57 | |
*** ethfci has quit IRC | 14:57 | |
pabelanger | looking | 14:57 |
pabelanger | and drinking coffee :) | 14:57 |
AJaeger | pabelanger: thanks - and enjoy! | 14:58 |
*** pblaho has quit IRC | 14:58 | |
*** ldnunes has quit IRC | 14:58 | |
*** bobh has quit IRC | 14:58 | |
AJaeger | and could I persuade some config-core to review https://review.openstack.org/#/c/515697/ and https://review.openstack.org/514053 as well, please? those fix document publishing | 14:59 |
*** ihrachys has joined #openstack-infra | 15:00 | |
pabelanger | AJaeger: -1 on 515708, want to see results from job run first. Once 515603 lands we can recheck | 15:01 |
*** ijw has joined #openstack-infra | 15:01 | |
*** ijw_ has joined #openstack-infra | 15:02 | |
AJaeger | pabelanger: thta's for zuul-jobs , see https://review.openstack.org/#/c/515593/ for positive result | 15:03 |
AJaeger | or would project-config usage find new problems in zuul-jobs? | 15:04 |
openstackgerrit | Vasyl Saienko proposed openstack-infra/project-config master: Remove legacy ironic-inspector jobs https://review.openstack.org/515737 | 15:05 |
pabelanger | AJaeger: project-config should find new issues in zuul-jobs, but we can turn the job voting | 15:05 |
pabelanger | thanks | 15:05 |
AJaeger | pabelanger: ok.. So, let's see what 515603 brings ;) | 15:06 |
*** marst has quit IRC | 15:06 | |
*** ijw has quit IRC | 15:06 | |
openstackgerrit | Merged openstack-infra/project-config master: Ansible-lint project-config https://review.openstack.org/515603 | 15:06 |
*** marst has joined #openstack-infra | 15:06 | |
*** baoli has joined #openstack-infra | 15:07 | |
*** dsariel has quit IRC | 15:07 | |
openstackgerrit | Michael Johnson proposed openstack-infra/project-config master: Remove migrated octavia zuul v3 jobs https://review.openstack.org/515486 | 15:07 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Run linters as voting https://review.openstack.org/515710 | 15:08 |
openstackgerrit | Vasyl Saienko proposed openstack-infra/openstack-zuul-jobs master: Cleanup ironic-inspector legacy jobs https://review.openstack.org/515738 | 15:08 |
openstackgerrit | Jose Luis Franco proposed openstack-infra/tripleo-ci master: WIP: Upgrade UC and OC using tripleo-upgrade role https://review.openstack.org/515643 | 15:09 |
AJaeger | fungi, pabelanger, clarkb , could you review https://review.openstack.org/515700 , please? That should fix sambetts' problem but I'm surprised to see this now | 15:09 |
openstackgerrit | Rabi Mishra proposed openstack-infra/devstack-gate master: Don't set CEILOMETER_BACKEND in localrc https://review.openstack.org/515734 | 15:09 |
*** ldnunes has joined #openstack-infra | 15:10 | |
openstackgerrit | Merged openstack-infra/project-config master: Make linter job voting on zuul-jobs https://review.openstack.org/515708 | 15:10 |
*** dave-mccowan has joined #openstack-infra | 15:11 | |
*** vhosakot has joined #openstack-infra | 15:11 | |
*** tosky has quit IRC | 15:12 | |
AJaeger | thanks everybody for reviewing my newton eol and ansible-lint changes! | 15:12 |
*** yamahata has joined #openstack-infra | 15:14 | |
frickler | I'm seeing a post failure for openstack-tox-cover running on designate-dashboard because the "tox -ecover" there is different and doesn't seem to produce a results dir, see http://logs.openstack.org/89/515689/1/check/openstack-tox-cover/2e35d62/job-output.txt.gz#_2017-10-27_11_56_15_616856 | 15:14 |
frickler | should I change openstack-tox-cover to not fail in that case or rather create a variant of that job? | 15:15 |
*** hemna_ has quit IRC | 15:15 | |
AJaeger | ansible-lint experts, we have failures for project-config at http://logs.openstack.org/86/515486/3/infra-check/project-config-zuul-jobs-linters/5f1d85a/job-output.txt.gz#_2017-10-27_15_13_36_715771 | 15:15 |
*** sree has joined #openstack-infra | 15:16 | |
fungi | frickler: what's the point of running a coverage job that doesn't generate results? seems like something that should be fixed in their repo probably | 15:16 |
*** iyamahat has joined #openstack-infra | 15:16 | |
frickler | fungi: I think that is because it is more javascript than python, but I'm not deep enough into that to decide. the other option would be to just remove the coverage job, like e.g. horizon seems to do | 15:17 |
pabelanger | AJaeger: you'll need to skip that, zuul_return isn't shipped by ansible. Another option is to install zuul, and add our libraries to ansible path | 15:18 |
*** links has joined #openstack-infra | 15:19 | |
fungi | frickler: right, i'm leaning toward fix things so that designate-dashboard's coverage job produces a report or stop running it. may be best to ask mugsie for input too | 15:19 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Make prepare-docs-for-afs useable for different paths https://review.openstack.org/515697 | 15:19 |
*** dave-mccowan has quit IRC | 15:21 | |
frickler | fungi: o.k., will do, thx | 15:21 |
*** dhinesh has joined #openstack-infra | 15:22 | |
*** armax_ has joined #openstack-infra | 15:22 | |
*** salv-orlando has quit IRC | 15:22 | |
*** armax has quit IRC | 15:22 | |
*** armax_ is now known as armax | 15:22 | |
*** salv-orlando has joined #openstack-infra | 15:23 | |
*** yamamoto has quit IRC | 15:24 | |
dhinesh | after installing nodepool 4.1, unable to do operations on it, i get this error https://pastebin.com/cDEQFrGU, is zookeeper required for it to work? | 15:25 |
*** cshastri has quit IRC | 15:25 | |
*** slaweq_ has quit IRC | 15:25 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: ansible-lint: install zuul https://review.openstack.org/515742 | 15:26 |
AJaeger | pabelanger: like this? ^ | 15:26 |
*** salv-orlando has quit IRC | 15:27 | |
*** ramishra has quit IRC | 15:27 | |
*** andreas_s has joined #openstack-infra | 15:27 | |
pabelanger | AJaeger: we should use required-projects, then pip install into save virtualenv. You'll then need to setup ANSIBLE_LIBRARY_PATH (I think) to point to the library folder for zuul | 15:29 |
AJaeger | pabelanger: I see... | 15:31 |
*** andreas_s has quit IRC | 15:31 | |
pabelanger | AJaeger: might not need to pip install, we could get away at just pointing to zuul library directory | 15:31 |
*** LindaWang has quit IRC | 15:31 | |
pabelanger | AJaeger: like we did for ANSIBLE_ROLES_PATH | 15:31 |
fungi | dirk: yes, <=0.3.x does not require zookeeper, >=0.4.x does: http://lists.openstack.org/pipermail/openstack-infra/2016-December/004972.html | 15:32 |
AJaeger | pabelanger: copying over what zuul-jobs does | 15:32 |
fungi | er, sorry dirk. dinesh apparently likes to ask questions in channel and then disappear | 15:32 |
AJaeger | Do we need ANSIBLE_CONFIG and set it to tests/ansible.cfg as well? | 15:32 |
fungi | dhinesh: yes, <=0.3.x does not require zookeeper, >=0.4.x does: http://lists.openstack.org/pipermail/openstack-infra/2016-December/004972.html | 15:33 |
fungi | i guess dhinesh did not leave the channel, i simply cannot type | 15:33 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: ansible-lint: install zuul https://review.openstack.org/515742 | 15:34 |
*** ykarel has quit IRC | 15:34 | |
fungi | AJaeger: any clue if there's supposed to be content at https://docs.openstack.org/pike/deploy/ ? | 15:35 |
dhinesh | fungi: yes is am well and alive :) , i will checkout the link | 15:35 |
*** kjackal_ has quit IRC | 15:35 | |
AJaeger | fungi: there was quite some discussion the last hours in the docu channel. | 15:36 |
fungi | AJaeger: thanks, i'll point the person asking at that discussion | 15:36 |
AJaeger | we found some bugs in our scripts that let to endless redirects, those are fixed and now we need to add them back in. So, known issue, needs some more work | 15:37 |
*** andreas_s has joined #openstack-infra | 15:37 | |
*** Goneri has quit IRC | 15:39 | |
jeblair | AJaeger, fungi: can you look at https://review.openstack.org/515732 when you have a moment? | 15:42 |
*** andreas_s has quit IRC | 15:46 | |
fungi | jeblair: is it at all worth pointing out the reason, being that once you create a new branch from master the branched configuration will apply to that branch, but the master configuration will keep getting applied to your earlier branches instead if you haven't backported? or is that just going to make things more confusing? | 15:47 |
*** andreas_s has joined #openstack-infra | 15:47 | |
fungi | i'm good with the proposed content as-is, fwiw | 15:47 |
jeblair | fungi: i'll propose a followup | 15:48 |
*** jaosorior has quit IRC | 15:48 | |
AJaeger | jeblair: that is fine! One suggestion inline on what I missed - once zuul has defined that a job is run, later variants cannot change that with files/irrelevant files | 15:48 |
jeblair | fungi: as i write it, i realize that i was about to repeat the sentence in the middle of teh paragraph. this could be tricky. :) | 15:49 |
fungi | jeblair: yeah, i guess that's pretty clear about it already | 15:50 |
jeblair | fungi: oh, the master configuration *won't* apply to branches if you haven't backported | 15:50 |
*** wolverineav has quit IRC | 15:50 | |
jeblair | fungi: (it does now, but won't once the new changes land; unless you set the pragma directive) | 15:50 |
fungi | jeblair: it won't? i thought branches inherited configuration from master when not otherwise supplied. i clearly need to brush back up on ths | 15:50 |
fungi | aha, i haven't looked at the pragma change yet, sorry | 15:51 |
*** wolverineav has joined #openstack-infra | 15:51 | |
jeblair | fungi: yeah that's basically untenable because of the thing AJaeger brings up -- you wouldn't be able to *undo* anything added to the master branch on stable. | 15:52 |
*** rhallisey has joined #openstack-infra | 15:52 | |
jeblair | fungi: (eg -- adding a new default service to devstack on master would affect stable if we left things this way) | 15:52 |
*** derekh has quit IRC | 15:52 | |
fungi | right. okay | 15:52 |
*** andreas_s has quit IRC | 15:52 | |
fungi | makes sense | 15:52 |
openstackgerrit | Doug Hellmann proposed openstack-infra/project-config master: fix release jobs for designate-dashboard and zaqar-ui https://review.openstack.org/515752 | 15:52 |
*** andreas_s has joined #openstack-infra | 15:53 | |
AJaeger | jeblair: so, I'm fine with change as is - just wonder whether my comment is worth adding. Could be a followup - then I'll +2A... | 15:53 |
jeblair | AJaeger: yeah, i'm looking at that now | 15:54 |
jlvillal | Review request: https://review.openstack.org/#/c/513444/ Add result_TIMED_OUT & result_RETRY_LIMIT as failure colors to Gerrit patch pages | 15:55 |
*** Goneri has joined #openstack-infra | 15:55 | |
*** sree has quit IRC | 15:55 | |
*** sree has joined #openstack-infra | 15:55 | |
jeblair | jlvillal: question on that | 15:55 |
jeblair | (inline) | 15:55 |
jlvillal | jeblair, | 15:55 |
AJaeger | bbl | 15:56 |
jlvillal | jeblair, Ah, good question. I'll add them! | 15:56 |
*** iyamahat has quit IRC | 15:56 | |
openstackgerrit | John L. Villalovos proposed openstack-infra/system-config master: Add result_TIMED_OUT & result_RETRY_LIMIT as failure colors https://review.openstack.org/513444 | 15:57 |
*** vsaienk0 has quit IRC | 15:57 | |
jlvillal | jeblair, As you can tell, I'm not a CSS expert :) | 15:57 |
*** iyamahat has joined #openstack-infra | 15:59 | |
openstackgerrit | James E. Blair proposed openstack-infra/infra-manual master: Zuul v3: clarify stable branches https://review.openstack.org/515732 | 15:59 |
jeblair | fungi, AJaeger: i tried to address both comments and also found something else i needed to change, so pushed up new rev ^ | 15:59 |
*** sree has quit IRC | 16:00 | |
dhinesh | fungi: the document suggests to have a separate one node setup for zookeeper and then point it to nodepool, am trying to make all my CI components into a single server, any suggestions around it? | 16:00 |
smcginnis | We got a new odd failure: http://lists.openstack.org/pipermail/release-job-failures/2017-October/000646.html | 16:01 |
smcginnis | Is this a zuul issue? | 16:01 |
*** yamamoto has joined #openstack-infra | 16:01 | |
*** andreas_s has quit IRC | 16:02 | |
*** Apoorva has joined #openstack-infra | 16:03 | |
fungi | dhinesh: i don't personally know of any reason why you couldn't run zookeeper on the same server unless resource constraints and scalability become a concern | 16:03 |
*** dave-mccowan has joined #openstack-infra | 16:04 | |
*** andreas_s has joined #openstack-infra | 16:04 | |
dhinesh | fungi: ok thanks | 16:05 |
jeblair | smcginnis: that's zuul saying that the publish-openstack-releasenotes job can't be modified to add openstack/horizon to it in a project stanza | 16:06 |
smcginnis | jeblair: Is there something we need to fix in that project's configuration? | 16:08 |
jeblair | smcginnis: well, the job has "final: True" set, which means that it can't be modified. | 16:10 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: ansible-lint: install zuul https://review.openstack.org/515742 | 16:10 |
jeblair | smcginnis: probably because if it were, someone could inherit from it and cause something other than the releasenotes to be published | 16:10 |
*** jascott1 has joined #openstack-infra | 16:10 | |
jeblair | smcginnis: but it also prevents adding new projects to 'required-projects' | 16:10 |
dhellmann | I wonder why they need horizon in order to build release notes | 16:11 |
dhellmann | maybe we need variants for this like for the pypi job template | 16:11 |
jeblair | dhellmann: yeah, not requiring horizon would be the ideal solution | 16:12 |
dhellmann | sure, we should start there | 16:13 |
smcginnis | Another failure: http://logs.openstack.org/80/802206a2fcbacd79ec3f7ce1fd2856c2d3927bcf/release/publish-openstack-python-tarball/f691362/job-output.txt.gz#_2017-10-27_16_10_08_045497 | 16:13 |
fungi | dhellmann: i expect the reason they "require" horizon to build release notes (or sdists/wheels, or documentation) is because their tox configuration attempts to install their project before doing those things and they declare that horizon is a runtime dependency required by all installations | 16:13 |
*** lucasagomes is now known as lucas-afk | 16:13 | |
jeblair | but otherwise, assuming we want to maintain the 'final' setting, we'll actually just need to duplicate the job rather than making a variant. (the new job can use the same playbooks) | 16:13 |
jeblair | so we'd duplicate https://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/jobs.yaml#n218 | 16:13 |
fungi | if we didn't perform those actions under tox or told tox to install a different set of software for those particular task environments, we could likely drop the dependency | 16:13 |
*** andreas_s has quit IRC | 16:14 | |
jeblair | it is also possible that 'required-projects' should be considered something that can be overridden in a project-pipeline | 16:15 |
jeblair | that would be a change to zuul, but it's an option. i just tried to be very conservative when i made the split. | 16:15 |
dhellmann | smcginnis : that 2nd one is a config issue in the repo: http://logs.openstack.org/80/802206a2fcbacd79ec3f7ce1fd2856c2d3927bcf/release/publish-openstack-python-tarball/f691362/job-output.txt.gz#_2017-10-27_16_09_52_540596 | 16:15 |
jeblair | basically anything that could potentially change what happens when the job runs is not allowed. | 16:15 |
*** andreas_s has joined #openstack-infra | 16:16 | |
smcginnis | Third one - release-openstack-javascript release-openstack-javascript : ERROR No valid playbook found | 16:16 |
dhellmann | in general for doc building we need to support dependencies for auto-generation of python api docs. That's not really the case for release notes, but that job uses the same requirements file and tox setup. | 16:16 |
*** hashar is now known as hasharAway | 16:18 | |
*** hemna_ has joined #openstack-infra | 16:18 | |
pabelanger | fungi: is that in the venv section of tox.ini? | 16:18 |
smcginnis | dhellmann: For that puppet-tripleo one, doesn't look like they've ever had a venv tox target? | 16:18 |
*** ijw_ has quit IRC | 16:18 | |
dhellmann | maybe it's running the wrong job | 16:19 |
fungi | pabelanger: well, in options inherited by testenv:venv at any rate | 16:19 |
*** robcresswell has joined #openstack-infra | 16:19 | |
pabelanger | dhellmann: smcginnis: yah, puppet jobs for release don't use tox, they have another puppet module build process | 16:19 |
*** tosky has joined #openstack-infra | 16:19 | |
*** iyamahat has quit IRC | 16:19 | |
fungi | pabelanger: but as dhellmann points out, a lot of projects do need to be minimally importable or perhaps even runnable to support some specific sphinx extensions which inspect their objects or output | 16:20 |
fungi | jeblair: mentioned that too | 16:20 |
jeblair | smcginnis: release-openstack-javascript specifies a playbook (playbooks/javascript/run) that doesn't exist | 16:20 |
*** andreas_s has quit IRC | 16:21 | |
*** e0ne has joined #openstack-infra | 16:21 | |
smcginnis | Hmm. Is this something new? | 16:22 |
*** catintheroof has quit IRC | 16:23 | |
*** salv-orlando has joined #openstack-infra | 16:23 | |
smcginnis | jeblair: Looks like that is probably supposed to be playbooks/javascript/tarball.yaml maybe? | 16:23 |
*** andreas_s has joined #openstack-infra | 16:23 | |
smcginnis | Hmm, nope. That appears to be for | 16:25 |
smcginnis | release-openstack-javascript | 16:25 |
smcginnis | Err, publish-openstack-javascript-tarball I mean. | 16:25 |
*** dave-mccowan has quit IRC | 16:26 | |
pabelanger | https://review.openstack.org/502185/ | 16:27 |
*** catintheroof has joined #openstack-infra | 16:27 | |
pabelanger | that's the original from mordred | 16:27 |
*** rhallisey has quit IRC | 16:27 | |
pabelanger | smcginnis: I think you might be right, playbooks/javascript/tarball.yaml | 16:27 |
*** salv-orlando has quit IRC | 16:27 | |
*** andreas_s has quit IRC | 16:28 | |
pabelanger | that file should likely be run.yaml | 16:28 |
pabelanger | don't see anything else using it | 16:28 |
dhellmann | so puppet-tripleo is correctly configured to use puppet-release-jobs but that template runs publish-openstack-python-tarball which is the job that failed | 16:28 |
smcginnis | pabelanger: We have two jobs: http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/zuul.d/project-templates.yaml#n547 | 16:29 |
smcginnis | Just as an example | 16:29 |
dhellmann | why would the puppet-release-jobs template use a python tarball job? | 16:29 |
*** ccamacho has quit IRC | 16:29 | |
smcginnis | pabelanger: The tarball one looks like it's correct for publish-openstack-javascript-tarball. | 16:29 |
*** dizquierdo has quit IRC | 16:29 | |
*** andreas_s has joined #openstack-infra | 16:29 | |
smcginnis | pabelanger: So I think we may just be missing a run playbook for release-openstack-javascript, though I don't know what it would be doing. | 16:30 |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Fix release-openstack-javascript job https://review.openstack.org/515772 | 16:30 |
pabelanger | smcginnis: ^should fix the release job | 16:31 |
smcginnis | pabelanger: That's going to break this: http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/zuul.d/jobs.yaml#n401 | 16:31 |
smcginnis | Right? | 16:31 |
pabelanger | smcginnis: no, that is another job in a different repo | 16:31 |
pabelanger | it has isn't own local playbook | 16:31 |
*** jamesden_ has joined #openstack-infra | 16:31 | |
pabelanger | jobs only look in the repo they are defined for playbooks | 16:32 |
*** jamesdenton has quit IRC | 16:32 | |
smcginnis | pabelanger: Ah, OK. Too many moving parts. | 16:32 |
pabelanger | so that is http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/javascript/tarball.yaml | 16:32 |
*** dizquierdo has joined #openstack-infra | 16:33 | |
smcginnis | pabelanger: Then while we wait for that, any ideas for the issue dhellmann pointed out above? puppet job trying to do a python tarball job. | 16:33 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: ansible-lint: install zuul https://review.openstack.org/515742 | 16:33 |
dhellmann | the puppet job template uses a python job to build tarballs for release | 16:34 |
*** Goneri has quit IRC | 16:34 | |
dhellmann | sorry, project template | 16:34 |
*** kjackal_ has joined #openstack-infra | 16:35 | |
*** catintheroof has quit IRC | 16:35 | |
*** ccamacho has joined #openstack-infra | 16:35 | |
* dhellmann wanders off in search of food | 16:36 | |
AJaeger | jeblair: great. One sentence confused me, -1 for that. | 16:36 |
pabelanger | smcginnis: dhellmann: yah, looking here in a moment | 16:36 |
*** dtantsur is now known as dtantsur|afk | 16:36 | |
*** catintheroof has joined #openstack-infra | 16:36 | |
openstackgerrit | Ronelle Landy proposed openstack-infra/tripleo-ci master: Use rdoproject-based settings for containers and dlrn_hash for images https://review.openstack.org/514668 | 16:38 |
clarkb | catching up, looks like most things are happy and/or being debugged? I'm going to take a look at e-r and why it still doesn't seem happy this morning after dmsimard's fixes for it merged | 16:38 |
*** links has quit IRC | 16:38 | |
*** catintheroof has quit IRC | 16:39 | |
smcginnis | clarkb: I have more release failures if you want those. :) | 16:39 |
smcginnis | http://logs.openstack.org/ef/ef1677a3f4ad19833ffeb9ed652a23a8fcc760c1/release/propose-update-constraints/fd633f3/job-output.txt.gz#_2017-10-27_16_20_10_541487 | 16:39 |
*** catintheroof has joined #openstack-infra | 16:39 | |
*** andreas_s has quit IRC | 16:40 | |
*** ccamacho has left #openstack-infra | 16:40 | |
*** sambetts is now known as sambetts|afk | 16:41 | |
pabelanger | smcginnis: dhellmann: looks like we need to build release-openstack-puppet jobs, seems we never did that | 16:41 |
*** hasharAway has quit IRC | 16:41 | |
smcginnis | Oh, that sounds like it could be an issue. :) | 16:42 |
pabelanger | yah, let me see what that would look like | 16:42 |
openstackgerrit | Merged openstack-infra/project-config master: Remove migrated octavia zuul v3 jobs https://review.openstack.org/515486 | 16:43 |
pabelanger | smcginnis: k, this is going to take some time. I'll start converting it to zuulv3 format after some food | 16:45 |
smcginnis | pabelanger: ++ Thanks | 16:45 |
openstackgerrit | Kaitlin Farr proposed openstack-infra/project-config master: Remove legacy Castellan jobs https://review.openstack.org/515774 | 16:46 |
*** rkukura has joined #openstack-infra | 16:48 | |
*** dave-mccowan has joined #openstack-infra | 16:50 | |
openstackgerrit | Kaitlin Farr proposed openstack-infra/openstack-zuul-jobs master: Remove legacy Castellan jobs https://review.openstack.org/515776 | 16:50 |
*** andreas_s has joined #openstack-infra | 16:51 | |
openstackgerrit | Kaitlin Farr proposed openstack-infra/project-config master: Remove legacy Castellan jobs https://review.openstack.org/515774 | 16:53 |
*** sshnaidm|off has quit IRC | 16:54 | |
openstackgerrit | Kaitlin Farr proposed openstack-infra/openstack-zuul-jobs master: Remove legacy Castellan jobs https://review.openstack.org/515776 | 16:54 |
*** trown is now known as trown|lunch | 16:54 | |
*** florianf has quit IRC | 16:54 | |
*** andreas_s has quit IRC | 16:56 | |
*** electrofelix has quit IRC | 16:56 | |
smcginnis | Still not sure where this "Waiting on logger" thing is coming from. | 16:57 |
smcginnis | http://logs.openstack.org/ef/ef1677a3f4ad19833ffeb9ed652a23a8fcc760c1/release/propose-update-constraints/fd633f3/job-output.txt.gz#_2017-10-27_16_20_10_541487 | 16:58 |
*** andreas_s has joined #openstack-infra | 16:58 | |
*** mhickey has quit IRC | 16:59 | |
clarkb | #status log Killed elastic recheck static status page update processes from october first to unlock processing lock. Status page updates seem to be processing now. | 17:00 |
openstackstatus | clarkb: finished logging | 17:00 |
clarkb | #status log Restarted elasticsearch on elasticsearch07 as it's process had crashed. Log doesn't give many clues as to why. Restarted log workers afterwards. | 17:00 |
openstackstatus | clarkb: finished logging | 17:00 |
*** ralonsoh has quit IRC | 17:00 | |
clarkb | I think with ^ we should start seeing e-r be happier now | 17:00 |
*** dave-mccowan has quit IRC | 17:03 | |
clarkb | smcginnis: thats the process on the remote nodes that streams the finger console data back to you | 17:03 |
*** caphrim007 has joined #openstack-infra | 17:03 | |
*** dave-mccowan has joined #openstack-infra | 17:03 | |
clarkb | mordred: jeblair ^ it looks like the logger process never started (maybe port was already taken?) and as a result it couldn't stop the logger and failed the task? | 17:03 |
openstackgerrit | James E. Blair proposed openstack-infra/infra-manual master: Zuul v3: clarify stable branches https://review.openstack.org/515732 | 17:04 |
jeblair | AJaeger, fungi: i chose to resolve that confusion by *removing* a sentence. we don't have to cover everything right there. :) | 17:04 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Ansible linting fixes https://review.openstack.org/515778 | 17:05 |
AJaeger | jeblair: ;) | 17:06 |
*** ykarel has joined #openstack-infra | 17:06 | |
*** ykarel is now known as ykarel|away | 17:06 | |
pabelanger | smcginnis: I think that is zuul_stream generating that | 17:06 |
*** andreas_s has quit IRC | 17:07 | |
*** dhinesh has quit IRC | 17:07 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: DNM: testing ansible-lint fixes https://review.openstack.org/515781 | 17:08 |
AJaeger | pabelanger, jeblair I think 515742 is ready now - and produces some lint issues. I fix some in 515778 | 17:08 |
*** kjackal_ has quit IRC | 17:09 | |
fungi | jeblair: wfm, thanks! | 17:10 |
openstackgerrit | Clark Boylan proposed openstack-infra/elastic-recheck master: Balance parens in all fails query https://review.openstack.org/515782 | 17:11 |
clarkb | dmsimard: ^ fyi | 17:11 |
clarkb | mtreinish: sdague if you are around review on ^ would be great, I think that may be the last thing to get e-r going properly again | 17:11 |
*** andreas_s has joined #openstack-infra | 17:13 | |
*** yamamoto has quit IRC | 17:13 | |
openstackgerrit | Petr Kovar proposed openstack-infra/irc-meetings master: Update docs team mtg time and channel https://review.openstack.org/515783 | 17:15 |
openstackgerrit | Merged openstack-infra/infra-manual master: Zuul v3: clarify stable branches https://review.openstack.org/515732 | 17:15 |
jeblair | clarkb: do you want to push in the zuul stable branch stack? starts at https://review.openstack.org/514459 | 17:16 |
clarkb | jeblair: let me look | 17:17 |
*** Goneri has joined #openstack-infra | 17:17 | |
*** inc0 has quit IRC | 17:17 | |
clarkb | also smcginnis' logger thing is from zuul/ansible/callback/zuul_stream.py where it appears the executor side of anisble is attempting to connect to the test node side logger daemon port | 17:17 |
*** rbrndt has quit IRC | 17:17 | |
*** inc0 has joined #openstack-infra | 17:17 | |
*** ianychoi_ has quit IRC | 17:17 | |
openstackgerrit | Petr Kovar proposed openstack-infra/irc-meetings master: Update docs team mtg time and channel https://review.openstack.org/515783 | 17:18 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul-jobs master: Ansible linting fixes https://review.openstack.org/515778 | 17:18 |
*** jpena is now known as jpena|off | 17:19 | |
jeblair | clarkb, smcginnis: inspection of ara or job-output.json may help with that | 17:19 |
*** andreas_s has quit IRC | 17:19 | |
jeblair | well, ara didn't report any problems starting zuul_console. | 17:20 |
*** yamamoto has joined #openstack-infra | 17:21 | |
AJaeger | argh, I broke docs publishing ;( On it... | 17:21 |
*** aarefiev is now known as aarefiev_afk | 17:21 | |
*** sshnaidm has joined #openstack-infra | 17:24 | |
*** kjackal_ has joined #openstack-infra | 17:24 | |
*** ldnunes has quit IRC | 17:24 | |
*** Swami has joined #openstack-infra | 17:29 | |
*** dave-mccowan has quit IRC | 17:29 | |
*** sshnaidm has quit IRC | 17:30 | |
*** kjackal_ has quit IRC | 17:30 | |
clarkb | jeblair: one small thing on https://review.openstack.org/#/c/514459/4 inline that may indicate a bug? I +2'd because I think it likely was missed in git add -p or something like that | 17:30 |
*** dhinesh has joined #openstack-infra | 17:31 | |
sdague | clarkb: +A | 17:31 |
clarkb | sdague: thanks | 17:31 |
*** ykarel|away has quit IRC | 17:32 | |
*** kjackal_ has joined #openstack-infra | 17:33 | |
*** rkukura has quit IRC | 17:34 | |
jeblair | clarkb: oh that's likely erroneous. how about i clean it up in a followup? | 17:35 |
*** ldnunes has joined #openstack-infra | 17:36 | |
clarkb | jeblair: sure | 17:38 |
openstackgerrit | Merged openstack-infra/elastic-recheck master: Balance parens in all fails query https://review.openstack.org/515782 | 17:38 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Fix prepare-docs-for-afs https://review.openstack.org/515791 | 17:39 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: fix publishing of deploy and install guide https://review.openstack.org/515792 | 17:40 |
AJaeger | config-core, could you review these two changes above, please? I broke *all* docs publishing ;( ^ | 17:40 |
*** xarses has joined #openstack-infra | 17:41 | |
*** ldnunes has quit IRC | 17:41 | |
fungi | AJaeger: both changes lgtm | 17:43 |
AJaeger | thanks, fungi | 17:43 |
*** rhallisey has joined #openstack-infra | 17:45 | |
*** iyamahat has joined #openstack-infra | 17:45 | |
*** yamamoto has quit IRC | 17:49 | |
*** kjackal_ has quit IRC | 17:49 | |
AJaeger | tests pass on https://review.openstack.org/515791 and https://review.openstack.org/515792 - anybody for a second review, please? | 17:52 |
*** yamahata has quit IRC | 17:52 | |
*** iyamahat has quit IRC | 17:52 | |
*** yamamoto has joined #openstack-infra | 17:53 | |
*** ldnunes has joined #openstack-infra | 17:54 | |
clarkb | jeblair: these are not fast reviews :) | 17:56 |
*** yamamoto has quit IRC | 17:59 | |
*** jbadiapa has quit IRC | 18:02 | |
AJaeger | jeblair: just reviewed the documentation changes on the stack, those are fine ;) | 18:02 |
clarkb | jeblair: question on https://review.openstack.org/#/c/511352/21 | 18:03 |
*** rkukura has joined #openstack-infra | 18:04 | |
*** harlowja has quit IRC | 18:05 | |
*** harlowja has joined #openstack-infra | 18:05 | |
openstackgerrit | Sean McGinnis proposed openstack-infra/project-config master: Create new repo cinder tempest plugin https://review.openstack.org/486303 | 18:13 |
openstackgerrit | Sean McGinnis proposed openstack-infra/project-config master: Add jobs for cinder-tempest-repo https://review.openstack.org/515797 | 18:13 |
jeblair | clarkb: replied; let me know if that clarifies or makes it worse :) | 18:14 |
*** Apoorva has quit IRC | 18:15 | |
*** e0ne has quit IRC | 18:15 | |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Create fetch-puppet-module-output role https://review.openstack.org/515798 | 18:16 |
clarkb | jeblair: ok so parent runs aren't run if you supply your own run? | 18:16 |
smcginnis | dhellmann: We're still waiting on this one for javascript jobs: https://review.openstack.org/#/c/515772/ | 18:16 |
*** jascott1 has quit IRC | 18:16 | |
jeblair | clarkb: correct. strict override. (and that behavior is unchanged here) | 18:16 |
clarkb | got it | 18:16 |
*** Apoorva has joined #openstack-infra | 18:17 | |
AJaeger | config-core, could you review https://review.openstack.org/#/c/515772/ to help release team, please? | 18:17 |
smcginnis | jeblair, clarkb: Sorry, I had stepped out. I see your responses above, but did figure out the Waiting on logger stuff? | 18:17 |
smcginnis | Is it an issue we need to fix yet, or just a transient thing? | 18:18 |
smcginnis | AJaeger: Thanks! | 18:18 |
clarkb | smcginnis: I haven't figured it out but my guess is it is a transient thing | 18:19 |
AJaeger | smcginnis: note that docs publishing is broken currently - we need https://review.openstack.org/515791 at least (releasenotes work) | 18:19 |
jeblair | smcginnis: let's assume transient for now | 18:19 |
smcginnis | pabelanger: Were you still working on release-openstack-puppet jobs? Just want to make sure I didn't miss a patch. | 18:19 |
clarkb | smcginnis: baiscally the test node didn't end up listing on the port we expected (so maybe firewall issue or port conflict I dunno) | 18:19 |
smcginnis | clarkb: OK, good. I was hoping it was. | 18:19 |
rm_work | hey -- who do you guys work with at OVH for their donated nodes? | 18:19 |
pabelanger | smcginnis: i am, starting with puppet-branch-tarball job first, almost have that finished. Then we can build on that into a puppet-tarball jobs, then release job | 18:20 |
smcginnis | pabelanger: OK, cool. Didn't want to pressure you, just wasn't sure if I missed anything. | 18:20 |
*** fultonj has quit IRC | 18:21 | |
rm_work | we're trying to track down an issue and we need a really quick data dump from the *host* of a vm... like, `uname -a` and `cat /proc/cpuinfo`; I assume you guys don't have visibility into their hosts, as we just treat them as a generic cloud provider, right? But, maybe there is someone I can talk to? | 18:21 |
openstackgerrit | Jeremy Stanley proposed openstack-infra/puppet-subunit2sql master: Install python-netifaces to avoid building https://review.openstack.org/515800 | 18:21 |
pabelanger | smcginnis: no, happy to help | 18:21 |
fungi | mtreinish: ^ i don't know when that regressed, and i dug around in oslo git histories for a while | 18:21 |
fungi | rm_work: _if_ a job runs in infra-cloud vanilla or chocolate, we've got you covered there | 18:22 |
clarkb | jeblair: and on https://review.openstack.org/#/c/515249/8/tests/unit/test_v3.py to make sure I understand the intended behavior. Untrusted playbooks still have secrets and we assert that they have access to them, but then we only run the jobs with those secrets in a "safe" post review manner | 18:22 |
clarkb | jeblair: that is why _checkUntrustedSecrets() shows secrets present, _gate test runs job(s) and _check test does not | 18:23 |
rm_work | fungi: err, i don't quite follow. as in, you have access to those hosts? | 18:23 |
clarkb | rm_work: it might help to describe the problem you are trying to address | 18:23 |
jeblair | clarkb: that one is intended to be a move of an existing test. i would have done it as its own change, but the old test used an internal interface, and you could not have written the old test in real zuul config yaml (the test used forward-inheritance) | 18:24 |
fungi | rm_work: the infra team operates infra-cloud, so we have access to the hosts running in it. but not for our other providers | 18:24 |
rm_work | so this is for OVH nodes | 18:24 |
rm_work | for example | 18:24 |
rm_work | http://logs.openstack.org/44/515544/2/check/octavia-v1-dsvm-scenario-multinode/235a97c/logs/libvirt/qemu/instance-00000002.txt.gz | 18:24 |
*** salv-orlando has joined #openstack-infra | 18:24 | |
fungi | rm_work: right, we don't have any access to the host layer at ovh, and i don't think we have any ovh operators present in irc either | 18:24 |
rm_work | ubuntu-xenial-ovh-gra1-0000526877 | 18:24 |
clarkb | rm_work: we don't support nested virt | 18:24 |
clarkb | rm_work: for this reason | 18:25 |
fungi | also, yes, that | 18:25 |
rm_work | ok, yes, thank you :P | 18:25 |
jeblair | clarkb: so there shouldn't be any changes there from the old one. | 18:25 |
rm_work | we know | 18:25 |
rm_work | but we believe it's a kernel bug | 18:25 |
rm_work | and we'd like to finish filing it as such | 18:25 |
jeblair | clarkb: but that test runs different jobs in the different pipelines because, yes, the jobs with untrusted secrets can't be run in check. | 18:25 |
rm_work | but we need just a couple of things from the underlying host | 18:25 |
clarkb | oh it surely is a kernel bug :) | 18:25 |
fungi | nested virtualization acceleration exposes a lot of hardware _and_ kernel bugs | 18:26 |
rm_work | specifically `uname -a` and `cat /proc/cpuinfo` | 18:26 |
rm_work | yeah, so i feel like the asks here aren't huge | 18:26 |
clarkb | rm_work: so in the past we've basically said we can't file those bugs because we don't have that visibility | 18:26 |
rm_work | we just need to find someone we can talk to there | 18:26 |
rm_work | right, we're not asking you to file it :) | 18:26 |
rm_work | I'm asking if you have a contact at OVH | 18:26 |
rm_work | so I can follow-up | 18:26 |
clarkb | I suppose you have the dpkg listing from the VM so you just need uuids from us? | 18:27 |
rm_work | i already knew what your response would be (we've done this a number of times) | 18:27 |
clarkb | rm_work: but ya we've talked to $clouds in the past and sorting this out tends to be far too much work for both sides | 18:27 |
rm_work | all we need is about 2 minutes of work | 18:28 |
rm_work | and a pastebin | 18:28 |
clarkb | so I don't really want to just set you loose on $cloud when we already have decided its not somethign we deal with | 18:28 |
*** vsaienk0 has joined #openstack-infra | 18:28 | |
rm_work | that's why i specifically didn't want to ask you to deal with it | 18:29 |
*** salv-orlando has quit IRC | 18:29 | |
clarkb | except you'll need instance uuids at the very least I imagine right? | 18:29 |
clarkb | so that the cloud can determine what hypervisor ran the VM in order to determine kernel and cpu | 18:29 |
rm_work | hmm, perhaps | 18:29 |
rm_work | is there a way to get that from logs? | 18:29 |
rm_work | we have a host | 18:29 |
clarkb | yes we can get the data but this is explicilty not supported :) | 18:30 |
rm_work | but i assume those are kinda dynamic and unrelated to the scheduling | 18:30 |
clarkb | fwiw speaking to Intel nova people at the ptg there seemed to be some interest in nova publishing a "this is how you make things actually work with nested virt and intel" document of some sort | 18:31 |
clarkb | ebcause apparently there are lots of tricks intel has sorted out themselves but that isn't necessarily common knowledge where people are running hypervisors | 18:31 |
openstackgerrit | Vasyl Saienko proposed openstack-infra/project-config master: Remove legacy ironic-inspector jobs https://review.openstack.org/515737 | 18:31 |
*** rbrndt has joined #openstack-infra | 18:32 | |
rm_work | out of curiosity, would it take longer or shorter than this conversation for you to just help me get the info I need? :P | 18:32 |
jeblair | ovh is a public cloud provider -- if this is something you're tracking down on your own, you can do that without our involvement. | 18:32 |
rm_work | jeblair: i don't need your direct involvment... i am asking for *contact info* | 18:32 |
rm_work | for whoever we deal with on their side | 18:32 |
clarkb | rm_work: ovh should publish contact info | 18:32 |
fungi | rm_work: "normal" support requests to tend to take weeks for us to hear back from ovh, so asking them for details from their hypervisors may never get any response. aside from trouble-tickets, the only real contact we have is with one of their vice presidents | 18:33 |
fungi | the one who authorized our pro-bono account | 18:33 |
rm_work | ok, so we don't have any specific contact, like an AM or something | 18:33 |
clarkb | and one of their asks when getting involved with infra donations is that we don't send all these little things in as support requests | 18:33 |
* rm_work sighs | 18:34 | |
jeblair | they give out free trial accounts all the time | 18:34 |
*** armaan has quit IRC | 18:34 | |
*** rtjure has quit IRC | 18:34 | |
fungi | rm_work: the contact we have with them is the same one listed at https://www.openstack.org/foundation/companies/profile/ovh-group | 18:34 |
jeblair | just sign up for an account, spin up a server, and open a ticket | 18:34 |
rm_work | alright, thanks | 18:35 |
*** rtjure has joined #openstack-infra | 18:35 | |
mtreinish | fungi: I expect it was when we landed this: https://review.openstack.org/#/c/492174/ we didn't touch the min version for a very long time until that landed | 18:36 |
AJaeger | jeblair, clarkb, could you review https://review.openstack.org/515791 to fix docs publishing again (Sorry, I broke it), please? And then also https://review.openstack.org/515792 (followup for two guides) and https://review.openstack.org/#/c/515772/ (release team help), please? | 18:36 |
AJaeger | all have one +2 already... | 18:36 |
fungi | mtreinish: thanks, i suspected that. there was also a long-lived oslo.db cap as well | 18:37 |
fungi | which i saw separately in the git log | 18:37 |
clarkb | AJaeger: commented | 18:37 |
fungi | mtreinish: anyway, that's the latest blocker for deploying the new worker | 18:38 |
mtreinish | fungi: anyway I +A'd the extra package in the puppet, seemed straightforward enough | 18:38 |
fungi | mtreinish: thanks! i'll try to fire up another once that merges | 18:38 |
clarkb | jeblair: should I go ahead and approve these changes as I go or do you want them to get in all at relatively the same time? | 18:38 |
*** dave-mccowan has joined #openstack-infra | 18:38 | |
*** rkukura has quit IRC | 18:39 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Fix prepare-docs-for-afs https://review.openstack.org/515791 | 18:39 |
*** rtjure has quit IRC | 18:39 | |
AJaeger | clarkb: indeed! THANKS! updated ^ | 18:39 |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Create publish-openstack-puppet-branch-tarball job https://review.openstack.org/515802 | 18:41 |
pabelanger | mnaser: EmilienM: dhellmann: smcginnis: ^ first attempt at puppet-branch-tarball job, we'll be able to build atop of that for release jobs. | 18:41 |
smcginnis | pabelanger: Thanks! | 18:41 |
AJaeger | fungi, could you review 515791 again, please? I fixed a critical typo that clarkb found | 18:41 |
fungi | AJaeger: clarkb: ouch, i can't believe i didn't spot that | 18:42 |
jeblair | clarkb: all at once would be best i think. | 18:42 |
clarkb | jeblair: ok | 18:42 |
*** vsaienk0 has quit IRC | 18:43 | |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Create publish-openstack-puppet-branch-tarball job https://review.openstack.org/515802 | 18:43 |
AJaeger | fungi: I shouldn't have written that ;( | 18:43 |
clarkb | jeblair: in https://review.openstack.org/#/c/515536/2/tests/fixtures/config/ansible/git/common-config/zuul.yaml some playbooks have .yaml suffixes and some don't | 18:43 |
*** rkukura has joined #openstack-infra | 18:43 | |
clarkb | jeblair: is there a right or a wrong way to do that? | 18:43 |
*** trown|lunch is now known as trown | 18:44 | |
clarkb | jeblair: for example the base-urls job | 18:44 |
*** dave-mccowan has quit IRC | 18:44 | |
AJaeger | clarkb: my understanding reading the docs: All shouldl have .yaml suffixes going forward | 18:44 |
*** rtjure has joined #openstack-infra | 18:45 | |
*** dprince has quit IRC | 18:48 | |
clarkb | jeblair: I guess if everything needs the .yaml that would be a separate change to update the non run playbooks? | 18:48 |
*** amoralej is now known as amoralej|off | 18:48 | |
*** rtjure has quit IRC | 18:50 | |
pabelanger | fungi: clarkb: AJaeger: mind looking at 515802 too, first zuulv3 jobs for puppet-branch-tarballs, we'll build on that for release jobs | 18:50 |
fungi | sure | 18:50 |
jeblair | clarkb: yep. i want to deprecate the non-extension form. with these changes, we support both for a transition period. when i updated the test fixtures, i went ahead and added .yaml to those to save time later. | 18:51 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Fix prepare-docs-for-afs https://review.openstack.org/515791 | 18:51 |
jeblair | clarkb: so yeah, once we remove the non-extension form, we'll need a similar change to update the stuff that was already there. | 18:51 |
*** smatzek has quit IRC | 18:52 | |
fungi | pabelanger: are the ansible-lint failures on 515802 expected? | 18:54 |
fungi | non-voting, but didn't know what the status is on whether we should be paying attention to those yet | 18:55 |
AJaeger | clarkb: could you put your eagle eyes on https://review.openstack.org/515792 as well, please? | 18:55 |
clarkb | dmsimard: we seem to be hitting the continue at https://git.openstack.org/cgit/openstack-infra/elastic-recheck/tree/elastic_recheck/cmd/uncategorized_fails.py#n365 which is why uncategorized e-r pages are not currently updating | 18:55 |
AJaeger | fungi: yes, expected - let me give you a few changes... | 18:55 |
fungi | thanks AJaeger! | 18:55 |
dmsimard | clarkb: I have to do to an appointment now, I won't have the time to check soon :/ | 18:56 |
AJaeger | fungi: https://review.openstack.org/515742 and https://review.openstack.org/515778 are steps to fix them. But more work is needed - that's why we made those non-voting so that we can iterate on them. | 18:56 |
AJaeger | fungi: we need 515742 in so that the job gives good results - and then we should check that new code does not introduce regressions. | 18:56 |
clarkb | dmsimard: thats fine, just passing info along | 18:56 |
AJaeger | help on remaining ansible issues is welcome, I've done what I could do easily... | 18:57 |
*** csomerville has quit IRC | 18:57 | |
*** csomerville has joined #openstack-infra | 18:57 | |
*** dhajare|mtg has quit IRC | 18:57 | |
AJaeger | now time to sign off - have a great weekend! | 18:57 |
fungi | thanks for all your help agani, AJaeger! | 18:58 |
fungi | s/agani/again/ | 18:58 |
smcginnis | pabelanger: Was there anything else after that puppet patch needed? Or should we be OK to try once that makes it through? | 18:58 |
*** yamamoto has joined #openstack-infra | 18:59 | |
openstackgerrit | Merged openstack-infra/project-config master: fix publishing of deploy and install guide https://review.openstack.org/515792 | 19:00 |
*** rbrndt has quit IRC | 19:02 | |
*** rbrndt has joined #openstack-infra | 19:02 | |
openstackgerrit | Ihar Hrachyshka proposed openstack-infra/irc-meetings master: Moved Neutron upgrades meeting 1h earlier https://review.openstack.org/515803 | 19:03 |
clarkb | jeblair: ok I have +2'd the entire stack that wasn't already approved | 19:05 |
openstackgerrit | Merged openstack-infra/zuul-jobs master: Create fetch-puppet-module-output role https://review.openstack.org/515798 | 19:05 |
clarkb | jeblair: I think you can approve them in the sets that you want (maybe all together at once) whenever you are ready | 19:05 |
*** yamamoto has quit IRC | 19:06 | |
pabelanger | infra-root: I have to run a few errands, but will return to continue work on puppet release jobs. | 19:06 |
fungi | thanks pabelanger! | 19:06 |
clarkb | I'm going to grab lunch now | 19:08 |
*** sree has joined #openstack-infra | 19:11 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add multi-branch support for project-templates https://review.openstack.org/515805 | 19:12 |
openstackgerrit | Merged openstack-infra/project-config master: Create publish-openstack-puppet-branch-tarball job https://review.openstack.org/515802 | 19:13 |
jeblair | clarkb: cool... i er, have one more change i'd like to get in when you're back from lunch. :) ^ | 19:13 |
*** dprince has joined #openstack-infra | 19:13 | |
jeblair | that one wasn't quite as urgent as the others, but with it, i think we've got the stable branch story pretty solid. | 19:14 |
fungi | ooh, this is fun... for some reason openstack-infra/puppet-subunit2sql got lumped into the tripleo gate queue | 19:15 |
* fungi fixes | 19:15 | |
*** sree has quit IRC | 19:16 | |
fungi | config-core: er, where do we (un)set that? | 19:16 |
jeblair | fungi: on the "project:" stanza for puppet-subunit2sql in project-config | 19:17 |
fungi | is it maybe getting set by a project-template then? | 19:18 |
jeblair | possibly | 19:19 |
fungi | and those are in osj, right? | 19:19 |
jeblair | yeah | 19:19 |
jeblair | i think that should be an anti-pattern | 19:20 |
jeblair | putting queues on project-templates | 19:20 |
*** andreas_s has joined #openstack-infra | 19:21 | |
fungi | hrm, i don't see it in project-templates.yaml | 19:21 |
jeblair | perhaps it should even be a config error :) | 19:21 |
jeblair | fungi: zuul.d/zuul-legacy-project-templates.yaml | 19:21 |
jeblair | fungi: was added in this change: https://review.openstack.org/512142 | 19:21 |
fungi | aha, thanks | 19:21 |
fungi | mnaser: ^ any concerns with me disconnecting the puppet-beaker-jobs and puppet-beaker-jobs-xenial job-templates from the tripleo change queue? they're also used on hundreds of infra puppet modules | 19:24 |
*** Goneri has quit IRC | 19:25 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/openstack-zuul-jobs master: Disjoin puppet-beaker-jobs-.* from tripleo queue https://review.openstack.org/515808 | 19:27 |
fungi | mnaser: ^ | 19:27 |
*** smatzek has joined #openstack-infra | 19:27 | |
*** Apoorva has quit IRC | 19:28 | |
*** Apoorva has joined #openstack-infra | 19:29 | |
*** catintheroof has quit IRC | 19:29 | |
*** e0ne has joined #openstack-infra | 19:29 | |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Add implied branch matchers on 'master' https://review.openstack.org/514459 | 19:30 |
*** e0ne has quit IRC | 19:30 | |
*** andreas_s has quit IRC | 19:31 | |
*** andreas_s has joined #openstack-infra | 19:32 | |
*** Apoorva has quit IRC | 19:33 | |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Switch to late-binding inheritance https://review.openstack.org/511352 | 19:33 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Move test_job_auth_inheritance to test_v3 https://review.openstack.org/515249 | 19:33 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Add pragma directive https://review.openstack.org/515483 | 19:33 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Support file extension in playbook path https://review.openstack.org/515535 | 19:33 |
*** tesseract has quit IRC | 19:33 | |
*** andreas_s has quit IRC | 19:37 | |
*** dave-mccowan has joined #openstack-infra | 19:37 | |
*** andreas_s has joined #openstack-infra | 19:38 | |
*** iyamahat has joined #openstack-infra | 19:40 | |
*** catintheroof has joined #openstack-infra | 19:41 | |
*** iyamahat_ has joined #openstack-infra | 19:42 | |
dhellmann | I'm seeing a parse error in a linter job on a project-config change that doesn't seem to be related to the error message. Is that a known issue? http://logs.openstack.org/52/515752/1/infra-check/project-config-zuul-jobs-linters/2116b8f/job-output.txt.gz | 19:43 |
*** Apoorva has joined #openstack-infra | 19:43 | |
dhellmann | the patch is https://review.openstack.org/#/c/515752/1 | 19:43 |
jeblair | dhellmann: yep; fix for that is either in progress or landed. safe to ignore. | 19:44 |
dhellmann | jeblair : ok, thanks | 19:45 |
jeblair | dhellmann: in progress, it looks like: https://review.openstack.org/515742 | 19:45 |
*** catintheroof has quit IRC | 19:45 | |
*** iyamahat has quit IRC | 19:45 | |
*** sree has joined #openstack-infra | 19:46 | |
*** smatzek has quit IRC | 19:47 | |
*** dave-mccowan has quit IRC | 19:49 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/zuul feature/zuulv3: Increase github delay to 10 seconds https://review.openstack.org/515812 | 19:49 |
AJaeger | dhellmann: the non-voting job is new - we're stabilizing it right now... | 19:50 |
mnaser | fungi i dont oppose to it, but i'd like mwhahaha or EmilienM to chime in about removing beaker jobs from tripleo queue. | 19:51 |
*** sree has quit IRC | 19:51 | |
mnaser | from puppet side of things, its not necessary, but it might affect tripleo more than anything | 19:52 |
dhellmann | AJaeger : the patch got a -1 because it failed | 19:53 |
dhellmann | oh, wait, no, it didn't | 19:53 |
dhellmann | sorry | 19:53 |
jeblair | mnaser: projects are added to queues rather than jobs, so we should add the projects you intended to that queue instead | 19:53 |
jeblair | mnaser: eg, all of the tripleo projects themselves, so we don't pull in the infra puppet projects. | 19:53 |
*** andreas_s has quit IRC | 19:54 | |
EmilienM | mnaser: beaker jobs? it has nothing to do with tripleo queue | 19:55 |
mnaser | jeblair https://review.openstack.org/#/c/512142/ - so does that mean patch is technically not correct? is there somewhere we can add projects to a queue? | 19:55 |
*** smatzek has joined #openstack-infra | 19:55 | |
fungi | mnaser: in the projects list | 19:55 |
mnaser | oooh i see what you mean | 19:56 |
jeblair | mnaser: it's *technically* correct, just probably not the best approach since lots of unrelated projects use those jobs | 19:56 |
mnaser | not in the job definiton | 19:56 |
mnaser | ok, makes sense | 19:56 |
jeblair | ("the best kind of correct!) | 19:56 |
*** andreas_s has joined #openstack-infra | 19:56 | |
mnaser | aha | 19:56 |
*** rcernin has joined #openstack-infra | 19:57 | |
mnaser | fungi: +2'd 515808 in that case | 19:57 |
smcginnis | config-core: Could we get a second on https://review.openstack.org/#/c/515752/ | 19:57 |
* fungi promotes jeblair to grade 37 bureaucrat | 19:57 | |
mnaser | fungi actually | 19:57 |
mnaser | fungi https://review.openstack.org/#/c/512142/ what do you think of reverting that entire patch | 19:57 |
mnaser | those are all generic puppet jobs | 19:57 |
jeblair | mnaser, EmilienM: we should get a replacement patch that adds projects to the tripleo queue prepared so we don't lose the queue | 19:58 |
fungi | mnaser: i'd be fine with that too. mostly just trying to make sure my puppet-subunit2sql change doesn't take 48 hours to merge | 19:58 |
*** sree has joined #openstack-infra | 19:58 | |
mnaser | fungi let me push up a patch to move the projects to the right tripleo queue | 19:59 |
mnaser | and then we can put the revert on top of it | 19:59 |
mnaser | does that sound ok? | 19:59 |
fungi | happy to help iterate on a more thorough fix, just trying to get a new subunit2sql worker deployed | 19:59 |
fungi | and was hoping not to have to set up a shadow module tree to accomplish that | 20:00 |
jeblair | fungi: there's a quick fix we can do | 20:00 |
fungi | sure, i could promote that change for now | 20:00 |
jeblair | i'll propose a patch that will let fungi move without disturbing the rest of the tripleo queue until mnaser's patches are ready | 20:00 |
*** andreas_s has quit IRC | 20:01 | |
mnaser | jeblair ok cool, mine should be up in a few minutes | 20:01 |
*** yamamoto has joined #openstack-infra | 20:02 | |
*** andreas_s has joined #openstack-infra | 20:02 | |
*** bobh has joined #openstack-infra | 20:02 | |
fungi | jeblair: separate/dropping project-templates? or can the project list override the project-templates for queue? | 20:02 |
openstackgerrit | James E. Blair proposed openstack-infra/project-config master: Put puppet-subunit2sql into its own queue https://review.openstack.org/515816 | 20:03 |
*** sree has quit IRC | 20:03 | |
jeblair | fungi: ^ first one wins | 20:03 |
fungi | the latter apparently | 20:03 |
fungi | thanks! | 20:03 |
*** bobh has quit IRC | 20:04 | |
jeblair | i propose we add the tripleo queue to the projects that need it, then revert 512142, then revert 515816. should be minimally disruptive for everyone. | 20:04 |
fungi | neat, so you add two project entries to accomplish that. yaml will allow duplicate list items? | 20:04 |
*** bobh has joined #openstack-infra | 20:05 | |
fungi | right, i guess duplicate list items is nothing outlandish now that i think about it | 20:05 |
jeblair | fungi: yeah, that's why zuul's config is a list rather than a dict. | 20:05 |
fungi | associative array keys would have been a problem, yeah | 20:05 |
clarkb | jeblair: looks like you WIP'd that last zuul change? | 20:05 |
jeblair | clarkb: yeah, i realized i missed something (details in #zuul) | 20:05 |
*** smatzek has quit IRC | 20:05 | |
jeblair | clarkb: i'm about halfway through addressing that | 20:06 |
*** mat128 has quit IRC | 20:06 | |
openstackgerrit | Merged openstack-infra/project-config master: fix release jobs for designate-dashboard and zaqar-ui https://review.openstack.org/515752 | 20:07 |
*** yamamoto has quit IRC | 20:08 | |
clarkb | ok | 20:08 |
mnaser | jeblair: what if i changed/edited the tripleo job project templates to run in the tripleo queue, would that be a good approach? | 20:08 |
clarkb | pabelanger: fungi ianw any reason to not use presentty for the project update presentation in sydney? | 20:09 |
mnaser | considering we won't weirdly list tripleo queue in the definition of a puppet job and the tripleo jobs control their queue? | 20:09 |
mnaser | s/puppet job/puppet project/ | 20:09 |
*** bobh has quit IRC | 20:09 | |
fungi | clarkb: since you're starting from the deck i used in boston and it's already presentty, seems reasonable to me | 20:10 |
*** rbrndt has quit IRC | 20:10 | |
fungi | also it helps show off that we're console nuts | 20:10 |
*** rbrndt has joined #openstack-infra | 20:10 | |
fungi | might tempt a few new recruits just from that ;) | 20:11 |
clarkb | fungi: well there are two presentations, the onboarding one I'm definitely just gonna reuse your boston deck from and then the project update deck where we get to talk about zuul and stuff | 20:11 |
*** ijw has joined #openstack-infra | 20:11 | |
clarkb | but probably easiest to just use a single tool for both cases | 20:11 |
fungi | clarkb: oh, right. color me confused. but yes, sounds swell to me | 20:12 |
jeblair | mnaser: i'm sort of inclined to want to have queue membership be explicit per-project in project-config, rather than something that you get automatically when you add a specific project-template. | 20:13 |
mnaser | jeblair: okay, i have no problems in either or, just thought i'd hear feedback | 20:14 |
jeblair | mnaser: mostly just from a perspective of not wanting surprises :). though it is more work. | 20:14 |
*** ldnunes has quit IRC | 20:14 | |
jeblair | mnaser: i'm not firmly opposed or anything. this is definitely a situation where we are figuring out the best way to use the new tool. :) | 20:14 |
*** andreas_s has quit IRC | 20:15 | |
*** ijw has quit IRC | 20:16 | |
*** andreas_s has joined #openstack-infra | 20:17 | |
*** wolverineav has quit IRC | 20:18 | |
*** wolverineav has joined #openstack-infra | 20:18 | |
*** hongbin has quit IRC | 20:20 | |
*** hashar has joined #openstack-infra | 20:22 | |
clarkb | fungi: https://review.openstack.org/#/c/515853/ fyi, also seems to work with whatever tox installs | 20:23 |
*** salv-orlando has joined #openstack-infra | 20:25 | |
*** yamahata has joined #openstack-infra | 20:25 | |
fungi | mnaser: for what it's worth, it definitely took a little time to track down what was causing the infra puppet modules to end up in the tripleo change queue, since it was applied indirectly through project-templates... but also avoiding doing it anywhere besides the projects list helps avoid confusion if you end up applying two project-templates which each declare a different change queue too | 20:25 |
*** hongbin has joined #openstack-infra | 20:27 | |
*** dprince has quit IRC | 20:27 | |
*** salv-orlando has quit IRC | 20:29 | |
*** andreas_s has quit IRC | 20:31 | |
*** cody-somerville has joined #openstack-infra | 20:31 | |
*** jascott1 has joined #openstack-infra | 20:32 | |
*** andreas_s has joined #openstack-infra | 20:32 | |
*** dave-mccowan has joined #openstack-infra | 20:33 | |
openstackgerrit | Mohammed Naser proposed openstack-infra/project-config master: Refactor TripleO jobs to use project-template for scenario001 https://review.openstack.org/515882 | 20:33 |
mnaser | step #1, cleanup ^ | 20:33 |
*** csomerville has quit IRC | 20:34 | |
*** kjackal_ has joined #openstack-infra | 20:34 | |
*** rhallisey has quit IRC | 20:35 | |
*** ijw has joined #openstack-infra | 20:35 | |
*** kgiusti has left #openstack-infra | 20:35 | |
*** salv-orlando has joined #openstack-infra | 20:35 | |
*** ijw has joined #openstack-infra | 20:36 | |
*** mdbooth has quit IRC | 20:37 | |
*** ari[m] has quit IRC | 20:37 | |
*** andreas_s has quit IRC | 20:37 | |
*** dave-mccowan has quit IRC | 20:38 | |
*** thorst has quit IRC | 20:38 | |
*** andreas_s has joined #openstack-infra | 20:38 | |
zxiiro | Does anyone know how to check documentation for a released version of something? For example I want to see JJB 1.6.2 documentation specificilly. | 20:39 |
zxiiro | the default url always gives you the latest master branch | 20:39 |
openstackgerrit | Mohammed Naser proposed openstack-infra/project-config master: Add modules tested by TripleO to their queue https://review.openstack.org/515903 | 20:39 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Add multi-branch support for project-templates https://review.openstack.org/515805 | 20:41 |
openstackgerrit | Mohammed Naser proposed openstack-infra/openstack-zuul-jobs master: Revert "move puppet jobs to tripleo queue" https://review.openstack.org/515918 | 20:42 |
*** andreas_s has quit IRC | 20:44 | |
*** ari[m] has joined #openstack-infra | 20:44 | |
jeblair | zxiiro: there's https://docs.openstack.org/infra/jenkins-job-builder/1.6/ | 20:44 |
*** mdbooth has joined #openstack-infra | 20:44 | |
*** yamahata has quit IRC | 20:44 | |
*** andreas_s has joined #openstack-infra | 20:44 | |
jeblair | zxiiro: i think that's based on branches, not releases though | 20:45 |
jeblair | which is why it's 1.6 and not 1.6.2 | 20:45 |
*** iyamahat_ has quit IRC | 20:45 | |
jeblair | zxiiro: we might be missing the release doc jobs on jjb. | 20:46 |
fungi | i was still not entirely convinced by the docs team's arguments that nobody wants to look at tag-specific versions of documentation (it only really makes sense in the openstack context where clients/libraries all get stable branches) | 20:46 |
mnaser | jeblair fungi https://review.openstack.org/#/q/topic:tripleo-queue | 20:47 |
zxiiro | jeblair: ah great, that's actually all i need | 20:47 |
fungi | jeblair: unless we undid it, all projects stopped running tag-specific docs publication because the jobs were simplified to no longer support it | 20:48 |
*** felipemonteiro__ has quit IRC | 20:48 | |
jeblair | fungi: even library projects? | 20:49 |
fungi | jeblair: yeah, because "proper" (read "openstack") libraries use stable branches | 20:49 |
*** andreas_s has quit IRC | 20:50 | |
*** bocon has joined #openstack-infra | 20:50 | |
*** andreas_s has joined #openstack-infra | 20:51 | |
mnaser | would infra be able to give us the ability to force merge a set of patches? for puppet build jobs to work, we needed to add "puppet" to our bindep.txt across all branches .. https://review.openstack.org/#/q/topic:bindep/puppet .. this is going to waste a lot of CI resources, could we have access to force merge it for now (or have infra do it for us?) to avoid the infra churn | 20:51 |
fungi | i've been meaning to come up with new jobs we can use in tag-based pipelines for branchless projects people consume releases of, just haven't gotten around to it | 20:51 |
jeblair | fungi: yeah, i think that will be appropriate for some projects | 20:52 |
*** jgr has quit IRC | 20:53 | |
jeblair | mnaser: that's a lot of changes. :) | 20:53 |
mnaser | jeblair yeah... well its a change for every puppet openstack repo across all branches :( | 20:53 |
*** lindycoder has quit IRC | 20:53 | |
*** dchen has quit IRC | 20:53 | |
jeblair | what is that, like 125 changes? | 20:53 |
mnaser | i dont mind having to do all the clicking if i can be given the ability to do that for puppet projects and do it for those only | 20:53 |
*** jgr has joined #openstack-infra | 20:54 | |
*** rlandy has quit IRC | 20:54 | |
fungi | mnaser: i guess what you're asking for is the ability to push changes directly into the git repositories so that they don't run any check/gate jobs at all? | 20:54 |
fungi | rather than proposing them for review first? | 20:54 |
mnaser | 33 changes per branch, master + 3 stable => 132 | 20:54 |
mnaser | well, either that, or if we can force merge those changes | 20:54 |
jeblair | fungi: they're already up for review, so i think ^ that | 20:55 |
fungi | ahh | 20:55 |
fungi | i didn't realize they'd already been proposed to gerrit | 20:55 |
mnaser | its ok, EmilienM is already gone to "somewhere with no ip addresses" so if the solution be easier not using the gerrit changes | 20:55 |
mnaser | we wont tell him | 20:55 |
mnaser | :P | 20:55 |
EmilienM | wat | 20:56 |
EmilienM | no i'm still here | 20:56 |
EmilienM | ++ for force push on these patches | 20:56 |
mnaser | :P | 20:56 |
EmilienM | if we can save resources here | 20:57 |
mnaser | and a lot of time, given tripleo queue is already so backed up and these jobs can take up to 1h-ish to run each | 20:57 |
jeblair | what change prompted this? | 20:57 |
mnaser | jeblair https://review.openstack.org/#/c/515802/ | 20:58 |
mnaser | we dont have puppet in our bindep so that job would fail for puppet modules | 20:58 |
*** Apoorva_ has joined #openstack-infra | 20:59 | |
jeblair | an alternative would be to install puppet in that playbook, then let all those bindep changes work their way in slowly, then remove it from the playbook. | 20:59 |
mnaser | i was thinking of creating something like an ensure-puppet type of role | 21:00 |
jeblair | yeah, though having it in bindep sounds like a good idea regardless, right? | 21:00 |
mnaser | but i feel like if we start using too much of those ensure-foo roles then whats the point of bindep.. so yeah its better to rely on bindep for that | 21:00 |
*** andreas_s has quit IRC | 21:01 | |
*** hemna_ has quit IRC | 21:01 | |
mnaser | ok, should i add a patch to install puppet in pre_tasks and we can let those changes make their way slowly but surely.. | 21:01 |
mnaser | and maybe with a note to drop it once everything in that topic merges | 21:02 |
*** Apoorva has quit IRC | 21:02 | |
mnaser | (this solves the issue, but still leaves a ton of wasted resources :<) | 21:02 |
*** andreas_s has joined #openstack-infra | 21:02 | |
*** camunoz has quit IRC | 21:04 | |
*** yamamoto has joined #openstack-infra | 21:05 | |
fungi | the human effort wasted in cleanup if something goes wrong is a lot more valuable than a few hundred server-hours spread out over some period where the tripleo queue has reached greater stability | 21:05 |
fungi | (obviously dumping those into the gate right now would not be an excellent choice) | 21:05 |
jeblair | mnaser: that's the slowest and safest route. while i can't see how force-merging these changes would cause a problem, i'm surprised every time we do it. if other infra-root and puppet-core folks would prefer to force-merge, i won't object, but it's not my first choice. | 21:06 |
*** trown is now known as trown|outtypewww | 21:06 | |
*** iyamahat has joined #openstack-infra | 21:06 | |
mnaser | here's where i'm at: its friday evening, weekend is ahead of us, we'll churn in as much as we can get | 21:06 |
mnaser | and see where we stand on monday | 21:07 |
mnaser | if we got 5 patches merged, let's re-evaluate... if there's only 5 left, then not an issue | 21:07 |
*** bocon has quit IRC | 21:07 | |
jeblair | sounds good | 21:07 |
*** smatzek has joined #openstack-infra | 21:07 | |
mnaser | and if i can have eyes on this before the weekend so it helps some of the stuff that is not in the tripleo queue: https://review.openstack.org/#/q/topic:tripleo-queue ! | 21:07 |
mnaser | ill focus on getting those in first | 21:08 |
*** andreas_s has quit IRC | 21:08 | |
*** andreas_s has joined #openstack-infra | 21:09 | |
*** yamamoto has quit IRC | 21:10 | |
jeblair | mnaser: regarding 515903 -- what about puppet-nova? | 21:10 |
EmilienM | I'll be 100% offline this weekend, just fyi | 21:12 |
*** marst has quit IRC | 21:15 | |
*** andreas_s has quit IRC | 21:18 | |
mnaser | jeblair i was looking for jobs which ran scenario in gate, puppet-nova only runs tripleo-ci in `check-tripleo` pipeline.. which doesnt seem to have any queues defined for it | 21:18 |
*** andreas_s has joined #openstack-infra | 21:19 | |
*** dizquierdo has quit IRC | 21:23 | |
*** leakypipes has quit IRC | 21:23 | |
*** andreas_s has quit IRC | 21:24 | |
*** yamahata has joined #openstack-infra | 21:25 | |
jeblair | if there are any infra-core folks who could review 515805 i'd appreciate it. | 21:27 |
*** rcernin has quit IRC | 21:28 | |
*** andreas_s has joined #openstack-infra | 21:29 | |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Update test fixtures to use explicit run https://review.openstack.org/515536 | 21:30 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Remove implied run https://review.openstack.org/515487 | 21:30 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Validate that a job has a run playbook on freeze https://review.openstack.org/515537 | 21:30 |
jeblair | 515805 is the last thing holding up the stable branch fix restart | 21:30 |
* clarkb looks | 21:31 | |
*** sree has joined #openstack-infra | 21:32 | |
*** dingyichen has joined #openstack-infra | 21:33 | |
*** esberglu has quit IRC | 21:34 | |
*** sree has quit IRC | 21:37 | |
*** andreas_s has quit IRC | 21:39 | |
*** hashar has quit IRC | 21:39 | |
*** jamesden_ is now known as jamesdenton | 21:41 | |
*** andreas_s has joined #openstack-infra | 21:41 | |
clarkb | jeblair: lgtm | 21:42 |
*** baoli has quit IRC | 21:42 | |
*** salv-orlando has quit IRC | 21:50 | |
*** wolverineav has quit IRC | 21:50 | |
*** salv-orlando has joined #openstack-infra | 21:50 | |
*** wolverineav has joined #openstack-infra | 21:50 | |
*** wolverineav has quit IRC | 21:54 | |
*** salv-orlando has quit IRC | 21:55 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 22:00 |
*** andreas_s has quit IRC | 22:01 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements https://review.openstack.org/503645 | 22:01 |
*** salv-orlando has joined #openstack-infra | 22:03 | |
*** ijw has quit IRC | 22:05 | |
*** ijw has joined #openstack-infra | 22:05 | |
*** yamamoto has joined #openstack-infra | 22:06 | |
*** andreas_s has joined #openstack-infra | 22:06 | |
*** smatzek has quit IRC | 22:09 | |
*** smatzek has joined #openstack-infra | 22:10 | |
*** smatzek has quit IRC | 22:11 | |
*** yamamoto has quit IRC | 22:11 | |
*** smatzek has joined #openstack-infra | 22:11 | |
*** iyamahat_ has joined #openstack-infra | 22:13 | |
*** jtomasek has quit IRC | 22:14 | |
*** dave-mccowan has joined #openstack-infra | 22:14 | |
openstackgerrit | Paul Belanger proposed openstack-infra/openstack-zuul-jobs master: Switch puppet-release-jobs to use puppet-branch-tarball job https://review.openstack.org/515957 | 22:15 |
*** iyamahat has quit IRC | 22:15 | |
clarkb | jeblair: I'll be around for the afternoon and early evening if you wnt to give it a go | 22:15 |
pabelanger | clarkb: fungi: mnaser: jeblair: smcginnis: ^ should start to let puppet jobs publish branch tarballs correctly. | 22:16 |
*** smatzek has quit IRC | 22:16 | |
clarkb | pabelanger: I'll add it to the review list | 22:16 |
clarkb | want to get a change up against publications for the project update first though | 22:16 |
*** andreas_s has quit IRC | 22:16 | |
pabelanger | sure | 22:17 |
pabelanger | clarkb: also, presentty wfm | 22:17 |
mnaser | https://review.openstack.org/#/q/topic:tripleo-queue <-- this would help us move some jobs out of puppet queue | 22:17 |
*** andreas_s has joined #openstack-infra | 22:17 | |
mnaser | to get the bindep changes to land easier | 22:18 |
mnaser | pabelanger is "publish-openstack-python-branch-tarball" in post for puppet jobs wrong in that case? | 22:19 |
pabelanger | looking | 22:19 |
pabelanger | mnaser: it is, but we need to write the release job still | 22:19 |
*** sree has joined #openstack-infra | 22:19 | |
pabelanger | that is what I am working | 22:19 |
pabelanger | but first want to make sure branch-tarballs are working, since we'll depend on that | 22:19 |
mnaser | pabelanger ok, because both "puppet-release-jobs" template is defined and "publish-openstack-python-branch-tarball" is manually listed in post for puppet projects, so ill drop "publish-openstack-python-branch-tarball" from the puppet projects.yaml | 22:20 |
mnaser | because i dont think they should be there | 22:20 |
mnaser | puppet-release-jobs template being there will mean that there is no functional change | 22:20 |
*** iyamahat__ has joined #openstack-infra | 22:21 | |
pabelanger | mnaser: right, you'll have to remove the manualy defined post pipelines too | 22:21 |
pabelanger | where is that? | 22:21 |
mnaser | pabelanger im seeing it for a few puppet modules in projects.yaml in project-config (removing them now) | 22:21 |
*** iyamahat__ has quit IRC | 22:21 | |
*** iyamahat has joined #openstack-infra | 22:22 | |
*** andreas_s has quit IRC | 22:22 | |
*** sree has quit IRC | 22:23 | |
*** thorst has joined #openstack-infra | 22:24 | |
*** dave-mccowan has quit IRC | 22:24 | |
*** panda|ruck is now known as panda|ruck|off | 22:24 | |
pabelanger | mnaser: yah, we should have coverage in puppet-release-jobs tempalte | 22:24 |
pabelanger | template* | 22:24 |
*** iyamahat_ has quit IRC | 22:24 | |
openstackgerrit | Merged openstack-infra/project-config master: Refactor TripleO jobs to use project-template for scenario001 https://review.openstack.org/515882 | 22:25 |
openstackgerrit | Mohammed Naser proposed openstack-infra/project-config master: Drop publish-openstack-python-branch-tarball from Puppet post https://review.openstack.org/515959 | 22:25 |
openstackgerrit | Merged openstack-infra/project-config master: Add modules tested by TripleO to their queue https://review.openstack.org/515903 | 22:26 |
mnaser | pabelanger btw, does this mean that we should technically drop "puppet-openstack-module-build" eventually? | 22:26 |
*** salv-orlando has quit IRC | 22:27 | |
mnaser | and use publish-openstack-puppet-branch-tarball ? | 22:27 |
pabelanger | no, we'll create a new job and add it to ozj, we might be able to result job name | 22:27 |
pabelanger | working on that after release jobs | 22:27 |
mnaser | ok cool, makes sense | 22:28 |
*** andreas_s has joined #openstack-infra | 22:28 | |
*** salv-orlando has joined #openstack-infra | 22:28 | |
*** thorst has quit IRC | 22:29 | |
jeblair | clarkb, fungi, mnaser: how does this look? https://etherpad.openstack.org/p/EODwIgRLgk | 22:30 |
*** iyamahat_ has joined #openstack-infra | 22:30 | |
mnaser | jeblair lgtm, makes perfect sense, maybe add [stable] tag? | 22:31 |
clarkb | ya lgtm | 22:31 |
jeblair | cool, will do. i'll send that as soon as we restart | 22:32 |
jeblair | which is not right now because the change bounced off the gate after getting an infra-cloud node. | 22:32 |
*** iyamahat__ has joined #openstack-infra | 22:33 | |
mnaser | :p | 22:33 |
*** iyamahat has quit IRC | 22:33 | |
*** salv-orlando has quit IRC | 22:35 | |
*** salv-orlando has joined #openstack-infra | 22:36 | |
*** iyamahat_ has quit IRC | 22:37 | |
*** sshnaidm has joined #openstack-infra | 22:38 | |
*** iyamahat__ has quit IRC | 22:38 | |
*** andreas_s has quit IRC | 22:38 | |
*** salv-orlando has quit IRC | 22:40 | |
*** jcoufal has quit IRC | 22:44 | |
*** wolverineav has joined #openstack-infra | 22:46 | |
fungi | sorry, entered an hour-long food prep and consumption tunnel without warning. back and catching up now | 22:48 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Revert "move puppet jobs to tripleo queue" https://review.openstack.org/515918 | 22:48 |
*** ijw has quit IRC | 22:48 | |
*** ijw has joined #openstack-infra | 22:49 | |
*** sdague has quit IRC | 22:50 | |
openstackgerrit | Merged openstack-infra/project-config master: ansible-lint: install zuul https://review.openstack.org/515742 | 22:51 |
clarkb | fungi: pabelanger ianw https://review.openstack.org/515950 | 22:53 |
clarkb | that is a first draft for the update presentation | 22:53 |
*** andreas_s has joined #openstack-infra | 22:54 | |
clarkb | jeblair: what would be some good zuulv4 features to call out on the zuul v4 future? | 22:54 |
*** rbrndt has quit IRC | 22:55 | |
clarkb | pabelanger: your change doesn't update pre-release and release, just post | 22:55 |
clarkb | pabelanger: is that intentional? | 22:55 |
clarkb | *your puppet change | 22:55 |
pabelanger | clarkb: it is, I am writing pre-release / release job now | 22:57 |
pabelanger | but wanted to first make sure post worked since it builds atop of it | 22:57 |
clarkb | got it | 22:57 |
clarkb | looks like fungi just approved | 22:57 |
*** andreas_s has quit IRC | 22:58 | |
pabelanger | great | 22:58 |
fungi | jeblair: draft announcement lgtm, thanks! | 23:00 |
*** tosky has quit IRC | 23:00 | |
pabelanger | clarkb: fungi: we also can land https://review.openstack.org/515959/ to, will remove duplicate jobs from puppet | 23:01 |
fungi | clarkb: yep, sorry didn't realize you were reviewing it as i was going through things i found in scrollback chronologically | 23:01 |
fungi | and i had a fair bit of context on that one owing to discussions in the -release channel | 23:01 |
fungi | plus, was basically a one-liner | 23:02 |
openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Switch puppet-release-jobs to use puppet-branch-tarball job https://review.openstack.org/515957 | 23:02 |
clarkb | pabelanger: done | 23:03 |
clarkb | fungi: np | 23:03 |
fungi | clarkb: maybe we don't want to make references to "zuul v4" per se, and freak everyone out. better to perhaps talk about "post-v3 improvements"? | 23:03 |
clarkb | hahaha | 23:03 |
clarkb | v3.1 ? | 23:03 |
pabelanger | thanks | 23:03 |
jeblair | clarkb: well, actually i have a whole list of things that still need to go into v3 before it's done... lemme find it | 23:07 |
*** yamamoto has joined #openstack-infra | 23:07 | |
*** xarses has quit IRC | 23:07 | |
clarkb | jeblair: that probably works better thanks | 23:08 |
jeblair | clarkb: http://paste.openstack.org/show/624856/ | 23:08 |
fungi | that might be better to talk about... the road to the 3.0.0 release | 23:08 |
jeblair | clarkb: my plan for early next week is to polish that up and send it out as a proposed update to our roadmap | 23:08 |
fungi | we're still very much beta-testing the future v3 | 23:08 |
jeblair | that's from the brainstorming etherpad at the ptg | 23:08 |
*** andreas_s has joined #openstack-infra | 23:08 | |
fungi | or at least gamma-testing release candidates | 23:09 |
clarkb | jeblair: perfect thanks | 23:09 |
openstackgerrit | Merged openstack-infra/project-config master: Drop publish-openstack-python-branch-tarball from Puppet post https://review.openstack.org/515959 | 23:09 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Add multi-branch support for project-templates https://review.openstack.org/515805 | 23:09 |
jeblair | \o/ back just in time :) | 23:09 |
*** ansiwen[q] has quit IRC | 23:10 | |
*** yamahata has quit IRC | 23:10 | |
*** yamamoto has quit IRC | 23:12 | |
pabelanger | okay, publish-openstack-puppet-branch-tarball worked: http://tarballs.openstack.org/puppet-qdr/ | 23:14 |
pabelanger | now for release job | 23:14 |
*** ansiwen[q] has joined #openstack-infra | 23:15 | |
*** andreas_s has quit IRC | 23:17 | |
jeblair | zuul has been upgraded on all the hosts | 23:18 |
jeblair | i'm going to install the repl on zuulv3 though real quick | 23:18 |
*** andreas_s has joined #openstack-infra | 23:19 | |
jeblair | i've saved queues and am going to restart zuulv3 scheduler and all executors now. it will be down for a few minutes for this. | 23:19 |
clarkb | fungi: jeblair pabelanger ianw ok new patchset to infra update deck with path to zuul v3 release | 23:20 |
clarkb | I grabbed an item out of each of jeblair's * term sections in the paste | 23:20 |
jeblair | clarkb: link to change? | 23:21 |
*** hongbin has quit IRC | 23:21 | |
clarkb | jeblair: https://review.openstack.org/#/c/515950 | 23:21 |
*** bauwser is now known as bauzas | 23:21 | |
*** ansiwen[q] has quit IRC | 23:22 | |
*** andreas_s has quit IRC | 23:24 | |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Creaate release-openstack-puppet job https://review.openstack.org/515968 | 23:26 |
pabelanger | jeblair: fungi: clarkb: mnaser: smcginnis: dhellmann: ^ should be our brand new zuulv3 release jobs for puppet modules | 23:27 |
*** thorst has joined #openstack-infra | 23:27 | |
pabelanger | actually, 1 sec. I think I see an issue | 23:27 |
openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Creaate release-openstack-puppet job https://review.openstack.org/515968 | 23:28 |
pabelanger | there, with signing goodness | 23:28 |
*** owalsh is now known as owalsh_pto | 23:29 | |
jeblair | still waiting on a few executors to finish cleaning up | 23:29 |
*** andreas_s has joined #openstack-infra | 23:30 | |
clarkb | jeblair: are you restarting the scheduler too? | 23:30 |
jeblair | clarkb: yes | 23:30 |
jeblair | there were changes to both | 23:30 |
clarkb | cool I'll check on my cgit thing afterweards | 23:30 |
pabelanger | i admit, now that we have our base jobs setup, it is getting faster to write new publishing jobs | 23:30 |
mnaser | pabelanger shouldnt that job have something which does the actual publish | 23:31 |
mnaser | or the post of the parent handles that? | 23:31 |
pabelanger | right, parent does that for us | 23:32 |
mnaser | ok cool | 23:32 |
jeblair | interesting... there are a lot of ssh processes sticking around now | 23:35 |
jeblair | they are controlpersist servers, even though the timeout for that is 60s | 23:35 |
fungi | pabelanger: was that renaming find copied from the v2 job, by any chance? | 23:36 |
pabelanger | fungi: yes | 23:36 |
clarkb | jeblair: last time I did restarts it was the ssh-agents not the sshclients | 23:36 |
pabelanger | fungi: crazy right? | 23:36 |
clarkb | and typically no more than a handful | 23:36 |
fungi | pabelanger: okay, i'll approve then. seems to have been written by someone who was either unaware of the rename command or afraid we'd try to port it to a platform where it was lacking | 23:37 |
jeblair | select(10, [3 5], [], NULL, NULL | 23:37 |
jeblair | a select with no timeout? :/ | 23:37 |
jeblair | we may need to try to keep track of those and clean them up ourselves :/ | 23:38 |
jeblair | SpamapS: ^ this is the ssh processes that implements the controlpersist server | 23:38 |
*** thorst has quit IRC | 23:38 | |
jeblair | clarkb: we actually didn't leake a single ssh-agent this time | 23:38 |
*** andreas_s has quit IRC | 23:38 | |
SpamapS | jeblair: gross | 23:39 |
fungi | yeah, when i was doing executor restarts these last couple of times, the stop left behind 50-100 ssh processes | 23:39 |
jeblair | ansible 'ze*' -m shell -a 'pkill -u zuul ssh' | 23:40 |
SpamapS | I guess they figured the select would die when the fifo was closed on the other side? | 23:40 |
fungi | that's basically what i was doing (though server by server as i was performing rolling restarts) | 23:40 |
SpamapS | it should result in an error | 23:40 |
jeblair | SpamapS: yeah. we have a way of finding network edge cases don't we? :) | 23:41 |
SpamapS | oh wait no these are just on disk sockets | 23:41 |
jeblair | ssh 8677 zuul 3u IPv4 2016013959 0t0 TCP ze08.openstack.org:42430->104.239.146.107:ssh (ESTABLISHED) | 23:41 |
jeblair | ssh 8677 zuul 5u unix 0xffff88000ff61000 0t0 2016014183 /var/lib/zuul/builds/383ff3e5a94b464991577d84635ac569/.ansible/cp/df1b1e19ba.FOTfgbjxywzxIW37 type=STREAM | 23:41 |
jeblair | SpamapS: ^ one of each it was waiting on | 23:41 |
jeblair | okay, i have cleaned up everything, starting executors now | 23:42 |
jeblair | starting scheduler | 23:42 |
*** andreas_s has joined #openstack-infra | 23:44 | |
openstackgerrit | Paul Belanger proposed openstack-infra/openstack-zuul-jobs master: Switch to release-openstack-puppet jobs for release-puppet-jobs https://review.openstack.org/515970 | 23:45 |
pabelanger | mnaser: fungi: clarkb: dhellmann: smcginnis: ^and that brings the new release jobs for puppet modules online | 23:45 |
jeblair | oh, hrm. the initial config loading now has to get all the branches before it does the cat jobs... that's annoying, but workable. but if it does that on *reconfiguration* that's unworkable. | 23:46 |
openstackgerrit | Paul Belanger proposed openstack-infra/openstack-zuul-jobs master: Switch to release-openstack-puppet jobs for release-puppet-jobs https://review.openstack.org/515970 | 23:46 |
*** Swami has quit IRC | 23:46 | |
jeblair | i'll look into that. | 23:46 |
*** thorst has joined #openstack-infra | 23:49 | |
*** caphrim007 has quit IRC | 23:49 | |
*** thorst has quit IRC | 23:49 | |
*** andreas_s has quit IRC | 23:49 | |
clarkb | you might need a different type of merge job to really quuckly list the branches? | 23:49 |
*** oanson has quit IRC | 23:54 | |
*** andreas_s has joined #openstack-infra | 23:54 | |
*** oanson has joined #openstack-infra | 23:55 | |
jeblair | yeah, that's an idea | 23:56 |
*** ianychoi has joined #openstack-infra | 23:56 | |
jeblair | this doesn't happen on dynamic reconfiguration. that's good, we should be able to press forward. | 23:57 |
jeblair | it looks like it added about 1 minute to the full reconfiguration time (went from 3m to 4m) | 23:57 |
jeblair | however, i think it's going to happen on tenant reconfiguration too (whenever a change lands) | 23:58 |
jeblair | that was running at about 1m | 23:58 |
jeblair | so it could end up doubling it | 23:58 |
jeblair | that's a moderate problem | 23:58 |
jeblair | i think maybe we should find a way to use cached data for tenant reconfig, but not on full reconfig | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!