| ianw | smcginnis: ok, http://zuulv3.openstack.org/static/stream.html?uuid=a85fce1e012d4c569212824c86e51cb8&logfile=console.log should see if it works ... | 00:00 |
|---|---|---|
| openstackgerrit | Sean McGinnis proposed openstack-infra/project-config master: Switch to pep8 jobs for cinder-specs https://review.openstack.org/520844 | 00:00 |
| smcginnis | ianw: Excellent, thanks! | 00:00 |
| *** armax has joined #openstack-infra | 00:02 | |
| ianw | doesn't seem to want to stream that job, finger issues maybe | 00:02 |
| *** esberglu has joined #openstack-infra | 00:04 | |
| ianw | hmm http://logs.openstack.org/43/520843/1/check/build-openstack-releasenotes/a85fce1/job-output.txt.gz ... it found the constraints file. the publish job didn't | 00:04 |
| *** smatzek has joined #openstack-infra | 00:07 | |
| *** esberglu has quit IRC | 00:08 | |
| *** gridinv has joined #openstack-infra | 00:09 | |
| *** xarses_ has quit IRC | 00:10 | |
| *** smatzek has quit IRC | 00:11 | |
| ianw | ahhh, i see | 00:11 |
| ianw | it's only the publish jobs | 00:12 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Install gettext for msgmerge https://review.openstack.org/520826 | 00:13 |
| *** gyee_ has quit IRC | 00:13 | |
| pabelanger | ianw: playbooks/publish/releasenotes.yaml is now wrong | 00:15 |
| pabelanger | it shouldn't be using fetch-tox-output, since tox support was dropped | 00:15 |
| pabelanger | will need to be updated to fetch logs from venv folder | 00:15 |
| pabelanger | or | 00:15 |
| pabelanger | create .tox/venv for the job | 00:15 |
| pabelanger | actually | 00:16 |
| ianw | pabelanger: yeah, the caller is passing the wrong vars | 00:16 |
| pabelanger | yah, I don't think fetch-tox-output will work, even if updated, it calls tox -l to find into about tox envlist | 00:17 |
| ianw | i feel like it might just work without it | 00:17 |
| pabelanger | k | 00:18 |
| openstackgerrit | Ian Wienand proposed openstack-infra/project-config master: Update releasenotes publish job https://review.openstack.org/520848 | 00:18 |
| ianw | pabelanger: ^ ? | 00:18 |
| *** armax has quit IRC | 00:19 | |
| openstackgerrit | wes hayutin proposed openstack-infra/elastic-recheck master: Adding query for bug #1731063 https://review.openstack.org/520574 | 00:20 |
| tbarron | ianw: https://review.openstack.org/#/c/520837 worked when I applied it locally | 00:20 |
| openstack | bug 1731063 in tripleo "CI: tempest TestVolumeBootPattern tests fail due to not being able to ssh to the VM" [Critical,Triaged] https://launchpad.net/bugs/1731063 | 00:20 |
| pabelanger | ianw: well, if we do that, then we have no more logs from venv environment. I would think there is something in venv/logs we want | 00:20 |
| ianw | won't it all just be blasted to stdout? | 00:21 |
| pabelanger | not sure, we should see what the old releasenotes did | 00:21 |
| pabelanger | it likely has info about python libs versions and such | 00:22 |
| *** claudiub|2 has quit IRC | 00:22 | |
| *** rlandy|rover|afk is now known as rlandy|rover | 00:23 | |
| *** slaweq has joined #openstack-infra | 00:24 | |
| ianw | pabelanger: yeah, i'm not really seeing where http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/roles/build-releasenotes/tasks/main.yaml stashes any particular logs | 00:25 |
| pabelanger | ianw: it would have been tox itself generating them | 00:25 |
| *** sdague has quit IRC | 00:25 | |
| pabelanger | so, if we are using virtualenv directly, does it generate anything? | 00:25 |
| pabelanger | possible it doesn't | 00:25 |
| ianw | but those logs would have essentially been the stdout of the jobs running under tox? | 00:25 |
| ianw | so we've just removed the middle-man :) | 00:26 |
| ianw | or the packages tox installed, etc | 00:26 |
| pabelanger | ianw: okay, just tested locally | 00:26 |
| pabelanger | there is no venv/logs folder | 00:26 |
| pabelanger | much only be a tox thing | 00:26 |
| pabelanger | so we can drop role I think | 00:26 |
| pabelanger | +2 | 00:27 |
| *** slaweq has quit IRC | 00:28 | |
| EmilienM | fungi: I see thx | 00:28 |
| ianw | ok, right. anything else on fire?! :) | 00:29 |
| *** andreas_s has joined #openstack-infra | 00:29 | |
| *** daidv has quit IRC | 00:30 | |
| *** daidv has joined #openstack-infra | 00:31 | |
| *** gouthamr has quit IRC | 00:31 | |
| *** markvoelker has quit IRC | 00:31 | |
| *** yuanying has joined #openstack-infra | 00:32 | |
| *** armax has joined #openstack-infra | 00:33 | |
| *** andreas_s has quit IRC | 00:33 | |
| *** zhurong has joined #openstack-infra | 00:34 | |
| *** armax_ has joined #openstack-infra | 00:34 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Fix afs_slug for centos-7 https://review.openstack.org/520850 | 00:34 |
| pabelanger | ianw: Hopefully LAST patch | 00:35 |
| pabelanger | ^ | 00:35 |
| pabelanger | fixes: mkdir: cannot create directory ‘/afs/.openstack.org/mirror/wheel/centos-7.4.1708-x86_64’: Permission denied | 00:35 |
| *** armax has quit IRC | 00:38 | |
| *** armax_ is now known as armax | 00:38 | |
| ianw | fungi: 292025 ... i read that and my immediate reaction is ABORT ABORT ABORT :) but if you can reword it to something like your comment saying the focus is on getting a review into gerrit, and bigger changes should be discussed before sending a patchset and we might reccommend a separate tool... | 00:38 |
| *** Sukhdev has joined #openstack-infra | 00:43 | |
| openstackgerrit | Merged openstack-infra/project-config master: Fix afs_slug for centos-7 https://review.openstack.org/520850 | 00:43 |
| *** markvoelker has joined #openstack-infra | 00:43 | |
| *** efoley has quit IRC | 00:44 | |
| *** Swami has quit IRC | 00:45 | |
| *** armax has quit IRC | 00:46 | |
| *** askb has quit IRC | 00:46 | |
| *** askb has joined #openstack-infra | 00:49 | |
| *** gmann_afk is now known as gmann | 00:50 | |
| *** askb has quit IRC | 00:53 | |
| *** cuongnv has joined #openstack-infra | 00:54 | |
| *** askb has joined #openstack-infra | 00:54 | |
| *** yamamoto_ has quit IRC | 00:57 | |
| *** aeng has quit IRC | 00:57 | |
| *** Apoorva has joined #openstack-infra | 01:02 | |
| *** yamamoto has joined #openstack-infra | 01:03 | |
| *** Apoorva has quit IRC | 01:06 | |
| *** xarses_ has joined #openstack-infra | 01:07 | |
| *** iyamahat has joined #openstack-infra | 01:08 | |
| *** Apoorva has joined #openstack-infra | 01:08 | |
| *** yamamoto has quit IRC | 01:08 | |
| *** gouthamr has joined #openstack-infra | 01:10 | |
| *** xarses_ has quit IRC | 01:23 | |
| *** caphrim007_ has joined #openstack-infra | 01:28 | |
| *** tinwood has quit IRC | 01:29 | |
| *** tinwood has joined #openstack-infra | 01:30 | |
| *** caphrim00_ has joined #openstack-infra | 01:31 | |
| *** caphrim007 has quit IRC | 01:32 | |
| *** caphrim007_ has quit IRC | 01:33 | |
| *** iyamahat has quit IRC | 01:34 | |
| *** zhurong has quit IRC | 01:34 | |
| *** caphrim00_ has quit IRC | 01:35 | |
| *** hongbin has quit IRC | 01:35 | |
| *** jaypipes has quit IRC | 01:36 | |
| *** Sukhdev has quit IRC | 01:38 | |
| *** liujiong has joined #openstack-infra | 01:38 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Use pip module to install reno and deps with --user https://review.openstack.org/520833 | 01:43 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Stop supporting releasenotes/requirements.txt https://review.openstack.org/520834 | 01:43 |
| *** gildub has quit IRC | 01:43 | |
| *** xarses_ has joined #openstack-infra | 01:44 | |
| *** yamamoto has joined #openstack-infra | 01:48 | |
| *** cshastri has joined #openstack-infra | 01:49 | |
| *** yamamoto has quit IRC | 01:53 | |
| *** Apoorva_ has joined #openstack-infra | 01:57 | |
| *** Apoorva has quit IRC | 02:00 | |
| *** Apoorva_ has quit IRC | 02:01 | |
| pabelanger | woot | 02:02 |
| pabelanger | http://logs.openstack.org/78/520178/5/periodic/release-wheel-mirror/fb52311/job-output.txt.gz | 02:02 |
| *** salv-orlando has joined #openstack-infra | 02:02 | |
| pabelanger | afs volume for centos-7 wheels now released | 02:02 |
| *** salv-orl_ has quit IRC | 02:05 | |
| ianw | it's that easy! | 02:07 |
| pabelanger | for some reason still building old branches like kilo, liberty | 02:08 |
| *** andreas_s has joined #openstack-infra | 02:09 | |
| ianw | it just does a grep for stable branches right? | 02:09 |
| *** kiennt26 has joined #openstack-infra | 02:09 | |
| ianw | http://logs.openstack.org/78/520178/5/periodic/build-wheel-mirror-centos-7/1328111/job-output.txt.gz#_2017-11-17_00_51_05_946114 | 02:11 |
| ianw | if requirements has it, we build it | 02:11 |
| pabelanger | Hmm | 02:11 |
| pabelanger | requirements should only have newton and above | 02:12 |
| pabelanger | wonder how old that image is | 02:12 |
| pabelanger | looks recent | 02:12 |
| *** armax has joined #openstack-infra | 02:13 | |
| pabelanger | http://git.openstack.org/cgit/openstack/requirements | 02:13 |
| pabelanger | newton, ocata, pike | 02:13 |
| pabelanger | other branches are eol | 02:13 |
| pabelanger | Oh | 02:13 |
| pabelanger | no | 02:13 |
| pabelanger | some how we still have old gitcache on images? | 02:14 |
| ianw | ahhh, i bet we don't get rid of odl branches | 02:14 |
| *** yamamoto has joined #openstack-infra | 02:14 | |
| ianw | jeez i have deja-vu on this | 02:14 |
| *** andreas_s has quit IRC | 02:14 | |
| pabelanger | nope, /opt/git looks right on centos-7 image | 02:15 |
| ianw | you gotta do "git fetch -p" | 02:15 |
| pabelanger | huh? | 02:16 |
| ianw | https://review.openstack.org/#/c/269374/ ... knew i'd seen this before | 02:17 |
| pabelanger | so, do we need to do that in our mirror-workspace-git-repos role? | 02:19 |
| ianw | http://logs.openstack.org/78/520178/5/periodic/build-wheel-mirror-centos-7/1328111/job-output.txt.gz#_2017-11-17_00_51_05_946114 is saying that the executor pushed those branches to the remote copy, right? | 02:22 |
| *** kota_ has quit IRC | 02:22 | |
| pabelanger | ianw: yah, so it seems zuul have the old branches | 02:23 |
| pabelanger | maybe in local git cache? | 02:23 |
| *** iyamahat has joined #openstack-infra | 02:24 | |
| *** slaweq has joined #openstack-infra | 02:24 | |
| ianw | but it's soooo old ... | 02:25 |
| pabelanger | have no idea | 02:26 |
| ianw | root@ze03:/var/lib/zuul/executor-git/git.openstack.org/openstack/requirements/.git/refs/heads/stable# ls | 02:27 |
| ianw | newton ocata pike | 02:27 |
| *** kota_ has joined #openstack-infra | 02:28 | |
| *** slaweq has quit IRC | 02:29 | |
| *** annp has joined #openstack-infra | 02:30 | |
| pabelanger | yah, have no idea | 02:33 |
| ianw | this is crazy?! | 02:34 |
| ianw | no reference in /opt/dib_cache/source-repositories/requirements_38c913bcdfc647288e87ce7f145972d064cf45d9/ | 02:34 |
| pabelanger | ianw: Oh | 02:34 |
| pabelanger | I think I see the issue | 02:34 |
| pabelanger | pabelanger@zm05:/var/lib/zuul/git/git.openstack.org/openstack/requirements/.git/refs/heads/stable$ ls | 02:35 |
| pabelanger | kilo liberty mitaka newton ocata pike | 02:35 |
| pabelanger | our zuul-mergers still have the old branches cached | 02:35 |
| ianw | ahhhh, they must not be doing --prune | 02:35 |
| pabelanger | yah, that sounds possible | 02:35 |
| pabelanger | sounds like a bug | 02:36 |
| *** caphrim007 has joined #openstack-infra | 02:36 | |
| ianw | # Ensure that we don't have stale remotes around | 02:38 |
| ianw | repo.prune() | 02:38 |
| ianw | it at least tries ... | 02:38 |
| *** ramishra has joined #openstack-infra | 02:39 | |
| pabelanger | ianw: looks to only be in zuul-cloner, not directly in merger | 02:41 |
| *** caphrim007 has quit IRC | 02:41 | |
| pabelanger | and we now use git push --mirror | 02:41 |
| pabelanger | so likely why we are now seeing it | 02:41 |
| ianw | yeah | 02:42 |
| *** namnh has joined #openstack-infra | 02:45 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul feature/zuulv3: Ensure the merger prunes on update https://review.openstack.org/520855 | 02:51 |
| *** yamahata has joined #openstack-infra | 02:51 | |
| ianw | pabelanger: ^ in lieu of a bug report ... see what others think | 02:52 |
| pabelanger | k | 02:53 |
| ianw | AJaeger: just to sync you on the releasenotes stuff when you're around... | 02:55 |
| *** armax has quit IRC | 02:55 | |
| ianw | we found a minor issue with a missing package -> fixed https://review.openstack.org/520826 | 02:55 |
| ianw | we found the publish job needed updating, that's proposed in -> https://review.openstack.org/520848 | 02:56 |
| ianw | maybe we could make things more robust with something an assert for missing constraints, see -> https://review.openstack.org/520839 | 02:57 |
| ianw | mordred had some changes too, which i think got merge failed out, let me take a poke at them | 02:57 |
| ianw | nope, they still apply ... https://review.openstack.org/#/c/520832 and up | 02:58 |
| *** andreww has joined #openstack-infra | 02:58 | |
| *** armax has joined #openstack-infra | 02:59 | |
| *** salv-orl_ has joined #openstack-infra | 03:00 | |
| *** xarses_ has quit IRC | 03:01 | |
| *** dhinesh has quit IRC | 03:01 | |
| *** xarses_ has joined #openstack-infra | 03:01 | |
| *** xarses has quit IRC | 03:01 | |
| *** xarses_ has quit IRC | 03:02 | |
| *** salv-orlando has quit IRC | 03:02 | |
| *** links has joined #openstack-infra | 03:03 | |
| *** xarses_ has joined #openstack-infra | 03:04 | |
| *** xarses_ has quit IRC | 03:04 | |
| *** gildub has joined #openstack-infra | 03:05 | |
| *** aeng has joined #openstack-infra | 03:07 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/git-review master: Add a note on Windows and Unicode https://review.openstack.org/520820 | 03:09 |
| ianw | now with less speling erorrs | 03:10 |
| fungi | i didn't even spot that and i reviewed both the source and rendered versions | 03:11 |
| *** gildub has quit IRC | 03:17 | |
| *** hongbin has joined #openstack-infra | 03:22 | |
| *** yamamoto has quit IRC | 03:23 | |
| *** yamamoto has joined #openstack-infra | 03:25 | |
| *** yamamoto has quit IRC | 03:28 | |
| ianw | fungi: is the wiki still emergency filed? | 03:29 |
| ianw | just checking backups on new host | 03:29 |
| *** jrist has quit IRC | 03:29 | |
| ianw | i've just fixed up etherpad01 and happy everything is backing up http://paste.openstack.org/show/626567/ | 03:30 |
| ianw | everything else | 03:31 |
| *** psachin has joined #openstack-infra | 03:31 | |
| *** andreas_s has joined #openstack-infra | 03:33 | |
| *** gildub has joined #openstack-infra | 03:35 | |
| *** jesusaur has quit IRC | 03:36 | |
| *** Sukhdev has joined #openstack-infra | 03:36 | |
| fungi | ianw: yeah, the wiki server's puppeting has never been completed, and the wiki.o.o hostname for now goes to a server named wiki-upgrade-test | 03:37 |
| fungi | wiki-dev.o.o is puppeted but not fully functional (yet anyway) | 03:37 |
| ianw | fungi: oh ... so i should just manually roll out a cron job update? when it's fully puppeted, it should "just work"? | 03:38 |
| *** andreas_s has quit IRC | 03:38 | |
| fungi | yeah, that seems safest | 03:39 |
| *** yamamoto has joined #openstack-infra | 03:40 | |
| ianw | ok, thanks | 03:40 |
| *** esberglu has joined #openstack-infra | 03:40 | |
| *** esberglu has quit IRC | 03:41 | |
| *** esberglu has joined #openstack-infra | 03:41 | |
| *** esberglu has quit IRC | 03:41 | |
| *** esberglu has joined #openstack-infra | 03:41 | |
| *** yamamoto has quit IRC | 03:45 | |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/git-review master: Add a note about contribution priorities https://review.openstack.org/292025 | 03:46 |
| fungi | ianw: ^ how's that for rephrasing? | 03:46 |
| *** esberglu has quit IRC | 03:46 | |
| *** caphrim007 has joined #openstack-infra | 03:48 | |
| *** andreas_s has joined #openstack-infra | 03:49 | |
| *** eharney has quit IRC | 03:49 | |
| *** eharney has joined #openstack-infra | 03:50 | |
| ianw | fungi: LGTM, thanks. the "changes will mostly be rejected" bit was just a bit too much for mine :) | 03:50 |
| fungi | yup, softened as requested | 03:51 |
| *** iyamahat has quit IRC | 03:52 | |
| *** andreas_s has quit IRC | 03:53 | |
| *** jrist has joined #openstack-infra | 03:53 | |
| *** rosmaita has quit IRC | 03:56 | |
| *** caphrim007_ has joined #openstack-infra | 03:59 | |
| *** caphrim007 has quit IRC | 04:02 | |
| *** coolsvap has joined #openstack-infra | 04:03 | |
| fungi | clarkb: to recap, let's see if we can include 292025, 519850 and 520820 before tagging git-review 1.26.0 | 04:06 |
| fungi | i'm not _too_ worried about releasing a tool like that on a friday (and i'm around all weekend if we need an emergency brownbag release) | 04:07 |
| *** eharney has quit IRC | 04:09 | |
| *** yamamoto has joined #openstack-infra | 04:10 | |
| *** yamamoto has quit IRC | 04:10 | |
| *** dave-mccowan has quit IRC | 04:19 | |
| *** dbecker has quit IRC | 04:23 | |
| *** jesusaur has joined #openstack-infra | 04:23 | |
| *** ykarel|away has joined #openstack-infra | 04:25 | |
| *** nikhil has quit IRC | 04:25 | |
| openstackgerrit | wes hayutin proposed openstack-infra/openstack-zuul-jobs master: feature027 is undercloud only, set to singlenode https://review.openstack.org/520877 | 04:29 |
| *** dbecker has joined #openstack-infra | 04:38 | |
| *** daidv has quit IRC | 04:42 | |
| *** cuongnv has quit IRC | 04:42 | |
| *** cuongnv has joined #openstack-infra | 04:42 | |
| *** daidv has joined #openstack-infra | 04:42 | |
| *** andreas_s has joined #openstack-infra | 04:50 | |
| *** udesale has joined #openstack-infra | 04:50 | |
| *** andreas_s has quit IRC | 04:54 | |
| *** jesusaurum has joined #openstack-infra | 05:00 | |
| *** kiennt26 has quit IRC | 05:01 | |
| *** jesusaur has quit IRC | 05:02 | |
| *** pgadiya has joined #openstack-infra | 05:05 | |
| *** yamamoto has joined #openstack-infra | 05:10 | |
| *** jaosorior has joined #openstack-infra | 05:12 | |
| openstackgerrit | Merged openstack-infra/project-config master: Update releasenotes publish job https://review.openstack.org/520848 | 05:15 |
| *** yamamoto has quit IRC | 05:17 | |
| *** janki has joined #openstack-infra | 05:22 | |
| *** claudiub has joined #openstack-infra | 05:22 | |
| *** hongbin has quit IRC | 05:23 | |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Split reno installation into a role https://review.openstack.org/520832 | 05:23 |
| *** armax has quit IRC | 05:23 | |
| *** Sukhdev has quit IRC | 05:26 | |
| *** gouthamr has quit IRC | 05:34 | |
| *** Sukhdev has joined #openstack-infra | 05:41 | |
| *** Sukhdev has quit IRC | 05:42 | |
| *** psachin_ has joined #openstack-infra | 06:00 | |
| *** psachin has quit IRC | 06:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack-infra/project-config master: Normalize projects.yaml https://review.openstack.org/520905 | 06:05 |
| *** rlandy|rover has quit IRC | 06:07 | |
| *** gildub has quit IRC | 06:08 | |
| AJaeger | ianw: thanks for summary and fixing! | 06:12 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Use new ensure-reno role https://review.openstack.org/520906 | 06:12 |
| AJaeger | ianw: we need now 520906 as well - since 520832 merged | 06:12 |
| *** kiennt26 has joined #openstack-infra | 06:14 | |
| AJaeger | config-core, please help reviewing ^ | 06:14 |
| *** krtaylor has quit IRC | 06:14 | |
| openstackgerrit | Merged openstack-infra/project-config master: Normalize projects.yaml https://review.openstack.org/520905 | 06:25 |
| *** slaweq has joined #openstack-infra | 06:26 | |
| *** armaan has joined #openstack-infra | 06:29 | |
| *** slaweq has quit IRC | 06:30 | |
| *** krtaylor has joined #openstack-infra | 06:31 | |
| *** aeng has quit IRC | 06:34 | |
| *** ijw has joined #openstack-infra | 06:36 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/jobs route https://review.openstack.org/503270 | 06:37 |
| *** zhurong has joined #openstack-infra | 06:41 | |
| *** ijw has quit IRC | 06:44 | |
| openstackgerrit | Rabi Mishra proposed openstack-infra/project-config master: Add jobs for heat-tempest-plugin https://review.openstack.org/520928 | 06:47 |
| *** markvoelker has quit IRC | 06:59 | |
| *** armaan has quit IRC | 07:01 | |
| openstackgerrit | zhurong proposed openstack-infra/project-config master: Remove legacy Solum jobs https://review.openstack.org/520961 | 07:02 |
| openstackgerrit | zhurong proposed openstack-infra/openstack-zuul-jobs master: Remove Solum legacy job https://review.openstack.org/520963 | 07:03 |
| *** armaan has joined #openstack-infra | 07:03 | |
| *** pcaruana has joined #openstack-infra | 07:04 | |
| *** armaan_ has joined #openstack-infra | 07:06 | |
| *** ykarel|away is now known as ykarel|lunch | 07:06 | |
| *** armaan has quit IRC | 07:09 | |
| *** ijw has joined #openstack-infra | 07:12 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/jobs route https://review.openstack.org/503270 | 07:17 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/builds route https://review.openstack.org/466561 | 07:17 |
| *** ijw has quit IRC | 07:17 | |
| *** rcernin has quit IRC | 07:18 | |
| *** esberglu has joined #openstack-infra | 07:18 | |
| *** esberglu has quit IRC | 07:19 | |
| *** esberglu has joined #openstack-infra | 07:19 | |
| *** esberglu has quit IRC | 07:19 | |
| *** psachin_ has quit IRC | 07:20 | |
| *** esberglu has joined #openstack-infra | 07:20 | |
| *** esberglu has quit IRC | 07:25 | |
| *** namnh has quit IRC | 07:27 | |
| *** andreas_s has joined #openstack-infra | 07:29 | |
| *** yamamoto has joined #openstack-infra | 07:29 | |
| *** andreas_s has quit IRC | 07:29 | |
| *** andreas_s has joined #openstack-infra | 07:29 | |
| *** psachin has joined #openstack-infra | 07:33 | |
| *** yamamoto has quit IRC | 07:34 | |
| *** yamamoto has joined #openstack-infra | 07:36 | |
| *** psachin has quit IRC | 07:38 | |
| *** psachin has joined #openstack-infra | 07:39 | |
| *** zhurong has quit IRC | 07:43 | |
| *** ijw has joined #openstack-infra | 07:47 | |
| *** kjackal has joined #openstack-infra | 07:51 | |
| *** florianf has joined #openstack-infra | 07:53 | |
| *** ijw has quit IRC | 07:53 | |
| *** kmalloc has quit IRC | 07:54 | |
| *** kjackal has quit IRC | 07:56 | |
| *** kjackal has joined #openstack-infra | 07:58 | |
| *** markvoelker has joined #openstack-infra | 08:00 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add jobs for heat-tempest-plugin https://review.openstack.org/520928 | 08:08 |
| *** pgadiya has quit IRC | 08:10 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add jobs for ansible-role-gearman https://review.openstack.org/519530 | 08:11 |
| openstackgerrit | Merged openstack-infra/project-config master: Don't index karaf logs https://review.openstack.org/520840 | 08:12 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: mqtt: Add basic reporter https://review.openstack.org/518279 | 08:21 |
| *** alexchadin has joined #openstack-infra | 08:24 | |
| *** armaan has joined #openstack-infra | 08:27 | |
| *** amoralej|off is now known as amoralej | 08:27 | |
| *** pgadiya has joined #openstack-infra | 08:27 | |
| *** ianychoi_ is now known as ianychoi | 08:27 | |
| *** dizquierdo has joined #openstack-infra | 08:30 | |
| *** armaan_ has quit IRC | 08:30 | |
| *** amotoki has joined #openstack-infra | 08:31 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci master: Import TripleO legacy zuul jobs https://review.openstack.org/514793 | 08:33 |
| *** yamamoto has quit IRC | 08:34 | |
| *** Hal has joined #openstack-infra | 08:36 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Refactor provider config to driver module https://review.openstack.org/488384 | 08:36 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement a static driver for Nodepool https://review.openstack.org/468624 | 08:36 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Implement an OpenContainer driver https://review.openstack.org/468753 | 08:36 |
| *** pgadiya has quit IRC | 08:40 | |
| *** yamamoto has joined #openstack-infra | 08:44 | |
| *** jpena|off is now known as jpena | 08:49 | |
| *** ralonsoh has joined #openstack-infra | 08:49 | |
| *** ccamacho has joined #openstack-infra | 08:49 | |
| *** yamamoto has quit IRC | 08:49 | |
| *** jpich has joined #openstack-infra | 08:51 | |
| *** pgadiya has joined #openstack-infra | 08:52 | |
| *** rossella_s has joined #openstack-infra | 08:59 | |
| openstackgerrit | Merged openstack-infra/project-config master: Use new ensure-reno role https://review.openstack.org/520906 | 09:01 |
| dirk | hwoarang: fungi : you're not hitting a connection limit, its just that rsync (which is a 2nd level server, not the primary staging server) is stuck syncing. see https://progress.opensuse.org/issues/27699 for details | 09:02 |
| *** wolverineav has joined #openstack-infra | 09:02 | |
| hwoarang | dirk: ah ok thanks! | 09:05 |
| *** alexchadin has quit IRC | 09:05 | |
| *** alexchadin has joined #openstack-infra | 09:05 | |
| *** slaweq has joined #openstack-infra | 09:06 | |
| *** pilgrimstack has joined #openstack-infra | 09:10 | |
| *** dbecker has quit IRC | 09:10 | |
| *** shardy has joined #openstack-infra | 09:11 | |
| *** yamamoto has joined #openstack-infra | 09:14 | |
| *** yamamoto has quit IRC | 09:19 | |
| *** yamamoto has joined #openstack-infra | 09:19 | |
| *** dizquierdo has quit IRC | 09:19 | |
| *** jascott1 has quit IRC | 09:19 | |
| *** jascott1 has joined #openstack-infra | 09:20 | |
| *** derekh has joined #openstack-infra | 09:20 | |
| *** rossella_s has quit IRC | 09:23 | |
| *** efoley has joined #openstack-infra | 09:23 | |
| *** jascott1 has quit IRC | 09:24 | |
| *** rossella_s has joined #openstack-infra | 09:26 | |
| *** e0ne has joined #openstack-infra | 09:29 | |
| *** lucas-afk is now known as lucasagomes | 09:33 | |
| *** hashar has joined #openstack-infra | 09:34 | |
| *** ykarel|lunch is now known as ykarel | 09:38 | |
| *** kiennt26 has quit IRC | 09:39 | |
| *** rossella_s has quit IRC | 09:45 | |
| *** rossella_s has joined #openstack-infra | 09:45 | |
| *** kjackal has quit IRC | 09:45 | |
| *** alexchadin has quit IRC | 09:46 | |
| *** alexchadin has joined #openstack-infra | 09:47 | |
| *** rossella_s has quit IRC | 09:55 | |
| *** rossella_s has joined #openstack-infra | 09:55 | |
| *** kjackal has joined #openstack-infra | 09:56 | |
| *** alexchadin has quit IRC | 09:57 | |
| *** alexchadin has joined #openstack-infra | 09:58 | |
| *** esberglu has joined #openstack-infra | 10:01 | |
| *** esberglu has quit IRC | 10:05 | |
| andreaf | AJaeger: it looks like the gate for openstack-infra/zuul-jobs is broken, I see the same failure on all patches http://logs.openstack.org/34/509234/8/check/tox-py35-on-zuul/2a5a417/job-output.txt.gz#_2017-11-17_08_15_10_116734 | 10:07 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Fix releasenotes publishing https://review.openstack.org/521048 | 10:07 |
| AJaeger | config-core, another fix needed, please review ^ | 10:07 |
| AJaeger | andreaf: no idea on that one - mordred was looking at it yesterday and confused as well, best ask him again later | 10:07 |
| andreaf | AJaeger: ok | 10:08 |
| *** liujiong has quit IRC | 10:09 | |
| *** cuongnv has quit IRC | 10:10 | |
| *** dizquierdo has joined #openstack-infra | 10:11 | |
| *** pilgrimstack has quit IRC | 10:17 | |
| *** pilgrimstack has joined #openstack-infra | 10:19 | |
| *** rossella_s has quit IRC | 10:24 | |
| *** rossella_s has joined #openstack-infra | 10:24 | |
| *** daidv has quit IRC | 10:26 | |
| rnoriega | hello guys! if you have any free slot for reviewing this: https://review.openstack.org/#/c/515408/ | 10:28 |
| rnoriega | thanks in advance!!! | 10:28 |
| *** slaweq_ has joined #openstack-infra | 10:28 | |
| *** cshastri has quit IRC | 10:31 | |
| *** LindaWang1 has joined #openstack-infra | 10:32 | |
| *** slaweq_ has quit IRC | 10:32 | |
| AJaeger | rnoriega: I was waiting for PTL review - but is neutron-l2gw an official project? | 10:33 |
| *** yamamoto has quit IRC | 10:36 | |
| *** gmann is now known as gmann_afk | 10:37 | |
| *** annp has quit IRC | 10:37 | |
| *** gildub has joined #openstack-infra | 10:43 | |
| *** yamamoto has joined #openstack-infra | 10:50 | |
| *** dbecker has joined #openstack-infra | 10:54 | |
| openstackgerrit | Ricardo Noriega proposed openstack-infra/project-config master: Add new repo for Neutron L2GW Tempest Plugin https://review.openstack.org/515408 | 10:56 |
| *** udesale has quit IRC | 10:56 | |
| *** dtantsur|afk is now known as dtantsur | 10:57 | |
| *** yamamoto has quit IRC | 10:59 | |
| *** efoley has quit IRC | 10:59 | |
| *** efoley_ has joined #openstack-infra | 10:59 | |
| frickler | AJaeger: andreaf: added some info about zuul-jobs failure to the etherpad, tldr: it fails against zuul master, passes with feature branch. branch selection appears to be random | 11:04 |
| *** alexchadin has quit IRC | 11:04 | |
| frickler | infra-root: ^^ | 11:05 |
| *** alexchadin has joined #openstack-infra | 11:05 | |
| andreaf | frickler: thanks - random branch selection sounds bad :S | 11:06 |
| *** ykarel is now known as ykarel|afk | 11:06 | |
| AJaeger | frickler: Oops ;( Thanks for investigating | 11:06 |
| *** pgadiya has quit IRC | 11:09 | |
| frickler | seems the "override-branch: feature/zuulv3" on the definition of tox-py35-on-zuul is a) deprecated and b) randomly not working | 11:10 |
| *** pgadiya has joined #openstack-infra | 11:10 | |
| openstackgerrit | Ricardo Noriega proposed openstack-infra/project-config master: Adding jobs to new networking-l2gw-tempest-plugin https://review.openstack.org/521061 | 11:12 |
| AJaeger | frickler: override-checkout is the new value but both _should_ work | 11:13 |
| *** claudiub|2 has joined #openstack-infra | 11:13 | |
| openstackgerrit | Ricardo Noriega proposed openstack-infra/project-config master: Add new repo for Neutron L2GW Tempest Plugin https://review.openstack.org/515408 | 11:13 |
| AJaeger | frickler: see also https://review.openstack.org/520236 | 11:13 |
| *** gildub has quit IRC | 11:14 | |
| *** claudiub has quit IRC | 11:15 | |
| *** dizquierdo has quit IRC | 11:16 | |
| *** alexchadin has quit IRC | 11:17 | |
| frickler | AJaeger: ah, missed a "git pull" for that, but then it looks like override-checkout may not be working better in fact, the latest failure was after that got merged | 11:20 |
| *** ldnunes has joined #openstack-infra | 11:24 | |
| *** ykarel|afk is now known as ykarel | 11:26 | |
| *** e0ne has quit IRC | 11:26 | |
| *** e0ne has joined #openstack-infra | 11:26 | |
| *** slaweq has quit IRC | 11:28 | |
| *** ihrachys has quit IRC | 11:36 | |
| *** pgadiya has quit IRC | 11:36 | |
| *** ihrachys has joined #openstack-infra | 11:36 | |
| *** armaan has quit IRC | 11:39 | |
| *** e0ne has quit IRC | 11:41 | |
| *** yamamoto has joined #openstack-infra | 11:44 | |
| *** e0ne has joined #openstack-infra | 11:44 | |
| rnoriega | AJaeger, sorry, didn't see your message | 11:48 |
| rnoriega | AJaeger, it's not under the Neutron stadium | 11:48 |
| rnoriega | AJaeger, do you ask this because maybe it's not worth to split the tempest plugin? | 11:49 |
| *** yamahata has quit IRC | 11:55 | |
| *** rhallisey has joined #openstack-infra | 12:00 | |
| *** ramishra has quit IRC | 12:08 | |
| *** dbecker has quit IRC | 12:08 | |
| *** sdague has joined #openstack-infra | 12:09 | |
| *** pbourke has quit IRC | 12:10 | |
| *** LindaWang1 has quit IRC | 12:10 | |
| *** janki has quit IRC | 12:11 | |
| *** pbourke has joined #openstack-infra | 12:12 | |
| *** dtantsur is now known as dtantsur|brb | 12:12 | |
| *** yamamoto has quit IRC | 12:13 | |
| *** udesale has joined #openstack-infra | 12:13 | |
| *** snuffkin has joined #openstack-infra | 12:20 | |
| *** shardy is now known as shardy_lunch | 12:22 | |
| AJaeger | rnoriega: for official projects we need PTL +1 - but since the repo is not an official one, we can move forward... | 12:28 |
| rnoriega | AJaeger, ack | 12:30 |
| rnoriega | thanks | 12:30 |
| *** efoley_ has quit IRC | 12:31 | |
| *** lucasagomes is now known as lucas-hungry | 12:33 | |
| *** yamamoto has joined #openstack-infra | 12:33 | |
| *** yamamoto has quit IRC | 12:37 | |
| *** smatzek has joined #openstack-infra | 12:39 | |
| *** apetrich has quit IRC | 12:41 | |
| *** clayton has quit IRC | 12:44 | |
| *** esberglu has joined #openstack-infra | 12:45 | |
| *** Douhet has quit IRC | 12:47 | |
| *** gcb has quit IRC | 12:47 | |
| *** esberglu has quit IRC | 12:49 | |
| *** jpena is now known as jpena|lunch | 12:50 | |
| *** jascott1 has joined #openstack-infra | 12:53 | |
| *** ldnunes has quit IRC | 12:54 | |
| *** efried is now known as fried_rice | 12:55 | |
| *** ldnunes has joined #openstack-infra | 12:55 | |
| *** jascott1 has quit IRC | 12:57 | |
| *** Douhet has joined #openstack-infra | 12:57 | |
| *** slaweq has joined #openstack-infra | 12:57 | |
| *** makowals has quit IRC | 12:59 | |
| AJaeger | config-core, could you review https://review.openstack.org/521048 https://review.openstack.org/520796 and https://review.openstack.org/520795 , please? Those are all for project-config. | 13:00 |
| *** ramishra has joined #openstack-infra | 13:00 | |
| *** psachin has quit IRC | 13:01 | |
| *** yamamoto has joined #openstack-infra | 13:01 | |
| *** yamamoto has quit IRC | 13:01 | |
| tbarron | manila gate is blocked due to centos7 job failures fixed by https://review.openstack.org/#/c/520837 | 13:02 |
| tbarron | fix is small & straightforward, TIA for review ^^^ | 13:02 |
| *** dtantsur|brb is now known as dtantsur | 13:03 | |
| *** yamamoto has joined #openstack-infra | 13:03 | |
| AJaeger | tbarron: that's devstack? Better ask on #openstack-qa ... | 13:04 |
| tbarron | AJaeger: ah yes, thanks. I just knew I could find ianw here last night :-) | 13:04 |
| *** cshastri has joined #openstack-infra | 13:06 | |
| *** dprince has joined #openstack-infra | 13:10 | |
| *** coolsvap has quit IRC | 13:12 | |
| *** dhill_ has quit IRC | 13:15 | |
| *** trown|outtypewww is now known as trown | 13:17 | |
| *** edmondsw has joined #openstack-infra | 13:17 | |
| *** ramishra has quit IRC | 13:17 | |
| *** shardy_lunch is now known as shardy | 13:18 | |
| *** links has quit IRC | 13:19 | |
| *** efoley has joined #openstack-infra | 13:20 | |
| *** ramishra has joined #openstack-infra | 13:22 | |
| *** rlandy has joined #openstack-infra | 13:29 | |
| *** clayton has joined #openstack-infra | 13:29 | |
| *** rlandy is now known as rlandy|rover | 13:29 | |
| *** janki has joined #openstack-infra | 13:30 | |
| *** lucas-hungry is now known as lucasagomes | 13:30 | |
| *** apetrich has joined #openstack-infra | 13:31 | |
| *** andreas_s has quit IRC | 13:33 | |
| *** andreas_s has joined #openstack-infra | 13:33 | |
| *** markvoelker has quit IRC | 13:34 | |
| *** markvoelker has joined #openstack-infra | 13:34 | |
| *** slaweq has quit IRC | 13:34 | |
| *** sbezverk has quit IRC | 13:35 | |
| *** sbezverk has joined #openstack-infra | 13:36 | |
| *** andreas_s has quit IRC | 13:38 | |
| *** clayton has quit IRC | 13:38 | |
| *** clayton has joined #openstack-infra | 13:39 | |
| *** jaypipes has joined #openstack-infra | 13:39 | |
| *** cshastri has quit IRC | 13:42 | |
| *** dklyle has quit IRC | 13:48 | |
| *** jpena|lunch is now known as jpena | 13:52 | |
| *** slaweq has joined #openstack-infra | 13:53 | |
| *** petevg is now known as petevg_afk | 13:55 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Use pip module to install reno and deps with --user https://review.openstack.org/520833 | 13:56 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Stop supporting releasenotes/requirements.txt https://review.openstack.org/520834 | 13:56 |
| *** camunoz has joined #openstack-infra | 14:01 | |
| *** dtantsur is now known as dtantsur|brb | 14:03 | |
| *** jkilpatr has quit IRC | 14:03 | |
| *** yamamoto has quit IRC | 14:04 | |
| AJaeger | mordred: could you +2A https://review.openstack.org/521048 to sync the settings for release notes, please? | 14:05 |
| *** makowals has joined #openstack-infra | 14:06 | |
| AJaeger | mordred: could you also review https://review.openstack.org/520071 , please? The translation jobs are not marked as final currently, so I expect this to work - just wonder whether there's another solution needed or whether we need to make them final... | 14:09 |
| *** dbecker has joined #openstack-infra | 14:10 | |
| *** gouthamr has joined #openstack-infra | 14:10 | |
| *** annp has joined #openstack-infra | 14:11 | |
| *** chandankumar is now known as chkumar|somewher | 14:13 | |
| *** jaypipes is now known as leakypipes | 14:13 | |
| *** ykarel is now known as ykarel|away | 14:14 | |
| *** rossella_s has quit IRC | 14:14 | |
| *** yamamoto has joined #openstack-infra | 14:15 | |
| *** jkilpatr has joined #openstack-infra | 14:16 | |
| *** rfolco is now known as rfolco_lunch | 14:17 | |
| *** rossella_s has joined #openstack-infra | 14:17 | |
| *** mriedem has joined #openstack-infra | 14:18 | |
| *** ykarel|away has quit IRC | 14:20 | |
| *** esberglu has joined #openstack-infra | 14:21 | |
| *** dave-mccowan has joined #openstack-infra | 14:21 | |
| *** annp has quit IRC | 14:22 | |
| *** CrayZee has joined #openstack-infra | 14:23 | |
| *** CrayZee has quit IRC | 14:23 | |
| *** dave-mcc_ has joined #openstack-infra | 14:24 | |
| *** mat128 has joined #openstack-infra | 14:24 | |
| *** rossella_s has quit IRC | 14:24 | |
| mordred | AJaeger: on it! | 14:25 |
| *** rossella_s has joined #openstack-infra | 14:25 | |
| mordred | AJaeger: for the translation jobs, we need the required projects because we're installing the code as part of doing the translations? | 14:26 |
| *** dave-mccowan has quit IRC | 14:26 | |
| mordred | oh - that first one makes me realize something ... | 14:27 |
| AJaeger | mmh, interesting - let me check something on translations... | 14:28 |
| AJaeger | we could change it indeed - we use " tox -e venv sphinx-build -b gettext -d releasenotes/build/doctrees" | 14:28 |
| AJaeger | and we just confirmed with releasenotes that we don't need it.... | 14:30 |
| * AJaeger will dig later into that... | 14:30 | |
| *** CrayZee has joined #openstack-infra | 14:31 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Run ensure-reno in pre, not run https://review.openstack.org/521087 | 14:31 |
| *** udesale has quit IRC | 14:31 | |
| mordred | AJaeger: ^^ the https://review.openstack.org/521048 patch above made me realize we should be doing that on the releasenotes stack | 14:31 |
| CrayZee | Hi Infra, I have a tiny patch on os-loganalyze, could anyone please review it? https://review.openstack.org/#/c/520320/ | 14:31 |
| mordred | AJaeger: yah - I think we shoudl change it to match what we're doing in releasenotes | 14:31 |
| *** smatzek has quit IRC | 14:31 | |
| openstackgerrit | Merged openstack-infra/project-config master: Fix releasenotes publishing https://review.openstack.org/521048 | 14:31 |
| *** smatzek has joined #openstack-infra | 14:32 | |
| mordred | AJaeger: you want me to do it? I haven't been super-useful recently :) | 14:32 |
| AJaeger | mordred: you need the same patch for the publish job as well | 14:32 |
| *** smatzek has quit IRC | 14:32 | |
| mordred | AJaeger: yah - coming right up | 14:32 |
| AJaeger | mordred: if you want to work on the translation jobs, would be appreciated | 14:32 |
| mordred | AJaeger: will do! | 14:32 |
| AJaeger | thanks | 14:32 |
| *** smatzek has joined #openstack-infra | 14:32 | |
| *** ramishra has quit IRC | 14:34 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Run ensure-reno in pre, not run https://review.openstack.org/521090 | 14:34 |
| *** ramishra has joined #openstack-infra | 14:35 | |
| mordred | AJaeger: the translation jobs are all in project-config right? | 14:36 |
| *** smatzek has quit IRC | 14:37 | |
| mordred | AJaeger: oh - SO ..... looking at the translations jobs, they make a venv with babel installed for doing actual translation things, but we're not installing babel for releasenotes - are we maybe missing something there? | 14:38 |
| mordred | fungi: whence you awaken, what's the process to migrate a project from launchpad to storyboard? | 14:39 |
| *** CrayZee has quit IRC | 14:39 | |
| *** CrayZee has joined #openstack-infra | 14:40 | |
| fungi | mordred: first update gerrit/projects.yaml to add the use-storyboard: true flag | 14:41 |
| fungi | mordred: once you see it appear in the projects list on sb, ssh onto storyboard.o.o and run `sudo storyboard-migrate --config-file /etc/storyboard/storyboard.conf --from-project $lp --to-project $sb` where $lp is the name of the project as lp knows it and $sb is the full repository name in gerrit (including namespace) | 14:43 |
| *** yamamoto has quit IRC | 14:44 | |
| * fungi was already awake, this has just been a busy week for not-openstack | 14:45 | |
| mordred | fungi: awesome, thanks | 14:45 |
| mordred | fungi: also, yah - this week has been a smidge brutal | 14:45 |
| fungi | well, i punted all our appointments and errands to after-summit, so this week has been devoted to catching up on all those | 14:46 |
| *** efoley_ has joined #openstack-infra | 14:46 | |
| *** efoley has quit IRC | 14:46 | |
| mordred | fungi: in good news though, we've merged the import of shade and os-client-config into openstacksdk and have also merged the main stack of invasive and breaking changes to sdk - there's still a good bit of work to do, but we seem to be in a good position for python-openstackclient to start consuming sdk - and for the shade layer to use the sdk layer for rest interactions - yay consolidation! | 14:47 |
| fungi | excellent! | 14:47 |
| fungi | hooray for better software _and_ less confusion for users | 14:47 |
| *** hongbin has joined #openstack-infra | 14:48 | |
| mordred | yah. I'm actually pretty excited about it - it's gonna give us all three interfaces - shade high-level resource-oriented, sdk OO service-oriented and direct per-service REST interactions all from one object | 14:48 |
| mordred | so it can stop being about which library you use and be about which function/method you use | 14:48 |
| fungi | (also a cli layer fairly consistent with them) | 14:49 |
| *** yamamoto has joined #openstack-infra | 14:50 | |
| mordred | ++ | 14:52 |
| AJaeger | mordred: we don't need babel for releasenotes, sphinx does it somehow magically ;) | 14:52 |
| fungi | we mainly just need babel for django-based projects, right? | 14:53 |
| AJaeger | For any project that's translated *code* | 14:53 |
| mordred | ahhhhhhh | 14:53 |
| mordred | that makes sense | 14:53 |
| AJaeger | So, releasenotes use sphinx -b gettext; code uses babel | 14:53 |
| AJaeger | sphinx -b gettext works with docs in general (releasenotes are just one variant) | 14:54 |
| fungi | so we do still need gettext, just not babel | 14:54 |
| *** yamamoto has quit IRC | 14:54 | |
| AJaeger | The translation scripts needs for e.g. horizon: sphinx with gettext for releasenotes and Babel for the code. | 14:55 |
| AJaeger | So, we need both. | 14:55 |
| fungi | i meant for rst-only content like releasenotes, specs, docs repos | 14:55 |
| AJaeger | mmh, I wonder why openstack-manuals still has Babel? Let me try building without it... | 14:56 |
| *** smatzek has joined #openstack-infra | 14:58 | |
| *** psachin has joined #openstack-infra | 14:59 | |
| *** smatzek has quit IRC | 14:59 | |
| *** yamahata has joined #openstack-infra | 14:59 | |
| *** smatzek has joined #openstack-infra | 14:59 | |
| AJaeger | fungi, mordred : Sphinx installs Babel as requirement... | 14:59 |
| *** iyamahat has joined #openstack-infra | 15:00 | |
| fungi | ahh | 15:00 |
| *** janki has quit IRC | 15:00 | |
| AJaeger | So, sphinx -b gettext uses Babel internally. And the OpenStack translations scripts call gettext commands to create e.g. from multiple pot files a single one. | 15:01 |
| *** dtantsur|brb is now known as dtantsur | 15:02 | |
| AJaeger | mordred: so, same approach as for releasenotes should work for requirements: A single venv with Sphinx, reno (use ensure-reno ;) - but for django repos, we need to install horizon as well. | 15:03 |
| *** kgiusti has joined #openstack-infra | 15:03 | |
| AJaeger | mordred: regarding https://review.openstack.org/#/c/520834 - we can remove this since no repo uses it. I added it since it's in the PTI, see my comment | 15:04 |
| mordred | AJaeger: cool - I'll send in a PTI patch | 15:04 |
| AJaeger | ;) | 15:05 |
| AJaeger | could I get a review for https://review.openstack.org/520795 and https://review.openstack.org/520796 ? These add additional required-projects that I noticed | 15:05 |
| mordred | AJaeger: so - I will now apologize - I'm about to spam you with many things to review ... | 15:05 |
| AJaeger | I'll take a break now to run some errands at home and look forward to review afterwards ;) | 15:06 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Run ensure-reno in pre, not run https://review.openstack.org/521090 | 15:06 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Use releasenotes pre playbook for translations https://review.openstack.org/521094 | 15:06 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Replace git:// with https:// https://review.openstack.org/521095 | 15:06 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Stop running tox-py35-on-zuul for now https://review.openstack.org/521096 | 15:06 |
| fungi | the release notes job changes are also being discussed in the release team meeting in #openstack-meeting right now | 15:07 |
| *** Apoorva has joined #openstack-infra | 15:08 | |
| *** eharney has joined #openstack-infra | 15:08 | |
| *** andreas_s has joined #openstack-infra | 15:09 | |
| *** dbecker has quit IRC | 15:10 | |
| *** rbrndt has joined #openstack-infra | 15:10 | |
| *** dbecker has joined #openstack-infra | 15:10 | |
| AJaeger | let me run my errands in a few minutes then... | 15:10 |
| *** armax has joined #openstack-infra | 15:11 | |
| *** udesale has joined #openstack-infra | 15:11 | |
| *** udesale has quit IRC | 15:12 | |
| *** andreas_s has quit IRC | 15:13 | |
| *** e0ne_ has joined #openstack-infra | 15:14 | |
| *** marst has joined #openstack-infra | 15:15 | |
| *** CrayZee has quit IRC | 15:15 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config master: Move TripleO jobs in-tree https://review.openstack.org/514778 | 15:15 |
| EmilienM | AJaeger, dmsimard: need re-+2 +A on ^ we had to remove a Depends-On on an abandonned patch. | 15:16 |
| AJaeger | EmilienM: no problem anymore, Zuul v3 would not block on it... | 15:16 |
| EmilienM | ok, I didn't know that | 15:16 |
| EmilienM | mwhahaha: ^ you were right - sorry. It's good to know at least... | 15:17 |
| *** e0ne has quit IRC | 15:17 | |
| odyssey4me | is it possible to have the jenkins vote removed from https://review.openstack.org/506142 as it's obscuring the zuul vote | 15:17 |
| odyssey4me | same with https://review.openstack.org/506203 | 15:17 |
| openstackgerrit | Merged openstack-infra/shade master: Cleanup objects that we create on behalf of images https://review.openstack.org/520652 | 15:17 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: ensure-reno: Install oslosphinx https://review.openstack.org/521104 | 15:18 |
| openstackgerrit | Monty Taylor proposed openstack-infra/project-config master: Use ensure-babel to make sure babel is installed https://review.openstack.org/521105 | 15:19 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add ensure-reno and ensure-babel roles https://review.openstack.org/521106 | 15:19 |
| EmilienM | mwhahaha: ^ https://review.openstack.org/#/c/521104/ fyi | 15:19 |
| fungi | odyssey4me: it won't if there's a new patchset or the change gets approved... however it may be time for us to do like we discussed pre-rollout and remove the jenkins user from the group which allows it to have verify votes. that will cause them to be hidden from all open changes | 15:19 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Make build-python-release job https://review.openstack.org/513925 | 15:19 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Remove old python-sdist job https://review.openstack.org/513926 | 15:19 |
| *** jaosorior has quit IRC | 15:19 | |
| fungi | odyssey4me: down-side is that changes which haven't been updated since the v3 rollout will need an explicit recheck before approving | 15:20 |
| odyssey4me | fungi oh you mean changes which have no zuul vote? that's fine to me, but clearly that's for my small set of patches | 15:21 |
| *** Apoorva has quit IRC | 15:21 | |
| *** bobh has joined #openstack-infra | 15:22 | |
| *** rosmaita has joined #openstack-infra | 15:23 | |
| *** david-lyle has joined #openstack-infra | 15:24 | |
| openstackgerrit | Merged openstack-infra/shade master: Add method to cleanup autocreated image objects https://review.openstack.org/520653 | 15:26 |
| *** ramishra has quit IRC | 15:28 | |
| openstackgerrit | Ronelle Landy proposed openstack-infra/tripleo-ci master: Add MTU adjustment parameters to multinode settings https://review.openstack.org/521112 | 15:30 |
| *** yamamoto has joined #openstack-infra | 15:35 | |
| *** ramishra has joined #openstack-infra | 15:38 | |
| *** yamamoto has quit IRC | 15:39 | |
| *** derekh has quit IRC | 15:40 | |
| *** ramishra has quit IRC | 15:43 | |
| openstackgerrit | Merged openstack-infra/project-config master: snaps: follow-up for snap-gnocchi creation https://review.openstack.org/518576 | 15:43 |
| openstackgerrit | Merged openstack-infra/project-config master: Fix networking-onos requirements https://review.openstack.org/520795 | 15:43 |
| openstackgerrit | Merged openstack-infra/project-config master: Fix castellan-ui https://review.openstack.org/520796 | 15:43 |
| openstackgerrit | Merged openstack-infra/project-config master: Add new repo for Neutron L2GW Tempest Plugin https://review.openstack.org/515408 | 15:43 |
| openstackgerrit | Merged openstack-infra/project-config master: Remove legacy Solum jobs https://review.openstack.org/520961 | 15:43 |
| openstackgerrit | Merged openstack-infra/project-config master: Remove release-notes variants https://review.openstack.org/520763 | 15:43 |
| *** ramishra has joined #openstack-infra | 15:45 | |
| openstackgerrit | Doug Hellmann proposed openstack-infra/openstack-zuul-jobs master: try to install more dependencies in the release notes job https://review.openstack.org/521119 | 15:51 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/openstack-zuul-jobs master: only install legacy dependencies if there is no new requirements file https://review.openstack.org/521120 | 15:51 |
| dhellmann | AJaeger , mordred , smcginnis , fungi , ttx: ^^ | 15:51 |
| *** caphrim007_ has quit IRC | 15:52 | |
| fungi | odyssey4me: i've added a topic to the infra meeting agenda to discuss dropping verify permission for the old jenkins user, which should globally solve the confusion you mentioned | 15:55 |
| EmilienM | AJaeger: could I have a review on https://review.openstack.org/#/c/514778/ again please? | 15:57 |
| EmilienM | AJaeger: all dependencies are in the gate | 15:58 |
| odyssey4me | thanks fungi - I've poked the teams for reviews, so hopefully for my patches they'll get reviewed anyway - but I expect that there may be a bunch of patches stagnating due to the same issue. | 16:00 |
| fungi | odyssey4me: yep, we basically predicted that, was just a matter of timing to figure out when the sweet spot was for dropping permissions for the old account | 16:00 |
| fungi | my opinion is now should be fine, but want to discuss when more interested parties are around and lots of people took this week as post-summit downtime | 16:01 |
| odyssey4me | fungi I'm guessing that going back to zuul v2.5 is pretty much not an option any more, so I guess it's time to say goodbye to the old friend | 16:02 |
| *** david-lyle has quit IRC | 16:02 | |
| *** david-lyle has joined #openstack-infra | 16:03 | |
| fungi | yeah, we've already resolved that we won't rollback as of a couple weeks ago. the timing was more about when a majority of active changes are likely to have gotten ci results from zuul v3's "zuul" user so that we don't leave a ton of changes without verify votes | 16:03 |
| fungi | at present zuul is considering that any verify +1 from the old jenkins user (a.k.a. zuul v2) is sufficient to be able to enqueue directly into the gate pipeline without a manual recheck | 16:04 |
| * fungi disappears for a bit to run more errands | 16:04 | |
| *** iyamahat has quit IRC | 16:05 | |
| *** iyamahat has joined #openstack-infra | 16:05 | |
| *** gyee_ has joined #openstack-infra | 16:09 | |
| *** hashar has quit IRC | 16:14 | |
| AJaeger | mordred: if you write the releasenotes email, feel free to ask me to review... | 16:15 |
| *** makowals has quit IRC | 16:19 | |
| AJaeger | mordred: do you want to merge https://review.openstack.org/#/c/521119/1/roles/ensure-reno/tasks/main.yaml - or solve it differently in your stack? | 16:19 |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Split build-wheel-mirror into publish-wheel-mirror https://review.openstack.org/521125 | 16:20 |
| *** yamamoto has joined #openstack-infra | 16:20 | |
| *** slaweq has quit IRC | 16:21 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Split build-wheel-mirror into publish-wheel-mirror https://review.openstack.org/521125 | 16:22 |
| EmilienM | in "jobs" layout, is "vars" extra-var in ansible? | 16:22 |
| *** ccamacho has quit IRC | 16:23 | |
| pabelanger | yes, which is passed via CLI | 16:24 |
| pabelanger | actually, not passed via CLI | 16:24 |
| pabelanger | just added into inventory file | 16:24 |
| AJaeger | clarkb: https://review.openstack.org/520171 *adds* files for logstash, I'm not reviewing yet... | 16:24 |
| *** yamamoto has quit IRC | 16:25 | |
| pabelanger | config-core: https://review.openstack.org/521125/ creates new publish-wheel-mirror jobs (which should release wheels to AFS) and build-wheel-mirror jobs (just build them). Allows us to now add the build-wheel-mirror job to check pipeline for openstack/requirements | 16:26 |
| *** ldnunes has quit IRC | 16:26 | |
| *** jaypipes has joined #openstack-infra | 16:27 | |
| *** leakypipes has quit IRC | 16:28 | |
| *** jaypipes is now known as leakypipes | 16:28 | |
| AJaeger | dhellmann, mordred : you have conflicting approaches to releasenotes/requirements.txt, see https://review.openstack.org/#/c/520834 and https://review.openstack.org/#/c/520834 https://review.openstack.org/#/c/520834 | 16:31 |
| AJaeger | right now *no* repo uses releasenotes/requirements.txt | 16:31 |
| AJaeger | config-core, cleanup time for releasenotes, please: https://review.openstack.org/#/c/520764/ | 16:33 |
| dhellmann | AJaeger : ah, good point. I'll rebase my patch on top of mordred's and drop the 2nd one | 16:33 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/openstack-zuul-jobs master: try to install more dependencies in the release notes job https://review.openstack.org/521119 | 16:36 |
| AJaeger | mordred: zuul-jobs looks fine now, do we really need https://review.openstack.org/521096 ? not sure why it suddenly works... | 16:36 |
| dhellmann | AJaeger : what's the deal with the --user flag in there? | 16:37 |
| dhellmann | should I be doing that instead of using a virtualenv? | 16:37 |
| AJaeger | dhellmann: yes, just gave you a -1 for that - but better ask mordred, it's his change that introduces that | 16:38 |
| *** shardy has quit IRC | 16:38 | |
| *** dhinesh has joined #openstack-infra | 16:38 | |
| AJaeger | mordred: thanks for all the changes, I think I reviewed everything now. If I missed any, please ping me - I'll be back online in an hour or so | 16:38 |
| *** jistr has quit IRC | 16:39 | |
| *** ldnunes has joined #openstack-infra | 16:39 | |
| *** shardy has joined #openstack-infra | 16:39 | |
| openstackgerrit | Doug Hellmann proposed openstack-infra/openstack-zuul-jobs master: try to install more dependencies in the release notes job https://review.openstack.org/521119 | 16:39 |
| AJaeger | EmilienM: please check https://review.openstack.org/520877 - does that conflict with your changes? | 16:39 |
| EmilienM | AJaeger: ok, in 5 min | 16:40 |
| EmilienM | pabelanger: do we have a doc that says how to run zuul v3 playbooks in a local env, let's say I want to test my changes in playbooks before pushing it | 16:41 |
| EmilienM | AJaeger: yes, don't merge that, I'll handle it myself in-tree | 16:41 |
| *** sdoran has joined #openstack-infra | 16:42 | |
| EmilienM | weshay: ^ please look https://review.openstack.org/520877 | 16:42 |
| weshay | k | 16:43 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Remove obsolete murano-firstapp docs job https://review.openstack.org/521127 | 16:43 |
| weshay | EmilienM, right.. I wasn't sure how you wanted to handle that, if we want to wait until your work is done that is fine | 16:43 |
| EmilienM | weshay: it's done now | 16:43 |
| EmilienM | weshay: send a patch to tripleo-ci | 16:43 |
| *** jistr has joined #openstack-infra | 16:43 | |
| weshay | ok.. thanks /me looks for it | 16:44 |
| AJaeger | mordred: zuul-jobs still fails ;( https://review.openstack.org/509234 | 16:44 |
| weshay | EmilienM, ok..need your help for a sec | 16:44 |
| *** iyamahat has quit IRC | 16:44 | |
| weshay | don't see the equiv line in tripleo-ci | 16:45 |
| EmilienM | weshay: in a call right now, brb | 16:45 |
| weshay | OH I see it | 16:45 |
| weshay | you already fixed it | 16:45 |
| weshay | EmilienM++ | 16:46 |
| pabelanger | EmilienM: not yet, but you can likely just run your playbooks using inventory file with localhost | 16:46 |
| EmilienM | weshay: https://github.com/openstack-infra/tripleo-ci/blob/master/zuul.d/undercloud-jobs.yaml#L24 | 16:46 |
| EmilienM | pabelanger: makes sense | 16:46 |
| pabelanger | EmilienM: that is something we still have to address | 16:46 |
| EmilienM | k | 16:46 |
| weshay | EmilienM, ya.. thanks | 16:46 |
| *** ijw has joined #openstack-infra | 16:46 | |
| *** efoley_ is now known as efoley | 16:46 | |
| * weshay is convinced EmilienM is a robot | 16:47 | |
| EmilienM | weshay: codesearch.openstack.org | 16:47 |
| weshay | ya | 16:47 |
| *** yamahata has quit IRC | 16:47 | |
| *** lucasagomes is now known as lucas-afk | 16:48 | |
| *** Apoorva has joined #openstack-infra | 16:51 | |
| *** ijw has quit IRC | 16:52 | |
| *** jascott1 has joined #openstack-infra | 16:54 | |
| *** HeOS has quit IRC | 16:57 | |
| *** jascott1 has quit IRC | 16:59 | |
| *** psachin has quit IRC | 17:02 | |
| *** iyamahat has joined #openstack-infra | 17:04 | |
| *** fried_rice is now known as fried_rolls | 17:06 | |
| *** gridinv has quit IRC | 17:06 | |
| *** Hal has quit IRC | 17:10 | |
| *** jascott1 has joined #openstack-infra | 17:11 | |
| *** iyamahat has quit IRC | 17:15 | |
| *** yamamoto has joined #openstack-infra | 17:18 | |
| *** yamamoto has quit IRC | 17:19 | |
| *** ykarel|away has joined #openstack-infra | 17:19 | |
| *** caphrim007 has joined #openstack-infra | 17:21 | |
| *** hemna_ has quit IRC | 17:23 | |
| *** shardy has quit IRC | 17:23 | |
| *** hemna_ has joined #openstack-infra | 17:23 | |
| *** shardy has joined #openstack-infra | 17:24 | |
| *** shardy has quit IRC | 17:25 | |
| *** rfolco_lunch is now known as rfolco | 17:25 | |
| *** dbecker has quit IRC | 17:26 | |
| *** andreas_s has joined #openstack-infra | 17:30 | |
| *** ralonsoh has quit IRC | 17:33 | |
| *** Apoorva has quit IRC | 17:34 | |
| *** andreas_s has quit IRC | 17:35 | |
| *** inc0 has quit IRC | 17:40 | |
| *** inc0 has joined #openstack-infra | 17:40 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Make build-python-release job https://review.openstack.org/513925 | 17:41 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Remove old python-sdist job https://review.openstack.org/513926 | 17:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Update ensure-reno to use requirements files https://review.openstack.org/521141 | 17:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 17:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Remove ensure-reno role https://review.openstack.org/521143 | 17:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Base build-openstack-releasenotes on build-reno-releasenotes https://review.openstack.org/521144 | 17:42 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Switch build-openstack-sphinx-docs to build-sphinx-docs https://review.openstack.org/521145 | 17:42 |
| *** dhinesh has quit IRC | 17:43 | |
| dhellmann | mordred : is that new set of patches in zuul-jobs a rewrite of the other releasenotes jobs? | 17:44 |
| dhellmann | I'm still a bit confused about why we have these things in so many different places | 17:44 |
| openstackgerrit | Ronelle Landy proposed openstack-infra/tripleo-ci master: Add MTU adjustment parameters to multinode settings https://review.openstack.org/521112 | 17:45 |
| *** salv-orlando has joined #openstack-infra | 17:46 | |
| *** salv-orl_ has quit IRC | 17:46 | |
| mordred | dhellmann: yes - it's part of consolidating them and collapsing them | 17:47 |
| mordred | dhellmann: there's an additional stack of work I wanted to finish this week but that's likely not going to happen so it'll be early next week now ... | 17:48 |
| *** dtantsur is now known as dtantsur|afk | 17:48 | |
| *** jpich has quit IRC | 17:48 | |
| AJaeger | dhellmann: basically zuul-jobs: Jobs that are usefull outside openstack as well; openstack-zuul-jobs: openstack specific ones - and project-config : jobs with access to secrets | 17:48 |
| mordred | dhellmann: but that will allow us to consolidate even more and only have ONE copy of a given artifact building job that we can use in both the build and the publish contexts | 17:48 |
| mordred | also - that - what AJaeger said :) | 17:48 |
| *** e0ne_ has quit IRC | 17:49 | |
| mordred | dhellmann: in zuulv3, since job config is in git, other zuul installatoins can (and are encouraged to) add https://git.openstack.org/openstack-infra/zuul-jobs to their own config - allowing us all to share job definitions as a wider zuul community - which basically makes the zuul-jobs repo like a stanadard library of generally applicable jobs | 17:50 |
| dhellmann | AJaeger, mordred : ok, maybe the thing that is tripping me up in I've seen some things defined part way between openstack-zuul-jobs and project-config | 17:51 |
| *** ykarel|away has quit IRC | 17:51 | |
| mordred | dhellmann: yah- that thing is related to how our base job and publicatoin stuff is set up righ tnow, which is the thing I'm hoping to fix soon | 17:51 |
| dhellmann | I do get the benefit of being able to share the job definitions and pushing them "upstream" as far as possible | 17:51 |
| openstackgerrit | Jamey McCabe proposed openstack-infra/irc-meetings master: New schedule and channel for LCOO meeting https://review.openstack.org/521147 | 17:52 |
| mordred | dhellmann: tl;dr we have the jobs themselves responsible for publication, which means we need secrets further down the inheritance stack which is making us need duplication between build and publish versions of things | 17:52 |
| dhellmann | could that be solved by defining the job at the most upstream level logical then inheriting from it in project-config and adding the secrets? | 17:53 |
| dhellmann | maybe that's what you're already working on | 17:53 |
| mordred | dhellmann: the change will be the definition of some interface directories on the remote nodes - and then putting the processing of fetching those things back and publishing them into the base job | 17:53 |
| mordred | dhellmann: yes, more or less, that's the idea | 17:53 |
| mordred | it's not actually too much work or too bad - it's just low enough in the stack that it needs some really careful attention and this week so far has been too distracting :) | 17:54 |
| *** jesusaurum has quit IRC | 17:54 | |
| dhellmann | yeah. maybe leaving well enough alone for a bit is a good idea. :-) | 17:55 |
| pabelanger | mordred: speaking of publishing: https://review.openstack.org/521125/ is the last step I think for our wheel-mirror build / publish. Right now everything is working on both ubuntu and centos, this just make it easier to bring online fedora / opensuse | 17:56 |
| AJaeger | mordred: nit on https://review.openstack.org/#/c/521141 | 17:57 |
| openstackgerrit | Luz Cazares proposed openstack-infra/project-config master: Add py35 jobs for refstack-client https://review.openstack.org/520814 | 17:58 |
| mordred | pabelanger: +3 | 17:58 |
| mordred | pabelanger: while I've got you ... would you mind looking at the stack that ends here: https://review.openstack.org/#/c/521096/ ? | 17:59 |
| mordred | AJaeger: good point ... will fix in a sec - maybe let's get the stop-running-tox-py35-on-zuul change landed before I push up those patches | 18:00 |
| pabelanger | mordred: sure, let me grab some coffee first | 18:00 |
| AJaeger | mordred: question on https://review.openstack.org/#/c/521142 | 18:02 |
| AJaeger | pabelanger: could you check https://review.openstack.org/521096 as well, please? | 18:03 |
| AJaeger | that blocks mordred's work | 18:03 |
| *** esberglu has quit IRC | 18:03 | |
| pabelanger | AJaeger: +3 | 18:03 |
| AJaeger | mordred: and the depends-on in https://review.openstack.org/#/c/521144 looks wrong to me | 18:04 |
| AJaeger | mordred: sorry, too many open tabs ;( | 18:04 |
| *** pilgrimstack has quit IRC | 18:05 | |
| mordred | AJaeger: tell me about it :) | 18:05 |
| openstackgerrit | Merged openstack-infra/project-config master: Split build-wheel-mirror into publish-wheel-mirror https://review.openstack.org/521125 | 18:05 |
| *** HeOS has joined #openstack-infra | 18:06 | |
| *** iyamahat has joined #openstack-infra | 18:06 | |
| *** davidlenwell_ is now known as davidlenwell | 18:06 | |
| AJaeger | pabelanger: please review https://review.openstack.org/#/c/521087/ -you just approved the project-config change and those are both needed | 18:06 |
| pabelanger | AJaeger: odd, missed them in related changes list | 18:07 |
| AJaeger | pabelanger: and the changes below it so that it can merge | 18:07 |
| pabelanger | looking | 18:07 |
| AJaeger | pabelanger: yeah, I only commented on the other part, not on the project-config one - wasn't obvious in the order you reviewed | 18:08 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/tripleo-ci master: Move scenario007-container to the gate https://review.openstack.org/521149 | 18:08 |
| *** camunoz has quit IRC | 18:08 | |
| *** lukebrowning_ has quit IRC | 18:08 | |
| openstackgerrit | Merged openstack-infra/project-config master: Run ensure-reno in pre, not run https://review.openstack.org/521090 | 18:09 |
| pabelanger | AJaeger: also, what are your thoughts on infra-check, do you think time has come to remove it? | 18:10 |
| EmilienM | pabelanger: can I get +A on https://review.openstack.org/#/c/514778/ please | 18:10 |
| EmilienM | it was approved before | 18:10 |
| EmilienM | but I removed one Depends-On | 18:10 |
| AJaeger | pabelanger: Yeah, I think we can remove it now. But we're not under heavy load either - and that's when it would be convenient ;) | 18:11 |
| AJaeger | pabelanger: I would +2 a change... | 18:11 |
| openstackgerrit | Merged openstack-infra/project-config master: Use releasenotes pre playbook for translations https://review.openstack.org/521094 | 18:11 |
| openstackgerrit | Merged openstack-infra/project-config master: Replace git:// with https:// https://review.openstack.org/521095 | 18:11 |
| openstackgerrit | Merged openstack-infra/project-config master: Stop running tox-py35-on-zuul for now https://review.openstack.org/521096 | 18:11 |
| AJaeger | mordred: I'll recheck zuul-jobs now ^ | 18:12 |
| pabelanger | AJaeger: yah, feels like we are getting into the swing of things now with project-config | 18:12 |
| *** jesusaurum has joined #openstack-infra | 18:13 | |
| *** jpena is now known as jpena|off | 18:14 | |
| *** efoley has quit IRC | 18:16 | |
| AJaeger | pabelanger: care to review some cleanups - https://review.openstack.org/520267 https://review.openstack.org/520684 https://review.openstack.org/520963 - nothing urgent... | 18:16 |
| *** Apoorva has joined #openstack-infra | 18:17 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add ensure-reno and ensure-babel roles https://review.openstack.org/521106 | 18:17 |
| *** Apoorva has quit IRC | 18:17 | |
| *** rbrndt has quit IRC | 18:18 | |
| *** Apoorva has joined #openstack-infra | 18:18 | |
| *** Swami has joined #openstack-infra | 18:19 | |
| *** yamamoto has joined #openstack-infra | 18:19 | |
| AJaeger | pabelanger, mordred, could either of you +2A https://review.openstack.org/516009 - to fix infra/publicatoins index build, please? | 18:19 |
| *** ramishra has quit IRC | 18:20 | |
| openstackgerrit | Merged openstack-infra/project-config master: Switch to pep8 jobs for cinder-specs https://review.openstack.org/520844 | 18:20 |
| *** jesusaurum has quit IRC | 18:22 | |
| *** camunoz has joined #openstack-infra | 18:22 | |
| *** HeOS has quit IRC | 18:22 | |
| *** jesusaurum has joined #openstack-infra | 18:24 | |
| AJaeger | sdague, clarkb, could you +2A this bashate change, please? https://review.openstack.org/#/c/516038/ | 18:24 |
| *** HeOS has joined #openstack-infra | 18:25 | |
| EmilienM | pabelanger: thx | 18:27 |
| *** yamamoto has quit IRC | 18:27 | |
| *** trown is now known as trown|lunch | 18:29 | |
| AJaeger | pabelanger: could you review https://review.openstack.org/#/c/516161/ - for ansible-role-shade, please? | 18:29 |
| AJaeger | config-core, we're blocked with release notes on review of https://review.openstack.org/#/c/520833 - that will allow the stack to merge | 18:30 |
| *** iyamahat has quit IRC | 18:31 | |
| AJaeger | config-core, we currently have broken release notes jobs *everywhere* and thus need the rest of mordred's stack to merge ^ | 18:31 |
| AJaeger | mordred: http://logs.openstack.org/58/520758/1/check/build-openstack-releasenotes/6660ce5/ is the failure | 18:31 |
| AJaeger | mordred: upstream-translation broken as well since only half of your changes are in - http://logs.openstack.org/48/48f35a35a47671008d60d346823bb96eb3fa0673/post/upstream-translation-update/b4129ea/ | 18:33 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add compress capabilities to stage artifacts https://review.openstack.org/509234 | 18:33 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add a generic process-test-results role https://review.openstack.org/509459 | 18:33 |
| clarkb | AJaeger: mordred where is .pip/bin or whatever --user installs to added to $PATH? | 18:33 |
| AJaeger | clarkb: haven't seen any change for that | 18:34 |
| clarkb | ok then I'm not sure just calling sphinx will work? | 18:34 |
| clarkb | since the installation wont be in the path | 18:35 |
| AJaeger | argh ;/ | 18:35 |
| AJaeger | mordred: ^ | 18:35 |
| *** iyamahat has joined #openstack-infra | 18:36 | |
| clarkb | this sort of change doesnt really fix anything either | 18:38 |
| clarkb | would be nice to focus on refactoring at the end of a fix stack | 18:38 |
| AJaeger | Is PATH set elsewhere? We install tox as user http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/ensure-tox/tasks/main.yaml | 18:39 |
| pabelanger | AJaeger: no | 18:45 |
| pabelanger | so | 18:45 |
| pabelanger | --user might cause path issues on CentOS | 18:45 |
| pabelanger | wait | 18:45 |
| pabelanger | no, tox works today | 18:46 |
| pabelanger | so, --user must be fine | 18:46 |
| *** camunoz_ has joined #openstack-infra | 18:46 | |
| *** camunoz has quit IRC | 18:47 | |
| AJaeger | pabelanger: or do we install tox on our images so that --user is not used? | 18:48 |
| openstackgerrit | Tom Barron proposed openstack-infra/project-config master: Switch to pep8 jobs for manila-specs https://review.openstack.org/521159 | 18:50 |
| pabelanger | AJaeger: ya, we pip install tox with DIB | 18:52 |
| pabelanger | so, we'd never use --user | 18:52 |
| sc` | when i approve a change, it seems to be needing some "fiddling" to get it to merge. is there something i should be doing that i'm not? | 18:52 |
| pabelanger | so, possible new reno changes will break with --user | 18:52 |
| pabelanger | we'll find out soon | 18:53 |
| sc` | i'll approve the change and it won't show up on the zuul status page | 18:53 |
| AJaeger | sc`: give us an example... | 18:53 |
| pabelanger | sc`: which change? | 18:53 |
| AJaeger | sc`: before you fiddle with it, please ;) | 18:53 |
| sc` | currently https://review.openstack.org/#/c/519375/ but i have several more | 18:53 |
| sc` | i've been going down the changes one at a time, and endured a little pain yesterday just to confirm it this morning | 18:54 |
| AJaeger | sc`: https://review.openstack.org/#/c/519374 needs to merge first | 18:54 |
| AJaeger | sc`: those are stacked on top of each other | 18:54 |
| AJaeger | sc`: see the "related chagnes" on the upper right | 18:54 |
| sc` | ah, right. i don't usually see stacked changes like that. i haven't done that in a while | 18:54 |
| sc` | thanks | 18:55 |
| AJaeger | sc`: if something doesn't merge, thinks to check first: 1) Stacked changes? 2) Depends-on - and 3) merge conflict | 18:55 |
| *** dhill_ has joined #openstack-infra | 18:56 | |
| AJaeger | sc`: you can leave the +A on the change, once 374 merges, 375 will merge as well... | 18:56 |
| sc` | yeah. when i learned to not stack changes, i almost never saw it in openstack-chef | 18:56 |
| sc` | i'm still relearning zuul | 18:56 |
| AJaeger | sc`: that's all gerrit ;) | 18:57 |
| sc` | also publicly-run gerrit | 18:57 |
| mordred | AJaeger: ooh - great question re: warning-is-error - lemme check | 18:58 |
| sc` | every time i make a change, it's on a fresh clone, so i don't see stacked changes in the gerrit instances i frequent too often. i hadn't seen stacked changes in openstack-chef in a couple years | 18:59 |
| *** iyamahat has quit IRC | 19:00 | |
| *** iyamahat has joined #openstack-infra | 19:00 | |
| AJaeger | mordred: we can postpone - let's figure out how to handle --user first and merge the rest of your stuff. | 19:00 |
| AJaeger | mordred: right now propose-translations and releasenote jobs are broken, let's give that prio | 19:00 |
| AJaeger | please | 19:00 |
| *** e0ne has joined #openstack-infra | 19:03 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: WIP: Convert from legacy to native devstack job https://review.openstack.org/520664 | 19:05 |
| mordred | AJaeger: yah, totally agree | 19:06 |
| mordred | AJaeger: I made a few more changes to the ensure-sphinx patch ... I will now apologize for it being so many files - but most of them are small | 19:07 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Update ensure-reno to use requirements files https://review.openstack.org/521141 | 19:08 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 19:08 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Make build-python-release job https://review.openstack.org/513925 | 19:08 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Remove old python-sdist job https://review.openstack.org/513926 | 19:08 |
| mordred | dhellmann: you know all the things about sphinx ... | 19:09 |
| mordred | dhellmann: our current practive of setting warning-is-error in setup.cfg ... is there a good way to do that if we're using sphinx-build and not python setup.py build_sphinx ? | 19:10 |
| AJaeger | mordred: you're not installing openstackdocstheme anymore in https://review.openstack.org/#/c/521142 - is that intented? | 19:12 |
| *** iyamahat has quit IRC | 19:13 | |
| mordred | AJaeger: yah - it should be being picked up because we're looking for doc/requirements.txt and then test-requirements.txt | 19:13 |
| mordred | the current projects should all have it in test-requirements.txt - but once people start using doc/requirements.txt that should also have openstackdocstheme in it, yeah? | 19:14 |
| *** rbrndt has joined #openstack-infra | 19:14 | |
| AJaeger | mordred: unfortunatley a few repos have it in requirements.txt, like charm-deployment-guide/requirements.txt or contributor-guide/requirements.txt - but that's just a handful... | 19:15 |
| *** e0ne has quit IRC | 19:15 | |
| fungi | infra-root: any chance i could get a couple quick reviews on system-config change https://review.openstack.org/519468 to add a new mailing list? | 19:16 |
| mordred | yah - well, at the moment those should also be fine because we're pip installing the repo itself which should also get requirements.txt | 19:16 |
| smcginnis | AJaeger, mordred: I just updated cinder-specs and found we only have a requirements.txt file there, no test-requirements.txt. | 19:16 |
| AJaeger | mordred: ah, ok | 19:16 |
| *** dhinesh has joined #openstack-infra | 19:16 | |
| *** pilgrimstack has joined #openstack-infra | 19:17 | |
| *** fried_rolls is now known as fried_rice | 19:17 | |
| mordred | smcginnis: that should be fine for now - we should probably add a doc/requirements.txt file at some point | 19:17 |
| smcginnis | mordred: I'll take care of that soon. Or soon-ish. | 19:18 |
| smcginnis | mordred: Oh, interesting. Just saw this though: https://review.openstack.org/#/c/468882/ | 19:19 |
| smcginnis | That should probably be reverted then? | 19:19 |
| mordred | we should probably make sure all the specs repos have a doc/requirements.txt - and then add a vars: install_package: false to the openstack-specs-jobs project-template for build-openstack-sphinx-jobs ... there's no need to try to install python code in the specs repos even if they do have a setup.cfg file | 19:19 |
| mordred | smcginnis: yah - I mean, it's not strictly necessary - it's more to enable not needing sphinx in the normal test-requirements (no need for sphinx to run unit tests) - and to not need potentially expensive test requirements for building docs ... | 19:21 |
| *** marst has quit IRC | 19:21 | |
| *** marst_ has joined #openstack-infra | 19:21 | |
| AJaeger | smcginnis: sphinx < 1.2? Don't revert directly, please | 19:23 |
| mordred | smcginnis: maybe once we get this stack landed we can propose a revert ... this makes me think - we should also update the update-requirements and check-requirements jobs to grok doc/requirements.txt too | 19:23 |
| mordred | AJaeger: yah | 19:23 |
| AJaeger | mordred: yes, indeed, we need to add it everywhere | 19:23 |
| AJaeger | mordred: part of your recent changes failed tests ;( | 19:24 |
| mordred | AJaeger: I think we can wait til next week to work on that - it's an optimization, the current stuff should all work with test-requirements.txt anyway | 19:24 |
| mordred | AJaeger: BOOO | 19:24 |
| AJaeger | mordred: yeah, one step at a time | 19:24 |
| *** marst_ has quit IRC | 19:27 | |
| fungi | clarkb: not sure if you saw my response last night, but let's see if we can include 292025, 519850 and 520820 before tagging git-review 1.26.0 | 19:30 |
| fungi | i'm not _too_ worried about releasing a tool like that on a friday (and i'm around all weekend if we need an emergency brownbag release) | 19:30 |
| *** amoralej is now known as amoralej|off | 19:34 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 19:34 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Make build-python-release job https://review.openstack.org/513925 | 19:34 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Remove old python-sdist job https://review.openstack.org/513926 | 19:34 |
| *** gouthamr has quit IRC | 19:35 | |
| *** Sukhdev has joined #openstack-infra | 19:38 | |
| AJaeger | mordred: which changes need to merge next to unbreak releasenotes building? | 19:39 |
| AJaeger | http://logs.openstack.org/86/520986/1/gate/build-openstack-releasenotes/4b5c766/job-output.txt.gz#_2017-11-17_18_39_55_266507 | 19:41 |
| mordred | AJaeger: looking | 19:43 |
| mordred | AJaeger: I believe https://review.openstack.org/#/c/521144/ ... maybe I should use that patch with a depends-on to verify? | 19:44 |
| AJaeger | mordred: just create a simple change that touches releasenotes directory? | 19:45 |
| mordred | AJaeger: k. will do. one sec. | 19:46 |
| EmilienM | fungi, clarkb: do you have any early feedback on how much storage consume tripleo CI regarding logs, we reduced again the amount of files lately. | 19:46 |
| AJaeger | mordred: that would tell us also whether --user works - since https://review.openstack.org/#/c/520833/ is the bottom of that stack | 19:46 |
| mordred | remote: https://review.openstack.org/521178 DNM Testing releasenotes job | 19:47 |
| mordred | AJaeger: ^^ | 19:47 |
| *** dprince has quit IRC | 19:48 | |
| fungi | EmilienM: we could do a spot check, but to get a representative sample across projects takes a while since it involves scanning the filesystem | 19:49 |
| fungi | i'll see what i can come up with | 19:49 |
| AJaeger | mordred: "error" in zuulv3.openstack.org for releasenotes job ;( | 19:50 |
| EmilienM | fungi: no problem | 19:51 |
| EmilienM | fungi: just let us know one day when you have numbers | 19:51 |
| AJaeger | mordred: I'm soon calling it a day - i suggest to not leave this broken, so either revert the changes to project-config for now or get a working set in... | 19:51 |
| EmilienM | fungi: we want to know if things are making progress | 19:51 |
| *** trown|lunch is now known as trown | 19:51 | |
| *** rosmaita has quit IRC | 19:52 | |
| dmsimard | Curious, what filesystem is logs.o.o running ? ext3? ext4? | 19:53 |
| *** iyamahat has joined #openstack-infra | 19:54 | |
| fungi | /dev/mapper/main-logs on /srv/static/logs type ext4 (rw,errors=remount-ro,barrier=0) | 19:54 |
| mordred | AJaeger: yah - I'm not going to stop until it's working | 19:54 |
| AJaeger | thanks, mordred ! | 19:55 |
| mordred | AJaeger: before you go - openstack/i18n uses a job called publish-openstack-sphinx-tox-docs - but uses build-openstack-sphinx-docs template for check/gate | 19:55 |
| mordred | AJaeger: do you know why or if that's intentional? | 19:55 |
| dmsimard | fungi: thanks! | 19:55 |
| dmsimard | fungi: the nobarrier is interesting.. has that bitten you before ? | 19:56 |
| *** jascott1_ has joined #openstack-infra | 19:56 | |
| *** iyamahat_ has joined #openstack-infra | 19:56 | |
| fungi | dmsimard: it's default for the rootfs on rackspace's ubuntu images, so we just cargo cult it assuming that it's to work around a lack of write caching on their side | 19:57 |
| AJaeger | mordred: for i18n, we cannot use sphinx-build as is - they translate the document as well. thus, we call tox -e docs instead | 19:57 |
| dmsimard | fungi: ok, thanks :) | 19:57 |
| AJaeger | http://git.openstack.org/cgit/openstack/i18n/tree/tox.ini#n25 | 19:58 |
| AJaeger | http://git.openstack.org/cgit/openstack/i18n/tree/tools/build-docs.sh is the file used for building | 19:58 |
| *** jascott1 has quit IRC | 19:58 | |
| fungi | er, i guess it's the other way around. barrier=0 (nobarrier) means we don't care what writes get cached | 19:58 |
| fungi | dmsimard: ^ | 19:58 |
| dmsimard | fungi: it disables the barrier, yeah | 19:58 |
| fungi | so presumably a performance boost at the potential cost of filesystem corruption | 19:59 |
| AJaeger | mordred: does that help? | 19:59 |
| fungi | anyway, rax uses it for their rootfs, and we just copy that for other filesystems | 19:59 |
| *** iyamahat has quit IRC | 20:00 | |
| AJaeger | mordred: the i18n team considers translating other manuals as well, so we might need to generalize this ;( | 20:00 |
| dmsimard | fungi: yeah, that's why I asked if it had bitten you ebfore | 20:01 |
| *** Apoorva has quit IRC | 20:01 | |
| *** iyamahat_ has quit IRC | 20:03 | |
| *** slaweq has joined #openstack-infra | 20:05 | |
| dhellmann | mordred : the -W option to sphinx-build turns that on but I'm not sure if there's a way to do it from within conf.py | 20:06 |
| *** iyamahat has joined #openstack-infra | 20:08 | |
| fungi | dhellmann: mordred: looks like build_sphinx.warning-is-error=1 in setup.cfg was added for sphinx >=1.5 | 20:11 |
| fungi | https://github.com/sphinx-doc/sphinx/commit/a9ffcf0 | 20:12 |
| fungi | first appears in the 1.5a1 tag | 20:12 |
| AJaeger | fungi, dhellmann: Challenge is with going from "python setup.py build_sphinx" to "sphinx-build ..." | 20:13 |
| fungi | as a means to control it without having to pass -W | 20:13 |
| AJaeger | the warning-is-error does not work with sphinx-build AFAIK | 20:13 |
| AJaeger | and mordred wants to use sphinx-build | 20:14 |
| fungi | looks like it should work if you add warning-is-error=1 to the [build_sphinx] section of setup.cfg and use sphinx 1.5 or later | 20:14 |
| *** makowals has joined #openstack-infra | 20:15 | |
| fungi | ahh, build-sphinx | 20:15 |
| fungi | er, sphinx-build | 20:15 |
| *** ldnunes has quit IRC | 20:15 | |
| *** jesusaurum has quit IRC | 20:15 | |
| AJaeger | fungi: https://review.openstack.org/#/c/521142/3/roles/sphinx/tasks/main.yaml - line 4 | 20:15 |
| *** HeOS has quit IRC | 20:15 | |
| fungi | yeah, may not be supported | 20:16 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Base build-openstack-releasenotes on build-reno-releasenotes https://review.openstack.org/521144 | 20:16 |
| AJaeger | fungi: I expect that this one will not respect the warning-is-error, so we need some way to set -W | 20:16 |
| *** jesusaurum has joined #openstack-infra | 20:17 | |
| AJaeger | maybe use python setup.py build_sphinx there for now and postpone this problem | 20:17 |
| *** bobh has quit IRC | 20:17 | |
| *** jesusaurum is now known as jesusaur | 20:17 | |
| mordred | well, we need to solve it for non-python anyway. it's easy enough to solve it in the gate jobs - but I'm not thrilled about having it be different when people run locally | 20:18 |
| *** rfolco has quit IRC | 20:18 | |
| fungi | sphinx release notes say 1.6.4 added an autodoc_warningiserror option | 20:19 |
| fungi | though that looks like just for overriding it in autodoc when you've already passed -W | 20:19 |
| mordred | fungi: yah - I think that's just for autodoc... there's also a config option to enable the nitpicky warnings ... there's just not conf.py setting for warnings-is-errors itself :( | 20:19 |
| *** jesusaur has quit IRC | 20:20 | |
| fungi | worthy of a bug report against sphinx i suppose... and a pr | 20:20 |
| mordred | yah | 20:20 |
| AJaeger | fungi: " Add autodoc_warningiserror to suppress the behavior of -W option during importing target modules on autodoc" - so that's the opposite of what we want and very specific | 20:21 |
| pabelanger | woot: http://zuulv3.openstack.org/static/stream.html?uuid=d8353d760b5140a1ba0d95fe309dc42c&logfile=console.log | 20:21 |
| pabelanger | check job for build-wheel-mirrors | 20:21 |
| mordred | pabelanger: woot! | 20:21 |
| openstackgerrit | Merged openstack-infra/system-config master: Add edge-computing mailing list https://review.openstack.org/519468 | 20:21 |
| pabelanger | <3 zuulv3 | 20:21 |
| mordred | AJaeger: http://zuulv3.openstack.org/static/stream.html?uuid=f56c3d2ab51a4915a17533d2cb60463e&logfile=console.log - build-openstack-releasenotes building | 20:21 |
| *** iyamahat has quit IRC | 20:21 | |
| * AJaeger watches ^ | 20:21 | |
| *** jesusaur has joined #openstack-infra | 20:22 | |
| pabelanger | okay, running errand for a bit, back later this evening | 20:22 |
| AJaeger | pabelanger: woot! yes, testing jobs is GREAT! | 20:22 |
| AJaeger | mordred: post failure? It's finished but queued again ;( | 20:24 |
| AJaeger | mordred: http://paste.openstack.org/show/626660/ - broken ansible ;( | 20:25 |
| AJaeger | mordred and also "stderr: Could not open requirements file: [Errno 2] No such file or directory: 'src/git.openstack.org/openstack/neutron/test-requirements.txt'" | 20:26 |
| * AJaeger waves good bye | 20:26 | |
| AJaeger | everybody have a great evening and weekend | 20:27 |
| fungi | have a good weekend AJaeger! | 20:27 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/system-config master: WIP: Add subunit2sql check database server and workers https://review.openstack.org/520192 | 20:29 |
| mtreinish | clarkb, fungi, mordred: ^^^ I think that's the right direction for setting up the second db for check queue data | 20:30 |
| mtreinish | my only open question is how to define the dependency on the upstream mysql puppet module | 20:30 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 20:31 |
| *** pilgrimstack has quit IRC | 20:33 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 20:35 |
| clarkb | fungi: ianw for 519850 do we expect that to break windows users if they don't have python 3.6 and have non ascii content? | 20:36 |
| mordred | mtreinish: modules.env in the root of system-config has the puppet module depends in it | 20:36 |
| mordred | mtreinish: we've got SOURCE_MODULES["https://github.com/puppetlabs/puppetlabs-mysql"]="3.11.0" currently | 20:36 |
| clarkb | mtreinish: mordred and individual modules can list their deps in the module json file | 20:36 |
| fungi | clarkb: only if they're running from tip of master. it's basically reverting a change which merged since the last release | 20:37 |
| clarkb | ah | 20:37 |
| fungi | which is why i'm in favor of merging before releasing | 20:37 |
| clarkb | in that case docs say use python3.6 sound good to me | 20:37 |
| fungi | so we don't release a regression | 20:37 |
| mordred | ++ | 20:37 |
| fungi | in fairness, when the change originally merged, python 3.6 did not yet eexist | 20:38 |
| fungi | exist | 20:38 |
| openstackgerrit | Jamey McCabe proposed openstack-infra/irc-meetings master: Updating LCOO schedule to new time and channel https://review.openstack.org/517449 | 20:38 |
| fungi | now that it does, that's a superior solution | 20:38 |
| clarkb | I take it rhbz is red hat bugzilla? | 20:40 |
| mtreinish | mordred, clarkb: cool, thanks | 20:40 |
| mtreinish | mordred: hmm, I building it with the docs for 5.1.0 (which is the latest version) | 20:41 |
| *** fultonj has quit IRC | 20:41 | |
| mtreinish | clarkb: well there isn't a separate module in the case, I'm just calling it in the openstack_project definition I added for subunit2sql db server | 20:41 |
| clarkb | fungi: ok I've approved the three git-review changes for making a release | 20:41 |
| *** dprince has joined #openstack-infra | 20:41 | |
| clarkb | mtreinish: ah in that case ya what mordred points at is what covers you | 20:42 |
| mtreinish | mordred: oh, it looks like they changed the name of it at some point: https://forge.puppet.com/puppetlabs/mysql | 20:43 |
| mtreinish | (it's not puppetlabs-mysql anymore) | 20:43 |
| mtreinish | oh, that's the git repo. nm, I just can't read | 20:44 |
| *** makowals has quit IRC | 20:44 | |
| fungi | clarkb: yeah, https://bugzilla.redhat.com/show_bug.cgi?id=1058167 | 20:44 |
| openstack | bugzilla.redhat.com bug 1058167 in git-review ""git review --setup" failed in Chinese locale" [Unspecified,Closed: errata] - Assigned to zaitcev | 20:44 |
| clarkb | fungi: we aren't regressing to that behavior here are we? | 20:45 |
| fungi | clarkb: not if git-review is installed under python >=3.6 aiui | 20:46 |
| clarkb | ya thats fine for windows because you just install active python or whatever, but on linux where you get distro python its more problematic | 20:47 |
| * clarkb looks to see what raises _GitReviewException | 20:48 | |
| clarkb | er no _ | 20:48 |
| mordred | clarkb: wow. did you ever think you'd say "it's fine on windows but on linux it's more problematic" ? | 20:48 |
| mtreinish | fungi, clarkb: I have python 3.6.3 installed on my systems... | 20:48 |
| fungi | clarkb: oh, you mean the fedora bug. i don't believe so, the commit message cites a subsequent change to fix that to stick with C locale | 20:49 |
| clarkb | fungi: ya reading more I think the universal_newlines implies its already been converted to the current locale | 20:50 |
| openstackgerrit | Merged openstack-infra/git-review master: Add a note about contribution priorities https://review.openstack.org/292025 | 20:50 |
| clarkb | and since those exceptions all come from the command execution we should be fine | 20:51 |
| *** Sukhdev has quit IRC | 20:51 | |
| clarkb | oh except | 20:51 |
| clarkb | hrm C is not utf8 friendly | 20:51 |
| *** eharney has quit IRC | 20:51 | |
| clarkb | which may be the underlying issue? | 20:51 |
| fungi | mtreinish: i have 2.7.14, 3.3.7, 3.4.7, 3.5.4, 3.6.3 and 3.7.0a2 installed on mine | 20:51 |
| clarkb | universal_newlines is going to assume ascii because we run under C? | 20:52 |
| ianw | clarkb: 519850 breaking windows ... maybe. tbh i'm not sure how good it was anyway; i'm guessing it might have just outputted encoded junk, nothing readable. is that better than a traceback anyway? | 20:52 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 20:52 |
| fungi | clarkb: the way i read it commands are executed under c, and then strings are passed to print() | 20:52 |
| clarkb | fungi: right but those strings will be ascii encoded because LANG is C? | 20:52 |
| clarkb | so we may just throw exceptions earlier when python tries to ingest the command output if it isn't valid ascii? | 20:53 |
| fungi | clarkb: but ascii is a subset of any the local encoding | 20:53 |
| mtreinish | fungi: oh, I meant as /usr/bin/python (sigh, arch) | 20:53 |
| clarkb | fungi: right, the problem is with universal_newlines and how it knows what to encode to though. Its uses your current locale iirc | 20:53 |
| mtreinish | fungi: how's the 3.7 alpha? | 20:54 |
| fungi | mtreinish: ahh, then yes my ~/bin/python3 i 3.3.6 but my ~/bin/python is 2.7.14 | 20:54 |
| clarkb | though C is only set on the subprocess so it may work based on the current parent process' locale? | 20:54 |
| fungi | mtreinish: i don't do much fancy with it, so no surprises thus far | 20:54 |
| fungi | clarkb: that's what i assumed... dunno what non-ascii could make it into the subprocess strings | 20:55 |
| clarkb | fungi: branch names, commit messages, etc | 20:55 |
| mordred | usernames in commit headers | 20:55 |
| fungi | yeah, local paths too | 20:55 |
| *** mat128 has quit IRC | 20:55 | |
| fungi | no idea how much of that we test | 20:56 |
| clarkb | so while git itself will only stick to C/ascii the content it writes may not | 20:56 |
| ianw | yes, that LANG setting is i think only for git itself, but when it outputs a review that's all in madarin or something, it's not translating | 20:56 |
| fungi | so results in mojibake or exceptions? | 20:57 |
| ianw | at the very best, i think it results in output that looks like https://imgur.com/a/kyN1U | 20:58 |
| ianw | that was when i was playing around with https://review.openstack.org/519797 | 20:59 |
| clarkb | I just tried by setting gitreview.username to snowman and that seems to have not exploded other than not being a valid gerrit username so now I have to entrer another one | 21:00 |
| *** iyamahat has joined #openstack-infra | 21:00 | |
| *** Apoorva has joined #openstack-infra | 21:01 | |
| ianw | brb, migrating my linode | 21:01 |
| *** ianw has quit IRC | 21:02 | |
| *** camunoz_ has quit IRC | 21:03 | |
| *** wolverineav has quit IRC | 21:05 | |
| *** wolverineav has joined #openstack-infra | 21:06 | |
| clarkb | I'm guessing that means the parent python process' locale is used to figure out the encoding and not the subprocess | 21:06 |
| clarkb | with universal newlines | 21:06 |
| openstackgerrit | Merged openstack-infra/git-review master: Fix output printing with python3 https://review.openstack.org/519850 | 21:07 |
| openstackgerrit | Merged openstack-infra/git-review master: Add a note on Windows and Unicode https://review.openstack.org/520820 | 21:07 |
| *** smatzek has quit IRC | 21:09 | |
| *** ianw has joined #openstack-infra | 21:09 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 21:09 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Switch build-openstack-sphinx-docs to build-sphinx-docs https://review.openstack.org/521145 | 21:10 |
| * mordred bangs head against computer | 21:10 | |
| *** tobiash has quit IRC | 21:13 | |
| bkero | mordred: you seem like you're having fun | 21:15 |
| fungi | metal health'll drive you mad | 21:16 |
| * fungi quietly riots in the corner | 21:17 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 21:17 |
| *** tobiash has joined #openstack-infra | 21:18 | |
| mordred | bkero: SO MUCH FUN | 21:18 |
| Shrews | fungi: i feel your noise | 21:18 |
| fungi | i don't doubt it | 21:18 |
| *** tobiash has quit IRC | 21:19 | |
| mordred | Shrews: I like to bring in da noise myself | 21:19 |
| *** tobiash has joined #openstack-infra | 21:21 | |
| fungi | how low can you go? | 21:21 |
| *** tobiash has quit IRC | 21:21 | |
| bkero | That song so good | 21:23 |
| * fungi enjoys the benassi mix of that | 21:23 | |
| clarkb | fungi: ianw my local testing seems to indicate its fine, however then I realized that I may be running it under python 3.6 :) | 21:23 |
| mordred | clarkb: heh | 21:24 |
| clarkb | so uh ya let me try this again | 21:24 |
| *** rlandy|rover has quit IRC | 21:24 | |
| clarkb | any other infra-root willing to review https://review.openstack.org/#/c/520672/ to help address logstash worker memory consumption? | 21:24 |
| fungi | lookin | 21:25 |
| fungi | ltm | 21:26 |
| fungi | lGtm | 21:26 |
| clarkb | ok python2.7 also seems happy | 21:28 |
| *** kjackal has quit IRC | 21:28 | |
| clarkb | via setting gitreview.username to snowman then running git review -s | 21:28 |
| clarkb | and confirmed that LANG=C git config --get gitreview.username returns snowman as is and doesn't do anything silly | 21:29 |
| clarkb | so I think we are ok to release git review as is? | 21:29 |
| fungi | yeah, happy to tag if there are no objections | 21:30 |
| *** Apoorva_ has joined #openstack-infra | 21:30 | |
| *** tobiash has joined #openstack-infra | 21:30 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 21:32 |
| *** salv-orl_ has joined #openstack-infra | 21:32 | |
| mordred | fungi, clarkb: sounds good ot me | 21:32 |
| *** Apoorva has quit IRC | 21:33 | |
| *** HeOS has joined #openstack-infra | 21:34 | |
| *** salv-orlando has quit IRC | 21:35 | |
| *** jascott1_ has quit IRC | 21:36 | |
| *** reed has quit IRC | 21:36 | |
| *** reed has joined #openstack-infra | 21:39 | |
| dmsimard | clarkb: <friday> who was a fan of Waffle House again ? https://i.redd.it/kmzap2dmflyz.jpg </friday> | 21:39 |
| *** HeOS has quit IRC | 21:39 | |
| *** jascott1 has joined #openstack-infra | 21:40 | |
| *** eharney has joined #openstack-infra | 21:42 | |
| *** andreas_s has joined #openstack-infra | 21:43 | |
| *** andreas_s has quit IRC | 21:47 | |
| *** florianf has quit IRC | 21:48 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 21:50 |
| dhellmann | hey, folks, I'm getting tons of spam email from @qq.com addresses to mailing list admin addresses again. I've set up a local filter, but I wonder if we can turn on whatever filtering we were doing on the list server again? | 21:50 |
| *** iyamahat has quit IRC | 21:50 | |
| *** trown is now known as trown|outtypewww | 21:51 | |
| dhellmann | fungi : ^^ if you're still online this late on a friday | 21:52 |
| *** Sukhdev has joined #openstack-infra | 21:53 | |
| clarkb | dhellmann: fungi says it should be fixed now in another channel | 21:55 |
| *** slaweq has quit IRC | 21:55 | |
| *** slaweq has joined #openstack-infra | 21:55 | |
| fungi | infra-puppet-core: anyone willing to approve https://review.openstack.org/484989 for puppet-exim? i just accidentally undid my hand-patching of that and accidentally allowed a few thousand spams through to list owners | 21:55 |
| fungi | dhellmann: what's insane is that i only inadvertently turned that off for a few minutes, and they're still bombing those addresses months after i started blackholing them all | 21:57 |
| clarkb | fungi: change is hand applied? | 21:57 |
| *** slaweq has quit IRC | 21:59 | |
| *** HeOS has joined #openstack-infra | 22:02 | |
| *** jtomasek has quit IRC | 22:02 | |
| clarkb | fungi: I've approved it largely on jeblairs +2 and the commit message making sense | 22:02 |
| fungi | yup | 22:03 |
| fungi | i've reapplied it now | 22:04 |
| *** Goneri has joined #openstack-infra | 22:04 | |
| fungi | i accidentally unapplied it when manually running kick.sh to get that new edge-computing ml created a few minutes ago | 22:04 |
| fungi | i had the presence of mind to copy and replace /etc/aliases but didn't realize the ordering change was still sitting out ther | 22:05 |
| fungi | e | 22:05 |
| smcginnis | Can someone tell me why this patch got a zuul +2 but is still Ready To Submit? | 22:06 |
| *** wolverineav has quit IRC | 22:06 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 22:08 |
| mordred | smcginnis: maybe - but you'll have to tell us which patch first | 22:09 |
| smcginnis | Hah! | 22:11 |
| smcginnis | https://review.openstack.org/#/c/481989/ | 22:11 |
| smcginnis | mordred: I suppose that might help. :) | 22:11 |
| mordred | smcginnis: how interesting ... | 22:13 |
| smcginnis | mordred: That's one way to put it I guess. | 22:13 |
| clarkb | its because its parent is patchset one of a multipled patchset chnage | 22:13 |
| clarkb | you can't merge if parented to an old patchset since old patchsets can never merge | 22:14 |
| clarkb | the little orange dot next to the parent sha1 tried to explain this if you hover on it | 22:14 |
| smcginnis | clarkb: Why doesn't it show a merge conflict like normal cases? | 22:14 |
| smcginnis | clarkb: You are saying it needs a rebase, right? | 22:14 |
| clarkb | because it isn't a merge conflict | 22:14 |
| clarkb | it is parented to something that will never merge | 22:15 |
| clarkb | which is different | 22:15 |
| smcginnis | Hmm, if I look up the parent commit, it is merged. | 22:15 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 22:15 |
| clarkb | smcginnis: patchset 2 merged which is a different commit | 22:15 |
| clarkb | compare the sha1s | 22:15 |
| clarkb | patchset 1 is the sha1 your commit is parented to | 22:15 |
| fungi | yeah, this is gerrit being picky basically | 22:16 |
| smcginnis | Super confusing. Wouldn't it just rebase it to the newer parent? | 22:16 |
| openstackgerrit | Alex Schultz proposed openstack-infra/tripleo-ci master: Move scenario003 to non-voting https://review.openstack.org/521205 | 22:16 |
| dhellmann | clarkb , fungi : thanks! | 22:17 |
| clarkb | smcginnis: gerrit doesn't make new commits for you | 22:17 |
| clarkb | smcginnis: there are cases where that wouldn't work, like if there was also a merge conflict | 22:17 |
| clarkb | or if the underlying patchsets changed enough that to two are no longer compatible | 22:18 |
| smcginnis | clarkb: I guess I still don't understand why this fails since we have other times where there are dependent patches and the parent patch goes through updates before merging. | 22:18 |
| smcginnis | Seems like it should be a more common issue then. | 22:18 |
| *** edmondsw has quit IRC | 22:19 | |
| clarkb | smcginnis: if you git review the stack everything gets updated | 22:19 |
| clarkb | this is how I work locally to make sure things don't get orphaned in this way | 22:19 |
| clarkb | basically i always edit from the tip of a stack and squash back as necessary | 22:19 |
| *** edmondsw has joined #openstack-infra | 22:19 | |
| clarkb | then when I push all changes in the series are up to date with latest parents | 22:20 |
| fungi | the git-restack tool is quite useful thee | 22:20 |
| fungi | ther | 22:20 |
| fungi | e | 22:20 |
| * fungi puts fist through terrible keyboard | 22:20 | |
| smcginnis | :) | 22:20 |
| *** fultonj has joined #openstack-infra | 22:21 | |
| *** kgiusti has left #openstack-infra | 22:23 | |
| *** edmondsw_ has joined #openstack-infra | 22:23 | |
| *** edmondsw has quit IRC | 22:24 | |
| *** Goneri has quit IRC | 22:24 | |
| *** wolverineav has joined #openstack-infra | 22:25 | |
| *** edmondsw_ has quit IRC | 22:27 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 22:27 |
| *** dave-mcc_ has quit IRC | 22:30 | |
| *** wolverineav has quit IRC | 22:31 | |
| *** wolverineav has joined #openstack-infra | 22:31 | |
| *** dprince has quit IRC | 22:33 | |
| *** fultonj has quit IRC | 22:34 | |
| *** wolverineav has quit IRC | 22:36 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 22:43 |
| mordred | wow. i'm truly unsmart today | 22:43 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 22:48 |
| *** iyamahat has joined #openstack-infra | 22:52 | |
| *** rossella_s has quit IRC | 22:59 | |
| *** rossella_s has joined #openstack-infra | 22:59 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 23:00 |
| *** yamamoto has joined #openstack-infra | 23:04 | |
| *** yamamoto has quit IRC | 23:05 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 23:06 |
| *** leakypipes has quit IRC | 23:10 | |
| *** iyamahat has quit IRC | 23:11 | |
| *** yamamoto has joined #openstack-infra | 23:11 | |
| *** iyamahat has joined #openstack-infra | 23:13 | |
| *** yamamoto has quit IRC | 23:16 | |
| fungi | looks like our infra docs job doesn't support being triggered by a tag: http://logs.openstack.org/f0/f0edfab81d49e8152bf8f1b2d4c94b7b3e863190/release/publish-openstack-sphinx-docs-infra/8ff4ca8/job-output.txt.gz#_2017-11-17_23_16_44_823492 | 23:20 |
| clarkb | fungi: we've only tagged the one time right? | 23:21 |
| clarkb | or does that job run against more tha system-config? | 23:21 |
| fungi | it's used by git-review | 23:21 |
| clarkb | ah | 23:21 |
| fungi | which is what it failed on | 23:22 |
| fungi | release itself seems to have worked though | 23:23 |
| johnsom | Known issue that the release notes gates are broken? http://logs.openstack.org/58/505158/44/check/build-openstack-releasenotes/7d0042b/job-output.txt.gz#_2017-11-17_22_54_09_362702 | 23:23 |
| *** wendar_ is now known as wendar | 23:24 | |
| fungi | #status log git-review 1.26.0 released, adding support for Gerrit 2.14 and Git 2.15: https://pypi.org/project/git-review/1.26.0/ | 23:24 |
| openstackstatus | fungi: finished logging | 23:24 |
| clarkb | johnsom: yes mordred has been working on it | 23:25 |
| johnsom | Ok | 23:26 |
| johnsom | Ah, probably waiting on this one: https://review.openstack.org/#/c/520833 | 23:27 |
| *** Apoorva_ has quit IRC | 23:27 | |
| *** Apoorva has joined #openstack-infra | 23:28 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add general sphinx and reno jobs and role https://review.openstack.org/521142 | 23:31 |
| mordred | johnsom: oh, I'm having myself a grand old time with it - apparently today is "can mordred get any small details correct" day | 23:33 |
| johnsom | Hahaha | 23:33 |
| mordred | johnsom: if you want to depends-on the thing that should be the fix - https://review.openstack.org/#/c/521145 almost completely worked this last time ... | 23:34 |
| mordred | I have confidence that *this* patch is gonna be the one ... | 23:34 |
| * johnsom crosses fingers for Monty | 23:35 | |
| *** gouthamr_ has joined #openstack-infra | 23:36 | |
| *** sdague has quit IRC | 23:40 | |
| *** iyamahat has quit IRC | 23:42 | |
| *** rbrndt has quit IRC | 23:43 | |
| *** edmondsw has joined #openstack-infra | 23:44 | |
| *** rbrndt has joined #openstack-infra | 23:44 | |
| *** rbrndt has quit IRC | 23:44 | |
| johnsom | mordred Still an issue: 2017-11-17 23:44:37.289290 | ubuntu-xenial | /bin/bash: /home/zuul/src/git.openstack.org/openstack/requirements/upper-constraints.txt: Permission denied | 23:47 |
| *** edmondsw has quit IRC | 23:48 | |
| johnsom | mordred With that depends on, even though we got that error, it completed. However our docs job failed: ImportError: No module named tools | 23:50 |
| johnsom | Which is odd that it ever passed actually. | 23:52 |
| johnsom | Oh, hmm, it's local code. | 23:54 |
| *** pcaruana has quit IRC | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
