| *** Sukhdev has quit IRC | 00:01 | |
| *** yamada-h has joined #openstack-infra | 00:01 | |
| *** otter768 has joined #openstack-infra | 00:01 | |
| *** sdake has quit IRC | 00:03 | |
| *** apmelton_away is now known as apmelton | 00:05 | |
| *** yamada-h has quit IRC | 00:06 | |
| *** salv-orlando has quit IRC | 00:07 | |
| *** achanda has quit IRC | 00:07 | |
| *** mtanino has quit IRC | 00:08 | |
| *** rkukura has quit IRC | 00:08 | |
| *** salv-orlando has joined #openstack-infra | 00:08 | |
| *** yolanda has joined #openstack-infra | 00:08 | |
| *** tsg_ has quit IRC | 00:11 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 00:12 |
|---|---|---|
| *** annegentle has joined #openstack-infra | 00:15 | |
| *** dannywilson has quit IRC | 00:15 | |
| *** ashleighfarnham has joined #openstack-infra | 00:15 | |
| *** otter768 has quit IRC | 00:19 | |
| *** annegentle has quit IRC | 00:20 | |
| *** samueldmq has joined #openstack-infra | 00:21 | |
| *** salv-orlando has quit IRC | 00:22 | |
| *** Krinkle|detached is now known as Krinkle | 00:23 | |
| *** yamamoto has joined #openstack-infra | 00:23 | |
| *** patrickeast has quit IRC | 00:24 | |
| *** ddieterly has joined #openstack-infra | 00:26 | |
| *** ddieterly has quit IRC | 00:26 | |
| *** amitgandhinz has joined #openstack-infra | 00:27 | |
| *** yamamoto has quit IRC | 00:28 | |
| *** gyee has quit IRC | 00:29 | |
| asselin | this nodepool dib command doesn't work in my setup: DEFAULT_QEMU_IMAGE_COMPAT_OPTIONS = "--qemu-img-options 'compat=0.10'" | 00:32 |
| greghaynes | we set that by default now | 00:33 |
| greghaynes | so it shouldnt be needed | 00:33 |
| asselin | greghaynes, yes...it used to optional..and now it's not | 00:33 |
| asselin | so my images break | 00:33 |
| greghaynes | how so? | 00:33 |
| *** tqtran_ has quit IRC | 00:34 | |
| asselin | greghaynes, http://paste.openstack.org/show/205451/ | 00:34 |
| *** yolanda has quit IRC | 00:35 | |
| greghaynes | asselin: what qemu-img version is installed? | 00:35 |
| asselin | greghaynes, http://paste.openstack.org/show/205452/ | 00:35 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/system-config: Move security.openstack.org to HTTPS https://review.openstack.org/155099 | 00:36 |
| cinerama | pleia2: if you choose to give your seal of approval to https://review.openstack.org/#/c/176578/ i want to hang around when we apply it to translate-dev | 00:36 |
| asselin | greghaynes, this is on ubuntu 12 | 00:37 |
| greghaynes | asselin: awesome | 00:37 |
| greghaynes | asselin: so yea, thats a bug | 00:38 |
| greghaynes | We didnt realize old qemu-img didnt support that argument | 00:38 |
| asselin | easy fix? | 00:38 |
| greghaynes | asselin: Yep, we should check the qemu-img version and set that arg appropriately | 00:40 |
| *** otter768 has joined #openstack-infra | 00:40 | |
| asselin | greghaynes, still not following why it wasn't left optional...? | 00:43 |
| greghaynes | asselin: its something that should be true everywhere and it was proving hard to figure out where the config for it should be | 00:44 |
| greghaynes | and really, theres not a good way for us to describe that in config - its based on the features of the build host not the target | 00:44 |
| greghaynes | except as a global config option | 00:44 |
| asselin | greghaynes, so e.g. a config option at the top of nodepool.yaml? | 00:46 |
| asselin | greghaynes, or a DIB_COMPAT option? | 00:47 |
| asselin | exported? | 00:47 |
| *** atan8 has quit IRC | 00:47 | |
| asselin | anyway I have to go home. I will think about it and take a closer look tomorrow | 00:47 |
| *** spzala has joined #openstack-infra | 00:48 | |
| *** nikhil_k is now known as nikhil_k-afk | 00:48 | |
| *** atan8 has joined #openstack-infra | 00:51 | |
| openstackgerrit | Merged openstack-dev/pbr: Be safe, don't let friends use RawConfigParser https://review.openstack.org/176997 | 00:53 |
| *** esker has quit IRC | 00:53 | |
| *** zhiwei has joined #openstack-infra | 00:53 | |
| *** michchap has quit IRC | 00:55 | |
| *** ddieterly has joined #openstack-infra | 00:55 | |
| krotscheck | mordred:, clarkb: Yeah, I'm currently working through the hook approach. It's pretty straightforward to be honest. The trickiest part is making sure your local version of node doesn't suck. | 00:56 |
| *** ashleighfarnham has quit IRC | 00:58 | |
| *** ddieterly has quit IRC | 00:58 | |
| openstackgerrit | Merged openstack-infra/system-config: Allow blank lines in node{} with apply test https://review.openstack.org/176482 | 00:59 |
| *** ddieterly has joined #openstack-infra | 01:00 | |
| *** wenlock has joined #openstack-infra | 01:00 | |
| *** jyuso1 has joined #openstack-infra | 01:03 | |
| greghaynes | asselin: clarkb https://review.openstack.org/#/c/177043/ | 01:03 |
| * greghaynes is getting tired of that | 01:04 | |
| *** SumitNaiksatam has quit IRC | 01:05 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 01:06 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/project-config: There is no Project Bootstrappers-core... https://review.openstack.org/177044 | 01:09 |
| *** spzala has quit IRC | 01:09 | |
| *** tiswanso has joined #openstack-infra | 01:10 | |
| greghaynes | if we merge that dib patch then we can just remove the nodepool logic about compat=0.10 entirely | 01:12 |
| *** tiswanso_ has joined #openstack-infra | 01:12 | |
| clarkb | ++ | 01:13 |
| *** tiswanso has quit IRC | 01:14 | |
| mordred | krotscheck: sweer | 01:15 |
| krotscheck | I'm having some trouble figuring out the intricacies of extending cmd.Command though | 01:16 |
| mordred | krotscheck: it's VERY strange | 01:16 |
| *** tiswanso_ has quit IRC | 01:17 | |
| mordred | krotscheck: there is a sequence where it constructs and "finalizes" args, and then another sequence where things get run | 01:17 |
| *** tiswanso has joined #openstack-infra | 01:17 | |
| mordred | krotscheck: it's one of the most obtuse things ever, and you essentially have to read down into distutils and also the setuptools layer to grok it | 01:18 |
| krotscheck | mordred: I'm guessing somewhere in the middle there it reads options from other places and assigns them to the instance. Feels... Java POJO-Mapping-ish | 01:18 |
| *** baoli has quit IRC | 01:18 | |
| krotscheck | mordred: Like, you have to tell it that there's a property that can be assigned there before it'll actually assign that property. | 01:18 |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 01:19 |
| openstackgerrit | greghaynes proposed openstack-infra/nodepool: Stop setting compat=0.10 for qemu-img https://review.openstack.org/177045 | 01:19 |
| mordred | krotscheck: yah. very gross | 01:19 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/system-config: Add a centos-6 node label https://review.openstack.org/171286 | 01:19 |
| *** bknudson has joined #openstack-infra | 01:21 | |
| *** ChuckC has quit IRC | 01:21 | |
| *** sdake has joined #openstack-infra | 01:22 | |
| *** atan8 has quit IRC | 01:23 | |
| *** sdake_ has quit IRC | 01:26 | |
| *** michchap has joined #openstack-infra | 01:26 | |
| *** flaper87 has quit IRC | 01:27 | |
| *** emagana has quit IRC | 01:27 | |
| *** flaper87 has joined #openstack-infra | 01:28 | |
| *** liusheng has joined #openstack-infra | 01:29 | |
| *** atan8 has joined #openstack-infra | 01:29 | |
| *** macjack has left #openstack-infra | 01:32 | |
| *** flaper87 has quit IRC | 01:33 | |
| *** zz_dimtruck is now known as dimtruck | 01:35 | |
| *** salv-orlando has joined #openstack-infra | 01:35 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 01:37 |
| *** nelsnelson has joined #openstack-infra | 01:38 | |
| *** flaper87 has joined #openstack-infra | 01:39 | |
| *** kaisers has quit IRC | 01:40 | |
| *** nelsnelson has quit IRC | 01:41 | |
| *** kaisers has joined #openstack-infra | 01:42 | |
| *** nelsnelson has joined #openstack-infra | 01:43 | |
| *** salv-orlando has quit IRC | 01:45 | |
| *** atan8 has quit IRC | 01:48 | |
| Kennan2 | hi, :clarkb | 01:49 |
| *** amitgandhinz has quit IRC | 01:49 | |
| Kennan2 | there ? | 01:49 |
| *** amitgandhinz has joined #openstack-infra | 01:49 | |
| *** ChuckC has joined #openstack-infra | 01:49 | |
| *** sdake_ has joined #openstack-infra | 01:50 | |
| Kennan2 | a question about how to enable our project (master branch) devstack gate run against other projects kilo branch ? | 01:50 |
| *** shardy_ has joined #openstack-infra | 01:50 | |
| Kennan2 | since our project not branched now | 01:50 |
| Kennan2 | I found the devstack gate job run against other projects all with master | 01:50 |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 01:50 |
| Kennan2 | I am not sure how to make it run against with kilo branch ? | 01:50 |
| Kennan2 | did I need configure something in devstack plugin.sh ? | 01:51 |
| Kennan2 | :clarkb are you here ? :) | 01:51 |
| *** shardy has quit IRC | 01:51 | |
| *** sdake has quit IRC | 01:53 | |
| *** ChuckC has quit IRC | 01:53 | |
| *** ChuckC has joined #openstack-infra | 01:54 | |
| *** garyk has quit IRC | 01:54 | |
| *** amitgandhinz has quit IRC | 01:54 | |
| Kennan2 | :lifeless there ? | 01:54 |
| *** garyk has joined #openstack-infra | 01:55 | |
| *** Kennan2 is now known as Kennan | 01:55 | |
| *** shardy_ has quit IRC | 01:55 | |
| *** shardy has joined #openstack-infra | 01:55 | |
| lifeless | Kennan: yes? | 01:56 |
| Kennan | HI :lifeless | 01:56 |
| Kennan | our project now not branched | 01:56 |
| Kennan | still master | 01:56 |
| *** asettle is now known as asettle-luncheon | 01:57 | |
| Kennan | but other projects in openstack (like nova etc, branched with kilo) | 01:57 |
| Kennan | we want our devstack gate job still run against other projects kilo | 01:57 |
| Kennan | do you know how to do that ? | 01:57 |
| *** apmelton is now known as apmelton_away | 01:57 | |
| Kennan | since if run again master, some requirements version can cause issue | 01:57 |
| lifeless | I'm not sure | 01:58 |
| Kennan | do you know what make devstack gate job run against kilo or master branch ? | 01:58 |
| Kennan | is it setup by jenkins ? | 01:58 |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 02:01 |
| *** Sam_Wan has quit IRC | 02:01 | |
| lifeless | Kennan: its all controlled by jjb | 02:01 |
| *** harlowja is now known as harlowja_away | 02:02 | |
| *** xianghuihui has joined #openstack-infra | 02:02 | |
| *** baoli has joined #openstack-infra | 02:03 | |
| krotscheck | Waaaait a sec. | 02:03 |
| krotscheck | The only way to add a command into the setuptools build chain is.... to monkeypatch the command? | 02:04 |
| krotscheck | seriously? | 02:04 |
| *** xianghuihui has left #openstack-infra | 02:06 | |
| lifeless | krotscheck: you reallllly don't want to stare down that path | 02:06 |
| krotscheck | lifeless: Iiiii'm starting to get that. | 02:07 |
| *** mwagner_lap has quit IRC | 02:07 | |
| *** michchap_ has joined #openstack-infra | 02:08 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 02:08 |
| * krotscheck walks away slowly | 02:10 | |
| *** baoli has quit IRC | 02:10 | |
| *** michchap has quit IRC | 02:12 | |
| *** baoli has joined #openstack-infra | 02:12 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 02:14 |
| *** rfolco has quit IRC | 02:15 | |
| *** salv-orlando has joined #openstack-infra | 02:15 | |
| *** ivar-lazzaro has quit IRC | 02:16 | |
| *** baoli has quit IRC | 02:19 | |
| *** tkelsey has joined #openstack-infra | 02:19 | |
| *** tqtran has joined #openstack-infra | 02:19 | |
| *** salv-orlando has quit IRC | 02:20 | |
| *** mwagner_lap has joined #openstack-infra | 02:21 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 02:23 |
| *** xyang1 has quit IRC | 02:23 | |
| *** tkelsey has quit IRC | 02:23 | |
| *** dannywilson has joined #openstack-infra | 02:23 | |
| *** dannywilson has quit IRC | 02:23 | |
| *** tqtran has quit IRC | 02:24 | |
| *** dannywilson has joined #openstack-infra | 02:24 | |
| stevemar | question - if we wanted to rename the project `python-openstackclient` to just `openstackclient` how much of a PITA is that? | 02:26 |
| stevemar | would it just be the same process as a new library? | 02:26 |
| openstackgerrit | Dan Prince proposed openstack-infra/tripleo-ci: puppet: use a single image for CI https://review.openstack.org/177060 | 02:29 |
| lifeless | stevemar: essentially yes | 02:29 |
| openstackgerrit | Timo Tijhof proposed openstack-infra/zuul: Status: Use vertical-align for progress bar instead of margin https://review.openstack.org/170058 | 02:29 |
| openstackgerrit | Timo Tijhof proposed openstack-infra/zuul: Status: Increase width of change-progress-row-left https://review.openstack.org/170059 | 02:29 |
| lifeless | stevemar: though, the impact on requirements in many projects may be a bit tedious | 02:29 |
| lifeless | stevemar: (doing point releases to stable branches with a different release name...) | 02:30 |
| stevemar | lifeless, shouldn't be *that* many projects depending on us, just a few of the other client libs and devstack | 02:31 |
| *** SumitNaiksatam has joined #openstack-infra | 02:31 | |
| lifeless | stevemar: its not the # :) | 02:31 |
| lifeless | stevemar: its the location | 02:31 |
| *** jyuso1 has quit IRC | 02:37 | |
| *** marun has joined #openstack-infra | 02:37 | |
| *** achanda has joined #openstack-infra | 02:37 | |
| *** wenlock has quit IRC | 02:39 | |
| *** unicell has quit IRC | 02:39 | |
| *** baoli has joined #openstack-infra | 02:40 | |
| *** baoli_ has joined #openstack-infra | 02:42 | |
| *** baoli has quit IRC | 02:42 | |
| dtroyer | lifeless: All of those would be packages, pypi and the like, though, right? As long as the last release remained they should work until updated | 02:43 |
| lifeless | dtroyer: sure, the thing I raised was doing point releases | 02:43 |
| lifeless | we've made stable branches of everything now AIUI | 02:43 |
| lifeless | so if you want to do a point release of python-openstackclient | 02:43 |
| dtroyer | we'd have to fake it in pypi | 02:43 |
| lifeless | you'd need the infra tooling for openstackclient to know how to publish to openstackclient for master and python-openstackclient for kilo | 02:44 |
| lifeless | or bypass it all | 02:44 |
| stevemar | oh i didn't think about that lifeless | 02:44 |
| ianw | TIL : how broken bash errexit is | 02:45 |
| dtroyer | in an emergency I'd want to do that, otherwise make the changes before needing a point release | 02:45 |
| * dtroyer crosses fingers | 02:45 | |
| lifeless | dtroyer: 18 months for kilo's lifetime right? :) | 02:45 |
| stevemar | dtroyer, i'm going to push a renamed version to my private repo | 02:46 |
| stevemar | though i have no idea how to maintain git history | 02:46 |
| dtroyer | lifeless: I think, or is it slightly longer? 21 maybe? | 02:46 |
| dtroyer | stevemar: that's doable but I don't know how infra does it | 02:47 |
| dtroyer | they've got it figured | 02:47 |
| stevemar | well tests pass, and pep8 | 02:48 |
| stevemar | all my renaming worked | 02:48 |
| *** wuhg has quit IRC | 02:49 | |
| dtroyer | what if we released in parallel for a time? Both names, same versions | 02:49 |
| stevemar | dtroyer, https://github.com/stevemart/openstackclient | 02:50 |
| stevemar | that sounds like a maintenance nightmare | 02:50 |
| dtroyer | if it was scripted, maybe not…only one repo | 02:50 |
| dtroyer | and only the code, document everything with the new name | 02:50 |
| dtroyer | just thinking out loud | 02:51 |
| dtroyer | damn, ruby installs s..l..o..w on a RasPi | 02:51 |
| *** bhuvan has quit IRC | 02:52 | |
| stevemar | dtroyer, slower than osc runs? | 02:54 |
| *** ayoung has quit IRC | 02:56 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 02:58 |
| *** tiswanso has quit IRC | 02:58 | |
| *** sdake has joined #openstack-infra | 03:00 | |
| *** marun has quit IRC | 03:01 | |
| openstackgerrit | Timo Tijhof proposed openstack-infra/zuul: Add cache-hardlink argument to cloner https://review.openstack.org/117626 | 03:01 |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 03:02 |
| *** yamada-h has joined #openstack-infra | 03:03 | |
| *** sdake_ has quit IRC | 03:04 | |
| *** sdake_ has joined #openstack-infra | 03:04 | |
| dtroyer | ianw: ugh, I think it was there already… | 03:05 |
| stevemar | dtroyer, do you use run_tests? | 03:06 |
| stevemar | or just tox? | 03:06 |
| dtroyer | in osc? just tox | 03:06 |
| stevemar | okay | 03:06 |
| stevemar | i'm going to propose we remove it | 03:06 |
| dtroyer | k | 03:07 |
| stevemar | dtroyer, what about the stuff in tools? (install_venv and such?) | 03:07 |
| *** yamada-h has quit IRC | 03:07 | |
| dtroyer | unsure if that's still needed. are they in the cookiecuter output? | 03:08 |
| *** sdake has quit IRC | 03:08 | |
| * stevemar shrugs | 03:08 | |
| dtroyer | doesn't look like it | 03:08 |
| stevemar | ha, OSC has 998 commits | 03:08 |
| stevemar | thats pretty cool | 03:09 |
| stevemar | we should rename it at 1000 :P | 03:09 |
| dtroyer | commit 1000 will be a giant replacement with roscoe, my go lcient ;) | 03:10 |
| dtroyer | or client even | 03:10 |
| stevemar | dtroyer, https://github.com/angst/roscoe ? | 03:11 |
| dtroyer | that's the early version | 03:11 |
| dtroyer | from when I threatend to use C and libcurl | 03:12 |
| stevemar | should have stuck with your threat | 03:13 |
| stevemar | dtroyer, bump: https://review.openstack.org/#/c/177066/1 | 03:13 |
| stevemar | also... https://review.openstack.org/#/c/177046/ | 03:14 |
| * stevemar just realized that we've been chatting in -infra, my bad | 03:14 | |
| dtroyer | stevemar: https://github.com/dtroyer/roscoe is more recent. I haven't pushed since I started re-working stackforge/golang-client last weekend | 03:14 |
| dtroyer | wanna see something really funny? | 03:14 |
| dtroyer | OSC's now vanished-from-the-scene father: https://github.com/dtroyer/drstack | 03:15 |
| ianw | dtroyer: it sort of is ... but did you know that "function || echo 'i failed' " will disable errexit in "function" ? it seems the path for every developer to knowing this is via about an hour of debugging | 03:15 |
| ianw | to find out that infact that is the POSIX way | 03:16 |
| dtroyer | ianw: I thought that was only in pipelines, not boolean operators | 03:16 |
| dtroyer | dang, we need to dump that shortcut if so | 03:16 |
| ianw | "... part of a command executed in a && or || list except the command following the final && or || ..." | 03:18 |
| ianw | don't google it, it will just make you angry looking at other people who are angry and bash developers who are also angry because they follow posix on it and, well, everyone is angry :) | 03:19 |
| *** samueldmq has quit IRC | 03:20 | |
| dtroyer | I don't need angry tonight, thanks for the warning | 03:21 |
| dtroyer | yay POSIX | 03:21 |
| *** yamahata has joined #openstack-infra | 03:23 | |
| *** yamamoto has joined #openstack-infra | 03:32 | |
| *** shashankhegde has quit IRC | 03:32 | |
| *** Longgeek has joined #openstack-infra | 03:33 | |
| *** ddieterly has quit IRC | 03:40 | |
| *** xianghuihui has joined #openstack-infra | 03:40 | |
| *** xianghuihui has quit IRC | 03:40 | |
| *** xianghuihui has joined #openstack-infra | 03:41 | |
| *** xianghuihui has quit IRC | 03:41 | |
| *** esker has joined #openstack-infra | 03:42 | |
| *** achanda has quit IRC | 03:44 | |
| openstackgerrit | Timo Tijhof proposed openstack-infra/zuul: Add cache-no-hardlinks argument to zuul-cloner https://review.openstack.org/117626 | 03:44 |
| openstackgerrit | Timo Tijhof proposed openstack-infra/zuul: Cloner: Implement cache-no-hardlinks argument https://review.openstack.org/117626 | 03:44 |
| *** esker has quit IRC | 03:47 | |
| *** Longgeek has quit IRC | 03:51 | |
| *** tsg has joined #openstack-infra | 03:52 | |
| *** otter768 has quit IRC | 03:54 | |
| *** asettle-luncheon is now known as asettle | 03:54 | |
| *** Krinkle is now known as Krinkle|detached | 03:59 | |
| *** Longgeek has joined #openstack-infra | 04:00 | |
| *** wenlock has joined #openstack-infra | 04:00 | |
| *** yamada-h has joined #openstack-infra | 04:04 | |
| *** baoli_ has quit IRC | 04:06 | |
| *** yamada-h has quit IRC | 04:08 | |
| *** maishsk_ has joined #openstack-infra | 04:14 | |
| maishsk_ | pleia2: Are you around? | 04:14 |
| *** salv-orlando has joined #openstack-infra | 04:18 | |
| *** yamamoto has quit IRC | 04:24 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 04:25 |
| *** esker has joined #openstack-infra | 04:25 | |
| *** salv-orlando has quit IRC | 04:25 | |
| *** subscope_ has joined #openstack-infra | 04:30 | |
| *** atan8 has joined #openstack-infra | 04:30 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 04:34 |
| *** achanda has joined #openstack-infra | 04:35 | |
| *** kaisers has quit IRC | 04:37 | |
| *** yamamoto has joined #openstack-infra | 04:38 | |
| *** Longgeek has quit IRC | 04:42 | |
| *** unicell has joined #openstack-infra | 04:42 | |
| *** emagana has joined #openstack-infra | 04:45 | |
| *** mrmartin has joined #openstack-infra | 04:45 | |
| maishsk_ | tristanC: pleia2: Ping!! | 04:46 |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 04:49 |
| *** markvoelker_ has quit IRC | 04:51 | |
| *** Longgeek has joined #openstack-infra | 04:52 | |
| *** wuhg has joined #openstack-infra | 04:52 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 04:52 |
| *** dhritishikhar_ has quit IRC | 04:53 | |
| *** tqtran has joined #openstack-infra | 04:53 | |
| *** kaisers has joined #openstack-infra | 04:54 | |
| *** nelsnelson has quit IRC | 04:55 | |
| *** dhritishikhar has joined #openstack-infra | 04:55 | |
| *** salv-orlando has joined #openstack-infra | 04:58 | |
| *** woodster_ has quit IRC | 05:00 | |
| *** macjack has joined #openstack-infra | 05:02 | |
| *** dannywilson has quit IRC | 05:02 | |
| *** salv-orlando has quit IRC | 05:05 | |
| *** AJaeger has joined #openstack-infra | 05:08 | |
| *** sarob has joined #openstack-infra | 05:09 | |
| sarob | i need to get https://review.openstack.org/#/c/176948/ merged | 05:10 |
| sarob | anyone up for reviews? | 05:10 |
| *** ddieterly has joined #openstack-infra | 05:11 | |
| morganfainberg | sarob: you might want to go with experimental instead of removed, you could then "check experimental" and see if it passes. (As a thought) | 05:12 |
| morganfainberg | But it doesn't run unless specifically asked. | 05:12 |
| sarob | ah, right | 05:12 |
| morganfainberg | :) | 05:12 |
| sarob | its more about im not ready to run tests yet | 05:13 |
| morganfainberg | Right. Makes sense. | 05:13 |
| sarob | and the ones applied to this repo would be the wrong ones | 05:13 |
| sarob | when we do start | 05:13 |
| morganfainberg | Ah that makes more sense then to remove. | 05:13 |
| sarob | thanks for the advice though | 05:14 |
| morganfainberg | Np | 05:14 |
| *** ddieterly has quit IRC | 05:16 | |
| *** tnovacik has joined #openstack-infra | 05:16 | |
| *** garyk has quit IRC | 05:16 | |
| *** maishsk_ has quit IRC | 05:17 | |
| *** deepakcs has joined #openstack-infra | 05:17 | |
| *** salv-orlando has joined #openstack-infra | 05:17 | |
| *** maishsk has joined #openstack-infra | 05:17 | |
| *** sks has joined #openstack-infra | 05:18 | |
| *** salv-orlando has quit IRC | 05:21 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/devstack-gate: [WIP] Add -e to gate wrap script https://review.openstack.org/177017 | 05:23 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Add Elastic Recheck to docs.openstack.org/infra https://review.openstack.org/177077 | 05:24 |
| AJaeger | sarob: is that only for this repo or also for the other akanada ones? | 05:25 |
| sarob | only for this repo | 05:25 |
| AJaeger | jhesketh, SergeyLukjanov: Could you review 176948, please? We need one more +2... | 05:26 |
| AJaeger | sarob: I pinged the other two core reviewers that might be awake ;) ^ | 05:26 |
| *** atan8 has quit IRC | 05:26 | |
| sarob | thanks! | 05:27 |
| *** wenlock has quit IRC | 05:27 | |
| jhesketh | AJaeger: looking | 05:28 |
| *** BharatK has joined #openstack-infra | 05:29 | |
| openstackgerrit | Andreas Jaeger proposed openstack/requirements: Remove doc repositories from projects.txt https://review.openstack.org/173201 | 05:29 |
| AJaeger | thanks, jhesketh ! | 05:29 |
| jhesketh | AJaeger, sarob: lgtm, approved | 05:29 |
| sarob | thanks people | 05:30 |
| sarob | one down, 500 to go | 05:30 |
| *** maishsk has quit IRC | 05:30 | |
| *** afazekas has quit IRC | 05:31 | |
| AJaeger | sarob: wait 30 mins to have it deployed... | 05:32 |
| sarob | right | 05:32 |
| sarob | its late here, i will in the wee hours | 05:33 |
| *** maishsk has joined #openstack-infra | 05:35 | |
| openstackgerrit | Di Xu proposed openstack-infra/jenkins-job-builder: Support deep merge of defaults https://review.openstack.org/150730 | 05:36 |
| *** tsg has quit IRC | 05:36 | |
| *** yolanda has joined #openstack-infra | 05:37 | |
| *** dimtruck is now known as zz_dimtruck | 05:40 | |
| *** hdd has joined #openstack-infra | 05:40 | |
| openstackgerrit | Merged openstack-infra/project-config: Update to akanda project https://review.openstack.org/176948 | 05:41 |
| *** e0ne has joined #openstack-infra | 05:43 | |
| *** e0ne is now known as e0ne_ | 05:43 | |
| *** e0ne_ is now known as e0ne | 05:43 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/requirements: Bump kombu version and add amqp version https://review.openstack.org/174930 | 05:44 |
| openstackgerrit | Merged openstack-infra/project-config: extends gerrit acls to allow creation of stable branches https://review.openstack.org/173302 | 05:44 |
| *** e0ne has quit IRC | 05:45 | |
| openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Move manage_projects logic to puppet-jeepyb https://review.openstack.org/171565 | 05:46 |
| openstackgerrit | Merged openstack-infra/project-config: Add publishing, check-requirement jobs to os-brick https://review.openstack.org/176996 | 05:48 |
| *** jyuso1 has joined #openstack-infra | 05:49 | |
| *** afazekas has joined #openstack-infra | 05:49 | |
| *** mrunge has joined #openstack-infra | 05:51 | |
| openstackgerrit | Di Xu proposed openstack-infra/jenkins-job-builder: Support deep merge of defaults https://review.openstack.org/150730 | 05:52 |
| *** sarob has quit IRC | 05:54 | |
| *** otter768 has joined #openstack-infra | 05:55 | |
| *** vlaza has joined #openstack-infra | 05:57 | |
| openstackgerrit | Merged openstack-infra/project-config: Enable branch creation for networking-cisco repo https://review.openstack.org/176724 | 05:57 |
| *** sarob has joined #openstack-infra | 05:59 | |
| *** scheuran has joined #openstack-infra | 06:00 | |
| *** otter768 has quit IRC | 06:00 | |
| *** david-ly_ has joined #openstack-infra | 06:01 | |
| *** david-lyle has quit IRC | 06:03 | |
| *** subscope_ has quit IRC | 06:11 | |
| *** ddieterly has joined #openstack-infra | 06:12 | |
| *** tqtran has quit IRC | 06:14 | |
| *** ddieterly has quit IRC | 06:16 | |
| *** dhritishikhar has quit IRC | 06:16 | |
| *** dhritishikhar has joined #openstack-infra | 06:16 | |
| *** unicell has quit IRC | 06:21 | |
| *** unicell has joined #openstack-infra | 06:23 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Fix translation compress for manuals https://review.openstack.org/177092 | 06:27 |
| *** maishsk has quit IRC | 06:29 | |
| *** stevemar has quit IRC | 06:32 | |
| AJaeger | jhesketh, SergeyLukjanov: If you're reviewing today, please consider 177092 ^ | 06:34 |
| jhesketh | AJaeger: lgtm but I can't test at the moment so only +1 | 06:37 |
| AJaeger | jhesketh: I tested locally - I should have tested initially on *all* repos... | 06:38 |
| AJaeger | jhesketh: thansk | 06:38 |
| *** chlong has quit IRC | 06:39 | |
| *** mpaolino has joined #openstack-infra | 06:42 | |
| * AJaeger just noticed debugging code in it - and will double check now | 06:42 | |
| openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Move manage_projects logic to puppet-jeepyb https://review.openstack.org/171565 | 06:44 |
| *** salv-orlando has joined #openstack-infra | 06:45 | |
| *** Ala has joined #openstack-infra | 06:50 | |
| *** armax has quit IRC | 06:51 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Fix translation compress for manuals https://review.openstack.org/177092 | 06:52 |
| *** sarob has quit IRC | 06:53 | |
| *** sarob_ has joined #openstack-infra | 06:54 | |
| *** salv-orlando has quit IRC | 06:55 | |
| *** sarob_ has quit IRC | 06:58 | |
| *** dtantsur|afk is now known as dtantsur | 07:02 | |
| *** luqas has joined #openstack-infra | 07:06 | |
| *** luqas has quit IRC | 07:06 | |
| *** yamada-h has joined #openstack-infra | 07:06 | |
| *** luqas has joined #openstack-infra | 07:06 | |
| *** coolsvap|afk is now known as coolsvap | 07:08 | |
| *** coolsvap is now known as coolsvap|afk | 07:09 | |
| *** tkelsey has joined #openstack-infra | 07:09 | |
| *** yamada-h has quit IRC | 07:10 | |
| *** salv-orlando has joined #openstack-infra | 07:12 | |
| openstackgerrit | David Pursehouse proposed openstack-infra/lodgeit: Add deploye WSGI with proper database path https://review.openstack.org/177098 | 07:12 |
| *** ddieterly has joined #openstack-infra | 07:13 | |
| *** tkelsey has quit IRC | 07:13 | |
| *** salv-orlando has quit IRC | 07:14 | |
| *** ddieterly has quit IRC | 07:17 | |
| *** sdake_ has quit IRC | 07:20 | |
| *** luqas has quit IRC | 07:22 | |
| *** tqtran has joined #openstack-infra | 07:23 | |
| *** achanda has quit IRC | 07:24 | |
| *** dizquierdo has joined #openstack-infra | 07:27 | |
| *** tqtran has quit IRC | 07:27 | |
| openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Move manage_projects logic to puppet-jeepyb https://review.openstack.org/171565 | 07:28 |
| *** wuhg has quit IRC | 07:29 | |
| *** Hal has joined #openstack-infra | 07:29 | |
| *** Hal is now known as Guest75026 | 07:30 | |
| openstackgerrit | David Pursehouse proposed openstack-infra/lodgeit: Update i18n files https://review.openstack.org/177102 | 07:30 |
| openstackgerrit | David Pursehouse proposed openstack-infra/lodgeit: Remove out-of-date paragraph from the 'not found' page https://review.openstack.org/177103 | 07:30 |
| *** arxcruz has joined #openstack-infra | 07:30 | |
| *** achanda has joined #openstack-infra | 07:36 | |
| *** [HeOS] has quit IRC | 07:39 | |
| *** salv-orlando has joined #openstack-infra | 07:40 | |
| *** tkelsey has joined #openstack-infra | 07:42 | |
| *** jistr has joined #openstack-infra | 07:44 | |
| *** scheuran has quit IRC | 07:45 | |
| *** tkelsey has quit IRC | 07:46 | |
| *** e0ne has joined #openstack-infra | 07:47 | |
| *** scheuran has joined #openstack-infra | 07:47 | |
| *** scheuran has quit IRC | 07:48 | |
| *** e0ne has quit IRC | 07:49 | |
| *** kaisers1 has joined #openstack-infra | 07:50 | |
| *** yamamoto has quit IRC | 07:50 | |
| *** yolanda has quit IRC | 07:51 | |
| *** zz_johnthetubagu is now known as johnthetubaguy | 07:51 | |
| *** jlanoux has joined #openstack-infra | 07:51 | |
| *** kaisers2 has joined #openstack-infra | 07:52 | |
| *** e0ne has joined #openstack-infra | 07:53 | |
| *** hichihara has quit IRC | 07:54 | |
| *** otter768 has joined #openstack-infra | 07:56 | |
| *** tonytan4ever has quit IRC | 07:57 | |
| *** achanda has quit IRC | 07:57 | |
| *** jcoufal has joined #openstack-infra | 07:57 | |
| *** kaisers2 has quit IRC | 07:59 | |
| *** dtantsur is now known as dtantsur|brb | 08:00 | |
| *** otter768 has quit IRC | 08:01 | |
| *** e0ne has quit IRC | 08:05 | |
| *** ajo has joined #openstack-infra | 08:06 | |
| *** e0ne has joined #openstack-infra | 08:09 | |
| *** ajo_ has joined #openstack-infra | 08:09 | |
| *** ajo has quit IRC | 08:09 | |
| *** jistr is now known as jistr|mtg | 08:09 | |
| *** dizquierdo has quit IRC | 08:09 | |
| *** e0ne has quit IRC | 08:11 | |
| *** hashar has joined #openstack-infra | 08:11 | |
| openstackgerrit | Salvatore Orlando proposed openstack-infra/elastic-recheck: Add a query for bug 1285323 https://review.openstack.org/177116 | 08:12 |
| openstack | bug 1285323 in grenade "Services fail to shut down on the old side of Grenade" [Undecided,In progress] https://launchpad.net/bugs/1285323 - Assigned to Sean Dague (sdague) | 08:12 |
| *** ddieterly has joined #openstack-infra | 08:13 | |
| *** fhubik has joined #openstack-infra | 08:14 | |
| *** ssam2 has joined #openstack-infra | 08:16 | |
| *** yamamoto has joined #openstack-infra | 08:18 | |
| *** hyakuhei has joined #openstack-infra | 08:18 | |
| *** ddieterly has quit IRC | 08:18 | |
| *** devvesa has joined #openstack-infra | 08:18 | |
| *** jyuso1 has quit IRC | 08:19 | |
| *** hyakuhei has quit IRC | 08:19 | |
| *** hyakuhei has joined #openstack-infra | 08:23 | |
| *** ihrachyshka has joined #openstack-infra | 08:23 | |
| *** _nadya_ has joined #openstack-infra | 08:24 | |
| *** gilliard is now known as gillliard | 08:24 | |
| *** mpaolino has quit IRC | 08:26 | |
| *** andreykurilin__ has joined #openstack-infra | 08:27 | |
| openstackgerrit | Davide Guerri proposed openstack-infra/shade: WiP: Add Neutron/Nova Floating IP support https://review.openstack.org/177036 | 08:27 |
| openstackgerrit | Julien Danjou proposed openstack-infra/project-config: Run Gnocchi devstack tests for stable/1.0 against Kilo https://review.openstack.org/176907 | 08:28 |
| *** MaxV has joined #openstack-infra | 08:29 | |
| openstackgerrit | Julien Danjou proposed openstack-infra/project-config: Publish Gnocchi to PyPI https://review.openstack.org/175801 | 08:33 |
| *** maishsk has joined #openstack-infra | 08:34 | |
| *** derekh has joined #openstack-infra | 08:35 | |
| *** hyakuhei has quit IRC | 08:36 | |
| *** weshay has joined #openstack-infra | 08:40 | |
| *** _nadya_ has quit IRC | 08:40 | |
| *** salv-orlando has quit IRC | 08:43 | |
| openstackgerrit | Merged openstack/requirements: Bump kombu version and add amqp version https://review.openstack.org/174930 | 08:44 |
| *** [HeOS] has joined #openstack-infra | 08:47 | |
| openstackgerrit | Victor Stinner proposed openstack-infra/jenkins-job-builder: Fix Builder.update_job() on Python 3 https://review.openstack.org/172238 | 08:50 |
| *** SumitNaiksatam has quit IRC | 08:51 | |
| *** ajo_ has quit IRC | 08:51 | |
| *** ajo has joined #openstack-infra | 08:59 | |
| *** jkt has quit IRC | 09:00 | |
| *** jkt has joined #openstack-infra | 09:01 | |
| openstackgerrit | Dawid Malinowski proposed openstack-infra/jenkins-job-builder: Add support for "Display Upstream Changes Plugin". https://review.openstack.org/176963 | 09:02 |
| *** _nadya_ has joined #openstack-infra | 09:05 | |
| *** fhubik is now known as fhubik_afk | 09:07 | |
| *** cody-somerville has joined #openstack-infra | 09:07 | |
| *** yamada-h has joined #openstack-infra | 09:08 | |
| *** e0ne has joined #openstack-infra | 09:10 | |
| *** ajo has quit IRC | 09:10 | |
| openstackgerrit | Davide Guerri proposed openstack-infra/shade: WiP: Add Neutron/Nova Floating IP support https://review.openstack.org/177036 | 09:10 |
| *** yamada-h has quit IRC | 09:12 | |
| *** ddieterly has joined #openstack-infra | 09:14 | |
| *** e0ne is now known as e0ne_ | 09:16 | |
| *** ajo has joined #openstack-infra | 09:16 | |
| *** yolanda has joined #openstack-infra | 09:18 | |
| *** ddieterly has quit IRC | 09:19 | |
| *** pelix has joined #openstack-infra | 09:20 | |
| *** e0ne_ has quit IRC | 09:21 | |
| *** jyuso1 has joined #openstack-infra | 09:28 | |
| *** jistr|mtg is now known as jistr | 09:32 | |
| *** teran has quit IRC | 09:33 | |
| *** jerryz has joined #openstack-infra | 09:33 | |
| *** teran has joined #openstack-infra | 09:33 | |
| *** teran has quit IRC | 09:33 | |
| *** e0ne has joined #openstack-infra | 09:35 | |
| *** salv-orlando has joined #openstack-infra | 09:35 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack-dev/hacking: Add support for flake8 off_by_default for optional checks https://review.openstack.org/134052 | 09:36 |
| *** sdake has joined #openstack-infra | 09:37 | |
| *** fhubik_afk is now known as fhubik | 09:38 | |
| *** sdake_ has joined #openstack-infra | 09:39 | |
| *** dizquierdo has joined #openstack-infra | 09:40 | |
| openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/requirements: Bump kombu version and add amqp version https://review.openstack.org/177138 | 09:43 |
| *** sdake has quit IRC | 09:43 | |
| *** hyakuhei has joined #openstack-infra | 09:44 | |
| *** Longgeek_ has joined #openstack-infra | 09:52 | |
| *** Longgeek has quit IRC | 09:52 | |
| Kennan | :Ajaeger or :fungi there | 09:55 |
| Kennan | ? | 09:55 |
| AJaeger | Hi Kennan ! | 09:55 |
| Kennan | Hi :Ajaeger | 09:55 |
| Kennan | I have one question about: as our project not branched yet | 09:55 |
| AJaeger | which repo? | 09:56 |
| Kennan | we not have kilo branch as others, magnum | 09:56 |
| Kennan | we want our code gate job still run aginast with stable/kilo | 09:56 |
| Kennan | do you know how to make that happen ? | 09:56 |
| *** yamamoto has quit IRC | 09:56 | |
| *** cdent has joined #openstack-infra | 09:57 | |
| AJaeger | the gate jobs are setup that they will run with any branch (unless you explictely disable it running somewhere) | 09:57 |
| AJaeger | So, once you have a stable/kilo branch, it should work from the infra side just fine | 09:57 |
| *** otter768 has joined #openstack-infra | 09:57 | |
| AJaeger | Btw. right now you do not have permssions in project-config file gerrit/acls/openstack/magnum.config setup for creation of branches | 09:58 |
| AJaeger | Kennan: please read this one here in general for release management: http://docs.openstack.org/infra/manual/drivers.html | 09:58 |
| Kennan | :Ajaeger so you means, if we only have master branch, it would definitely run against with devstack (master branch for all projects, nova etc.) | 10:00 |
| *** yamamoto has joined #openstack-infra | 10:00 | |
| Kennan | we can not make it run stable/kilo ? | 10:00 |
| Kennan | for example, https://review.openstack.org/#/c/176907/ the Julien Danjou proposed to run their branch ? | 10:02 |
| Kennan | is this a working way to solve such issue ? | 10:02 |
| *** otter768 has quit IRC | 10:03 | |
| AJaeger | Kennan: that's a question you better discuss with others, can't help here. sdague, fungi, or clarkb should be around in 3hours to discuss | 10:04 |
| Kennan | ok. sure. thanks guy. | 10:04 |
| *** Guest75026 has quit IRC | 10:04 | |
| *** flaper87 has quit IRC | 10:05 | |
| *** flaper87 has joined #openstack-infra | 10:05 | |
| *** markvoelker has joined #openstack-infra | 10:06 | |
| *** pc_m has joined #openstack-infra | 10:09 | |
| *** pc_m has quit IRC | 10:10 | |
| *** fhubik has quit IRC | 10:11 | |
| *** markvoelker has quit IRC | 10:11 | |
| *** fhubik has joined #openstack-infra | 10:11 | |
| *** dhritishikhar has quit IRC | 10:13 | |
| *** dhritishikhar has joined #openstack-infra | 10:15 | |
| *** ddieterly has joined #openstack-infra | 10:15 | |
| *** hyakuhei has quit IRC | 10:15 | |
| *** fhubik has quit IRC | 10:16 | |
| *** fhubik_afk has joined #openstack-infra | 10:16 | |
| *** fhubik_afk is now known as fhubik | 10:16 | |
| *** pc_m has joined #openstack-infra | 10:18 | |
| *** ddieterly has quit IRC | 10:19 | |
| *** Bsony has joined #openstack-infra | 10:20 | |
| *** Longgeek_ has quit IRC | 10:22 | |
| *** Longgeek has joined #openstack-infra | 10:23 | |
| *** maishsk has quit IRC | 10:24 | |
| *** samueldmq has joined #openstack-infra | 10:25 | |
| *** andreykurilin__ has quit IRC | 10:26 | |
| *** maishsk_ has joined #openstack-infra | 10:26 | |
| *** maishsk_ has quit IRC | 10:31 | |
| *** tnovacik has quit IRC | 10:31 | |
| *** maishsk has joined #openstack-infra | 10:32 | |
| *** garyk has joined #openstack-infra | 10:33 | |
| *** dtantsur|brb is now known as dtantsur | 10:33 | |
| openstackgerrit | Julien Danjou proposed openstack-infra/project-config: Publish Gnocchi to PyPI https://review.openstack.org/175801 | 10:34 |
| *** teran has joined #openstack-infra | 10:35 | |
| *** mrmartin has quit IRC | 10:36 | |
| *** fhubik_afk has joined #openstack-infra | 10:37 | |
| *** teran_ has joined #openstack-infra | 10:38 | |
| *** teran__ has joined #openstack-infra | 10:40 | |
| *** fhubik has quit IRC | 10:41 | |
| *** teran has quit IRC | 10:41 | |
| *** teran has joined #openstack-infra | 10:42 | |
| *** teran_ has quit IRC | 10:44 | |
| *** kashyap has joined #openstack-infra | 10:44 | |
| kashyap | Hi, anyone remember top off their heads what's the oldest version of QEMU that's being run in the Gate? | 10:45 |
| kashyap | Maybe I should check in one of the CI run logs. | 10:45 |
| *** teran__ has quit IRC | 10:45 | |
| openstackgerrit | Salvatore Orlando proposed openstack-infra/elastic-recheck: Add query for bug 1448067 https://review.openstack.org/177169 | 10:50 |
| openstack | bug 1448067 in neutron "Failed to shutdown neutron" [Undecided,New] https://launchpad.net/bugs/1448067 | 10:50 |
| *** sdake_ has quit IRC | 10:53 | |
| mordred | dtroyer, lifeless: it's not that bad actually - what we'd do is publish a new thing to pypi called openstackclient - and then publish a new release of python-openstackclient that is an empty project except has a depend on openstackclient | 10:56 |
| mordred | dtroyer: we did it for the quantum/neutron rename, and that had _way_ more intricate depends | 10:58 |
| *** dguerri is now known as _dguerri | 10:59 | |
| *** isviridov_away is now known as isviridov | 11:00 | |
| *** _dguerri is now known as dguerri | 11:00 | |
| *** maishsk has quit IRC | 11:00 | |
| *** _nadya_ has quit IRC | 11:00 | |
| *** dhritishikhar has quit IRC | 11:03 | |
| *** hashar is now known as hasharLunch | 11:04 | |
| *** markvoelker has joined #openstack-infra | 11:07 | |
| AJaeger | jd__: ping | 11:07 |
| *** zhiwei has quit IRC | 11:08 | |
| sdague | salv-orlando: so https://launchpad.net/bugs/1285323 isn't actually the bug any more | 11:09 |
| openstack | Launchpad bug 1285323 in grenade "Services fail to shut down on the old side of Grenade" [Undecided,In progress] - Assigned to Sean Dague (sdague) | 11:09 |
| *** hdd has quit IRC | 11:10 | |
| *** markvoelker has quit IRC | 11:12 | |
| *** e0ne is now known as e0ne_ | 11:13 | |
| *** hdd has joined #openstack-infra | 11:15 | |
| *** yamamoto has quit IRC | 11:15 | |
| openstackgerrit | Sean Dague proposed openstack-infra/elastic-recheck: Add a query for bug 1446583 https://review.openstack.org/177116 | 11:15 |
| openstack | bug 1446583 in oslo-incubator "services no longer reliably stop in stable/kilo" [Critical,Fix committed] https://launchpad.net/bugs/1446583 - Assigned to Julien Danjou (jdanjou) | 11:16 |
| *** ddieterly has joined #openstack-infra | 11:16 | |
| *** alexpilotti has joined #openstack-infra | 11:18 | |
| openstackgerrit | Giulio Fidente proposed openstack-infra/tripleo-ci: Bump up delorean to allow for openstack-dashboard installation https://review.openstack.org/177176 | 11:18 |
| *** cdent has quit IRC | 11:20 | |
| *** ddieterly has quit IRC | 11:20 | |
| *** dguerri is now known as _dguerri | 11:20 | |
| AJaeger | sdague: Good morning! I had to rebase the doc requirements patch, could you +2 again, please? https://review.openstack.org/#/c/173201/ | 11:22 |
| *** wznoinsk has quit IRC | 11:22 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Add a query for bug 1446583 https://review.openstack.org/177116 | 11:22 |
| openstack | bug 1446583 in oslo-incubator "services no longer reliably stop in stable/kilo" [Critical,Fix committed] https://launchpad.net/bugs/1446583 - Assigned to Julien Danjou (jdanjou) | 11:22 |
| sdague | done | 11:22 |
| *** BharatK has quit IRC | 11:23 | |
| AJaeger | thanks, sdague | 11:23 |
| *** e0ne_ has quit IRC | 11:23 | |
| *** dguitarbite has quit IRC | 11:24 | |
| *** dguitarbite has joined #openstack-infra | 11:25 | |
| *** _nadya_ has joined #openstack-infra | 11:27 | |
| *** marcusvrn has quit IRC | 11:28 | |
| *** marcusvrn has joined #openstack-infra | 11:28 | |
| *** yamahata has quit IRC | 11:31 | |
| *** _dguerri is now known as dguerri | 11:37 | |
| openstackgerrit | Merged openstack-dev/hacking: Updated from global requirements https://review.openstack.org/176820 | 11:40 |
| *** ldnunes has joined #openstack-infra | 11:40 | |
| openstackgerrit | Merged openstack-infra/os-loganalyze: Add support for non-text files https://review.openstack.org/107267 | 11:41 |
| openstackgerrit | Merged openstack-infra/os-loganalyze: Tidy up generators into contained objects https://review.openstack.org/172711 | 11:41 |
| *** e0ne has joined #openstack-infra | 11:43 | |
| *** dprince has joined #openstack-infra | 11:49 | |
| *** fhubik_afk has quit IRC | 11:50 | |
| openstackgerrit | Sean Dague proposed openstack/requirements: create a separate section for pinned requirements https://review.openstack.org/177193 | 11:51 |
| *** funzo has joined #openstack-infra | 11:51 | |
| *** dguitarbite has quit IRC | 11:53 | |
| salv-orlando | sdague: do we have a new one? I used that because I did a search on lp and it kind of matched the failure mode fairly well | 11:54 |
| *** markvoelker has joined #openstack-infra | 11:54 | |
| salv-orlando | ok it's 1446583 then | 11:55 |
| *** tnovacik has joined #openstack-infra | 11:55 | |
| sdague | yep | 11:55 |
| sdague | the logging changed a little bit with the grenade refactor so you needed "die" in there as well | 11:57 |
| salv-orlando | sdague: https://bugs.launchpad.net/neutron/+bug/1448067 | 11:57 |
| openstack | Launchpad bug 1448067 in neutron "Failed to shutdown neutron" [Undecided,New] | 11:57 |
| openstackgerrit | Dan Prince proposed openstack-infra/tripleo-ci: puppet: use a single image for CI https://review.openstack.org/177060 | 11:57 |
| salv-orlando | I ignored bug 1446583 so I thought there were different manifestations | 11:57 |
| openstack | bug 1446583 in oslo-incubator "services no longer reliably stop in stable/kilo" [Critical,Fix committed] https://launchpad.net/bugs/1446583 - Assigned to Julien Danjou (jdanjou) | 11:57 |
| salv-orlando | I guess this second bug can be marked as duplicate then | 11:57 |
| *** otter768 has joined #openstack-infra | 11:58 | |
| sdague | yeh, so that was actually kind of a different bug | 11:58 |
| sdague | that was just a code bug in grenade that slipped through | 11:58 |
| jd__ | AJaeger: pong | 11:59 |
| sdague | when we started upgrading from stable/kilo instead of stable/juno | 11:59 |
| sdague | that was fixed with - https://review.openstack.org/#/c/176723/ | 11:59 |
| salv-orlando | sdague: thank I will close this second bug then | 11:59 |
| sdague | salv-orlando: just did | 12:00 |
| sdague | with the review posted | 12:00 |
| *** jerryz has quit IRC | 12:00 | |
| salv-orlando | sdague: cool. I should remember that pressing f5 on your browser window is a good thing to do! | 12:00 |
| *** hdd has quit IRC | 12:00 | |
| sdague | yeh, sorry, I've been knee deep in this bug for a week, and was just trying to figure out ways past it. I did realize this was blocking neutron yesterday, left a message in channel early on to hopefully alert people to it. | 12:01 |
| *** EmilienM has quit IRC | 12:01 | |
| *** EmilienM has joined #openstack-infra | 12:01 | |
| sdague | so if you've got neutron patches that failed on grenade in the last 2 days, I'd say just blanket recheck them now | 12:02 |
| *** otter768 has quit IRC | 12:02 | |
| *** scheuran has joined #openstack-infra | 12:06 | |
| openstackgerrit | Julien Danjou proposed openstack-infra/project-config: Publish Gnocchi to PyPI https://review.openstack.org/175801 | 12:07 |
| *** kgiusti has joined #openstack-infra | 12:09 | |
| openstackgerrit | Jon-Paul Sullivan (jp_at_hp) proposed openstack-infra/zuul: Reorder zuul-cloner checkout strategy https://review.openstack.org/177201 | 12:09 |
| openstackgerrit | Thierry Carrez proposed openstack-infra/release-tools: Update rccut and rcdelivery to match Kilo process https://review.openstack.org/177202 | 12:10 |
| *** dboik_ has quit IRC | 12:10 | |
| AJaeger | jd__: I commented in gerrit already, let me review your newest patch now... | 12:10 |
| jd__ | AJaeger: yeah thanks for your help | 12:10 |
| *** dizquierdo has quit IRC | 12:11 | |
| *** mpaolino has joined #openstack-infra | 12:11 | |
| *** hdd has joined #openstack-infra | 12:11 | |
| openstackgerrit | Serg Melikyan proposed openstack/requirements: Add python-muranoclient to requirements https://review.openstack.org/177204 | 12:12 |
| openstackgerrit | Merged openstack-infra/release-tools: Use fill method to simplify string wrapping https://review.openstack.org/172993 | 12:13 |
| openstackgerrit | Serg Melikyan proposed openstack/requirements: Add python-muranoclient to requirements https://review.openstack.org/177204 | 12:13 |
| openstackgerrit | Serg Melikyan proposed openstack/requirements: Add python-muranoclient to requirements https://review.openstack.org/177205 | 12:14 |
| openstackgerrit | Merged openstack-infra/release-tools: Fix lp project to repo name conversion https://review.openstack.org/176013 | 12:15 |
| *** maishsk has joined #openstack-infra | 12:15 | |
| *** ddieterly has joined #openstack-infra | 12:16 | |
| openstackgerrit | Merged openstack-infra/release-tools: warn instead of failing with missing bug url https://review.openstack.org/176027 | 12:17 |
| AJaeger | fungi, sdague, pleia2: Could you review and approve https://review.openstack.org/177092 today, please? The translation import worked fine on openstack-manuals but not on the other repos... | 12:17 |
| sdague | AJaeger: +2 | 12:18 |
| AJaeger | thanks, sdague ! | 12:20 |
| *** ddieterly has quit IRC | 12:21 | |
| AJaeger | fungi, good morning to you and thanks as well! | 12:21 |
| *** nfedotov has joined #openstack-infra | 12:21 | |
| *** tnovacik has quit IRC | 12:22 | |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add Neutron QoS Meeting https://review.openstack.org/176193 | 12:24 |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add Monasca Meeting details https://review.openstack.org/176194 | 12:24 |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add OpenStack Community Infrastructure Meeting https://review.openstack.org/176195 | 12:24 |
| openstackgerrit | Merged openstack/requirements: Remove doc repositories from projects.txt https://review.openstack.org/173201 | 12:25 |
| openstackgerrit | Merged openstack/requirements: Recap ceilometerclient for stable/juno https://review.openstack.org/173148 | 12:25 |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add translation test meeting https://review.openstack.org/176196 | 12:25 |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add Install guide meeting https://review.openstack.org/176197 | 12:25 |
| *** hdd has quit IRC | 12:25 | |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add Networking Gudie update hangout https://review.openstack.org/176198 | 12:26 |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add the Win the Enterprise meetings https://review.openstack.org/176199 | 12:26 |
| openstackgerrit | Merged openstack-infra/project-config: Publish Gnocchi to PyPI https://review.openstack.org/175801 | 12:26 |
| *** _nadya_ has quit IRC | 12:26 | |
| *** dboik has joined #openstack-infra | 12:27 | |
| *** dboik has quit IRC | 12:27 | |
| *** baoli has joined #openstack-infra | 12:27 | |
| *** dboik has joined #openstack-infra | 12:27 | |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add Akanda Team Meeting https://review.openstack.org/176200 | 12:27 |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add Neutron Service Chaining Meeting https://review.openstack.org/176201 | 12:27 |
| openstackgerrit | Merged openstack-infra/irc-meetings: Add log WG meeting https://review.openstack.org/176202 | 12:27 |
| *** devvesa has quit IRC | 12:29 | |
| cloudnull | morning. | 12:29 |
| cloudnull | quick question, the console logs from logs.openstack.org seem to be throwing an ISE. is there a maintenance going on? Example: http://logs.openstack.org/31/177131/2/check/os-ansible-deployment-dsvm-check-commit/317acde/console.html | 12:29 |
| cloudnull | it seems to only effect "console.html" | 12:29 |
| *** Hal has joined #openstack-infra | 12:30 | |
| *** Hal is now known as Guest47857 | 12:30 | |
| *** zhiwei has joined #openstack-infra | 12:30 | |
| openstackgerrit | Merged openstack-infra/project-config: Fix translation compress for manuals https://review.openstack.org/177092 | 12:31 |
| fungi | sdague: looks like we just merged a couple of changes to os-loganalyze, could one of those be causing that do you think? | 12:35 |
| *** dboik_ has joined #openstack-infra | 12:35 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Decrease PO files for imports (horizon+friends) https://review.openstack.org/176943 | 12:35 |
| fungi | i'll start digging deeper in a bit if nobody beats me to it | 12:35 |
| *** baoli has quit IRC | 12:35 | |
| *** dboik_ has quit IRC | 12:35 | |
| cloudnull | thanks fungi ! | 12:36 |
| *** isviridov is now known as isviridov_away | 12:36 | |
| *** dboik_ has joined #openstack-infra | 12:36 | |
| *** dguitarbite has joined #openstack-infra | 12:36 | |
| *** isviridov_away is now known as isviridov | 12:37 | |
| *** dboik has quit IRC | 12:38 | |
| sdague | fungi: url? | 12:38 |
| sdague | fungi: it could be | 12:38 |
| sdague | can we spot revert and see? | 12:39 |
| sdague | or can you look at the logs? | 12:39 |
| sdague | yeh, it looks like all console.html files are dead | 12:39 |
| sdague | yeh, all kinds of files are dead | 12:40 |
| *** baoli has joined #openstack-infra | 12:40 | |
| sdague | fungi: I think having apache server logs here is kind of required | 12:40 |
| *** yolanda has quit IRC | 12:40 | |
| openstackgerrit | Sean Dague proposed openstack-infra/os-loganalyze: Revert "Tidy up generators into contained objects" https://review.openstack.org/177218 | 12:41 |
| ttx | yep, just stumbled on same problem, 500 | 12:41 |
| sdague | yep | 12:41 |
| sdague | so, until fungi gets back with apache errors, it's a guessing game | 12:41 |
| ttx | I guess... Moon phase | 12:42 |
| *** bknudson has quit IRC | 12:42 | |
| sdague | no, I approved some things this morning that I assumed were more tested than they actually were | 12:42 |
| ttx | sdague: and I just found the motivation to look into gate-barbican-devstack-dsvm and see why it fails 60% of the time | 12:42 |
| ttx | I guess I'll recehck it in the mean time :) | 12:43 |
| *** isviridov is now known as isviridov_away | 12:46 | |
| BobBall | krtaylor / asselin: Ping :) Could you let me know when you're around? | 12:48 |
| sdague | ok, well I'm going to start reverting the patches one at a time and see | 12:48 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add inventory command to shade https://review.openstack.org/175891 | 12:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Have get_image_name return an image_name https://review.openstack.org/176549 | 12:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Raise a shade exception on broken volumes https://review.openstack.org/176550 | 12:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Remove REST links from inventory metadata https://review.openstack.org/175903 | 12:49 |
| openstackgerrit | Sean Dague proposed openstack-infra/os-loganalyze: Revert "Add support for non-text files" https://review.openstack.org/177221 | 12:49 |
| openstackgerrit | Merged openstack-infra/os-loganalyze: Revert "Tidy up generators into contained objects" https://review.openstack.org/177218 | 12:49 |
| *** dkranz has joined #openstack-infra | 12:49 | |
| openstackgerrit | Sean Dague proposed openstack-infra/os-loganalyze: Revert "Add support for non-text files" https://review.openstack.org/177221 | 12:49 |
| *** isviridov_away is now known as isviridov | 12:50 | |
| *** xyang1 has joined #openstack-infra | 12:50 | |
| *** mwagner_lap has quit IRC | 12:51 | |
| *** isviridov is now known as isviridov_away | 12:51 | |
| AJaeger | fungi, docs.openstack.org/infra/index.html does not yet exist since the new job merged. I noticed that we have Elastic Research already published but not mentioned, so wrote https://review.openstack.org/177077 . Nothing urgent for review - we still miss some projects. | 12:52 |
| *** _nadya_ has joined #openstack-infra | 12:52 | |
| *** e0ne is now known as e0ne_ | 12:53 | |
| *** rlandy has joined #openstack-infra | 12:53 | |
| openstackgerrit | Merged openstack/requirements: Bump eventlet version to 0.17.3 https://review.openstack.org/172132 | 12:55 |
| *** e0ne_ is now known as e0ne | 12:55 | |
| *** afazekas has quit IRC | 12:55 | |
| *** dboik_ has quit IRC | 12:55 | |
| *** dboik has joined #openstack-infra | 12:56 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack-dev/hacking: Updated from global requirements https://review.openstack.org/177229 | 12:56 |
| *** dboik has quit IRC | 12:56 | |
| *** bswartz has quit IRC | 12:56 | |
| fungi | AJaeger: right the job that creates and uploads the index only fires when the files which create it are modified (file pattern restrictions for it in layout.yaml) so i'll probably need to manually trigger that unless we have any pending changes for tox.ini or other matching files | 12:57 |
| *** mrmartin has joined #openstack-infra | 12:57 | |
| *** dboik has joined #openstack-infra | 12:57 | |
| AJaeger | fungi: exactly - so 177077 will be such a trigger | 12:57 |
| fungi | AJaeger: also the individual documents only get uploaded when their respective projects have commits merge to them, so some of them may just not have a high rate of code churn | 12:57 |
| *** _nadya_ has quit IRC | 12:57 | |
| marcusvrn | hi all! Does someone know what is the openstack Jenkins CI version? | 12:57 |
| sdague | fungi: did you find anything in the apache logs, or should I just keep pushing reverts until we get back to something good? | 12:57 |
| fungi | sdague: looking at the server logs now | 12:58 |
| *** alaski is now known as lascii | 12:58 | |
| AJaeger | fungi: Yeah, we need to check which ones are missing and somehow push them... | 12:58 |
| fungi | i was stepping away from the computer just as cloudnull spotted the error | 12:58 |
| cloudnull | yea, sorry about that :) | 12:59 |
| sdague | nothing wrong with spotting errors | 12:59 |
| sdague | I reverted the top of 2 patches | 12:59 |
| fungi | AttributeError: 'module' object has no attribute 'get_file_generator' | 12:59 |
| sdague | figured I'd wait until top of the hour to figure out if it deployed the fix | 13:00 |
| fungi | i'll clean up a full traceback | 13:00 |
| sdague | ok, so the patch I reverted and landed removed that | 13:01 |
| sdague | when will the revert take? | 13:01 |
| *** funzo has quit IRC | 13:01 | |
| fungi | http://paste.openstack.org/show/205683/ | 13:02 |
| *** jamesmcarthur has joined #openstack-infra | 13:02 | |
| fungi | with clarkb's speedups, puppet lag is down to ~half an hour now | 13:02 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add thread sync points to Task https://review.openstack.org/176459 | 13:03 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a docstring to the Task class https://review.openstack.org/177235 | 13:03 |
| fungi | but i can pull it earlier manually | 13:03 |
| mordred | harlowja_away, Shrews, SpamapS: ^^ my original comments on harlowja_away's patch were wrong and he was right. we do need that code | 13:03 |
| sdague | fungi: yeh, if you could make sure it's refreshed, that would be nice | 13:03 |
| *** yamamoto has joined #openstack-infra | 13:04 | |
| *** gordc has joined #openstack-infra | 13:04 | |
| *** ajo has quit IRC | 13:05 | |
| fungi | sdague: that seems to have fixed it--thanks! | 13:06 |
| *** radez_g0n3 is now known as radez | 13:07 | |
| *** dizquierdo has joined #openstack-infra | 13:08 | |
| *** yamamoto has quit IRC | 13:08 | |
| AJaeger | sdague: thanks! | 13:10 |
| *** doug-fish has joined #openstack-infra | 13:13 | |
| *** afazekas has joined #openstack-infra | 13:13 | |
| *** weshay has quit IRC | 13:13 | |
| *** dustins has joined #openstack-infra | 13:13 | |
| *** ajo has joined #openstack-infra | 13:14 | |
| *** eharney has joined #openstack-infra | 13:15 | |
| fungi | i think civs just got 'stacked | 13:16 |
| fungi | i'm having timeouts reaching it | 13:17 |
| fungi | there it goes, finally | 13:17 |
| *** maishsk has quit IRC | 13:21 | |
| *** mpaolino has quit IRC | 13:22 | |
| AJaeger | fungi: I just voted fine... | 13:22 |
| AJaeger | is everybody voting directly? ;) | 13:23 |
| *** zz_jgrimm is now known as jgrimm | 13:23 | |
| *** mpaolino has joined #openstack-infra | 13:23 | |
| *** esker has quit IRC | 13:23 | |
| fungi | i suspect a thundering herd among the people who do it immediately just to get it out of the way | 13:23 |
| *** peristeri has joined #openstack-infra | 13:25 | |
| *** jgrimm is now known as zz_jgrimm | 13:25 | |
| AJaeger | yeah, so that they don't forget and vote one hour after it closes... | 13:25 |
| *** dhritishikhar has joined #openstack-infra | 13:25 | |
| *** bswartz has joined #openstack-infra | 13:25 | |
| openstackgerrit | Thierry Carrez proposed openstack-infra/release-tools: Update rccut and rcdelivery to match Kilo process https://review.openstack.org/177202 | 13:27 |
| *** xyang1 has quit IRC | 13:27 | |
| *** mattfarina has joined #openstack-infra | 13:27 | |
| *** xyang1 has joined #openstack-infra | 13:28 | |
| sgordon | i have a dumb Q, sorry if the wrong place to ask it, a while back we created with the help of infra a stackforge/telcowg-usecases project in gerrit (yay!) - when i view all projects in gertty however it doesn't show up | 13:28 |
| *** kaisers1 has quit IRC | 13:28 | |
| sgordon | im wondering if this is a symptom of us missing something during project setup | 13:28 |
| AJaeger | sgordon: it exists and has content: http://git.openstack.org/cgit/stackforge/telcowg-usecases/ | 13:28 |
| *** kaisers has quit IRC | 13:28 | |
| AJaeger | So, just a gertty problem? | 13:29 |
| AJaeger | sgordon: jeblair has written gertty, you might want to wait another hour to ask him | 13:29 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Update flavor API for new get/list/search API https://review.openstack.org/176931 | 13:30 |
| *** kaisers has joined #openstack-infra | 13:30 | |
| *** kaisers1 has joined #openstack-infra | 13:30 | |
| sgordon | AJaeger, maybe but i dont have this issue with any other new repos | 13:31 |
| sgordon | they show up automagically | 13:31 |
| *** yolanda has joined #openstack-infra | 13:31 | |
| fungi | sgordon: new repos may not show up without a gertty restart? i haven't tested that | 13:33 |
| *** erlon has joined #openstack-infra | 13:33 | |
| *** AJaeger has quit IRC | 13:34 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Update flavor API for new get/list/search API https://review.openstack.org/176931 | 13:34 |
| *** ayoung_ has joined #openstack-infra | 13:34 | |
| *** mriedem_away is now known as mriedem | 13:35 | |
| *** ddieterly has joined #openstack-infra | 13:38 | |
| sdague | clarkb: when you get up - https://review.openstack.org/#/c/177193/1/global-requirements.txt,cm | 13:39 |
| openstackgerrit | Merged openstack/requirements: Relax pysendfile requirement https://review.openstack.org/165969 | 13:42 |
| *** tiswanso has joined #openstack-infra | 13:43 | |
| *** bknudson has joined #openstack-infra | 13:44 | |
| openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Move manage_projects logic to puppet-jeepyb https://review.openstack.org/171565 | 13:45 |
| *** ayoung_ is now known as ayoung | 13:46 | |
| openstackgerrit | Sean Dague proposed openstack/requirements: create a separate section for pinned requirements https://review.openstack.org/177193 | 13:47 |
| openstackgerrit | Sean Dague proposed openstack/requirements: relax final pins https://review.openstack.org/177266 | 13:47 |
| *** smccully has quit IRC | 13:48 | |
| *** jamesmcarthur has quit IRC | 13:49 | |
| *** smccully has joined #openstack-infra | 13:49 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: zuul/layout: make gate-puppet-.*-puppet-syntax-future voting https://review.openstack.org/177267 | 13:50 |
| *** armax has joined #openstack-infra | 13:50 | |
| openstackgerrit | Merged openstack-infra/devstack-gate: add liberty to feature grid https://review.openstack.org/175493 | 13:50 |
| *** apmelton_away is now known as apmelton | 13:50 | |
| *** nikhil_k-afk is now known as nikhil_k | 13:51 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 13:52 | |
| *** armax has quit IRC | 13:52 | |
| *** kashyap has left #openstack-infra | 13:53 | |
| *** armax has joined #openstack-infra | 13:53 | |
| *** mwagner_lap has joined #openstack-infra | 13:53 | |
| openstackgerrit | Zhenguo Niu proposed openstack/requirements: Update python-cinderclient to 1.2.0 https://review.openstack.org/177270 | 13:54 |
| *** mriedem has quit IRC | 13:56 | |
| sdague | fungi: is there any way to make apache logs available for situations like the os-loganalyze fail in some protected way? just because I was debugging blind on that. | 13:56 |
| *** esker has joined #openstack-infra | 13:57 | |
| *** stevemar has joined #openstack-infra | 13:57 | |
| odyssey4me | hi guys - a note on hpcloud vs rax images - it seems that hpcloud has no backports apt source included, whereas rax does | 13:57 |
| *** dansmith is now known as superdan | 13:59 | |
| *** otter768 has joined #openstack-infra | 13:59 | |
| *** emagana has quit IRC | 14:00 | |
| *** mpaolino has quit IRC | 14:00 | |
| *** deepakcs has quit IRC | 14:00 | |
| *** dguerri is now known as _dguerri | 14:01 | |
| *** yolanda has quit IRC | 14:01 | |
| *** tqtran has joined #openstack-infra | 14:01 | |
| *** asselin_ has joined #openstack-infra | 14:01 | |
| *** mriedem has joined #openstack-infra | 14:01 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:03 | |
| *** alex_xu has quit IRC | 14:03 | |
| *** heyongli has quit IRC | 14:03 | |
| *** otter768 has quit IRC | 14:04 | |
| sdague | odyssey4me: that's ... possible | 14:04 |
| sdague | odyssey4me: is there a specific issue? | 14:04 |
| *** signed8bit has joined #openstack-infra | 14:04 | |
| *** _dguerri is now known as dguerri | 14:05 | |
| odyssey4me | sdague we're noticing it for the swift erasure encoding apt packages which come from the backports repository | 14:05 |
| mordred | gotcha | 14:05 |
| *** jp_at_hp has joined #openstack-infra | 14:05 | |
| mordred | this is in unittests or devstack/functional/integration? | 14:06 |
| *** ChuckC_ has joined #openstack-infra | 14:06 | |
| mordred | because we're VERY close to having devstack nodes all be the same across all clouds | 14:06 |
| odyssey4me | sdague https://bugs.launchpad.net/openstack-ansible/+bug/1448152 | 14:06 |
| openstack | Launchpad bug 1448152 in openstack-ansible trunk "swift erasure encoding library installation failures" [Undecided,New] | 14:06 |
| mordred | it'll be slightly longer for unittest nodes to be the same | 14:06 |
| *** jp_at_hp has quit IRC | 14:06 | |
| *** jp_at_hp has joined #openstack-infra | 14:06 | |
| *** jp_at_hp has joined #openstack-infra | 14:06 | |
| *** ChuckC has quit IRC | 14:06 | |
| jp_at_hp | Can I get some feedback on https://review.openstack.org/#/c/177201/ from anyone please? mordred - would you know who would be best to talk to about zuul-cloner? | 14:07 |
| mordred | jp_at_hp: clarkb should be up soon | 14:08 |
| *** sks has quit IRC | 14:08 | |
| *** hasharLunch is now known as hashar | 14:08 | |
| *** afazekas has quit IRC | 14:09 | |
| *** jp_at_hp1 has joined #openstack-infra | 14:09 | |
| mordred | jp_at_hp: although looking at that change, you'll probably want feedback from sdague and jeblair too | 14:09 |
| jp_at_hp1 | thanks mordred | 14:11 |
| clarkb | I'm reading the old code and the order the commit msg has and dont see the difference | 14:11 |
| *** vhoward has left #openstack-infra | 14:11 | |
| *** jp_at_hp has quit IRC | 14:11 | |
| *** vhoward has joined #openstack-infra | 14:11 | |
| clarkb | well nevermind then? | 14:12 |
| clarkb | oh two clients | 14:12 |
| clarkb | its too early | 14:12 |
| clarkb | zuul ref -> specified branch -> master is what the old code does | 14:12 |
| clarkb | how is thia different? | 14:12 |
| *** amitgandhinz has joined #openstack-infra | 14:13 | |
| mordred | the old code at least claims to do "zuul ref for specified branch -> zuul ref for master -> tip of specified branch -> tip of master" | 14:13 |
| *** ChuckC_ has quit IRC | 14:13 | |
| *** armax has quit IRC | 14:13 | |
| mordred | the new node claims to do "zuul ref for specified branch -> tip of specified branch -> zuul ref for master -> tip of master" | 14:13 |
| *** armax has joined #openstack-infra | 14:14 | |
| clarkb | zuul ref for master isnt a thing iirc. only zuul ref | 14:14 |
| mordred | well, the comments in the old code list it as a thing | 14:14 |
| clarkb | you get a unique ref branch not multiples | 14:14 |
| * mordred has not read the actual flow - is just reading the comments | 14:14 | |
| *** alex_xu has joined #openstack-infra | 14:14 | |
| *** heyongli has joined #openstack-infra | 14:14 | |
| mordred | fungi: ManagerStoppedException: Manager hpcloud-b1 is no longer running | 14:14 |
| mordred | fungi: do you happen to know what might cause that? | 14:14 |
| mordred | fungi: hp is investigating times where we fail to boot nodes or time them out and delete them - and found a couple of places where the node booted properly and then we deleted it | 14:15 |
| fungi | mordred: um... no? but i'll see if i can identify a list of potential causes | 14:15 |
| clarkb | oh wait nvm thats how grenade works so I am probably wrong | 14:15 |
| mordred | fungi: in investigating the given node, 72fdbe6b-f240-4237-93df-952ba99151a4, I found that to be the cause for deleting it | 14:16 |
| mordred | clarkb: moar kofffeee | 14:16 |
| sdague | mordred: what's the use case here, I'm concerned it will do the wrong thing with upstream | 14:16 |
| *** scheuran has quit IRC | 14:16 | |
| mordred | sdague: meet jp_at_hp1 . jp_at_hp1 meet sdague | 14:16 |
| * mordred is just a matchmaker | 14:16 | |
| clarkb | ah I see it does it for botg zuul refs in the old code too | 14:17 |
| *** tonytan4ever has joined #openstack-infra | 14:17 | |
| sdague | because zuul ref's should be uniqueish, right? | 14:17 |
| mtreinish | mordred: https://www.youtube.com/watch?v=59Hj7bp38f8 | 14:17 |
| clarkb | so I am not really sure what the desired change of behavior is | 14:17 |
| sdague | mtreinish: if that's fiddler on the roof, I'm going to have to hit you | 14:17 |
| clarkb | jp_at_hp1 can you expand on that? | 14:17 |
| mtreinish | sdague: heh, then I'll have to hide | 14:17 |
| clarkb | going by comment diff its flipping 2 and 3 | 14:18 |
| fungi | so apparently a nodepool.task_manager.TaskManager object starts out running when created, and looks like that only changes when its stop() method is called | 14:18 |
| *** Ala has quit IRC | 14:18 | |
| fungi | so looking for places calling stop on it | 14:18 |
| jp_at_hp1 | clarkb sdague - so... When performing CI on a product that uses multiple repositories to create it's final built artefacts, we want to run CI in the gate in a way that is most consistent. | 14:19 |
| openstackgerrit | Merged openstack-infra/shade: Have get_image_name return an image_name https://review.openstack.org/176549 | 14:19 |
| *** zz_johnthetubagu is now known as johnthetubaguy | 14:20 | |
| fungi | inside nodepool.NodePool.reconfigureManagers there's a call to stop a manager when reconfiguring | 14:20 |
| jp_at_hp1 | If I try and take a simple example of 2 repos A and B, each of which build on different branches, master and not-master | 14:20 |
| *** Ala has joined #openstack-infra | 14:20 | |
| sdague | jp_at_hp1: so that assumes that *every* repo that's being used with zuul has the same branch structure | 14:20 |
| fungi | mordred: is it possible the incidents you found happened during a reconfiguration? | 14:20 |
| jp_at_hp1 | sdague: not at all | 14:20 |
| *** esker has quit IRC | 14:21 | |
| sdague | jp_at_hp1: so what happens if you are testing projects1, 2, 3. refs A (on 1), B on (2), C on (3) | 14:21 |
| sdague | on branch foo | 14:21 |
| fungi | sdague: on making apache logs public, not sure. are there good log scrubbers out there that can sanitize those properly? | 14:21 |
| sdague | but C doesn't have branch foo | 14:22 |
| jp_at_hp1 | so if I make a change to repo A and a change to repo B, on the master and not-master branches respectively, what I want to happen in the gate is that my test of the later change runs with both those zuul refs. In both the old and new code that will happen. | 14:22 |
| sdague | you then don't test C, right? | 14:22 |
| sdague | honestly, I think we need jeblair for this. I can never keep the whole model in my head. | 14:23 |
| jp_at_hp1 | sdague: typing too slow. sometimes human conversation is easier :( | 14:23 |
| *** wenlock has joined #openstack-infra | 14:23 | |
| openstackgerrit | Merged openstack-infra/shade: Remove REST links from inventory metadata https://review.openstack.org/175903 | 14:23 |
| jp_at_hp1 | sdague: I don't think your example is quite true. | 14:23 |
| *** mikap has joined #openstack-infra | 14:24 | |
| jp_at_hp1 | I think I can show that through my example... | 14:24 |
| *** mikap has quit IRC | 14:24 | |
| jp_at_hp1 | so in your example, ref C on (3) is on some branch other than foo? | 14:25 |
| *** hdd has joined #openstack-infra | 14:25 | |
| jp_at_hp1 | or is it specifically on the fallback branch of 'master'? | 14:25 |
| *** woodster_ has joined #openstack-infra | 14:26 | |
| *** tqtran has quit IRC | 14:26 | |
| clarkb | I don't think it matters, it will checkout the indicated branch in either case | 14:26 |
| *** mtanino has joined #openstack-infra | 14:27 | |
| clarkb | the scenario that is important to consider is what ref of B is checked out when testing C | 14:27 |
| clarkb | (I think) | 14:27 |
| *** ChuckC_ has joined #openstack-infra | 14:28 | |
| jp_at_hp1 | so... | 14:28 |
| jp_at_hp1 | in my example of 2 repos | 14:28 |
| jp_at_hp1 | A and B, each with master and non-master as the branch we want to use | 14:28 |
| jp_at_hp1 | when I build I want to always use those 2 branches. | 14:29 |
| jp_at_hp1 | A on master and B on non-master | 14:29 |
| clarkb | that breaks an assumption that zuul makes | 14:29 |
| sdague | right, so you assume that all projects have the same branch structure | 14:29 |
| clarkb | (the always bit) | 14:29 |
| jp_at_hp1 | no | 14:30 |
| jp_at_hp1 | I don't see that as true at all | 14:30 |
| clarkb | yes you can do overrides to change the indicated branch | 14:30 |
| jp_at_hp1 | OK - I know what the problem is | 14:30 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: Update make_library_stable_branch.sh to set defaultbranch https://review.openstack.org/173919 | 14:30 |
| fungi | mordred: looks like there was a new provider manager created for each of the hpcloud providers at 2015-04-23 17:48 which is also when 72fdbe6b-f240-4237-93df-952ba99151a4 failed to be added | 14:30 |
| clarkb | but zuuls always default behavior is to test a change to B on non master against A on non master | 14:30 |
| *** dguitarbite has quit IRC | 14:30 | |
| *** yolanda has joined #openstack-infra | 14:30 | |
| jp_at_hp1 | When talking about the zuul-cloner, I am explicitly thinking of using the mapping ability to choose branches for things. | 14:30 |
| jp_at_hp1 | As I read the docs, zuul doesn't care about branches | 14:31 |
| clarkb | zuul absolutely cares about branches | 14:31 |
| jp_at_hp1 | it makes a unique ID for every change in the pipeline, which is all projects and all branches in the pipeline | 14:31 |
| clarkb | jp_at_hp1: but then it only hands out a single zuul_branch to the job | 14:32 |
| jp_at_hp1 | then the cloner makes the choice about what to test through the use of the mapping table first, zuul_branch (where the main change under test is) second | 14:32 |
| clarkb | so what it sounds like you would like to do is override zuul_branch in all cases to change the indicated branch | 14:32 |
| *** esker has joined #openstack-infra | 14:32 | |
| jp_at_hp1 | no - this is getting quite off topic. And in fact even here I can show incorrect behaviour of current code, I think... | 14:32 |
| clarkb | jp_at_hp1: I am trying to understand the use case so that I understand your example | 14:33 |
| *** zz_jgrimm is now known as jgrimm | 14:33 | |
| jp_at_hp1 | stable branches | 14:33 |
| clarkb | is ^ not something that you are doing? force A to master regardless and B to non master regardless? | 14:33 |
| jp_at_hp1 | that is a use-case that is broken by current code | 14:33 |
| clarkb | stable branches are working for us just fine I think | 14:33 |
| jp_at_hp1 | the current code will do the wrong thing | 14:33 |
| clarkb | but it may be there is a corner case when you do the above overriding | 14:33 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: Add script to update .gitreview files https://review.openstack.org/174594 | 14:34 |
| clarkb | (also we aren't using this with a large set of stable branches yet, its mostly master to master everywhere iirc) | 14:34 |
| jp_at_hp1 | no - I'm pretty sure that (based off the comment) the current code *should* be broken., | 14:34 |
| clarkb | so back to an example | 14:34 |
| clarkb | A change to master is pushed, B depends on A change to stable/foo is pushed | 14:34 |
| jp_at_hp1 | yes - I'm 100% sure that in majority of cases my change is corerct. Really I want to know what the use case of the current code is | 14:34 |
| sdague | so... we're running 3 different stable branches upstream, + 3 different combinations of 2 branch jobs in grenade. And we're testing the right things. | 14:35 |
| clarkb | zuul-cloner will checkout B at stable/foo/zuulref, zuul-cloner will checkout A at stable/foo/zuulref | 14:35 |
| clarkb | jp_at_hp1: ^ that should work | 14:35 |
| clarkb | now if A does not have a stable/foo it will try to checkout master/zuulref | 14:35 |
| clarkb | that should also work | 14:36 |
| sdague | so it would be good to know what is working around this at our level if you think the code is going to give the wrong results | 14:36 |
| clarkb | sdague: well we aren't zuul clonering grenade yet but the intent is to do so | 14:36 |
| sdague | clarkb: oh, gotcha | 14:36 |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Generalize fingerprint for horizon bug 1447476 https://review.openstack.org/177290 | 14:37 |
| openstack | bug 1447476 in OpenStack Dashboard (Horizon) "No module named pathlib" [Critical,Fix released] https://launchpad.net/bugs/1447476 | 14:37 |
| clarkb | if A has been forced to use master via an override we should still checkout B at stable/foo/zuulref and A at master/zuulref | 14:37 |
| jp_at_hp1 | clarkb: you're not quite right there (unless it was just the comment that threw me off | 14:37 |
| clarkb | jp_at_hp1: I feel like I am missing the fourth use case to make your change clock | 14:37 |
| clarkb | er click | 14:37 |
| clarkb | jp_at_hp1: indicated branch will be master, zuul will checkout master/zuulref before anything else for that project | 14:38 |
| jp_at_hp1 | repo A we want not-master | 14:38 |
| clarkb | sure we can s/master/foobranch/ and I htink the answer is still the same | 14:38 |
| jp_at_hp1 | So if there is a change going through on A/master, but not on A/not-master, when we test we will be testing using rpo A/master/zuulref where what we *actually* want is A/not-master | 14:39 |
| jp_at_hp1 | precedence in first order is branch, second order if there is a zuulref for that branch | 14:40 |
| clarkb | jp_at_hp1: so thats actually a question for zuuls mergers and not the cloner | 14:40 |
| jp_at_hp1 | no - that is the code in the cloner that I fixed | 14:40 |
| clarkb | jp_at_hp1: if the merger makes a ref for A/not-master then your example is wrong | 14:40 |
| clarkb | and I don't know if it does that or not | 14:40 |
| jp_at_hp1 | no - it doesn't make a ref for A/not-master, because there is no change going through the gate on that repo | 14:41 |
| clarkb | iirc it will only if there is a change in the list on that branch somewhere | 14:41 |
| clarkb | ok | 14:41 |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Generalize fingerprint for horizon bug 1447476 https://review.openstack.org/177290 | 14:41 |
| jp_at_hp1 | but if a change is going through on A/master, it will make a ref for that, right? | 14:41 |
| openstack | bug 1447476 in OpenStack Dashboard (Horizon) "No module named pathlib" [Critical,Fix released] https://launchpad.net/bugs/1447476 | 14:41 |
| clarkb | jp_at_hp1: yes it definitely makes a ref on any branch for which there is a change | 14:41 |
| jp_at_hp1 | and then the job using the cloner will pull the master ref, which it doesn't want, instead of the indicated branch. | 14:41 |
| clarkb | https://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/functions.sh#n315 is what I am looking at now to see if this behavior is preexisting | 14:43 |
| clarkb | https://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/functions.sh#n343 appears to be the key bit there | 14:44 |
| clarkb | which is mimicked on lines ~139 of the old side of your change's diff | 14:44 |
| jp_at_hp1 | clarkb: cool - that function does exactly what my patch does... | 14:44 |
| jp_at_hp1 | so that basically says my patch is correct? | 14:45 |
| clarkb | jp_at_hp1: not sure yet | 14:45 |
| clarkb | jp_at_hp1: I think zuul-cloner attempts this behavior | 14:45 |
| jp_at_hp1 | well, I read that code, and my change makes zuul-cloner behave exactly like that function | 14:45 |
| clarkb | because on 139 it checks if the branch exists | 14:45 |
| clarkb | so in your example it should say yes I have branch stable/foo then make that the fallback zuul ref branch, it will fail to fetch that because no ref is made | 14:46 |
| jp_at_hp1 | look at the final conditional block in the set_project function | 14:47 |
| clarkb | then it should hit the else on 161 and checkout the fallback branch even though there was a master zuul ref | 14:47 |
| clarkb | so I think zuul-cloner already does what you want it to there | 14:47 |
| jp_at_hp1 | nope | 14:47 |
| clarkb | jp_at_hp1: but you never hit that else | 14:48 |
| clarkb | on 142 | 14:48 |
| clarkb | so master isn't even considered | 14:48 |
| jp_at_hp1 | Hmmm. | 14:48 |
| jp_at_hp1 | So either the comment or reading of the code is wrong | 14:48 |
| clarkb | as long as you have properly indicated you want a different branch and that branch exists the behavior you want appears present | 14:48 |
| clarkb | the comment is correct | 14:49 |
| *** zz_dimtruck is now known as dimtruck | 14:49 | |
| clarkb | its just not going to be evaluated completely on each run | 14:49 |
| jp_at_hp1 | no - the comment is wrong | 14:49 |
| phschwartz | Has anyone seen issues with jjb when not using the absolute latest jenkins stable where runs fail with a 400 when doing a job create? | 14:49 |
| clarkb | so it might be better to write a slightly more verbose comment with better branching | 14:49 |
| *** ashleighfarnham has joined #openstack-infra | 14:49 | |
| jp_at_hp1 | it doesn't consider a zuul ref on master ahead of the tip of the indicated branch | 14:49 |
| *** jamesmcarthur has joined #openstack-infra | 14:49 | |
| jp_at_hp1 | and I'm still not 100 % convinved :( | 14:50 |
| clarkb | jp_at_hp1: it does if the indicated branch does not exist | 14:50 |
| jp_at_hp1 | gimme 5 to read the code again. | 14:50 |
| clarkb | jp_at_hp1: so there is a branch | 14:50 |
| clarkb | which is why I think there is confusion, its not a fall through set of steps | 14:50 |
| jp_at_hp1 | clarkb: the comment is this: | 14:50 |
| jp_at_hp1 | The reference lookup is attempted in this order: 1) Zuul reference for the indicated branch 2) Zuul reference for the master branch 3) The tip of the indicated branch 4) The tip of the master branch | 14:51 |
| jp_at_hp1 | that is plain wrong if the actual behaviour is correct | 14:51 |
| clarkb | yes and 2) is IFF indicated branch does not exist | 14:51 |
| clarkb | otherwise its still correct | 14:51 |
| jp_at_hp1 | And I'll be happier to just correct the comment, if it is behaving properly ;-) | 14:51 |
| clarkb | er 4) is also IFF the indiicated branch does not exist | 14:51 |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Generalize fingerprint for horizon bug 1447476 https://review.openstack.org/177290 | 14:51 |
| openstack | bug 1447476 in OpenStack Dashboard (Horizon) "No module named pathlib" [Critical,Fix released] https://launchpad.net/bugs/1447476 | 14:51 |
| *** vhoward has quit IRC | 14:51 | |
| jp_at_hp1 | yes. So the comment *is* *wrong*. Phew - I'm not mad. | 14:52 |
| clarkb | its incomplete | 14:52 |
| clarkb | the order is correct | 14:52 |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for bug 1448067 https://review.openstack.org/177169 | 14:52 |
| *** vhoward has joined #openstack-infra | 14:52 | |
| openstack | bug 1448067 in grenade "Failed to shutdown neutron" [Undecided,Fix released] https://launchpad.net/bugs/1448067 - Assigned to Sean Dague (sdague) | 14:52 |
| clarkb | but there is a second condition that determines whether or not things are evaluated at all | 14:52 |
| *** arxcruz has quit IRC | 14:53 | |
| jp_at_hp1 | clarkb: shades of wrong are still wrong | 14:53 |
| clarkb | sureI am not saying don't fix it | 14:53 |
| clarkb | but the thesis we started with is the order was wrong | 14:53 |
| *** samuelBartel has joined #openstack-infra | 14:53 | |
| clarkb | whih is not the case | 14:53 |
| *** julim has joined #openstack-infra | 14:54 | |
| jp_at_hp1 | true - but that has nothing to do with the fact the comment is wrong ;-) | 14:54 |
| *** sabeen has joined #openstack-infra | 14:55 | |
| jp_at_hp1 | Glad the code is correct. I think a few comments to make it more decipherable will be good - I'll amend my review. | 14:55 |
| *** nelsnelson has joined #openstack-infra | 14:57 | |
| *** nelsnelson has quit IRC | 14:58 | |
| *** _nadya_ has joined #openstack-infra | 14:59 | |
| *** dangers_away is now known as dangers | 15:00 | |
| *** doug-fish has left #openstack-infra | 15:01 | |
| jp_at_hp1 | clarkb: maybe I should just change my commit message to say it makes the existing code more readable? ;-) | 15:01 |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Remove query for bug 1348204 https://review.openstack.org/177299 | 15:01 |
| openstack | bug 1327218 in OpenStack Compute (nova) "duplicate for #1348204 Volume detach failure because of invalid bdm.connection_info" [High,Fix released] https://launchpad.net/bugs/1327218 - Assigned to Matt Riedemann (mriedem) | 15:01 |
| clarkb | jp_at_hp1: yes, especially since the order change there isn't quite right either | 15:02 |
| clarkb | comment should read something like if indicated_branch exists: 1) try zuul ref for indicated branch 2) checkout indicated branch else 1) try zuul ref for master 2) checkout master | 15:03 |
| *** achanda has joined #openstack-infra | 15:03 | |
| jp_at_hp1 | yeah - I have changed that comment, that's easy | 15:03 |
| jp_at_hp1 | but the actual code just seems so over-complicated for no particular reason | 15:04 |
| clarkb | ya it may be simpler to basically write it as ^ | 15:04 |
| jp_at_hp1 | trying to add comments to it to describe what it is actually doing in a sane way is making my head hurt | 15:04 |
| *** Bsony_ has joined #openstack-infra | 15:04 | |
| *** Bsony has quit IRC | 15:06 | |
| jp_at_hp1 | so... fallback_zuul_ref is totally unnecessary if you check for the branch before getting it | 15:06 |
| jp_at_hp1 | that also simplifies the if statement at the end | 15:06 |
| jp_at_hp1 | because the flow is... | 15:06 |
| clarkb | ya you can just make zuul_ref and set it appropriately based on the existence of the branch | 15:07 |
| jp_at_hp1 | check for upstream branch, if not set branch to master | 15:07 |
| jp_at_hp1 | check for zuul ref on branch | 15:07 |
| jp_at_hp1 | use branch | 15:07 |
| *** davideagnello has quit IRC | 15:07 | |
| jp_at_hp1 | clarkb: ^^^ that is the change to make, isn't it? | 15:07 |
| clarkb | basically | 15:08 |
| *** achanda has quit IRC | 15:08 | |
| clarkb | I guess we are assuming upstream branch is the predetermined indicated branch | 15:08 |
| clarkb | so ya | 15:08 |
| jp_at_hp1 | yes | 15:08 |
| jp_at_hp1 | well - the code setting indicated branch is good, and won't change | 15:08 |
| clarkb | right its the checking for its existence after and setting the branch to operate on and the zuul ref | 15:09 |
| jp_at_hp1 | yup | 15:09 |
| jp_at_hp1 | ok - new patch in 5 - simples | 15:09 |
| *** sandywalsh has quit IRC | 15:10 | |
| *** dkranz has quit IRC | 15:11 | |
| *** dhritishikhar_ has joined #openstack-infra | 15:12 | |
| *** e0ne is now known as e0ne_ | 15:12 | |
| *** dhritishikhar has quit IRC | 15:12 | |
| *** dkranz has joined #openstack-infra | 15:12 | |
| *** jamesmcarthur has quit IRC | 15:12 | |
| openstackgerrit | Matthew Treinish proposed openstack/requirements: Add cap for tempest-lib so reqs syncs don't break the world https://review.openstack.org/177306 | 15:13 |
| *** baoli has quit IRC | 15:13 | |
| *** e0ne_ is now known as e0ne | 15:13 | |
| *** sabeen3 has joined #openstack-infra | 15:14 | |
| openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Abort failed cherry-pick and reverts https://review.openstack.org/173940 | 15:15 |
| *** doug-fish has joined #openstack-infra | 15:15 | |
| *** sabeen has quit IRC | 15:15 | |
| *** dannywilson has joined #openstack-infra | 15:16 | |
| *** Bsony_ has quit IRC | 15:17 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Remove query for bug 1348204 https://review.openstack.org/177299 | 15:18 |
| openstack | bug 1327218 in OpenStack Compute (nova) "duplicate for #1348204 Volume detach failure because of invalid bdm.connection_info" [High,Fix released] https://launchpad.net/bugs/1327218 - Assigned to Matt Riedemann (mriedem) | 15:18 |
| *** dhritishikhar_ has quit IRC | 15:18 | |
| *** david-ly_ is now known as david-lyle | 15:22 | |
| *** ihrachyshka has quit IRC | 15:23 | |
| *** hashar has quit IRC | 15:23 | |
| openstackgerrit | Julien Danjou proposed openstack-infra/project-config: Run Gnocchi devstack tests for stable/1.0 against Kilo https://review.openstack.org/176907 | 15:25 |
| *** nfedotov has quit IRC | 15:27 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add inventory command to shade https://review.openstack.org/175891 | 15:28 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a method for getting an endpoint https://review.openstack.org/177314 | 15:28 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Make keystone endpoints work like the others https://review.openstack.org/177315 | 15:28 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add early service fail and active check method https://review.openstack.org/177316 | 15:28 |
| *** cody-somerville has quit IRC | 15:29 | |
| *** e0ne is now known as e0ne_ | 15:29 | |
| fungi | heading out to lunch... back in a bit | 15:30 |
| fungi | btw, last call for infra outreachy internship ideas and mentor volunteers https://etherpad.openstack.org/p/infra-liberty-outreachy | 15:30 |
| *** e0ne_ is now known as e0ne | 15:30 | |
| *** rm_work|away is now known as rm_work | 15:30 | |
| fungi | when i get back from lunch i'm going to try to get up with vkmc about next steps | 15:30 |
| fungi | pleia2: jeblair: clarkb: mordred: SergeyLukjanov: jhesketh: everyone else ^ | 15:31 |
| mordred | fungi: I'm having lunch with jeblair in about 30 minutes - he's otherwise offline today | 15:31 |
| *** jlvillal has quit IRC | 15:31 | |
| fungi | mordred: i expected so | 15:31 |
| mordred | fungi: I'll try to grab suggestions from him at lunch? | 15:32 |
| fungi | we've been warned ;) | 15:32 |
| fungi | yes, please! | 15:32 |
| fungi | okay, back in a while | 15:33 |
| *** marcusvrn has quit IRC | 15:34 | |
| zaro | morning | 15:38 |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Convert hiera configuration to support public data https://review.openstack.org/177320 | 15:38 |
| *** liam_ has joined #openstack-infra | 15:39 | |
| *** liam_ is now known as Guest67926 | 15:39 | |
| *** Swami has joined #openstack-infra | 15:39 | |
| anteaya | morning zaro | 15:40 |
| *** cody-somerville has joined #openstack-infra | 15:40 | |
| *** andreykurilin__ has joined #openstack-infra | 15:40 | |
| *** vlaza has quit IRC | 15:41 | |
| *** marcusvrn has joined #openstack-infra | 15:41 | |
| *** yolanda has quit IRC | 15:43 | |
| *** Guest67926 has quit IRC | 15:43 | |
| *** Bsony has joined #openstack-infra | 15:43 | |
| *** ilyashakhat has quit IRC | 15:43 | |
| nibalizer | clarkb: you might find that interesting ^^ implements a really old spec | 15:44 |
| *** ilyashakhat has joined #openstack-infra | 15:44 | |
| *** MarkAtwood has joined #openstack-infra | 15:44 | |
| *** cdent has joined #openstack-infra | 15:45 | |
| *** MaxV has quit IRC | 15:45 | |
| *** _nadya_ has quit IRC | 15:46 | |
| *** cody-somerville has quit IRC | 15:48 | |
| mordred | nibalizer: so - that assumes that there will be one and only one hieradata dir yeah? | 15:48 |
| anteaya | fungi: I have added some thoughts | 15:48 |
| mordred | and it will be in /opt/system-config | 15:48 |
| nibalizer | mordred: yes | 15:48 |
| mordred | nibalizer: does hiera do symlinks well? | 15:49 |
| nibalizer | if that doesn't jive with whatever else is floating in /opt/system-config let me know | 15:49 |
| *** marcusvrn has quit IRC | 15:49 | |
| nibalizer | mordred: i think it can follow them ya | 15:49 |
| zaro | clarkb: just read the gerrit post, that's good news. | 15:49 |
| nibalizer | could test pretty easy | 15:49 |
| mordred | nibalizer: like, can we make /opt/system-config/hieradata a symlink to /etc/puppet/hieradata | 15:49 |
| clarkb | zaro: well fungi pointed out why it may be bad news too :) but it is good that mina may not be at fault | 15:49 |
| clarkb | zaro: that may be an exploitable DoS | 15:49 |
| mordred | nibalizer: main question there being that I'd be slightly scared to have a git checkout managed by puppet have a directory inside of it that wasn't part of that git repo and that losing would be bad | 15:50 |
| mordred | nibalizer: but if we can symlink, then ossum | 15:50 |
| nibalizer | mordred: huh so is /opt/system-config the repo or is that just a container and /opt/system-config/production is the repo? | 15:50 |
| mordred | oh - one sec | 15:50 |
| mordred | yah. I suck | 15:51 |
| mordred | nibalizer: you're right - /opt/system-config/production is the repo | 15:51 |
| *** sdake has joined #openstack-infra | 15:51 | |
| *** marcusvrn has joined #openstack-infra | 15:51 | |
| mordred | nibalizer: so this would put a common.yaml, a group and an fqdn dir in the root of the system-config repo, yeah? | 15:52 |
| nibalizer | i think hiera/{those things} | 15:52 |
| nibalizer | at least thats what i meant to do | 15:52 |
| mordred | oh - wow. I cannot read | 15:52 |
| mordred | yup. I see it | 15:53 |
| *** dtantsur is now known as dtantsur|afk | 15:53 | |
| nibalizer | ah yea i screwed up the comments at the bottom | 15:53 |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Convert hiera configuration to support public data https://review.openstack.org/177320 | 15:53 |
| mordred | your public data comment at the bottom | 15:53 |
| mordred | yeah | 15:53 |
| nibalizer | the fix ^ | 15:53 |
| mordred | yup. that reads well | 15:53 |
| mordred | I like it | 15:53 |
| mordred | it might be worthwhile adding a comment to the commit message that applying this change will need a manual maint task to be performed | 15:54 |
| mordred | actually - we could add a symlink right now, and then at some point if we like it, we could move the hieradata dir to /opt if we wanted | 15:54 |
| mordred | so that the patch can land whenever | 15:54 |
| mordred | nibalizer: actually - can we put environment in the private one too? we have those | 15:55 |
| thingee | clarkb: this landed. I think we spoke about the 0.1.0 tag having to manually published to pypi? https://review.openstack.org/#/c/176996/ | 15:56 |
| mordred | so "hierdata/%{::environment}/common" | 15:56 |
| mordred | nibalizer: ^^ | 15:56 |
| nibalizer | mordred: totes | 15:56 |
| * nibalizer will respin | 15:56 | |
| mordred | nibalizer: that way I can just do ln -s /etc/puppet/hieradata /opt/system-config | 15:56 |
| clarkb | thingee: yup we will need to trigger those jobs for you | 15:56 |
| clarkb | pleia2: when you wake is doing ^ something you are interested in? | 15:56 |
| clarkb | pleia2: I can walk you through it | 15:57 |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Convert hiera configuration to support public data https://review.openstack.org/177320 | 15:58 |
| thingee | clarkb, pleia2 thanks :) | 15:58 |
| mordred | nibalizer: +2 from me - nicely done! | 15:59 |
| mordred | nibalizer: I knew that one of these days we'd find the elegant way to do it | 15:59 |
| nibalizer | ty, ty its been a while since that spec has been looked at | 15:59 |
| mordred | yup | 15:59 |
| *** mriedem is now known as mriedem_nom | 15:59 | |
| *** marcusvrn has quit IRC | 16:00 | |
| *** otter768 has joined #openstack-infra | 16:00 | |
| zaro | pelix: what is your opinion on this? https://review.openstack.org/#/c/75514 | 16:00 |
| clarkb | so thats a dangerous change | 16:00 |
| *** asselin_ has quit IRC | 16:00 | |
| *** andreykurilin__ has quit IRC | 16:00 | |
| clarkb | because we use default hiera values for testing, then we move the hiera data away | 16:01 |
| *** jlanoux has quit IRC | 16:01 | |
| mordred | clarkb: what's dangerous about that/ | 16:01 |
| *** baoli has joined #openstack-infra | 16:01 | |
| mordred | ? | 16:01 |
| clarkb | nibalizer: mordred what is going to move the hiera data? just a manual step? | 16:01 |
| mordred | clarkb: yah | 16:01 |
| clarkb | mordred: we will reset all our DB passwords to known values | 16:01 |
| clarkb | mordred: because hiera will say "no data here" then use the default | 16:01 |
| mordred | oh - so - I've already made a symlink | 16:02 |
| mordred | so there will be zero seconds of time when the data is not in a location that we know about | 16:02 |
| mordred | because doing things in a different order would be suck | 16:02 |
| clarkb | ok, I am just calling it out because that change does not call it out | 16:03 |
| mordred | ++ | 16:03 |
| clarkb | and our downstream users may want to know this | 16:03 |
| nibalizer | clarkb: i dont understand your concern yet | 16:03 |
| clarkb | since site.pp is apparently a thing others use | 16:03 |
| mordred | cool - just making sure we were worried about the same thing | 16:03 |
| mordred | nibalizer: if the change lands without the corresponding hieradata move | 16:03 |
| clarkb | nibalizer: if we change the source of hiera data without also having data in that location the default hiera lookup values will be used | 16:03 |
| mordred | nibalizer: then a puppet run could happen in production that would wind up setting things to defaults | 16:03 |
| nibalizer | ah yea | 16:03 |
| clarkb | nibalizer: this will reset all database passwords and do a swarm of other nastyness | 16:03 |
| nibalizer | well cp works too | 16:04 |
| nibalizer | followed by rm | 16:04 |
| *** baoli_ has joined #openstack-infra | 16:04 | |
| anteaya | I'm doing some archeology work in the interest of untangling nova-net and neutron, some conversations took place on the mailing list in the first half of 2012 and our lists.openstack.org archives only go back to May 2012: http://lists.openstack.org/pipermail/openstack-dev/ | 16:04 |
| clarkb | sure, point is how to make sure that we don't pwn a bunch of downstreams (and ourselves though mordred seems to have taken care of us) | 16:04 |
| anteaya | might there be any other location where the -dev list was archived? | 16:04 |
| clarkb | anteaya: on launchpad | 16:04 |
| samueldmq | fungi, ping - you around ? | 16:04 |
| clarkb | anteaya: I think it was called openstack-dev there as well | 16:04 |
| *** otter768 has quit IRC | 16:04 | |
| samueldmq | fungi, iirc, you said me days ago that we had experimental jobs for keystone v3, am I right ? | 16:05 |
| zaro | pelix: it's a big jjb change, my concern is that unit test might not be enough. | 16:05 |
| *** marcusvrn has joined #openstack-infra | 16:05 | |
| nibalizer | clarkb: i think this is a systemic problem with passing those defaults in via site.pp ... ill do some thinking on a way to not do that | 16:05 |
| *** baoli has quit IRC | 16:06 | |
| odyssey4me | it would appear that the zuul status page isn't updating | 16:06 |
| * mordred does not really like the passing of defaults, just for the record | 16:06 | |
| *** Ala has quit IRC | 16:07 | |
| anteaya | clarkb: thank you | 16:07 |
| clarkb | odyssey4me: it appears to be updating for me (note the two queue counters in the top left, zuul is chewing on a backlog of events) | 16:07 |
| *** e0ne has quit IRC | 16:07 | |
| odyssey4me | the jobs do appear to be transitioning between jenkins and gerrit, so perhaps this is a graphite issue? | 16:07 |
| anteaya | clarkb: yes: https://lists.launchpad.net/openstack/thrd101.html | 16:07 |
| clarkb | mordred: nibalizer ya maybe we can just stop doing that entirely | 16:07 |
| odyssey4me | ah ok clarkb cool - just wanted to give a heads-up | 16:07 |
| *** smccully has quit IRC | 16:08 | |
| openstackgerrit | Sean Dague proposed openstack/requirements: update openstackclient requirement to 1.0.3 https://review.openstack.org/177337 | 16:08 |
| pelix | zaro: sure, guess the question is what additional tests do we want to add to give the level of confidence? | 16:09 |
| openstackgerrit | Merged openstack-infra/project-config: Make oslo.db py3 job voting https://review.openstack.org/176356 | 16:09 |
| *** gyee has joined #openstack-infra | 16:09 | |
| anteaya | I think I should go for a walk before heading into the dust of the past | 16:09 |
| pelix | zaro: I think the performance benefits are definitely worth investing in adding threading code to JJB, it's just a question of what is needed to ensure correct behaviour | 16:10 |
| openstackgerrit | Merged openstack-infra/project-config: Using Neutron network by default in gate-murano-congress-devstack-dsvm https://review.openstack.org/176398 | 16:10 |
| *** ZZelle has quit IRC | 16:11 | |
| clarkb | nibalizer: mordred two comments | 16:11 |
| *** smccully has joined #openstack-infra | 16:12 | |
| *** marun has joined #openstack-infra | 16:12 | |
| *** sandywalsh has joined #openstack-infra | 16:12 | |
| *** dprince has quit IRC | 16:13 | |
| clarkb | time for breakfast, pleia2 ping if/when you are interested in triggering zuul post jobs directly | 16:13 |
| openstackgerrit | Jon-Paul Sullivan (jp_at_hp) proposed openstack-infra/zuul: Make the cloner strategy clearer https://review.openstack.org/177201 | 16:18 |
| *** jogo is now known as flashgordon | 16:18 | |
| *** tjones1 has joined #openstack-infra | 16:19 | |
| asselin | BobBall, hi | 16:20 |
| jp_at_hp1 | clarkb: thanks a mill for the discussion earlier - I have an updated patch up that I think makes things clearer... | 16:20 |
| *** unicell has quit IRC | 16:20 | |
| *** tjones2 has joined #openstack-infra | 16:21 | |
| *** tjones1 has quit IRC | 16:22 | |
| *** rm_work is now known as rm_work|away | 16:22 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add inventory command to shade https://review.openstack.org/175891 | 16:23 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Raise a shade exception on broken volumes https://review.openstack.org/176550 | 16:23 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a method for getting an endpoint https://review.openstack.org/177314 | 16:23 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Make keystone endpoints work like the others https://review.openstack.org/177315 | 16:23 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add early service fail and active check method https://review.openstack.org/177316 | 16:23 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Replace e.message with str(e) https://review.openstack.org/177346 | 16:23 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Call super in OpenStackCloudException https://review.openstack.org/177347 | 16:23 |
| BobBall | #asselin: XenProject has been accepted for a BoF for the summit around the setting up and use a of 3rd party CI. The original submission has been lost (not quite sure how) so we wanted to take the opportunity to re-imagine the BoF, given the recent developments as well; we were thinking it would be most useful to have a group of us talking about and sharing what we each think are the best practices for setting one up. What are your t | 16:24 |
| *** dizquierdo has quit IRC | 16:28 | |
| zaro | pelix: i've tested and it looks good. do you feel the unit tests are adequate? | 16:29 |
| asselin | BobBall, what's BoF? | 16:29 |
| *** tjones2 has quit IRC | 16:29 | |
| asselin | and your comment got cut off. "What are your t" | 16:29 |
| asselin | BobBall, sorry back in a bit | 16:30 |
| BobBall | Birds of a feather; My understanding it's just a group of people with the same interests getting together to talk about a particular topic (i.e. not a formal presentation etc) | 16:30 |
| *** tjones1 has joined #openstack-infra | 16:30 | |
| *** tjones1 has quit IRC | 16:30 | |
| BobBall | asselin: Just said "What are your thoughts on that" :) | 16:30 |
| BobBall | np! | 16:30 |
| *** tjones1 has joined #openstack-infra | 16:30 | |
| *** jcoufal has quit IRC | 16:31 | |
| *** tjones1 has joined #openstack-infra | 16:32 | |
| *** tsg_ has joined #openstack-infra | 16:32 | |
| *** tjones1 has left #openstack-infra | 16:32 | |
| *** patrickeast has joined #openstack-infra | 16:32 | |
| *** shashankhegde has joined #openstack-infra | 16:32 | |
| *** Guest47857 has quit IRC | 16:32 | |
| *** jcoufal has joined #openstack-infra | 16:33 | |
| *** jcoufal has quit IRC | 16:36 | |
| *** jistr has quit IRC | 16:36 | |
| krtaylor | BobBall, we have been talking about a third party CI working group session for weeks, here is the etherpad -> https://etherpad.openstack.org/p/liberty-third-party-ci-working-group | 16:36 |
| rakhmerov | hi fungi, I wasn't able to attend the last infra team meeting but I read its log. So what do I need to do to move mistral to openstack space? | 16:36 |
| krtaylor | BobBall, it is proposed as a cross-project session -> https://docs.google.com/spreadsheets/d/1vCTZBJKCMZ2xBhglnuK3ciKo3E8UMFo5S5lmIAYMCSE/ | 16:36 |
| pelix | zaro: I think it's good enough, unless we can think of something else to test right now, I'd vote for adding the patch (once the conflict is resolved) and asking people to stress it a bit to shake out edge cases that we can't forsee right now. | 16:37 |
| BobBall | Yes; I've seen that and having a formal session is great. I was wondering if having a BoF on just sharing what each of us has done and what we think should be the best practices would be a useful complement to the working group session. My understanding of the original BoF submission was that it is effectively superceeded by that session | 16:37 |
| krtaylor | BobBall, part of this is being discussed as a infra design session (downstream-puppet) with asselin driving it | 16:37 |
| krtaylor | BobBall, a BoF is fine, but that is exactly the focus of the working group session | 16:39 |
| krtaylor | BobBall, if the cross-project session is not selected, then a BoF would be an excellent backup | 16:39 |
| krtaylor | BobBall, I do not envision the cross-project session to be a formal "presentation" format, a working session or worst case fish bowl if too big for all at one table | 16:40 |
| *** yamamoto has joined #openstack-infra | 16:42 | |
| BobBall | I saw a BoF to be maybe covering one of the 'improved ways to share' what we've each been doing but of course, as discussions often do, I can see that many of the things in that proposed session would be of interest. | 16:42 |
| *** unicell has joined #openstack-infra | 16:42 | |
| *** wenlock_ has joined #openstack-infra | 16:42 | |
| krtaylor | I do not know what ttx schedule is for choosing cross-project sessions, but I'd prefer that to get decided first | 16:43 |
| krtaylor | BobBall, is the BoF a done deal? | 16:43 |
| BobBall | It's been approved to happen, yes | 16:43 |
| openstackgerrit | Julien Danjou proposed openstack-dev/pbr: Support Unicode in requirements file https://review.openstack.org/177354 | 16:44 |
| krtaylor | BobBall, ah, ok, well then, if I might suggest that a show and tell for best practices be directed toward one of the working group meetings | 16:44 |
| *** dimtruck is now known as zz_dimtruck | 16:45 | |
| *** nelsnelson has joined #openstack-infra | 16:45 | |
| krtaylor | BobBall, we have a "Highlighting a System" section of the meeting that is geared up to share these topics | 16:45 |
| krotscheck | clarkb: So, after thinking about it, I actually agree with mordred that pbr should be aware of javascript things. To me it's less a question of "hey pbr should only do python things", and more a question of "package maintainers already trust pbr, and thus anything submitted there will be held to a higher standard to maintain that trust". For example, by enforcing the --single_version_externally_managed flag | 16:45 |
| *** nelsnelson has quit IRC | 16:46 | |
| krtaylor | BobBall, I'd suggest using f2f time to work out details of how to share, and brainstorming - just a thought | 16:46 |
| *** yamamoto has quit IRC | 16:46 | |
| krotscheck | clarkb: Also, because a "Python web app" is actually a thing people build frequently | 16:46 |
| clarkb | I was more concerned about the reasonableness aspect. particularly how to get the js back out again | 16:47 |
| clarkb | we can certainly be opinionated about that as pbr is full of opinions but I am not sure there is a good answer to that question | 16:47 |
| *** baoli_ has quit IRC | 16:47 | |
| *** baoli has joined #openstack-infra | 16:48 | |
| *** derekh has quit IRC | 16:48 | |
| *** tjones2 has joined #openstack-infra | 16:48 | |
| clarkb | do we not pull it out and expect webapps to refer to it with pkg_resources and serve it directly? or do we make a default install location on disk? options seems not good | 16:49 |
| clarkb | putting the js in the sdist is straightforward, run bower the include as package data | 16:49 |
| BobBall | krtaylor: I have to say I'm getting confused the more I think about it; it seems that the BoF is to be part of the main summit schedule not design summit, but clearly the appropriate audience for a 3rd party CI discussion is more the devs than users... *confused* | 16:51 |
| morganfainberg | samueldmq, we had experimental jobs for python3 in keystone not keystone v3 afaik | 16:51 |
| clarkb | krotscheck do you have plans for the install step? | 16:51 |
| *** achanda has joined #openstack-infra | 16:52 | |
| krotscheck | clarkb: I'm currently writing a hook which checks for the existence of a bower.json file and adds the necessary logic to resolve those dependencies on pip install. | 16:52 |
| clarkb | if the plan is to make package data then have python serve it, would it make more sense to just commit the files and sdist as package data? | 16:52 |
| krotscheck | clarkb: yes. | 16:53 |
| krotscheck | clarkb: YES. | 16:53 |
| *** zz_dimtruck is now known as dimtruck | 16:53 | |
| krotscheck | clarkb: But rock and hard place. | 16:53 |
| clarkb | ok then why the pbr change? | 16:53 |
| *** sdake has quit IRC | 16:53 | |
| samueldmq | morganfainberg, hmm, maybe I have confused myself then | 16:53 |
| *** Swami_ has joined #openstack-infra | 16:53 | |
| samueldmq | morganfainberg, I thought I had talked about that with fungi | 16:53 |
| samueldmq | thanks | 16:53 |
| *** davideagnello has joined #openstack-infra | 16:54 | |
| krotscheck | clarkb: Because I'm exploring options for how to make javascript things packageable, and if this is the path we end up going down I want to make sure it's feasible. | 16:54 |
| krotscheck | clarkb: I'm doing my research for vancouver. | 16:54 |
| *** bnemec is now known as beekneemech | 16:54 | |
| clarkb | krotscheck also keep in mind there are two steps to install, sdist to make a tarball (this is the easy bit) and the install step of copying files to where they will live on disk | 16:54 |
| *** dannywilson has quit IRC | 16:55 | |
| *** Swami has quit IRC | 16:55 | |
| *** dannywilson has joined #openstack-infra | 16:55 | |
| krotscheck | clarkb: Yeah, the various install steps aren't that well documented, so right now I've made the assumption that 'install' is the place to do that. | 16:55 |
| clarkb | I think the install command implies sdist and runs it for you | 16:56 |
| krotscheck | clarkb: Where this logic lives is certainly up for discussion. At the moment I need to have this hook make a best-effort to detect/install node & npm. | 16:56 |
| krotscheck | clarkb: And then it needs to invoke 'npm install', which resovles dependencies and anything else configured in the package. | 16:56 |
| *** pblaho has quit IRC | 16:57 | |
| krotscheck | clarkb: The assumption is that after that's done, all the files will be where they need to be. | 16:57 |
| clarkb | thats a bad assumption | 16:57 |
| morganfainberg | samueldmq, once devstack can deploy a v3-only keystone we will do an experimental job | 16:57 |
| clarkb | they will end up in the sdist | 16:57 |
| morganfainberg | samueldmq, then we can chase down all the other fixes needed. | 16:57 |
| rakhmerov | sorry guys for jumping in.. clarkb: do you know who can help with stackforge -> openstack renaming? | 16:57 |
| krotscheck | clarkb: *waves hands around* | 16:57 |
| clarkb | but apache/nginx/etc wont know how to get at those files to serve them | 16:58 |
| *** sarob has joined #openstack-infra | 16:58 | |
| krotscheck | clarkb: Ok, if you were writing this hook, and the end goal is "pip install ." results in a fully functional web app, what would you do? | 16:58 |
| *** nelsnelson has joined #openstack-infra | 16:58 | |
| krotscheck | clarkb: (With the caveat that whoever runs pip install needs to have reasonable clue on how to setup a vhost/etc. | 16:58 |
| clarkb | krotscheck put files in an sdist so that the sdist step works. this allows ypu to pypi | 16:58 |
| clarkb | but I have no good answer for installing the js at install time | 16:59 |
| samueldmq | morganfainberg, well, to deploy a v3 only keystone is just to remove the /v2.0 apps in paste-ini | 16:59 |
| samueldmq | morganfainberg, that's how I did manually | 16:59 |
| morganfainberg | samueldmq, no there is a lot of devstack stuff that uses v2 apis even if v3 is specified in stack.sh | 16:59 |
| clarkb | the simplest approach is to force applications to serve the js out of their packages | 16:59 |
| morganfainberg | samueldmq, i'm mostly done with a fix to solve this. | 16:59 |
| morganfainberg | samueldmq, but it's whack-a-mole | 16:59 |
| pleia2 | clarkb: sorry, have a bit of a cold so I'm slower moving this morning, have you triggered the zuul jobs yet? | 16:59 |
| *** gmurphy has left #openstack-infra | 16:59 | |
| * fungi is back. the party can resume ;) | 17:00 | |
| morganfainberg | pleia2, no! Feel better! | 17:00 |
| fungi | anteaya: thanks for the input! | 17:00 |
| morganfainberg | pleia2, not allowed to be sick! :) | 17:00 |
| clarkb | but that enforces an opinion on hosting static files that I expect most to dislike | 17:00 |
| clarkb | pleia2 I have not | 17:00 |
| pleia2 | morganfainberg: my husband was home from work 3 days because of it, I had hoped to retain immunity, alas! | 17:00 |
| clarkb | also feel better | 17:00 |
| samueldmq | morganfainberg, great ! you're coming with devstack and I am coming with the working services with v3 (just testing so far) | 17:00 |
| morganfainberg | pleia2, oh gah! seriously feel better!! | 17:00 |
| fungi | samueldmq: i don't recall mentioning experimental keystone jobs for either v3 api or python 3.x, but i also have a terrible memory | 17:00 |
| samueldmq | morganfainberg, and we will converge soon :) | 17:00 |
| pleia2 | clarkb: I'm feeling ok, let's walk through this :) | 17:00 |
| openstackgerrit | Jon-Paul Sullivan (jp_at_hp) proposed openstack-infra/zuul: Make the cloner strategy clearer https://review.openstack.org/177201 | 17:01 |
| morganfainberg | fungi, we had py3 for a while.. we removed them because $REASONS or made them expirimental | 17:01 |
| samueldmq | fungi, k, maybe I got confused :p sorry | 17:01 |
| krotscheck | clarkb: Is there a place where the order of commands is documented? | 17:01 |
| morganfainberg | fungi, but i expect we will be py3 compat in liberty (fully). we are ~2 libraries away afaict | 17:01 |
| morganfainberg | both with py3 friendly alternatives | 17:01 |
| clarkb | krotscheck: I have no idea, setuptools (and distutils) are dark magic and I find most of the details are undocumetned | 17:02 |
| clarkb | krotscheck: setuptools does have docs though | 17:02 |
| *** tsg_ has quit IRC | 17:02 | |
| clarkb | krotscheck: anyways I think an important thing is to have the sdist step package up the files so that if someone installs off of pypi they do not need npm and node and bower, they just need python | 17:02 |
| *** tsg_ has joined #openstack-infra | 17:02 | |
| krotscheck | clarkb: I agree. | 17:02 |
| *** vhoward has left #openstack-infra | 17:03 | |
| krotscheck | clarkb: And hopefully the sdist command will check for the necessary tools. | 17:03 |
| fungi | rakhmerov: if nobody's talked to you yet, mainly you just need to git grep in the openstack-infra/project-config repo for places where your project name is mentioned, and then update any occurrences of stackforge with openstack and submit it for review. also link that review url on our meeting agenda wiki page so we can discuss it at one of our meetings when we plan our next gerrit maintenance (which | 17:03 |
| fungi | won't be until kilo is released, so as not to impact the release process at this crucial stage) | 17:03 |
| *** marcusvrn1 has joined #openstack-infra | 17:04 | |
| rakhmerov | fungi, I have already done all that | 17:04 |
| krotscheck | Know what? I'm going to let my subconscious chew on this one until monday and do storyboard things instead. | 17:04 |
| *** marcusvrn has quit IRC | 17:04 | |
| clarkb | krotscheck: for the sdist step you would implement a hook into there (either as a pbr hook or in pbr directly) that makes sure all of the files are present then makes sure the files are added to the package as package data | 17:04 |
| fungi | clarkb: krotscheck: you might be interested in the recent thread on distutils-sig about classifying python-packaged file types (for fhs support, but more generally so that platform-specific deployment locations can be declared by the platforms where the package is being repackaged or installed) | 17:04 |
| clarkb | krotscheck: that should get everything into the pckage. | 17:05 |
| *** yolanda has joined #openstack-infra | 17:05 | |
| clarkb | but then on the install side you have to decide where those files go | 17:05 |
| clarkb | dstufft did say warehouse is just serving the js directly from python and not putting the files anywhere outside of the pckage itself | 17:05 |
| rakhmerov | fungi, the patch is https://review.openstack.org/#/c/175328/ and at the last meeting I was told to go to this channel directly and talk to some of you about the further steps | 17:05 |
| rakhmerov | so I'm here | 17:05 |
| clarkb | that will work as long as your python application is aware that it is its job to serve the static files and that they live in its package data | 17:06 |
| krotscheck | clarkb: Well, there are different patterns there, which the setup config file should handle. | 17:06 |
| *** shashankhegde has quit IRC | 17:06 | |
| krotscheck | clarkb: Something like a config block with a install_my_static_files_here option or something | 17:06 |
| *** ssam2 has quit IRC | 17:07 | |
| fungi | rakhmerov: yep, those were the further steps | 17:07 |
| fungi | rakhmerov: now we just need to wait for an opportunity to schedule the work around it | 17:07 |
| clarkb | krotscheck: I am not sure if you can feed setup.py config at install time. Or rather I don't think that has been implemented anywhere so don't have good prior art | 17:07 |
| *** [HeOS] has quit IRC | 17:07 | |
| rakhmerov | fungi, ok, I'll be waiting | 17:07 |
| rakhmerov | thanks | 17:07 |
| *** HeOS has joined #openstack-infra | 17:08 | |
| fungi | rakhmerov: please add it at https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting#Agenda_for_next_meeting | 17:08 |
| clarkb | krotscheck: pbr can probably do it but I would be surprised if setuptools had direct support for that | 17:08 |
| *** dimtruck is now known as zz_dimtruck | 17:08 | |
| krotscheck | clarkb: pbr does it via hooks. | 17:08 |
| rakhmerov | fungi, ok, I'll do right away | 17:08 |
| fungi | rakhmerov: after the meeting i updated the agenda moving your topic to the renames section, but at the time didn't know the url | 17:08 |
| krotscheck | clarkb: I believe the config block is passed through. I might be wrong | 17:08 |
| clarkb | krotscheck: pbr packages don't take any config at install time iirc | 17:08 |
| rakhmerov | fungi: got it | 17:09 |
| clarkb | krotscheck: the config is baked at sdist | 17:09 |
| fungi | rakhmerov: so where i have "project-config review link goes here" just replace that with the url | 17:09 |
| *** mriedem_nom is now known as mriedem_rmorse | 17:09 | |
| *** mriedem_rmorse is now known as mriedem | 17:09 | |
| *** tsg_ has quit IRC | 17:10 | |
| *** tnovacik has joined #openstack-infra | 17:10 | |
| rakhmerov | fungi: done | 17:10 |
| *** tsg_ has joined #openstack-infra | 17:10 | |
| krotscheck | clarkb: Well, given that distros end up rebuilding their own things anyway, I'm not too concerned about that. Their version of insert-pythonlib-here could install to /var/www, anothers to /lib/www, etc. The pip version... well, that's up to the package maintainer really. | 17:11 |
| krotscheck | I mean the code maintainer | 17:11 |
| clarkb | krotscheck: its not distros you worry about its anyone doing `pip install refstack` | 17:11 |
| pleia2 | clarkb: assuming it's the right script, I have notes on using /opt/zuul/tools/trigger-job.py but crafting the appropriate trigger job is something I'm still learning | 17:11 |
| clarkb | because that will operate on the sdist (or whl) on pypi | 17:11 |
| clarkb | pleia2: kk let me pull of the tag and see what the params should look like | 17:12 |
| *** tjones2 has quit IRC | 17:12 | |
| *** sdake has joined #openstack-infra | 17:12 | |
| krotscheck | clarkb: Yep, not worried about that. Installing something like refstack is going to require additional configuration anyway, which means file locations will need to be documented no matter what. | 17:12 |
| *** rm_work|away is now known as rm_work | 17:12 | |
| pleia2 | clarkb: the easy bits, --job publish-to-pypi --project openstack/os-brick --pipeline post (right?) | 17:13 |
| clarkb | pleia2: close, the job is actually going to be the tarball job first, we then wait for the tarball to be on tarballs.openstack.org then we run the pypi upload job | 17:13 |
| krotscheck | clarkb: And since the preferred install of something like that is in a private venv anyway, anyone configuring the webserver has access to set the uid/gid on the files anyway and can make sure insert-webserver-of-choice can host it. | 17:13 |
| clarkb | pleia2: we also have to feed it the tag as the ref iirc | 17:13 |
| krotscheck | (Unless they're installing as root, in which case I have no pity for them) | 17:14 |
| pleia2 | clarkb: nods | 17:14 |
| clarkb | krotscheck: sure, you can install it that way and rely on documentation to have people configure webservers | 17:14 |
| clarkb | or node or whatever is using the js | 17:14 |
| *** jp_at_hp1 has quit IRC | 17:15 | |
| clarkb | pleia2: my only other real concern is I think there may be a different script to use beacuse the fixes for something never got commited to git :/ | 17:15 |
| clarkb | pleia2: but I think that isn't actually a problem | 17:16 |
| pleia2 | clarkb: trigger-job.py does seem pretty reliant on having git hashes | 17:17 |
| clarkb | pleia2: ya I think we feed it the git hash of the commit the tag points to, pbr then generates the version properly because of git | 17:17 |
| clarkb | one sec making the command now | 17:18 |
| pleia2 | ok | 17:18 |
| *** dprince has joined #openstack-infra | 17:18 | |
| pleia2 | aha, publish-to-pypi is a template, need to break it down into it's jobs for this | 17:18 |
| *** tjones1 has joined #openstack-infra | 17:19 | |
| pleia2 | cinerama: whenever you're around, I can approve 176578 and we'll keep an eye on how it does | 17:20 |
| fungi | pleia2: clarkb: btw if you grep through my .bash_history on zuul for trigger-job.py you'll see plenty of release pipeline examples | 17:20 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/infra-manual: Add clarification on questions in reviews https://review.openstack.org/177364 | 17:20 |
| clarkb | `/opt/zuul/tools/trigger-job.py --job os-brick-tarball --project openstack/os-brick --pipeline release --refname 0.1.0 --newrev 0b89d9b3e7d899f318eadbeb866f6f675f20ead0 then we need a logpath | 17:21 |
| clarkb | fungi: oh thats handy | 17:21 |
| cinerama | pleia2: i am here | 17:21 |
| *** bhuvan has joined #openstack-infra | 17:21 | |
| clarkb | ah the newrev should be for the tag itself and refname is refs/tags/0.1.0 | 17:22 |
| fungi | yep | 17:22 |
| clarkb | pleia2: ok constructing the actual command for tarball job now | 17:22 |
| *** ociuhandu has joined #openstack-infra | 17:22 | |
| pleia2 | clarkb: ok, once you have it, we can maybe walk through where you got each value so I can do a better job of looking them up myself | 17:23 |
| *** zz_dimtruck is now known as dimtruck | 17:23 | |
| fungi | normally when you're retriggering a job which already ran, it's a lot easier because you can just look in jenkins for the parameters which were passed in the previous run | 17:24 |
| * pleia2 nods | 17:24 | |
| *** dguerri is now known as _dguerri | 17:24 | |
| clarkb | sudo /opt/zuul/tools/trigger-job.py --url=http://review.openstack.org/p --job os-brick-tarball --project openstack/os-brick --pipeline release --newrev 89a6b0ae7a61ffa697555306b154119915f0cac0 --refname refs/tags/0.1.0 --logpath 89/89a6b0ae7a61ffa697555306b154119915f0cac0/release/os-brick-tarball/manual-run-1 | 17:24 |
| clarkb | yup definitely can walk through what each of those things is | 17:24 |
| fungi | so while i _do_ know how to figure out the various values to pass, it's often easier to just cut-n-paste them from the failed/broken/whatever run (this case where the job never ran/existed at all is a little harder) | 17:25 |
| *** _dguerri is now known as dguerri | 17:25 | |
| pleia2 | clarkb: do we need a --url ? | 17:26 |
| clarkb | pleia2: url is the base url to fetch the ref from. it will be joined with the value of project to get the full path | 17:26 |
| pleia2 | ok | 17:26 |
| *** salv-orlando has quit IRC | 17:26 | |
| clarkb | pleia2: job is the name of the job, you can work backwards from the zuul layout to find what the name is for that I actually hopped on jenkins.o.o and searched for os-brick because that was quick | 17:26 |
| clarkb | project is the project name and needs the org prefix thing | 17:26 |
| clarkb | pipeline is release because that is the pipeline these jobs will run in the zuul layout | 17:27 |
| clarkb | you have to work back from the template to get that | 17:27 |
| *** Krinkle|detached is now known as Krinkle | 17:27 | |
| pleia2 | clarkb: oh, I see the url now, in my previous work with this it was a zm0* url, not review | 17:27 |
| clarkb | pleia2: ya we can't use zm0X because these jobs were never triggered and won't have refs on any zm0X, but gerrit iwll have them so we use it instead | 17:27 |
| pleia2 | gotcha | 17:27 |
| fungi | right, normally zuul will have asked a merger to prepare the refs for a job | 17:28 |
| fungi | just not in this case | 17:28 |
| clarkb | pleia2: newrev is the sha1 ofthe tag, not the sha1 of the commit the tag points to. You can get that from git by doing a `git show-ref` | 17:28 |
| fungi | luckily for post/tag/pre-release/release pipelines the zuul ref is of little consequence since it should be identical to something in gerrit or our git mirrors | 17:28 |
| clarkb | pleia2: I figured that out by looking at fungi's examples on zuul.o.o | 17:28 |
| clarkb | I also figured out that the refname needed the full path of refs/tags/0.1.0 by looking at that example | 17:29 |
| fungi | you can also look at the jenkins parameters from recent runs of another similar job for a different repo for more examples | 17:29 |
| *** melwitt has joined #openstack-infra | 17:29 | |
| clarkb | pleia2: finally the log path is where to copy the files, this is mostly arbitrary but best to make it follow the pattern the job normally would've taken. That is first_2_of_sha1/sha1/pipeline/jobname/someuuid | 17:30 |
| clarkb | this really makes me wnat to write a patch to zuul enqueue to handle releaes | 17:30 |
| pleia2 | ok, confirming now that I can look all of these up properly and get the same results as you | 17:30 |
| *** cdent has quit IRC | 17:30 | |
| clarkb | awesome | 17:30 |
| clarkb | pleia2: since the tarball job must run before the upload job we can start with the above command then when it is done update the command to run the upload job | 17:31 |
| clarkb | and we can confirm that the tarball is on tarballs.openstack.org before running the upload as well | 17:31 |
| * pleia2 nods | 17:32 | |
| clarkb | since enqueue doesn't grok all this release stuff yet an short term thing to do may be to push a patch to the trigger script with better help text for each of these fields | 17:33 |
| pleia2 | clarkb: ok, easy enough, all lgtm | 17:34 |
| pleia2 | I can run this on zuul.o.o now if we're all good | 17:34 |
| clarkb | pleia2: yup go for it | 17:34 |
| pleia2 | running, no errors yet, but I think --url=http... should have been url http... | 17:38 |
| clarkb | pleia2: I think argparse will do the correct thing for us either way | 17:39 |
| pleia2 | finished ok, and we now have http://tarballs.openstack.org/os-brick/ | 17:39 |
| clarkb | woot | 17:39 |
| clarkb | logs don't show up, so lets find the jenkins console log before the next thing | 17:39 |
| clarkb | (its likely because we didn't pass in swift stuff | 17:39 |
| pleia2 | ok, just start searching through the jenkinsXXs? | 17:40 |
| clarkb | ya :/ https://jenkins06.openstack.org/job/os-brick-tarball/ is the base link and just increment the number as you go | 17:40 |
| clarkb | 06 is the winner looks like | 17:40 |
| fungi | passing in swift stuff would i think in this case also mean doing the credential issuing dance zuul normally performs, so likely not worth trying | 17:40 |
| clarkb | and confirmed that swift log uploads bailed out because we didn't pass the info (second to last line of the console log) | 17:41 |
| *** tqtran has joined #openstack-infra | 17:41 | |
| pleia2 | ah | 17:41 |
| clarkb | fungi: ++ for these sorts of cleanups I am not super worried about it | 17:41 |
| fungi | and yeah, i can't think of any uses of the trigger-job script which wouldn't better be served by adding support to the enqueue subcommand | 17:41 |
| fungi | oh, other than "testing gearman" | 17:42 |
| *** shashankhegde has joined #openstack-infra | 17:42 | |
| clarkb | pleia2: I have also confirmed the tarball and whl are present at http://tarballs.openstack.org/os-brick/ | 17:42 |
| *** r-daneel has joined #openstack-infra | 17:42 | |
| clarkb | so I think we are ready to run the next job which is the one that wil upload to pypi | 17:42 |
| pleia2 | great | 17:42 |
| clarkb | for that we want to run the 'os-brick-pypi-both-upload' job | 17:43 |
| pleia2 | similar command but --job os-brink-pypi-wheel-upload ? | 17:43 |
| clarkb | pleia2: you should be able to just replace os-brick-tarball with ^ in the previous command | 17:43 |
| pleia2 | cool | 17:43 |
| cinerama | pleia2: feel free to pull the trigger on that | 17:43 |
| clarkb | pleia2: and in the log url (though that likely won't matter) | 17:43 |
| pleia2 | ah yes, log url too | 17:43 |
| cinerama | pleia2: worst case we can nuke everything & reinstall | 17:43 |
| *** prad_ has joined #openstack-infra | 17:44 | |
| pleia2 | clarkb: both-upload or wheel-upload? (template I'm looking at has wheel) | 17:44 |
| *** prad has quit IRC | 17:44 | |
| *** prad_ is now known as prad | 17:44 | |
| clarkb | pleia2: for pre-release we do only wheel, for release we do both-upload | 17:44 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: Add update_machine method https://review.openstack.org/173996 | 17:44 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: Add Ironic machine power state pass-through https://review.openstack.org/172284 | 17:44 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: Add Ironic maintenance state pass-through https://review.openstack.org/172283 | 17:44 |
| pleia2 | ok, so we want both-upload here | 17:44 |
| clarkb | pleia2: the reason for that was old pip doesn't know how to install wheels and new pip won't install pre releases by default | 17:45 |
| clarkb | pleia2: so by uploading wheels only for pre releases we ensure no one uses a pre release without explicitly wanting it | 17:45 |
| fungi | however it turns out that plan has a downside | 17:45 |
| fungi | pypi still shows the wheel-only prerelease if we hide all but the most recent release | 17:45 |
| fungi | we've also all but abandoned use of pre-release tags anyway | 17:46 |
| clarkb | ya | 17:46 |
| pleia2 | alright, doing this one now | 17:47 |
| *** cody-somerville has joined #openstack-infra | 17:47 | |
| *** cody-somerville has quit IRC | 17:47 | |
| *** cody-somerville has joined #openstack-infra | 17:47 | |
| *** rm_work is now known as rm_work|away | 17:47 | |
| *** dhritishikhar has joined #openstack-infra | 17:47 | |
| *** bcourt has quit IRC | 17:48 | |
| pleia2 | seems good https://pypi.python.org/pypi/os-brick/ | 17:49 |
| clarkb | yup lgtm | 17:49 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: Add Ironic maintenance state pass-through https://review.openstack.org/172283 | 17:49 |
| clarkb | pleia2: thansk for the help | 17:49 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: Add Ironic machine power state pass-through https://review.openstack.org/172284 | 17:49 |
| *** bcourt has joined #openstack-infra | 17:49 | |
| pleia2 | clarkb: thank you! | 17:50 |
| clarkb | should probably let thingee know and have cinder/brick do any checking they want to do on their side | 17:50 |
| clarkb | but this looks gone to me | 17:50 |
| clarkb | s/gone/done/ | 17:50 |
| thingee | pleia2, clarkb thanks!! | 17:50 |
| *** harlowja_away is now known as harlowja | 17:51 | |
| *** tnovacik has quit IRC | 17:51 | |
| pleia2 | cinerama: approved, fingers crossed :) | 17:52 |
| harlowja | mordred u were wrong and i was right on https://review.openstack.org/#/c/176459/ ? cool | 17:53 |
| harlowja | that whole shade task thing is similar to https://docs.python.org/dev/library/concurrent.futures.html#future-objects (and similar to taskflow task also, ha) | 17:54 |
| harlowja | perhaps u guys should use taskflow :-P | 17:54 |
| openstackgerrit | Matt Riedemann proposed openstack-infra/system-config: Index screen-barbican logs https://review.openstack.org/177371 | 17:55 |
| openstackgerrit | Mike Perez proposed openstack/requirements: Adding os-brick for managing local volumes https://review.openstack.org/177372 | 17:55 |
| openstackgerrit | Merged openstack-infra/puppet-zanata: Update to 3.6.2 https://review.openstack.org/176578 | 17:56 |
| clarkb | harlowja: except I think this is all synchronous | 17:57 |
| harlowja | *for now* ha | 17:57 |
| clarkb | harlowja: the idea being that your api rate limit should control the execution rate | 17:57 |
| openstackgerrit | Merged openstack-infra/project-config: Set the proper homepage for akanda repositories. https://review.openstack.org/176386 | 17:57 |
| openstackgerrit | Merged openstack-infra/project-config: Set the proper homepage for akanda repositories. https://review.openstack.org/176386 | 17:57 |
| clarkb | no its intentional | 17:57 |
| mrmartin | re | 17:57 |
| clarkb | you can't just async full speed ahead at oepnstack, the api explodes and badness happens | 17:57 |
| *** btran has joined #openstack-infra | 17:58 | |
| harlowja | ya, thats why https://review.openstack.org/#/c/172246/12/oslo_utils/speedutils.py is useful for folks (hopefully) | 17:58 |
| harlowja | https://github.com/openstack/taskflow/blob/master/taskflow/types/futures.py#L184 (also synchronous, but using the same futures interface) | 17:59 |
| harlowja | anyways, maybe ^ not relevant for u guys, either way | 18:00 |
| *** dhritishikhar has quit IRC | 18:00 | |
| clarkb | speedutils doesn't seem to natively grok thread safe queues? | 18:01 |
| *** otter768 has joined #openstack-infra | 18:01 | |
| harlowja | unsure what u thinking | 18:01 |
| clarkb | Queue.Queue.get(timeout) raises exceptions | 18:01 |
| harlowja | it just limits the rate at which some wrapped iterator returns values | 18:01 |
| clarkb | which would bubble out and break your iteration | 18:01 |
| harlowja | ok ? | 18:02 |
| clarkb | I guess you can just handle that by restarting iteration after whatever timeout handling needs to happen | 18:02 |
| clarkb | harlowja: right it might be nice to say stopiteration cleanly in that case | 18:02 |
| harlowja | hmmm, think i know what u mean | 18:02 |
| harlowja | future improvements then :-P | 18:02 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/system-config: Use common jenkins master implementation https://review.openstack.org/177373 | 18:02 |
| flashgordon | pleia2: for getting out the vote | 18:03 |
| flashgordon | pleia2: can you re-email people 24 after the polls open if they didn't vote? | 18:04 |
| flashgordon | as in is that possible | 18:04 |
| *** dhritishikhar has joined #openstack-infra | 18:04 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/puppet-openstackci: Create a jenkins server like http://jenkins.openstack.org https://review.openstack.org/175571 | 18:05 |
| *** ZZelle has joined #openstack-infra | 18:05 | |
| *** otter768 has quit IRC | 18:05 | |
| pleia2 | flashgordon: there isn't currently a mechanism for that in civs, it would all be manual | 18:06 |
| pleia2 | we can resend individual ballots one by one | 18:06 |
| *** sandywalsh has quit IRC | 18:06 | |
| pleia2 | we can send reminders to the whole dev list, and I'm on twitter patrol with it (busy morning!) | 18:07 |
| *** sandywalsh has joined #openstack-infra | 18:08 | |
| *** hemna has joined #openstack-infra | 18:08 | |
| *** dguerri is now known as _dguerri | 18:09 | |
| *** erikmwilson is now known as Guest8853 | 18:11 | |
| *** erikmwilson has joined #openstack-infra | 18:11 | |
| *** erikmwilson_ has joined #openstack-infra | 18:11 | |
| *** Swami has joined #openstack-infra | 18:12 | |
| *** erikmwilson is now known as Guest7383 | 18:12 | |
| *** erikmwil_ has joined #openstack-infra | 18:12 | |
| *** Guest7383 has quit IRC | 18:12 | |
| *** erikmwil_ is now known as erikmwilson | 18:12 | |
| *** HeOS has quit IRC | 18:13 | |
| *** Swami_ has quit IRC | 18:13 | |
| *** vinkman has joined #openstack-infra | 18:13 | |
| openstackgerrit | Joshua Harlow proposed openstack-infra/release-tools: Include 'source_url' in notes and use readme parser function https://review.openstack.org/176528 | 18:14 |
| *** vinkman has left #openstack-infra | 18:14 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 18:15 | |
| *** ivar-lazzaro has joined #openstack-infra | 18:16 | |
| SpamapS | fungi: well played (re +2 comment ;) | 18:16 |
| openstackgerrit | Mike Perez proposed openstack/requirements: Adding os-brick for managing initiators https://review.openstack.org/177372 | 18:16 |
| fungi | SpamapS: well played (re improving our documentation) | 18:17 |
| *** ivar-lazzaro has quit IRC | 18:19 | |
| *** tjones1 has quit IRC | 18:19 | |
| *** ivar-lazzaro has joined #openstack-infra | 18:20 | |
| tristanC | asselin: greeting sir, I'd like your opinion on the jenkins master refactor I proposed here: https://review.openstack.org/175571/ | 18:20 |
| openstackgerrit | Thai Tran proposed openstack-infra/project-config: Create a repository for stackforge/xstatic-nvd3 https://review.openstack.org/176565 | 18:20 |
| asselin | tristanC, hi, thanks for the ping. I will take a look! | 18:21 |
| openstackgerrit | Thai Tran proposed openstack-infra/project-config: Create a repository for stackforge/xstatic-angular-nvd3-directives https://review.openstack.org/176563 | 18:22 |
| tristanC | cool, feedback are most welcome, puppet stuff are pretty new to me :) | 18:22 |
| clarkb | SpamapS: fungi I was hoping not to get sucked into that thread but I mostly disagree with the idea that I can't -1 because I have questions | 18:22 |
| clarkb | I wrote feedback on the docs update | 18:23 |
| cinerama | pleia2: the footer still says we're on 3.6.0...how often does puppet run? | 18:23 |
| pleia2 | cinerama: you can keep an eye on it by going to http://puppetboard.openstack.org/nodes | 18:23 |
| clarkb | as a core reviewer for a project it is my job to understand the changes going into it. If I don't understand them that is worthy of a -1 because something needs to be addressed. Either the proposal itself, my understanding of the code, documentation etc | 18:23 |
| SpamapS | clarkb: I'm glad to have your dissent, and I think through the two positions we'll find goal posts that the readers can aim to land somewhere between. :) | 18:23 |
| pleia2 | cinerama: hm, seems we're not running puppet there regularly for some reason | 18:24 |
| clarkb | left a suggestion that -1's come with concrete actionable feedback though so others can be confident if concerns have been addressed even if the original source of the -1 is no longer active/present/revieweing | 18:24 |
| fungi | clarkb: that sounds like concrete, actionable feedback | 18:24 |
| cinerama | clarkb, SpamapS: so my contention with -1 for questions is in a situation where the reviewer is asking a question about how a third-party tool or library works. it is frustrating to get held up on that | 18:25 |
| SpamapS | I wonder if we should also time-out -1's | 18:25 |
| *** tsg_ has quit IRC | 18:25 | |
| clarkb | fungi: I could do "I don't understand why this code change is made. Can you explain why function foo() needs to be called here?" | 18:25 |
| SpamapS | like, if a -1 has been sitting there for 45 days, and author has responded, we could just remove it. | 18:25 |
| fungi | clarkb: and i agree, as i said in the thread, we should avoid questions for the sake of rhetoric and hyperbole, because they don't translate well in an international/cross-cultural context | 18:25 |
| clarkb | fungi: so not a nebulous "I don't understand" but instead a concrete "this is what I don't get, please explain it to the rest of us" | 18:25 |
| pleia2 | clarkb, fungi - did I make a mistake/miss a step when setting up translate-dev, or did we turn off puppet somewhere along the way? | 18:26 |
| cinerama | clarkb, SpamapS: if the reviewer is -1 on grounds of the code in the patch itself i think that is fair enough | 18:26 |
| clarkb | pleia2: oh! I think we still haven't accepted the ssh key | 18:26 |
| clarkb | pleia2: so ansible is failing to log in | 18:26 |
| clarkb | pleia2: on the puppet master you can check /var/log/puppet_run_all.log iirc to check and see what ansible says about that host | 18:26 |
| pleia2 | clarkb: thanks, checking | 18:26 |
| clarkb | pleia2: if that is the case you can just accept the key, eithe run ssh or use keyscan | 18:26 |
| fungi | clarkb: and i do think that we should probably ask that the change itself be made more clear or include appropriate commentary, rather than us being confused and having it explained only in review comments which almost nobody is likely to see again once it merges | 18:26 |
| SpamapS | clarkb: That sounds like a request for code documentation, and could be worded as such. :) | 18:27 |
| *** salv-orlando has joined #openstack-infra | 18:27 | |
| clarkb | SpamapS: but it may not be documentation that is necessary | 18:27 |
| SpamapS | clarkb: I do think we could improve my prose to make it clear that questions with specific needs are appropriate for -1. | 18:27 |
| fungi | pleia2: yeah, our launch readme probably needs to mention that you should sudo ssh to the fqdn at the end of the launch process and accept the host key | 18:28 |
| clarkb | SpamapS: maybe its a matter of pointing me at learning how ansible modules work | 18:28 |
| *** tjones1 has joined #openstack-infra | 18:28 | |
| pleia2 | fungi: nods, all this fancy new ansible stuff | 18:28 |
| fungi | pleia2: as usual, our documentation lags progress | 18:28 |
| clarkb | SpamapS: or maybe the change itself is invalid but without prompting with a question and giving any incentive to address it nothing happens, code merges and now everything is unhappy | 18:29 |
| SpamapS | clarkb: Yeah, I think you've found the line and are tap-dancing on it. IMO if the answer to your question is that you need to learn about X, you kind of wasted the contributor's time and should have +0'd. If however the answer is "Oh dear I forgot to mention that this depends on Ansible modules which is implicit here, I'll add it to the commit message, and here's where the docs are for you Clark", then it was a good -1. | 18:29 |
| *** tjones1 has quit IRC | 18:30 | |
| *** tjones1 has joined #openstack-infra | 18:30 | |
| *** salv-orlando has quit IRC | 18:30 | |
| clarkb | SpamapS: and your suggestion is that we should all avoid any confrontation because -1s are being see nthat way and just use 0 | 18:31 |
| cinerama | pleia2: oh by the way, we should be able to configure the log location now for wildfly, so let me know if you want it to live somewhere else and we can make that change | 18:31 |
| *** tjones1 has left #openstack-infra | 18:31 | |
| *** patrickeast has quit IRC | 18:31 | |
| pleia2 | cinerama: I think /var/log/wildfly would make the most sense (easy on our brains if we're looking for it) | 18:31 |
| *** patrickeast has joined #openstack-infra | 18:31 | |
| clarkb | anyways, I see code review as a conversation, there is no hard rule for when to -1 or -2 or 0 or +1 or +2 | 18:31 |
| *** tjones1 has joined #openstack-infra | 18:31 | |
| SpamapS | clarkb: My suggestion is that we inform reviewers of the expectations, and that they use their brain to decide +0 or -1. We have not informed them of where the line is yet, so their brains have been making poor choices at times. | 18:32 |
| SpamapS | And if there's two lines, and a big grey area, that is _totally fine_ | 18:32 |
| SpamapS | Just right now the whole thing is grey. | 18:32 |
| pleia2 | host key accepted, next time puppet tries to run it should be ok | 18:33 |
| cinerama | partytown | 18:33 |
| *** dhritishikhar has quit IRC | 18:34 | |
| *** dhritishikhar has joined #openstack-infra | 18:34 | |
| *** rm_work|away is now known as rm_work | 18:34 | |
| clarkb | crinkle: can you expand on what you want to wait for before 176343 merges? | 18:34 |
| clarkb | crinkle: comment on the change is fine | 18:34 |
| crinkle | clarkb: sure | 18:35 |
| clarkb | SpamapS: I think that by having hard fast rules we replace grey areas with well defined areas that don't fit into their alloted space on the graph | 18:35 |
| *** ajo has quit IRC | 18:36 | |
| clarkb | SpamapS: we lready observe that many different projects have different review conventions | 18:36 |
| clarkb | these have grown somewhat organically and lkely address different domain specific issues | 18:36 |
| SpamapS | clarkb: I think I'm more willing to trade some review quality for review efficiency than you are. :) | 18:37 |
| clarkb | for example infra won't approve changes to production without someone willing to babysit for example | 18:37 |
| *** ayoung is now known as ayoung-bye | 18:37 | |
| clarkb | tripleo is much more fond of -1ing for nitpicks than infra | 18:37 |
| *** ayoung-bye has quit IRC | 18:37 | |
| greghaynes | heh, not sure about that :p | 18:37 |
| *** hashar has joined #openstack-infra | 18:37 | |
| clarkb | but these different conventions work in the different domains | 18:37 |
| clarkb | greghaynes: maybe its changed | 18:37 |
| clarkb | greghaynes: but statistically I think tripleo had one of the highest negative review rates of any project | 18:38 |
| clarkb | but thats fine if it results in the reviews that you want to get as a project | 18:38 |
| greghaynes | IMO we can probably just focus on the egregiously bad case, identify that, and let people figure out how far that case extends | 18:38 |
| greghaynes | which is -1 "how does this work" | 18:39 |
| mordred | oh my | 18:39 |
| greghaynes | or maybe "-1 I didnt read this explain it to me" | 18:39 |
| greghaynes | :p | 18:39 |
| mordred | I get back from lunch and we're discussing -1 habits | 18:39 |
| * greghaynes -1s mordred's lunch decision | 18:40 | |
| fungi | i have to agree, i have gotten plenty of poor -1 reviews which just say "why did you use [x] here?" without even continuing "...instead of [y]?" or "...given that it causes [z]?" | 18:40 |
| *** cody-somerville has quit IRC | 18:40 | |
| *** dhritishikhar has quit IRC | 18:40 | |
| fungi | this is not a programming proficiency quiz, it's code review. if you have suggestions on how i could have done it better then please provide them ;) | 18:40 |
| cinerama | yes, i think we should try to avoid assumptions like "everyone knows [foo] is considered harmful, i shouldn't have to explain to you why" | 18:41 |
| SpamapS | clarkb: the nit picking in tripleo has subsided. It was a cultural thing and not actually useful. | 18:41 |
| mordred | clarkb, SpamapS: one of the things I'd love is if it was possible to ping someone in a review thread | 18:41 |
| *** dhritishikhar has joined #openstack-infra | 18:41 | |
| greghaynes | yessss | 18:41 |
| SpamapS | mordred: yeah, that would go well with the timeout I was suggesting. | 18:42 |
| mordred | like, clarkb -1's something, I respond and reference him and it shows up on his queueu again | 18:42 |
| SpamapS | mordred: ping starts clock.. clock runs out.. -1 is removed | 18:42 |
| *** jlvillal has joined #openstack-infra | 18:42 | |
| mordred | because my problem is that if I leave a -1 with a question, I'll literally never see the response | 18:42 |
| mordred | which is why Italk to most people about questions in irc | 18:42 |
| pleia2 | cinerama: Zanata 3.6.2 \o/ | 18:42 |
| greghaynes | ++ | 18:42 |
| greghaynes | I have the same issue | 18:42 |
| cinerama | pleia2: yay we didn't break it! | 18:42 |
| mordred | of course, then I have to go back and add in what we learned in IRC as a comment so other people know | 18:43 |
| cinerama | pleia2: i did test it but my install has less datur | 18:43 |
| *** signed8bit is now known as signed8bit_ZZZzz | 18:43 | |
| pleia2 | cinerama: still, good work with testing | 18:43 |
| SpamapS | mordred: I find that to be rather natural really. "Hey we talkd elsewhere and it was awesome and we're good. +1" | 18:43 |
| mordred | yah. that works well | 18:43 |
| mordred | but what doesn't work well at least for me is having a back and forth on a review beacuse I miss thigns | 18:44 |
| mordred | but that might just be me | 18:44 |
| greghaynes | Theres the cases where people dont ping in IRC though, and theres really not a great way for me to realize someone replied to my question | 18:44 |
| *** hodos has joined #openstack-infra | 18:44 | |
| mordred | yah | 18:44 |
| mordred | that's my problem | 18:44 |
| greghaynes | so it gets ignored, then I feel like a jerk | 18:44 |
| flashgordon | fungi: epic quote | 18:44 |
| mordred | SpamapS: what do you think the repr in that Exception should look like? | 18:44 |
| mordred | flashgordon: did I miss a good fungi? | 18:45 |
| flashgordon | very good | 18:45 |
| mordred | where? | 18:45 |
| flashgordon | http://lists.openstack.org/pipermail/openstack-dev/2015-April/062583.html | 18:45 |
| SpamapS | mordred: a common convention is that it should equal code that could be eval'd to reproduce a similar/the same object. | 18:45 |
| *** Longgeek has quit IRC | 18:45 | |
| cinerama | pleia2: so i'm working on the proposal slave scripts & obviously i'm kind of putting somethign out there which i expect, nay, DEMAND people rip to bits so we get it right, but i'm wondering if we have showstoppers at the moment in terms of what the CLI gives us to work with | 18:46 |
| SpamapS | mordred: so before the super change, it did.. it was OpenStackCloudException("noooo","99") | 18:46 |
| *** _nadya_ has joined #openstack-infra | 18:46 | |
| cinerama | pleia2: for example, would love to have your eyes on it but i don't see a way to specify percentage completion of what we pull down | 18:47 |
| harlowja | SpamapS clarkb i was also talking with folks internally (at y!) about having a nit-pick button (that like has weight -0.1) ; so if 10 nitpicks == -1 | 18:47 |
| harlowja | someone should add that :) | 18:47 |
| SpamapS | harlowja: no god no | 18:47 |
| harlowja | lol | 18:47 |
| SpamapS | people will get to 7 and start digging deep | 18:47 |
| harlowja | the ocd people then would have a button to click | 18:47 |
| SpamapS | harlowja: and click and click and click and click | 18:47 |
| harlowja | ha | 18:48 |
| SpamapS | They can click the close-window button for all I'm concerned | 18:48 |
| harlowja | maybe the button should do that then | 18:48 |
| harlowja | ? | 18:48 |
| hodos | for thirdparty ci: which backend should I test if my driver got removed???? | 18:48 |
| harlowja | pretty sure some javascript can do that, lol | 18:48 |
| SpamapS | As a recovering nit-picker.. I don't want to even acknowledge the practice. :) | 18:48 |
| fungi | the ocd button should also increment when you click it, so that you can count along | 18:48 |
| harlowja | :) | 18:48 |
| hodos | just LVM ? | 18:48 |
| harlowja | SpamapS do u go to nit-picker anonymous ? | 18:48 |
| mordred | SpamapS: also, I don't understand your comments about needing to send all of the arguments to super ... | 18:49 |
| harlowja | *as a recovering nit-picker | 18:49 |
| SpamapS | Like I still do reviews the way I used to, but now I go back and delete half the comments because they're not actually useful to anybody but the pedant inside me. | 18:49 |
| clarkb | I have just approved 176618 if there is a reason to not run that test in the gate hurry up and let me know :) | 18:49 |
| SpamapS | mordred: super(OpenStackCloudException, self).__init__(message, extra_data) | 18:49 |
| clarkb | (but it should prevent an issue we actually ran into so I would prefer running that test in the gate) | 18:49 |
| mordred | SpamapS: but OpenStackCloudException does not support extra_data ? | 18:49 |
| asselin | hodos, your driver is cinder? | 18:50 |
| openstackgerrit | Merged openstack-infra/infra-manual: How To Avoid Merging Specific Files https://review.openstack.org/177034 | 18:50 |
| SpamapS | mordred: it does so | 18:50 |
| *** rm_work is now known as rm_work|away | 18:50 | |
| mordred | SpamapS: it does??? | 18:50 |
| hodos | asselin: yes, couple of nexenta drivers for cinder. got removed due to missing CI | 18:50 |
| clarkb | hodos: asselin: Iwould assume you would test with your driver reapplied to the code base so that you can show the driver works with CI | 18:51 |
| SpamapS | mordred: http://git.openstack.org/cgit/openstack-infra/shade/tree/shade/__init__.py#n61 | 18:51 |
| *** _nadya_ has quit IRC | 18:51 | |
| mordred | SpamapS: that's an arg I added to __init__ | 18:51 |
| fungi | "shade/tree" is so awesome btw | 18:51 |
| SpamapS | mordred: Exception's signature is *args | 18:51 |
| SpamapS | fungi: we need peach too | 18:51 |
| mordred | SpamapS: gotcha | 18:51 |
| cinerama | fwiw, in the +0 reviews i've done with suggestions-i-don't-think-should-necessarily-stop-merging people generally do tend to acknowledge them | 18:51 |
| SpamapS | ok, tummy wants blue crab burger | 18:53 |
| * SpamapS goes to find it | 18:53 | |
| fungi | that sounds amazing | 18:53 |
| SpamapS | fungi: it looks amazing too http://files.ctctcdn.com/d5c5d16b001/689021a8-efee-493c-a1a1-0f0a37db186d.jpg | 18:53 |
| fungi | i get crabcake sandwiches around here pretty often | 18:53 |
| clarkb | mtreinish: I htink you are an expert on tempest regex stuff, can you look at 176662 and tell me if the regex change there is appropriate? | 18:54 |
| fungi | lotsa crab on/around the island | 18:54 |
| clarkb | mtreinish: specifically does that say all tests that match both strings? which should be no tests | 18:54 |
| hodos | clarkb: asselin: ok...that's an assumption or guideline ? | 18:55 |
| *** erikmwilson has quit IRC | 18:55 | |
| reed | where should I log a RFE for the sandbox doc? storyboard? | 18:55 |
| *** erikmwilson_ is now known as erikmwilson | 18:55 | |
| clarkb | hodos: I can't speak for cinder, but if the point is to show that you are testing your stuff then you will want to test it with that driver | 18:55 |
| *** dangers is now known as dangers_away | 18:55 | |
| clarkb | reed: sandbox doc being documetation around using git with gerrit and having sandbox be the arena? | 18:56 |
| mtreinish | clarkb: it gets passed as args to testr directly | 18:56 |
| *** rm_work|away is now known as rm_work | 18:56 | |
| hodos | clarkb: thanks, let me ask for cinder | 18:56 |
| reed | clarkb, yes :) | 18:56 |
| mtreinish | and testr ors all the regexes it gets | 18:56 |
| mtreinish | so that should work | 18:56 |
| clarkb | mtreinish: does that mean all tests will work or is it an XOR evaluation | 18:56 |
| openstackgerrit | Merged openstack-infra/project-config: Run project-config-gerrit job in gate check queue https://review.openstack.org/176618 | 18:57 |
| fungi | clarkb: presumably http://docs.openstack.org/infra/manual/sandbox.html | 18:57 |
| reed | http://docs.openstack.org/infra/manual/drivers.html#feature-branches is probably missing some details | 18:57 |
| clarkb | mtreinish: er, what I am trying to ask is does that OR such that both work or XOR so that only one set does. I guess not XOR is what makes sense | 18:57 |
| mtreinish | clarkb: that should be all tests they want to run should work. It's OR not XOR | 18:57 |
| asselin | hodos, it's documented in the cinder wiki FAQ | 18:57 |
| clarkb | mtreinish: thanks | 18:57 |
| mtreinish | clarkb: you can always do a testr list-tests on tempest with the regex | 18:57 |
| clarkb | mtreinish: I thought list tests idn't apply the regex | 18:58 |
| reed | sandbox.html may offer a playground, with sandbox the git repo offering a feature branch to fool around with | 18:58 |
| reed | or maybe it's only me that to be thick | 18:58 |
| clarkb | reed: well I added a feature branch to sandox | 18:58 |
| reed | thanks | 18:59 |
| clarkb | we may need to update acls for the merge commits | 18:59 |
| mtreinish | clarkb: http://paste.openstack.org/show/205780/ | 18:59 |
| reed | before I forget, I'd like to add a bug/RFE to test things | 18:59 |
| mtreinish | clarkb: nope it does, it's kinda useful for testing regexes | 18:59 |
| openstackgerrit | Merged openstack-infra/project-config: Run full tempest on centos7 https://review.openstack.org/176628 | 18:59 |
| *** akshai has joined #openstack-infra | 18:59 | |
| mtreinish | especially because I always screw them up | 19:00 |
| reed | clarkb, probably not needed since the basic use case I have in mind is of a new contributor who will not have to do the merge, right? | 19:00 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: puppet: run beaker on both trusty & centos7 https://review.openstack.org/175434 | 19:02 |
| clarkb | reed: ya merge commits should only come from a project core/release manager | 19:02 |
| clarkb | reed: everyone else won't need to worry about them they will just push to the branch | 19:02 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: update make_library_stable_branch.sh to set defaultbranch https://review.openstack.org/173919 | 19:03 |
| *** spzala has joined #openstack-infra | 19:03 | |
| reed | cool | 19:03 |
| reed | so, is storyboad the place to log such RFE? | 19:03 |
| clarkb | mtreinish: neat for some reason I didn't think list tests applied the regex and the regex was applied post listing | 19:04 |
| clarkb | reed: sure | 19:04 |
| mtreinish | clarkb: oh, that's the internal mechism | 19:04 |
| mtreinish | it does full discovery and then does a regex match afterwords | 19:04 |
| *** patrickeast has quit IRC | 19:04 | |
| mtreinish | or afterwards I guess :) | 19:04 |
| clarkb | gotcha | 19:04 |
| reed | https://storyboard.openstack.org/#!/story/2000246 | 19:05 |
| openstackgerrit | Merged openstack-infra/project-config: Add testr_results report https://review.openstack.org/176662 | 19:06 |
| reed | i cannot see comments on storyboard | 19:06 |
| reed | now I can | 19:06 |
| reed | needed to refresh | 19:06 |
| reed | ok, I'm good to go to lunch | 19:06 |
| *** dannywilson has quit IRC | 19:08 | |
| *** tsg_ has joined #openstack-infra | 19:09 | |
| *** shashankhegde has quit IRC | 19:10 | |
| *** dannywilson has joined #openstack-infra | 19:10 | |
| *** jamesmcarthur has joined #openstack-infra | 19:11 | |
| anteaya | superdan: thanks for you reply to lennyb | 19:12 |
| *** ZZelle_ has joined #openstack-infra | 19:12 | |
| anteaya | your | 19:12 |
| *** ZZelle_ has quit IRC | 19:12 | |
| superdan | anteaya: np | 19:13 |
| *** hdd has quit IRC | 19:13 | |
| *** eharney has quit IRC | 19:15 | |
| *** tjones2 has joined #openstack-infra | 19:16 | |
| *** jamesmcarthur has quit IRC | 19:16 | |
| *** mrunge has quit IRC | 19:17 | |
| lifeless | mordred: except python-quantumclient wasn't doing stable branches w/point releases :) | 19:19 |
| lifeless | mordred: still, I'll take your word :) | 19:19 |
| *** ddieterly has quit IRC | 19:19 | |
| openstackgerrit | Jan Klare proposed openstack-infra/project-config: move gate-.*-chef-rake job and run it branch specific https://review.openstack.org/176674 | 19:20 |
| *** ddieterly has joined #openstack-infra | 19:21 | |
| *** rm_work is now known as rm_work|away | 19:22 | |
| lifeless | jd__: unicode requirements? !? | 19:24 |
| jd__ | lifeless: lol yeah I thought we were in 2015 but finally I gave up :) | 19:24 |
| *** dustins has quit IRC | 19:24 | |
| lifeless | jd__: I'm surprised 2.6 failed and 2.7 didn't | 19:25 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: Check more Puppet modules with tripleo-f20puppet-nonha https://review.openstack.org/177392 | 19:25 |
| EmilienM | dprince: ^ | 19:25 |
| lifeless | jd__: well, surprised it failed *there*, vs higher up. | 19:25 |
| jd__ | lifeless: I imagine io.open has a different behaviour by default? | 19:25 |
| lifeless | jd__: FWIW I'd be entirely happy to make pbr str-bytes clean | 19:25 |
| lifeless | jd__: [and I do need to support 2.6 still, for the testing ecosystem] | 19:25 |
| lifeless | jd__: oh, I see what you did | 19:26 |
| lifeless | jd__: so in 2.6, the docs https://docs.python.org/2.6/library/io.html say The io module provides the Python interfaces to stream handling. The built-in open() function is defined in this module. | 19:27 |
| pleia2 | cinerama: missed your comment earlier, the percentages thing may be a problem, since we only commit the translations to the repository w/ gerrit once they are a percentage completed, if Zanata really can't provide these statistics to the client (and we can't get them to write it in), we may have to write a script on the proposal slave to download everything and then determine percentages there | 19:27 |
| mordred | lifeless: how much longer do you need to support 2.6 in pbr? | 19:27 |
| pleia2 | (oops, that line may have cut off) | 19:27 |
| mordred | lifeless: given that 2.6 has been EOL'd for a while ... | 19:28 |
| lifeless | mordred: indefinitely, but I'll happy provide the resources to do so | 19:28 |
| lifeless | mordred: 2.6 is still in widespread use | 19:28 |
| mordred | lifeless: k. I mean, I'm mostly just asking because infra will stop having 2.6 capable build hosts once openstack kills 2.6 support | 19:28 |
| *** signed8bit_ZZZzz is now known as signed8bit | 19:28 | |
| cinerama | pleia2: the web UI provides the percentage completion so it's not like it's not available | 19:28 |
| mordred | so at some point we should figure out what support it looks like | 19:29 |
| jd__ | lifeless: ah yeah so that makes sense that it fails :( I could try to specify an encoding but I don't think it's really good? | 19:29 |
| lifeless | mordred: so, 2.6 in a container is sufficient to test | 19:29 |
| pleia2 | cinerama: yeah, so maybe a good thing to ask them about exposing via the client API? | 19:29 |
| lifeless | jd__: I think it will be a bit of a tarpit | 19:29 |
| *** esker has quit IRC | 19:29 | |
| lifeless | jd__: since setuptools internals and distutils internals we use are likely not hygenic yet | 19:29 |
| lifeless | jd__: so there'd be a chunk of pre-call conversions needed to avoid blowups in other code bases | 19:30 |
| *** _dguerri is now known as dguerri | 19:30 | |
| lifeless | mordred: you should look at https://caremad.io/2015/04/a-year-of-pypi-downloads/ | 19:30 |
| cinerama | pleia2: yup | 19:30 |
| lifeless | mordred: 2.6 is still ~15% of pypi downloads. I'm tempted to argue we should preserve client support for 2.6 generally on that basis | 19:31 |
| *** teran has quit IRC | 19:31 | |
| lifeless | mordred: since if folk are crazy enough to use it... | 19:31 |
| *** teran has joined #openstack-infra | 19:31 | |
| pleia2 | cinerama: I *believe* that's the big one WRT downloads, off the top of my head I don't think there are any others | 19:31 |
| lifeless | jd__: how did you think to test this? was it a comment or an actual package with unicode in it ? | 19:32 |
| *** akshai has quit IRC | 19:33 | |
| lifeless | mordred: whats the limiting factor on infra having 2.6 capable hosts? | 19:33 |
| jd__ | lifeless: it was a comment | 19:33 |
| *** tjones1 has quit IRC | 19:33 | |
| mordred | lifeless: on the other hand - perhaps EOL'ing client support on 2.6 might help convince people to move off it | 19:33 |
| jd__ | lifeless: I don't think package with unicode is going to work :) I'm not that optimistic :) | 19:33 |
| mordred | lifeless: an OS that includes it | 19:33 |
| mordred | lifeless: that still gets security updates | 19:33 |
| lifeless | mordred: would https://launchpad.net/~fkrull/+archive/ubuntu/deadsnakes be ok? | 19:33 |
| mordred | lifeless: we don't really run from PPAs except under extreme duress, no | 19:33 |
| mordred | and also - who is doing security fixes for 2.6 in that repo? | 19:33 |
| clarkb | nick wrote a blog on this iirc | 19:34 |
| mordred | I mean, at some point EOL has to mean EOL | 19:34 |
| clarkb | and said please stop supporting python2.6 | 19:34 |
| mordred | and I'd like to think that three-years after EOL is good enough | 19:34 |
| mordred | and thing need to start breaking | 19:34 |
| mordred | so I agree with nick | 19:34 |
| lifeless | so nicks thing said | 19:34 |
| lifeless | if you're a project, stop caring, let redistributors care | 19:34 |
| mordred | yup | 19:34 |
| lifeless | its their job | 19:34 |
| mordred | we're a project | 19:34 |
| mordred | I think we do the community a service by ceasing to care | 19:35 |
| lifeless | mmm | 19:35 |
| mordred | sometimes it takes outside help to get out of a bad relationship | 19:35 |
| lifeless | so unless a redistributor pops up to care | 19:35 |
| *** teran has quit IRC | 19:35 | |
| mordred | sure - but I can't do anything about that | 19:35 |
| lifeless | I'm going to be personally affected if pbr breaks on 2.6 before I EOL 2.6 caring in testtools/testscenarios/testr/subunit/linecache2/unittest2/traceback2/mock | 19:36 |
| mordred | lifeless: well, we tend to not land pbr changes quickly | 19:36 |
| *** jtriley_ has joined #openstack-infra | 19:36 | |
| *** dhritishikhar has quit IRC | 19:36 | |
| lifeless | mordred: I know, its more the ohfuckitbroke concerns around not being tested | 19:36 |
| mordred | nod | 19:36 |
| jtriley_ | hello all - just signed up for review.openstack.org and having issues running git review -s | 19:36 |
| mtreinish | fungi, clarkb: is there an easy way to query which projects I have +2 on? | 19:37 |
| *** isviridov_away is now known as isviridov | 19:37 | |
| jtriley_ | signed the ICLA and added my ssh key | 19:37 |
| jtriley_ | and set my username in review.openstack.org | 19:37 |
| lifeless | mordred: so, whats the concern around security updates for a python thats only used to run a test suite as an unprivileged user ? | 19:37 |
| openstackgerrit | Elizabeth K. Joseph proposed openstack-infra/system-config: Add instruction to verify new server fingerprint https://review.openstack.org/177396 | 19:37 |
| clarkb | mtreinish yes, label:code-review,+2 iirc | 19:37 |
| lifeless | mordred: [putting aside the PPA aspect for a second] | 19:37 |
| clarkb | mtrienish oh wait | 19:37 |
| clarkb | mtrienish I misread you want to know if you can +2, I dont think so | 19:38 |
| lifeless | you need to query the acls | 19:38 |
| lifeless | and team memberships | 19:38 |
| lifeless | Ithink its accessible | 19:38 |
| clarkb | it is its not easy | 19:38 |
| mtreinish | lifeless: yeah, I was looking (through the ui at least) and it didn't seem like something exposed easily | 19:39 |
| *** prad has quit IRC | 19:39 | |
| mtreinish | because of the group heiarchies | 19:39 |
| lifeless | mtreinish: https://review.openstack.org/#/admin/projects/openstack-dev/devstack,access <- acls | 19:40 |
| *** prad has joined #openstack-infra | 19:40 | |
| lifeless | mtreinish: and from there to groups, and spider out from there. | 19:40 |
| fungi | jtriley_: also signed up as an individual member of the openstack foundation and confirmed that supplying contact information in gerrit didn't return an error? and do you have (or did you set) a username in gerrit? | 19:40 |
| fungi | jtriley_: confirm that the username field at https://review.openstack.org/#/settings/ matches what you're trying to use with git review | 19:41 |
| greghaynes | What is the ubuntu apt mirror we should be using during test jobs? | 19:41 |
| mtreinish | lifeless: yeah I realize that. I was hoping to do it in reverse though. Ie given a user find the access permissions on all projects | 19:41 |
| lifeless | if i wanted to make a variation of the pbr integration job that had a different pip version | 19:41 |
| fungi | jtriley_: and that https://review.openstack.org/#/settings/contact says something like "Contact information last updated on [some recent date and time]." | 19:41 |
| lifeless | mtreinish: so I suspect checking out the committed form of the acls is that | 19:41 |
| mordred | greghaynes: we don't have one / set one | 19:42 |
| *** tjones1 has joined #openstack-infra | 19:42 | |
| greghaynes | :( | 19:42 |
| lifeless | mtreinish: or I may be imaginging something | 19:42 |
| mordred | greghaynes: it's been a thing we've discussed, but so far is not ugly enough of a problem that anyone has worked on it | 19:42 |
| greghaynes | I just noticed that the trove dib jobs are using archive.ubuntu.com | 19:42 |
| lifeless | whats the right git repo to hack on these days to add a variation of a job | 19:42 |
| lifeless | infra/system-config ? | 19:42 |
| greghaynes | lifeless: jobs are in project-config | 19:43 |
| *** rm_work|away is now known as rm_work | 19:43 | |
| *** ajo has joined #openstack-infra | 19:43 | |
| fungi | jtriley_: mainly double-check that you followed the steps at http://docs.openstack.org/infra/manual/developers.html#account-setup in sequence | 19:43 |
| lifeless | slow net is slow Receiving objects: 25% (13626/54502), 3.38 MiB | 94.00 KiB/s | 19:44 |
| fungi | mordred: lifeless: i'm not _too_ concerned with the security posture of random debs used in running jobs as we're moving toward no longer preinstalling them anyway (just pre-caching them) | 19:45 |
| fungi | and also not talking about installing them on long-running systems, production servers, whatever | 19:46 |
| mordred | patch bomb incoming | 19:46 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add inventory command to shade https://review.openstack.org/175891 | 19:46 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Raise a shade exception on broken volumes https://review.openstack.org/176550 | 19:46 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a method for getting an endpoint https://review.openstack.org/177314 | 19:46 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Make keystone endpoints work like the others https://review.openstack.org/177315 | 19:46 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add early service fail and active check method https://review.openstack.org/177316 | 19:46 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Call super in OpenStackCloudException https://review.openstack.org/177347 | 19:46 |
| clarkb | I have somehow forgotten that I should get a thermometer for the smoker | 19:46 |
| mordred | clarkb: FAIL | 19:46 |
| * clarkb runs off to find lunch and kitchen gadgets | 19:46 | |
| greghaynes | clarkb: :O | 19:46 |
| mordred | greghaynes, SpamapS: ^^ I haz fixed | 19:47 |
| greghaynes | clarkb: I have my thermometer for brewing | 19:47 |
| fungi | the only real security concern there i guess is that by saying "commit tested against package <whatever>" could make downstream consumers think we're endorsing the security of that package | 19:47 |
| greghaynes | clarkb: I also have a thermocouple we can hack on :p | 19:47 |
| clarkb | greghaynes: will you be awake at 5am tomorrow? I guess hacking needs to happen before that | 19:47 |
| clarkb | meh I will just buy a remote probe thermometer | 19:48 |
| *** eharney has joined #openstack-infra | 19:48 | |
| fungi | they're cheap these days | 19:48 |
| morganfainberg | clarkb, fungi, jeblair, pleia2: I'd like to get a branch created for pycadf for stable/juno | 19:48 |
| morganfainberg | the commit it should be based on is: https://github.com/openstack/pycadf/commit/52727bcea3a98e72331e748ce5f9e3a111a64cd1 | 19:48 |
| morganfainberg | cc ttx ^ | 19:49 |
| morganfainberg | this is to fix keystonemiddleware's failing | 19:49 |
| *** dangers_away is now known as dangers | 19:49 | |
| fungi | dhellmann: ^ do you or other release managers want to be doing that or okay with one of us doing it? | 19:49 |
| morganfainberg | gordc, bkudson, ^^ | 19:49 |
| * fungi hopes ttx has checked out for the weekend | 19:49 | |
| morganfainberg | fungi, seriously i do to | 19:49 |
| lifeless | oh man | 19:49 |
| lifeless | export PIP_PROCESS_DEPENDENCY_LINKS=1 | 19:49 |
| lifeless | we so have to kill that | 19:49 |
| lifeless | in our tests | 19:50 |
| fungi | morganfainberg: i can do it easily enough, but let's give it a few minutes for someone to respond. i don't want to step on any release management toes | 19:50 |
| morganfainberg | fungi, thanks. | 19:50 |
| morganfainberg | fungi, totally. | 19:50 |
| morganfainberg | fungi, :) | 19:50 |
| lifeless | clarkb: I need a small hint for jjb | 19:50 |
| lifeless | clarkb: I want to add a variation of check-pbr-installation-dsvm | 19:50 |
| lifeless | clarkb: that uses a different pip version | 19:50 |
| bknudson | once that's created we'll probably want the proposal bot to update its requirements | 19:51 |
| lifeless | clarkb: do I encode that into the name | 19:51 |
| dhellmann | fungi: I'm OK with you doing it, but I also have time to help morganfainberg if you're busy with other things | 19:51 |
| bknudson | and then we can also see if the tests work. | 19:51 |
| lifeless | clarkb: like ...-upstream in the layout and then pull that out with a {} variable in requirements.yaml ? | 19:51 |
| *** isviridov is now known as isviridov_away | 19:51 | |
| dhellmann | fungi, morganfainberg: you need to create the branch based on a release tag, right? | 19:52 |
| bknudson | 0.6.0 is commit 52727bcea3... | 19:52 |
| jtriley_ | followed all that as far as i can tell - meh i'll try again later | 19:52 |
| gordc | dhellmann: yep. it'd be off 0.6.0 https://github.com/openstack/pycadf/releases/tag/0.6.0 | 19:52 |
| morganfainberg | dhellmann, ^ | 19:52 |
| morganfainberg | yeah | 19:52 |
| dhellmann | gordc, morganfainberg : ok, just double checking | 19:53 |
| fungi | dhellmann: please take care of it, if you don't mind. it's less that i don't have time, but more that i like the release managers handling this for official projects | 19:53 |
| dhellmann | fungi: sure thing | 19:53 |
| clarkb | lifeless ya needs to be a new name but likely you can use existing template | 19:53 |
| fungi | dhellmann: and thanks! | 19:53 |
| dhellmann | gordc, morganfainberg : so this is stable/juno at 0.6.0, right? | 19:54 |
| clarkb | lifeless but update the job in the template to switch on the parameter passed to the name | 19:54 |
| *** Bsony has quit IRC | 19:54 | |
| *** nelsnelson has quit IRC | 19:54 | |
| gordc | i guess we should also add a disclaimer the 0.6.1 release would be a single patch which moves a oslo.messaging req from requirements.txt to test-requirements.txt? | 19:54 |
| morganfainberg | dhellmann: I defer to gordc and bknudson for specifics here. thye've been fighting this. | 19:54 |
| gordc | dhellmann: yep | 19:54 |
| lifeless | clarkb: tell you what, I'll blindly do it and you tell me in review, I tink thats most efficient for all concerned | 19:54 |
| dhellmann | gordc: a requirements change is really a semver minimum version bump :-/ | 19:54 |
| fungi | jtriley_: can you paste the error you're getting at http://paste.openstack.org/ and link the resulting url here? it might help us tell you where the problem is | 19:54 |
| dhellmann | gordc, morganfainberg : let's move to openstack-relmgr-office to cut down on the noise here | 19:54 |
| dhellmann | bknudson: ^^ | 19:55 |
| gordc | moving | 19:55 |
| mordred | lifeless: what's doing PIP_PROCESS_DEPENDENCY_LINKS? | 19:55 |
| jtriley_ | fungi: sure it's git-review telling me could not connect to gerrit and asking for my username | 19:55 |
| lifeless | mordred: grep for it in project-config | 19:55 |
| jtriley_ | not much to see | 19:55 |
| sc` | dumb question: jenkins is supposed to be automatically added as a reviewer to every review, but it's not on a change i recently submitted. should that be of concern? | 19:55 |
| *** isviridov_away is now known as isviridov | 19:56 | |
| *** HeOS has joined #openstack-infra | 19:56 | |
| jtriley_ | fungi: Trying again with ssh://jtriley@review.openstack.org:29418/stackforge/kolla.git 'we dont know where your gerrit is' | 19:56 |
| fungi | jtriley_: that can sometimes be a misleading error indicating that the test push it tried failed for other reasons. what ssh username are you connecting with? | 19:56 |
| jtriley_ | same as my irc (without the _) | 19:56 |
| fungi | ah, i see it above | 19:57 |
| jtriley_ | i cloned the repo from github which the project owner does as well apparently | 19:57 |
| pleia2 | grabbing some lunch, bbiab | 19:57 |
| *** tsg_ has quit IRC | 19:57 | |
| fungi | jtriley_: i see that username corresponds to account_id 16110 in gerrit. can you confirm that's the account id number you see at https://review.openstack.org/#/settings/ | 19:58 |
| jtriley_ | yep | 19:58 |
| jtriley_ | i see it | 19:58 |
| openstackgerrit | Merged openstack-infra/project-config: Add python34 unit test job to tempest experiemental https://review.openstack.org/176869 | 19:58 |
| *** tnovacik has joined #openstack-infra | 19:58 | |
| *** patrickeast has joined #openstack-infra | 19:59 | |
| fungi | jtriley_: it looks like that account has not supplied contact information. go to https://review.openstack.org/#/settings/contact and let me know if you see "Contact information last updated on <some date and time>." | 19:59 |
| jtriley_ | nope | 19:59 |
| fungi | jtriley_: normally you're prompted for that when you agree to the icla, but might have missed entering it for some reason | 20:00 |
| jtriley_ | ok i generally avoid that type of info if it will let me ;) | 20:00 |
| jtriley_ | which i guess it did.. | 20:00 |
| fungi | jtriley_: go ahead and put something (anything really) in the mailing address and country fields there and save it | 20:00 |
| jtriley_ | done | 20:00 |
| openstackgerrit | lifeless proposed openstack-infra/project-config: Test pbr against unreleased pip. https://review.openstack.org/177423 | 20:00 |
| openstackgerrit | Aurélien Chabot proposed openstack-infra/git-review: Add an option to add reviewer to submitted changes https://review.openstack.org/177424 | 20:01 |
| jtriley_ | that did it! | 20:01 |
| jtriley_ | fungi: thank you very much | 20:01 |
| *** tjones2 has quit IRC | 20:01 | |
| fungi | jtriley_: what's probably happening is that git-review is being told during its test push that the kolla repository's cla requires contact information, but git-review is hiding that error and assuming your username is wrong. i believe it's fixed in git-review master but not released yet | 20:01 |
| *** otter768 has joined #openstack-infra | 20:02 | |
| *** marcusvrn1 has quit IRC | 20:02 | |
| jtriley_ | ah ok good to know thanks again for your help | 20:02 |
| *** tsg_ has joined #openstack-infra | 20:02 | |
| fungi | jtriley_: absolutely--any time! happy hacking | 20:02 |
| *** ociuhandu has quit IRC | 20:02 | |
| sdake | fungi as usual #openstack-infra is a wealth of knowlege :) tia :) | 20:03 |
| fungi | sdake: it's only because we beat our heads against these particular problems until they're imprinted into our skulls | 20:03 |
| sdake | fungi yup I know :) | 20:03 |
| sdake | fungi I see it all day long when I lurk in this channel :) | 20:04 |
| fungi | tomorrow we'll fix everything, i promise. it's too bad tomorrow never comes | 20:04 |
| sdake | fungi there is always tomororw :) | 20:04 |
| sdake | (I use that on my kids all the time :) | 20:05 |
| sdake | "daddy I'll do it tomorrow" | 20:05 |
| sdake | "no you will do it today!" | 20:05 |
| fungi | one of the bars i frequent has a big sign on the wall that says "free beer tomorrow" | 20:05 |
| *** otter768 has quit IRC | 20:06 | |
| *** kgiusti has left #openstack-infra | 20:08 | |
| lifeless | clarkb: ^ thoughts appreciated | 20:09 |
| *** jtriley_ has quit IRC | 20:09 | |
| *** haypo has joined #openstack-infra | 20:09 | |
| fungi | sc`: which review? got a link? | 20:09 |
| sc` | fungi: looks like something knocked loose. after about 10 minutes it showed up | 20:10 |
| fungi | sc`: jenkins won't show up as a reviewer until the first time it finishes testing the review and leaves a comment/vote on it | 20:11 |
| sc` | got it. still getting my head wrapped around how it all works | 20:12 |
| fungi | sc`: sometimes it can take a few minutes to run test jobs (or even to process the upload event if it's particularly busy) | 20:12 |
| fungi | lifeless: i suspect one or two of the jobs are going to barf on the stray hard tab in your yaml | 20:13 |
| *** jgrimm is now known as zz_jgrimm | 20:14 | |
| *** akshai has joined #openstack-infra | 20:15 | |
| haypo | hi. i don't understand why gate-tempest-dsvm-largeops-src-oslo.messaging fails with https://launchpad.net/bugs/1403284 : my change on requirements.txt should not impact the installed eventlet version: https://review.openstack.org/#/c/172135/7/requirements.txt -- i saw these tracebacks: http://pastealacon.com/37368 | 20:15 |
| openstack | Launchpad bug 1403284 in devstack "mysql (1040, 'Too many connections')" [Undecided,In progress] - Assigned to Joe Gordon (jogo) | 20:15 |
| fungi | lifeless: curious, have you tested running devstack-gate with PBR_PIP_VERSION exported like that in the calling environment? | 20:16 |
| haypo | it looks like "Bug 1403284 - mysql (1040, 'Too many connections')" only occured on my changeset: http://status.openstack.org/elastic-recheck/#1403284 | 20:16 |
| lifeless | fungi: narf, thanks | 20:17 |
| openstackgerrit | lifeless proposed openstack-infra/project-config: Test pbr against unreleased pip. https://review.openstack.org/177423 | 20:17 |
| lifeless | fungi: I'm adding the glue to integration.sh now | 20:17 |
| *** rm_work is now known as rm_work|away | 20:17 | |
| *** Bsony has joined #openstack-infra | 20:17 | |
| fungi | lifeless: also this could be done by extending the existing job-template similar to how we do the {branch-designator} parameter suffix on other d-g jobs, but what you have there isn't technically wrong | 20:17 |
| lifeless | fungi: yeah, I couldn't see an easy cargo cult | 20:18 |
| lifeless | fungi: so wanted a reviewer to say 'it should look like <details>' | 20:18 |
| fungi | it's not crucial in my opinion | 20:18 |
| lifeless | fungi: I'm not happy with what I have, is all I'm saying | 20:18 |
| fungi | but i can add notes if you're interested in trying to keep one template there | 20:18 |
| lifeless | there are currently two with the only difference being s/requirements/pbr/ in one command AFAICT | 20:19 |
| lifeless | that offends me :). making it be just one would be great. | 20:19 |
| lifeless | my patch adds a third | 20:19 |
| openstackgerrit | Merged openstack-infra/project-config: Add qemu-img to dib functests https://review.openstack.org/176965 | 20:19 |
| *** rm_work|away is now known as rm_work | 20:19 | |
| lifeless | but I am perhaps overly nuts on this | 20:19 |
| lifeless | certainly I'd like to keep the two pbr specific ones unified | 20:19 |
| fungi | lifeless: agreed, reduplicating nearly identical jobs is what templates were intended to solve ;) | 20:19 |
| *** woodster_ has quit IRC | 20:20 | |
| *** mrmartin has quit IRC | 20:21 | |
| *** ibiris is now known as ibiris_away | 20:22 | |
| *** Bsony has quit IRC | 20:23 | |
| *** shashankhegde has joined #openstack-infra | 20:25 | |
| lifeless | huh | 20:25 |
| lifeless | I think integration.sh is broken | 20:25 |
| *** hashar has quit IRC | 20:25 | |
| fungi | oh? | 20:26 |
| lifeless | I need to check a couple thing | 20:26 |
| fungi | that wouldn't _entirely_ surprise me | 20:26 |
| lifeless | I think it runs | 20:26 |
| lifeless | but I'm not sure that it populates the wheelhouse it thinks it is | 20:26 |
| fungi | also remember that script has been forked and has different versions in pbr vs requirements repos now | 20:26 |
| lifeless | oh gawd | 20:26 |
| lifeless | why can't we have nice things? | 20:26 |
| *** Bsony has joined #openstack-infra | 20:27 | |
| lifeless | yeah WHEELHOUSE isn't a pip variable | 20:27 |
| lifeless | we're not writing our wheels to the wheelhouse we've made | 20:27 |
| mordred | AWESOME | 20:27 |
| lifeless | we're writing them to ./wheelhouse | 20:27 |
| fungi | that could certainly impact performance | 20:28 |
| lifeless | yes | 20:28 |
| lifeless | this first | 20:28 |
| mordred | lifeless: I look forward to it sucking less | 20:28 |
| lifeless | then that | 20:28 |
| *** tiswanso has quit IRC | 20:29 | |
| *** Swami has quit IRC | 20:29 | |
| lifeless | yeah, these things need to be options to pip | 20:29 |
| lifeless | $WAT | 20:29 |
| fungi | though some of the challenge is that passing pip options through tox without modifying tox.ini is tough | 20:30 |
| fungi | i guess pip.conf is the other answer there | 20:30 |
| lifeless | this is the integration tests though | 20:30 |
| fungi | ahh, yeah in that case i don't think we're relying on tox | 20:30 |
| lifeless | like | 20:31 |
| lifeless | we're not going to run project tests | 20:31 |
| lifeless | we're going to run tempest | 20:31 |
| lifeless | having put everything together with a new pbr || new pip | 20:31 |
| lifeless | AIUI | 20:31 |
| *** peristeri has quit IRC | 20:31 | |
| lifeless | or maybe its just referring to using the dsvm image | 20:31 |
| *** Bsony has quit IRC | 20:32 | |
| mordred | yeah - it just uses the image | 20:33 |
| mordred | SpamapS: python2.7 does NOT like my changes to the Exception | 20:34 |
| *** teran has joined #openstack-infra | 20:35 | |
| mordred | oh. hah | 20:35 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add inventory command to shade https://review.openstack.org/175891 | 20:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Raise a shade exception on broken volumes https://review.openstack.org/176550 | 20:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a method for getting an endpoint https://review.openstack.org/177314 | 20:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Make keystone endpoints work like the others https://review.openstack.org/177315 | 20:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add early service fail and active check method https://review.openstack.org/177316 | 20:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Call super in OpenStackCloudException https://review.openstack.org/177347 | 20:36 |
| * mordred is fascinated as to why that did not break python3.4 | 20:36 | |
| lifeless | also the next pip release will build wheels itself | 20:36 |
| lifeless | so all the manual wheel caching can be tossed | 20:36 |
| mordred | lifeless: \o/ | 20:37 |
| *** dangers is now known as dangers_away | 20:37 | |
| *** dangers_away is now known as dangers | 20:38 | |
| fungi | lifeless: some comments added to 177423 on extending the current template | 20:38 |
| fungi | and also on how to fix the current job failure | 20:38 |
| *** esker has joined #openstack-infra | 20:40 | |
| *** woodster_ has joined #openstack-infra | 20:40 | |
| lifeless | mordred: indeed :) | 20:40 |
| lifeless | mordred: yeah, I did that :) | 20:41 |
| * mordred hands lifeless a used packet of seasoning | 20:43 | |
| lifeless | fungi: why requirements-jobs ? | 20:43 |
| mordred | lifeless: because | 20:43 |
| SpamapS | mordred: oh? | 20:43 |
| lifeless | fungi: I don't want to test that a change to global-requirements works with upsream tests | 20:43 |
| fungi | lifeless: it's a pseudo project name | 20:44 |
| mordred | SpamapS: enjoy the diff of diffs between the last two patchsets of 177347 | 20:44 |
| fungi | lifeless: it can be renamed to gralpy if we want | 20:44 |
| fungi | in fact i'd posit that gralpy would be better anyway | 20:44 |
| *** mattfarina has quit IRC | 20:45 | |
| *** isviridov is now known as isviridov_away | 20:45 | |
| SpamapS | mordred: thats exactly what I was just looking at | 20:45 |
| SpamapS | mordred: why do you need to build up real_args ? | 20:45 |
| openstackgerrit | lifeless proposed openstack-infra/project-config: Test pbr against unreleased pip. https://review.openstack.org/177423 | 20:45 |
| lifeless | fungi: I don't know what that means, but doing what you say anyway | 20:45 |
| SpamapS | mordred: why not just pass the two that you expect to super? | 20:45 |
| fungi | lifeless: about the only thing project entries do in jjb is provide a convenient mechanism to fill in the {name} built-in parameter | 20:46 |
| fungi | lifeless: particularly so as to pass them into job-templates you're instantiating | 20:46 |
| mordred | SpamapS: I dunno | 20:46 |
| fungi | lifeless: for example there's another one in there called devstack-jobs. it's not a real git repo either, just another dummy project name used for collecting job-template instantiations | 20:47 |
| *** rm_work is now known as rm_work|away | 20:47 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Call super in OpenStackCloudException https://review.openstack.org/177347 | 20:47 |
| *** rm_work|away is now known as rm_work | 20:47 | |
| mordred | SpamapS: like that? | 20:47 |
| SpamapS | mordred: yes like that. :-D | 20:48 |
| mordred | k. I push rest of stack | 20:48 |
| SpamapS | mordred: assuming that actually works... :) | 20:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add inventory command to shade https://review.openstack.org/175891 | 20:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Raise a shade exception on broken volumes https://review.openstack.org/176550 | 20:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a method for getting an endpoint https://review.openstack.org/177314 | 20:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Make keystone endpoints work like the others https://review.openstack.org/177315 | 20:49 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add early service fail and active check method https://review.openstack.org/177316 | 20:49 |
| mordred | SpamapS: only one way to find out | 20:49 |
| *** samuelBartel has quit IRC | 20:49 | |
| lifeless | mordred: why are you calling super ? | 20:50 |
| lifeless | mordred: also, why map(repr) instead of %r in the format ? | 20:52 |
| *** pelix has quit IRC | 20:52 | |
| *** BobH1 has quit IRC | 20:53 | |
| fungi | lifeless: much closer, but my explanation about how to add what you need to the jenkins/jobs/projects.yaml was pretty lacking, sorry. clarifications in the latest patchset | 20:54 |
| *** spzala has quit IRC | 20:56 | |
| *** BobH has quit IRC | 20:57 | |
| *** rm_work is now known as rm_work|away | 20:57 | |
| *** dguerri is now known as _dguerri | 20:57 | |
| *** armax has quit IRC | 20:57 | |
| bknudson | so we created a branch stable/juno for pycadf... is proposal bot going to come along and update requirements automatically? | 20:58 |
| *** patrickeast has quit IRC | 20:58 | |
| mordred | lifeless: %r looks like it would add [] around the args, which would not be happymaking | 20:58 |
| fungi | bknudson: the next time a change merges to the stable/juno branch of the openstack/requirements repo it should | 20:59 |
| mordred | lifeless: and calling super because I'm only adding one param myself, and the other one should be have as standard Exception does | 20:59 |
| bknudson | fungi: that might never happen. | 20:59 |
| *** tnovacik has quit IRC | 21:00 | |
| fungi | bknudson: in the meantime, if you're in a hurry, you could run the update.py script from requirements against that branch of pycadf and propose the resulting diff yourself (after checking that it makes sense of course) | 21:00 |
| bknudson | ohh | 21:00 |
| mordred | SpamapS: if I stab you in the face in a friendly way, don't take it the wrong way | 21:00 |
| fungi | bknudson: anything else in that particular sausage factory you have questions about, don't hesitate to ask | 21:01 |
| *** Swami has joined #openstack-infra | 21:01 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add inventory command to shade https://review.openstack.org/175891 | 21:02 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Raise a shade exception on broken volumes https://review.openstack.org/176550 | 21:02 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a method for getting an endpoint https://review.openstack.org/177314 | 21:02 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Make keystone endpoints work like the others https://review.openstack.org/177315 | 21:02 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add early service fail and active check method https://review.openstack.org/177316 | 21:02 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Call super in OpenStackCloudException https://review.openstack.org/177347 | 21:02 |
| SpamapS | mordred: my face hasn't been stabbed in a while. ;) | 21:02 |
| * mordred uses all of the test nodes ... | 21:02 | |
| mordred | SpamapS: ok. I think that's finally a winner | 21:02 |
| anteaya | mordred: have you been running low on cats? | 21:02 |
| *** gyee has quit IRC | 21:02 | |
| fungi | mordred: you're just stress-testing hpcloud, right? | 21:02 |
| mordred | anteaya: I try to use them only on special occasions | 21:02 |
| mordred | fungi: yup | 21:02 |
| *** gyee has joined #openstack-infra | 21:02 | |
| anteaya | mordred: I feel that is such a nicer form of communication | 21:02 |
| anteaya | ah | 21:02 |
| * anteaya feels bad about the stabbing | 21:03 | |
| mordred | anteaya: I did hand lifeless a used packet of seasoning earlier | 21:03 |
| *** tsg_ has quit IRC | 21:03 | |
| *** bswartz has quit IRC | 21:03 | |
| fungi | irl? | 21:03 |
| anteaya | I missed that | 21:03 |
| mordred | fungi: nope | 21:03 |
| mordred | fungi: irc | 21:03 |
| fungi | oh, too bad | 21:03 |
| anteaya | such I nice gift | 21:03 |
| fungi | 'cause i totally pictured it happening in meatspace | 21:03 |
| anteaya | ha ha ha | 21:04 |
| *** ayoung has joined #openstack-infra | 21:04 | |
| *** isviridov_away is now known as isviridov | 21:04 | |
| *** sdake_ has joined #openstack-infra | 21:04 | |
| *** tsg has joined #openstack-infra | 21:04 | |
| openstackgerrit | lifeless proposed openstack-infra/project-config: Test pbr against unreleased pip. https://review.openstack.org/177423 | 21:05 |
| lifeless | fungi: thanks | 21:05 |
| lifeless | fungi: I appreciate the hand holding immensely | 21:05 |
| *** dhritishikhar has joined #openstack-infra | 21:05 | |
| mordred | fungi: now I kind of want to hand lifeless a used packet of seasoning in person | 21:05 |
| fungi | lifeless: other than the hard tabs your editor seems to really like inserting in yaml, i think that should pass our project-config jobs now | 21:06 |
| lifeless | fungi: so frustrating | 21:06 |
| mordred | my editor is the WORST at yaml, btw | 21:06 |
| *** andreykurilin__ has joined #openstack-infra | 21:06 | |
| fungi | is it like worstcat for yaml? | 21:07 |
| mordred | it constantly wants to reindent lines in the wrong way | 21:07 |
| mordred | fungi: YES | 21:07 |
| openstackgerrit | lifeless proposed openstack-infra/project-config: Test pbr against unreleased pip. https://review.openstack.org/177423 | 21:07 |
| lifeless | I need to add a yaml rule for my vimrc | 21:07 |
| lifeless | anyhow | 21:07 |
| *** salv-orlando has joined #openstack-infra | 21:07 | |
| mordred | http://41.media.tumblr.com/7accf821d3ea0c5e8c5c8d39a8bd0f78/tumblr_nljc9lRnc61tlnaoto1_400.jpg | 21:07 |
| mordred | lifeless: if you find good vim settings to make yaml not suck, please share | 21:07 |
| mordred | lifeless: I don't have tab problems, but I do have bad autoindent problems | 21:08 |
| *** sdake has quit IRC | 21:08 | |
| lifeless | mordred: I'll fiddle when I have a spare cycle | 21:08 |
| * mordred hands lifeless a bicycle he found out on the street, ignoring the half-eaten sandwich left in the basket | 21:09 | |
| fungi | lifeless: lgtm. now it's clarkb's turn to tell me i've steered you in an entirely wrong direction or something | 21:09 |
| SpamapS | mordred: that cat sandwich looks disgusting. | 21:09 |
| cinerama | pleia2: oh SNAP. we're down. can you look? | 21:10 |
| *** jtriley has joined #openstack-infra | 21:10 | |
| *** tonytan4ever has quit IRC | 21:10 | |
| fungi | cinerama: i can take a look if she's convalescing with her plague at the moment | 21:10 |
| fungi | cinerama: translate-dev.o.o yeah? | 21:11 |
| cinerama | fungi: lemme put on my plague doctor mask | 21:11 |
| cinerama | fungi: yes | 21:11 |
| fungi | apache's running at least | 21:11 |
| fungi | cinerama: i can get to the login screen at least | 21:12 |
| fungi | what specifically is broken? | 21:12 |
| *** salv-orl_ has joined #openstack-infra | 21:13 | |
| cinerama | fungi: oh weird, now it's working. before it was just showing the wildfly default screen. i noticed because the client was having a sad | 21:13 |
| cinerama | fungi: it might be worth seeing if there's something of interest in /var/log/wildfly/console.log | 21:13 |
| fungi | i'll take a peek. might be cats in the gears | 21:13 |
| openstackgerrit | lifeless proposed openstack-dev/pbr: Honour new test variable PIPVERSION https://review.openstack.org/177457 | 21:13 |
| *** salv-orlando has quit IRC | 21:14 | |
| *** dboik_ has joined #openstack-infra | 21:15 | |
| lifeless | fungi: $tmpdir/wheelhouse/bin/pip | 21:15 |
| *** akshai has quit IRC | 21:15 | |
| fungi | cinerama: the earliest timestamp in that file is 2015-04-24 21:09:58 | 21:15 |
| fungi | cinerama: i smell a restart | 21:15 |
| yolanda | mordred, jeblair, would you mind taking a look at https://review.openstack.org/#/c/139598/ ? it's becoming a bit urgent from our side, and already has a +2 and several positive comments | 21:15 |
| yolanda | nodepool image testing | 21:15 |
| fungi | lifeless: what's the question? | 21:16 |
| lifeless | fungi: Can lifeless copy-paste correctly. | 21:16 |
| lifeless | fungi: https://jenkins06.openstack.org/job/gate-project-config-layout/842/console | 21:16 |
| fungi | oh | 21:16 |
| *** ldnunes has quit IRC | 21:16 | |
| lifeless | oh, I see it | 21:17 |
| openstackgerrit | lifeless proposed openstack-infra/project-config: Test pbr against unreleased pip. https://review.openstack.org/177423 | 21:17 |
| mordred | yolanda: I think it needs a jeblair | 21:18 |
| anteaya | just one jeblair? | 21:18 |
| *** dboik has quit IRC | 21:18 | |
| mordred | anteaya: how many jeblairs does it take? | 21:19 |
| anteaya | usually less than one | 21:19 |
| *** sdake has joined #openstack-infra | 21:19 | |
| *** dboik_ has quit IRC | 21:19 | |
| anteaya | was picturing many jeblairs | 21:19 |
| anteaya | the libraries they would organize | 21:19 |
| mordred | is a jeblair a metric or imperial unit? | 21:19 |
| anteaya | I would put money on metric myself | 21:20 |
| *** salv-orl_ has quit IRC | 21:20 | |
| yolanda | mordred, or any other infra cores , any of you guys are good to review that :) | 21:20 |
| fungi | cinerama: so it appears that wildfly restarted, though the server itself seems fine. also the rotated log doesn't seem to contain any indication of why it went down/restarted | 21:20 |
| fungi | cinerama: no sign of oom condition in dmesg or anything | 21:20 |
| fungi | cinerama: puppet applied a new configuration right then and restarted it | 21:21 |
| yolanda | mordred, i just need to start moving that a bit, even if that's not approved but i have good feedback on the direction | 21:21 |
| cinerama | fungi: wondering if puppet ran again | 21:21 |
| *** ajo has quit IRC | 21:21 | |
| fungi | cinerama: mystery solved | 21:21 |
| fungi | Apr 24 21:10:22 translate-dev puppet-agent[14575]: (/Stage[main]/Wildfly::Service/Service[wildfly]) Triggered 'refresh' from 1 events | 21:21 |
| cinerama | fungi: great, now we can all hang out with the gang at $local_establishment and end on a freeze frame | 21:21 |
| *** haypo has quit IRC | 21:22 | |
| fungi | cinerama: it would have gotten away with it too, if it weren't for us meddling kids | 21:22 |
| openstackgerrit | Brant Knudson proposed openstack/requirements: Bandit should not be updated automatically https://review.openstack.org/177460 | 21:23 |
| *** sdake_ has quit IRC | 21:23 | |
| *** viglesias has quit IRC | 21:25 | |
| *** ajo has joined #openstack-infra | 21:25 | |
| *** e0ne has joined #openstack-infra | 21:27 | |
| *** e0ne has quit IRC | 21:29 | |
| *** viglesias has joined #openstack-infra | 21:31 | |
| lifeless | fungi: 177423 is green | 21:31 |
| lifeless | mordred: / clarkb: ^ could I get a +A ? | 21:31 |
| *** e0ne has joined #openstack-infra | 21:33 | |
| clarkb | yolanda reviewed the spec | 21:34 |
| *** ChuckC_ is now known as ChuckC | 21:34 | |
| yolanda | clarkb, thx, let me take a look | 21:35 |
| *** radez is now known as radez_g0n3 | 21:36 | |
| yolanda | clarkb, what do you mean with I don't know that nodepool should rerun tests multiple times. You can just have the test itself retry if that is desired. | 21:36 |
| yolanda | ? | 21:36 |
| clarkb | lifeless you added a periodic job to jenkins but zuul layout doesnt seem to run it. Is that intentional? | 21:36 |
| yolanda | my idea was to add a retry setting,that would make nodepool retry the test if failed, the desired amount of times | 21:37 |
| clarkb | yolanda put the for loop in the script that runs on the node not in nodepool | 21:37 |
| yolanda | clarkb, so you mean add the number of retries in the same script is going to test the image? | 21:37 |
| clarkb | yolanda in my mind the interface here should be boot need, run script, check return code | 21:37 |
| clarkb | its simple and matches existing interfaces | 21:37 |
| clarkb | we already do this for image builds and node ready checks | 21:38 |
| yolanda | ok, i'm fine with that. And for running the test, you should just spin up the node, and run the test as part of that spin up ? you meant that' | 21:39 |
| yolanda | ? | 21:39 |
| clarkb | ya basically do like the ready check script but for an image state transition | 21:39 |
| lifeless | clarkb: my intent was to run it. | 21:39 |
| yolanda | i'm ok with your opinion, even much more simple than my thought | 21:39 |
| clarkb | I dont like making jenkins and gearman required here since neither are required today | 21:39 |
| lifeless | clarkb: I am just entirely paged out all of this | 21:39 |
| *** signed8bit has quit IRC | 21:40 | |
| yolanda | i'll just amend it now and push, as I want to move that forward as soon as possible | 21:40 |
| fungi | clarkb: good point, i missed that should have been periodic-whatever in the periodic pipeline | 21:41 |
| clarkb | it should allow you to optionally use jenkins or anything else | 21:41 |
| *** akshai has joined #openstack-infra | 21:41 | |
| clarkb | lifeless fungi I have to drive noe but lgtm other than not running the periodic job | 21:41 |
| *** dangers is now known as dangers_away | 21:41 | |
| lifeless | morded has +A | 21:42 |
| lifeless | I will follow up with the periodic thing | 21:42 |
| *** dangers_away is now known as dangers | 21:42 | |
| lifeless | zuul/layout you say | 21:42 |
| fungi | lifeless: yeah, so the periodic instantiation of that job-template would simply allow you to have a different job name for the version you put in the periodic pipeline in layout.yaml | 21:43 |
| *** miqui has quit IRC | 21:43 | |
| lifeless | fungi: oh, so it will run periodically now? | 21:44 |
| fungi | lifeless: yeah, i don't think that extra stanza in projects.yaml is actually needed | 21:44 |
| *** gordc has quit IRC | 21:44 | |
| lifeless | ok | 21:45 |
| lifeless | then we'll wait and see | 21:45 |
| openstackgerrit | Merged openstack-infra/project-config: Test pbr against unreleased pip. https://review.openstack.org/177423 | 21:45 |
| fungi | lifeless: we have some jobs where the lack of a git commit to test on is a problem and so there are separate periodic jobs/job-templates to deal with that situation | 21:45 |
| fungi | lifeless: but without having thought through this particularly well, i suspect the pbr integration test job won't care | 21:45 |
| lifeless | how long till that new commit is active? | 21:46 |
| fungi | it's _possible_ devstack-gate will just get all unhappy, but we can wait and see i suppose | 21:46 |
| lifeless | I need to recheck the new change in pbr against it | 21:46 |
| *** hdd has joined #openstack-infra | 21:46 | |
| fungi | lifeless: best guess is ~30-45 minutes before the jobs are created on the jenkins servers and zuul has reconfigured to know to run them | 21:46 |
| lifeless | ok | 21:46 |
| lifeless | I'll pop back in a bit then | 21:46 |
| *** mriedem has quit IRC | 21:47 | |
| *** bknudson has quit IRC | 21:49 | |
| *** smccully has quit IRC | 21:52 | |
| *** stevemar has quit IRC | 21:53 | |
| *** dhritishikhar has quit IRC | 21:53 | |
| *** dboik has joined #openstack-infra | 21:54 | |
| *** prad has quit IRC | 21:57 | |
| *** prad has joined #openstack-infra | 21:58 | |
| *** e0ne has quit IRC | 22:00 | |
| *** baoli has quit IRC | 22:00 | |
| openstackgerrit | yolanda.robla proposed openstack-infra/infra-specs: Add nodepool image testing spec. https://review.openstack.org/139598 | 22:02 |
| yolanda | clarkb, i pushed a new version. Many thx for your feedback | 22:02 |
| *** jtriley has quit IRC | 22:03 | |
| *** otter768 has joined #openstack-infra | 22:03 | |
| *** mwagner_lap has quit IRC | 22:03 | |
| *** hdd has quit IRC | 22:04 | |
| *** jtriley has joined #openstack-infra | 22:06 | |
| *** otter768 has quit IRC | 22:07 | |
| *** yamamoto has joined #openstack-infra | 22:08 | |
| *** patrickeast has joined #openstack-infra | 22:09 | |
| *** isviridov is now known as isviridov_away | 22:11 | |
| *** sdake_ has joined #openstack-infra | 22:11 | |
| *** Kennan2 has joined #openstack-infra | 22:12 | |
| *** Kennan has quit IRC | 22:12 | |
| *** _dguerri is now known as dguerri | 22:12 | |
| *** yamamoto has quit IRC | 22:13 | |
| *** lascii is now known as alaski | 22:13 | |
| *** r-daneel has quit IRC | 22:15 | |
| *** sdake has quit IRC | 22:15 | |
| *** bswartz has joined #openstack-infra | 22:15 | |
| clarkb | fungi: re 177044 why would project bootstrappers have any explicit perms? They can basically do anything already | 22:16 |
| openstackgerrit | Davide Guerri proposed openstack-infra/shade: WiP: Add Neutron/Nova Floating IP support https://review.openstack.org/177036 | 22:16 |
| clarkb | fungi: does exclusive group perms override the magical do everything permissions for project boostrappers? | 22:17 |
| fungi | clarkb: exclusive group permissions takes it away if not explicitly mentioned in that section | 22:17 |
| fungi | we might actually consider dropping that in per-project acls though anyway, just because administrators can work around it by temporarily adding themselves to whatever group has access to what they need | 22:18 |
| *** teran has quit IRC | 22:18 | |
| clarkb | fungi: I just tested and that doesn't appear to be the case | 22:18 |
| *** akshai has quit IRC | 22:18 | |
| clarkb | add yourself to project-bootstrappers then look at https://review.openstack.org/#/c/177378/1,publish | 22:18 |
| yolanda | mordred, clarkb, how is status will the multi-threaded nodepool change i did? do you think it won't be useful now that hpcloud is performing better? i was still interested in giving that a try | 22:18 |
| clarkb | thats a stable/kilo barbican change | 22:18 |
| clarkb | yolanda: it may still be useful, we had to pull it out of production because we think it may have caused hpcloud to fall over (though I was afk for that so may have misunderstood) | 22:19 |
| fungi | for example, the other day i needed to manually submit a change on devstack stable/juno and had to manually add myself to devstack-core to do it. just adding myself to project bootstrappers was insufficient | 22:19 |
| yolanda | oh really? my two queue change made hpcloud fail? | 22:19 |
| yolanda | why? | 22:19 |
| clarkb | fungi: oh nevermind its only code-review that is currently affected | 22:19 |
| clarkb | yolanda: all we have is the inference that we put it in place and a few hours later hpcloud said turn it off | 22:20 |
| clarkb | yolanda: same thing happened when we tried jeblairs change | 22:20 |
| yolanda | oh man... | 22:20 |
| pleia2 | cinerama: ah, we'll just have to keep an eye on what triggers restarts (since they take a couple minutes and can be disruptive), some of the updates can probably pile up until we have a maintenance window in production | 22:20 |
| clarkb | yolanda: you probably need to talk to mordred about it bceause I don't know much more than that | 22:21 |
| yolanda | clarkb, so i had it running for 2 days on my cloud account, and was performing so good . But of course you have much more volume than me | 22:21 |
| mordred | aro? | 22:21 |
| yolanda | mordred, do you know more? | 22:21 |
| mordred | no - I do not | 22:21 |
| mordred | sorry | 22:21 |
| clarkb | mordred: we put in yolanda's two queue change for nodepool so that list nodes would be separated and we very quickly disabled hpcloud after | 22:21 |
| clarkb | mordred: this was the first time we disabled hpcloud recently and I think they told you to do it | 22:22 |
| *** dguerri is now known as _dguerri | 22:22 | |
| yolanda | clarkb, i thought it was due to maintenance' | 22:22 |
| yolanda | ? | 22:22 |
| mordred | oh - yeah - I'm not sure I realized we'd put in yolanda's change | 22:22 |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: Remove heat from default configuration https://review.openstack.org/170608 | 22:22 |
| *** jtriley has quit IRC | 22:22 | |
| mordred | so I think it might have just been happenstance | 22:22 |
| mordred | their system was just steadily degrading and hit the point where our load was too much | 22:22 |
| clarkb | fungi: I just approved the project bootstrappers cleanup | 22:22 |
| yolanda | so i wanted to give that a try but i don't want to make hpcloud explode | 22:22 |
| mordred | thankfully that's fixed | 22:22 |
| yolanda | mordred, are you giving that patch another try? | 22:23 |
| clarkb | mordred: it may be, except the time before last that we did similarto nodepool we also had to turn it off | 22:23 |
| mordred | clarkb: nod | 22:23 |
| *** sdake has joined #openstack-infra | 22:23 | |
| fungi | clarkb: thanks. that was my bad for not noticing the normalize script was insane and had done that to the acls when i ran it before submitting the stable/kilo overrides before | 22:23 |
| clarkb | but maybe everything is happy again post cleanup | 22:23 |
| mordred | clarkb: it may be worth trying again post cleanup | 22:23 |
| clarkb | mordred: ya | 22:23 |
| mordred | otoh - the patch shouldn't be needed in a world where the cloud isn't broken in the first pace | 22:23 |
| mordred | place | 22:23 |
| yolanda | mordred, i know, it's not so broken now, but still a bit slow | 22:24 |
| openstackgerrit | Merged openstack-infra/project-config: Add coverage job to cue check pipeline https://review.openstack.org/177022 | 22:24 |
| clarkb | yolanda: also I think jeblair had comments, have those been addressed? | 22:24 |
| yolanda | looking at numbers it still makes a bit of sense | 22:24 |
| clarkb | maybe its best to try it again before rewriting it if possible | 22:24 |
| yolanda | clarkb, i think so, unless i missed something new | 22:24 |
| clarkb | yolanda: it was just the task interface stuff | 22:25 |
| clarkb | which isn't that new | 22:25 |
| yolanda | ah, the stop | 22:25 |
| yolanda | i did the change | 22:25 |
| *** hyakuhei has joined #openstack-infra | 22:25 | |
| *** rlandy has quit IRC | 22:25 | |
| *** yamahata has joined #openstack-infra | 22:25 | |
| *** dimtruck is now known as zz_dimtruck | 22:25 | |
| yolanda | clarkb, mordred, so i added a stop method to the taskmanager | 22:26 |
| *** sdake_ has quit IRC | 22:27 | |
| mordred | yolanda: is that a change that shade would need to support? | 22:27 |
| clarkb | that was the concern with the first pass | 22:27 |
| *** dprince has quit IRC | 22:28 | |
| yolanda | mordred, as far as i know, shade will allow a taskmanager to be passed | 22:28 |
| yolanda | and all the updates are done into taskmanager, i'm not doing changes on Task anymore | 22:28 |
| openstackgerrit | Merged openstack-infra/project-config: There is no Project Bootstrappers-core... https://review.openstack.org/177044 | 22:30 |
| *** jtriley has joined #openstack-infra | 22:30 | |
| clarkb | anyone else want to review devstack-gate's foray into using ansible? https://review.openstack.org/#/c/172614/ | 22:31 |
| clarkb | fungi: pleia2 jhesketh SergeyLukjanov ^ otherwise I am going to approve (it has necessary +2s) | 22:32 |
| lifeless | folk interested in pip/pbr - see https://jenkins05.openstack.org/job/check-pbr-installation-dsvm-upstream/1/console | 22:32 |
| lifeless | for automatic wheel building at work | 22:32 |
| mordred | neat! | 22:32 |
| clarkb | lifeless: any idea why its doing `pip wheel ''` ? | 22:32 |
| clarkb | lifeless: it seems to barf on that | 22:32 |
| * clarkb is guesing a bad bash for loop? | 22:33 | |
| pleia2 | clarkb: having a look | 22:33 |
| openstackgerrit | Merged openstack-infra/project-config: zuul/layout: make gate-puppet-.*-puppet-syntax-future voting https://review.openstack.org/177267 | 22:33 |
| clarkb | pleia2: danke | 22:34 |
| openstackgerrit | Joe Gordon proposed openstack-infra/project-config: Add tempest-dsvm-sahara https://review.openstack.org/167779 | 22:35 |
| openstackgerrit | Joe Gordon proposed openstack-infra/project-config: Add sahara tempest job to sahara and tempest https://review.openstack.org/167780 | 22:35 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add inventory command to shade https://review.openstack.org/175891 | 22:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Raise a shade exception on broken volumes https://review.openstack.org/176550 | 22:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a method for getting an endpoint https://review.openstack.org/177314 | 22:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Make keystone endpoints work like the others https://review.openstack.org/177315 | 22:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add early service fail and active check method https://review.openstack.org/177316 | 22:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Split exceptions into their own file https://review.openstack.org/177476 | 22:36 |
| mordred | SpamapS, Shrews: ^^ I split out exceptions into their own file | 22:36 |
| clarkb | hrm jd__ is likely out for the weekend /me approves and leaves a comment | 22:37 |
| yolanda | clarkb, mordred, leaving for today, just drop some comments here if you have some more feedback about this change. I really wish i could talk to you more often, spending some time in the chat at your timezone is so useful | 22:37 |
| *** thinrichs has joined #openstack-infra | 22:37 | |
| yolanda | i raised the comment with jeblair this week, communication with upstream is one of my main concerns | 22:38 |
| mordred | yolanda: ++ | 22:38 |
| mordred | yolanda: I totally agree - it's hard when the overlap is bad | 22:38 |
| clarkb | silly timezones | 22:38 |
| yolanda | i will end up working at nights :) | 22:38 |
| clarkb | I tell you what though, it was a lot easier for me to stay up late and work with apac before I started having all these scheduled appointments during "normal" work hours | 22:39 |
| clarkb | unfortunately if you are seeing the doctor at 10pm thats usuall a very bad thing | 22:39 |
| mordred | clarkb: if you get a sec - 177476 and the two before it are boring and I'd love to clear them out of the queue - the ones after it I need to add some tests for ... but I'm not adding unit tests to test str(e) ... | 22:40 |
| *** ayoung is now known as ayoung-noms | 22:40 | |
| yolanda | so today i'm just talking with you because i'm on GMT time , an hour later than in Spain, and not with family. But it's still a crazy hour, so imagine in a day to day in spanish time, is so hard to have a chat with you all | 22:40 |
| *** harlowja_ has joined #openstack-infra | 22:41 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: puppet: run beaker on both trusty & centos7 https://review.openstack.org/175434 | 22:41 |
| mordred | yolanda: yah - it's one of the reasons I want more folks in europe timezones so that there are more people working together - but also since it's so hard to talk I fear growing two split teams ... it's a hard problem we have to solve | 22:41 |
| *** erlon has quit IRC | 22:41 | |
| *** harlowja has quit IRC | 22:41 | |
| yolanda | mordred, so you have some folks interested on that, including myself, we'd love to resolve that situation | 22:42 |
| lifeless | hmm no, that hasn't stuck quite right | 22:42 |
| yolanda | not sure on how that works on other projects, i suppose it should be the same problem everywhere? | 22:42 |
| fungi | lifeless: have an example run you can link? | 22:43 |
| *** Swami_ has joined #openstack-infra | 22:43 | |
| thinrichs | Hi all. If someone has a free minute, I'm having some trouble pushing tags/branches to gerrit (for the first time). Possibly something silly. | 22:43 |
| *** esker has quit IRC | 22:43 | |
| *** amitgandhinz has quit IRC | 22:44 | |
| fungi | thinrichs: can you put a transcript of the commands/errors on paste.openstack.org and link them here? | 22:44 |
| thinrichs | fungi: sure. | 22:44 |
| lifeless | fungi: I'm looking in https://jenkins05.openstack.org/job/check-pbr-installation-dsvm-upstream/1/consoleFull for PBR_PIP_VERSION | 22:44 |
| lifeless | fungi: which should show up in the +if [ -n "${PBR_PIP_VERSION:-}" ]; then | 22:45 |
| lifeless | test | 22:45 |
| lifeless | which does pip wheel -w | 22:45 |
| lifeless | but there's no pip wheel -w | 22:45 |
| thinrichs | fungi: http://paste.openstack.org/show/205831/ | 22:45 |
| *** rbradfor has quit IRC | 22:45 | |
| mordred | yolanda: I think we're a slightly different situation, because we're a little more protective of the core status due to the production systems hanging off of the +2 ... that doesn't mean we're doing it right currently - but I think it's at least slightly harder for us to solve than for the other teams | 22:46 |
| thinrichs | fungi: I'm a member of the congress-release on gerrit | 22:46 |
| lifeless | fungi: and we should be installing a pip wheel from a file | 22:46 |
| mordred | yolanda: that said- I'm very excited that you and others are interested and willing to help us work on resolving it | 22:46 |
| lifeless | fungi: not just wheel | 22:46 |
| *** Swami has quit IRC | 22:46 | |
| clarkb | mordred: see comment on https://review.openstack.org/#/c/177347/ my test there shows that the str() may not do what you want it to do | 22:47 |
| mordred | clarkb: really? | 22:47 |
| * mordred glares at SpamapS | 22:47 | |
| clarkb | mordred: ya it will print None | 22:47 |
| yolanda | mordred, yes, i was not talking about cores really, but do anything that we can to improve that communication and improve speed of reviews | 22:47 |
| *** marun has quit IRC | 22:47 | |
| clarkb | mordred: which you are explicitl trying to not do | 22:47 |
| fungi | thinrichs: yeah, permissions look right. i'll double-check that there's not a second account for you or something | 22:47 |
| clarkb | mordred: but double check my manual test case | 22:47 |
| yolanda | from our team we really feel we need to improve on that | 22:47 |
| thinrichs | fungi: I changed my email at some point. Could that be the problem? | 22:47 |
| mordred | clarkb: no - you're right - I had a workaround for that in a previous patch, and I think I baby-with-the-bathwatered something | 22:48 |
| mordred | thanks | 22:48 |
| clarkb | mordred: np | 22:48 |
| *** hyakuhei has quit IRC | 22:48 | |
| openstackgerrit | Merged openstack-infra/project-config: Run Gnocchi devstack tests for stable/1.0 against Kilo https://review.openstack.org/176907 | 22:49 |
| *** hamzy has quit IRC | 22:49 | |
| *** hyakuhei has joined #openstack-infra | 22:49 | |
| *** ajo has quit IRC | 22:50 | |
| *** hyakuhei has quit IRC | 22:50 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Call super in OpenStackCloudException https://review.openstack.org/177347 | 22:50 |
| mordred | clarkb: how about that ^^ ? | 22:50 |
| lifeless | clarkb: the '' is probably from grep -v '^#' $REPODIR/requirements/global-requirements.txt | while read req | 22:50 |
| *** woodster_ has quit IRC | 22:50 | |
| mtreinish | mordred: is that raised if OpenStack dies? | 22:50 |
| lifeless | mordred: why do you call pip wheel in a loop | 22:50 |
| lifeless | mordred: rather than 'pip wheel -r global-requirements.txt' ? | 22:50 |
| *** xyang1 has quit IRC | 22:50 | |
| clarkb | mordred: that should fix it +2 | 22:51 |
| fungi | thinrichs: oh! you didn't sign the tag | 22:51 |
| mordred | clarkb: thanks! | 22:51 |
| fungi | thinrichs: http://docs.openstack.org/infra/manual/drivers.html#tagging-a-release | 22:51 |
| mordred | lifeless: uhm ... there was a reason | 22:51 |
| clarkb | lifeless: mordred yes the reason is if one fails you want to continue and do the others | 22:52 |
| mordred | lifeless: in some contexts it was to get as many things as possible - beacuse pip fails on the first | 22:52 |
| thinrichs | fungi: I was about to ask that! So once I sign it, I shouldn't have any issues? | 22:52 |
| *** dhritishikhar has joined #openstack-infra | 22:52 | |
| lifeless | clarkb: pip itself does that | 22:52 |
| clarkb | lifeless: it didn't before | 22:52 |
| mordred | lifeless: if it does, it's new that it does | 22:52 |
| clarkb | lifeless: it would do everthing up to the fail then bail out iirc | 22:52 |
| mordred | yup | 22:52 |
| mordred | it very much did that | 22:52 |
| SpamapS | mordred: +1'd as well.. And that is, btw, the answer to "why aren't you just passing extra_data to super" ;) | 22:52 |
| fungi | thinrichs: you'll need to `git tag -d 2015.1.0rc1` to delete the current one in your local repo, and then try again using the command we have documented there in the infra manual | 22:52 |
| mordred | SpamapS: :) | 22:53 |
| fungi | thinrichs: but yeah, otherwise everything looks okay with your account and permissions | 22:53 |
| * mordred hands SpamapS a super | 22:53 | |
| lifeless | mordred: clarkb: possibly a bug; the intent in pip is clearly to run them all and then fail at the end | 22:53 |
| *** sdake_ has joined #openstack-infra | 22:53 | |
| mordred | lifeless: neat | 22:53 |
| *** achanda has quit IRC | 22:53 | |
| lifeless | mordred: # Return True if all builds were successful | 22:53 |
| mordred | I love bugs | 22:53 |
| lifeless | return len(build_failure) == 0 | 22:53 |
| lifeless | and 'Failed to build %s', | 22:53 |
| lifeless | ' '.join([req.name for req in build_failure]), | 22:53 |
| lifeless | anyhow, that can wait | 22:54 |
| thinrichs | fungi: Thanks! Didn't occur to me that signing would be required. Will remember next time. | 22:54 |
| lifeless | it will fix that '' issue | 22:54 |
| fungi | thinrichs: yep, if you look at your gerrit acl, the control is specific to signed tags only. unsigned annotated and lightweight tags are not allowed by that acl | 22:55 |
| *** sdake__ has joined #openstack-infra | 22:55 | |
| fungi | thinrichs: that's mainly because people releasing software via git tags should be signing the tags, so having the acl that way helps keep us from accidentally forgetting to sign them | 22:56 |
| *** sdake has quit IRC | 22:56 | |
| thinrichs | fungi: makes sense. Thanks again. | 22:56 |
| fungi | thinrichs: any time! | 22:57 |
| *** apmelton is now known as apmelton_away | 22:58 | |
| *** sdake_ has quit IRC | 22:59 | |
| fungi | clarkb: regarding your comment on 176907, i was going to ask the same thing until i reread the commit message and realized that it seems like they're saying they expect to have multiple branches of gnocci they want to test against the same branches of openstack | 22:59 |
| clarkb | mordred: for the noqa at https://review.openstack.org/#/c/177476/1/shade/__init__.py would it be beter to blanket disable that rule or are exceptions a special case? | 22:59 |
| clarkb | mordred: I am trying to remember if that is a hacking rule and I think it is | 22:59 |
| clarkb | which makes me think turning it off is ok | 22:59 |
| mordred | clarkb: I generally like that rule ... but also I didn't really want to shove exc. everywhere | 22:59 |
| lifeless | fungi: 2015-04-24 22:09:05.643 | + -upstream = -upstream | 23:00 |
| lifeless | 2015-04-24 22:09:05.644 | /tmp/hudson8862369368516885738.sh: line 8: -upstream: command not found | 23:00 |
| lifeless | fungi: that appears to be the problem | 23:00 |
| fungi | clarkb: so e.g., stable/1.0 and stable/2.0 against openstack stable/kilo but stable/3.0 against master or something | 23:00 |
| fungi | lifeless: oh, it stripped the quotes! | 23:00 |
| clarkb | fungi: gotcha | 23:00 |
| mordred | clarkb: how about I follow up with a patch that either removes the rule or uses exc. everywhere? | 23:00 |
| lifeless | fungi: would - if "{variant}" = "-upstream"; then | 23:00 |
| lifeless | + if [ "{variant}" = "-upstream" ]; then | 23:00 |
| lifeless | help ? | 23:00 |
| fungi | lifeless: er, nevermind. i forgot the word "test" there didn't i? | 23:00 |
| lifeless | fungi: or should I push the eval down into integration.sh ? | 23:01 |
| fungi | lifeless: or square brackets, yeah | 23:01 |
| fungi | lifeless: well, you'll still need to get that parameter into the environment i think | 23:01 |
| mordred | clarkb: I need to rebase through that patch anyway | 23:01 |
| greghaynes | clarkb: youd like this https://review.openstack.org/#/c/166650/15/elements/debian/package-installs.yaml | 23:01 |
| greghaynes | er | 23:01 |
| mordred | clarkb: I'll just fix it this pass | 23:01 |
| greghaynes | wrong link | 23:01 |
| clarkb | mordred: I just -1'd it for the rebase too | 23:01 |
| greghaynes | clarkb: http://logs.openstack.org/43/177043/1/check/gate-dib-dsvm-functests-devstack-precise/e06633d/console.html#_2015-04-24_22_02_52_966 | 23:02 |
| openstackgerrit | lifeless proposed openstack-infra/project-config: Better shell. https://review.openstack.org/177482 | 23:02 |
| fungi | lifeless: so if you want to do the eval in the script, you need to export a shell variable with that, which probably makes the script not stand alone as well | 23:02 |
| greghaynes | clarkb: that link | 23:02 |
| lifeless | fungi: the script barely is :) | 23:02 |
| clarkb | greghaynes: \o/ | 23:02 |
| fungi | fair enough | 23:02 |
| *** sarob has quit IRC | 23:02 | |
| lifeless | now I just need two cores to hammer that through | 23:03 |
| clarkb | lifeless: looking | 23:03 |
| lifeless | and then go away for 45m and come back | 23:03 |
| clarkb | silly test | 23:03 |
| *** woodster_ has joined #openstack-infra | 23:03 | |
| fungi | lifeless: also my apologies for inflicting my terrible mailercode on you | 23:04 |
| lifeless | fungi: its ok, I should have applied my brain cap more too | 23:04 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Split exceptions into their own file https://review.openstack.org/177476 | 23:06 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Raise a shade exception on broken volumes https://review.openstack.org/176550 | 23:07 |
| mordred | clarkb: there ya go | 23:07 |
| *** hemna is now known as hemnafk | 23:09 | |
| clarkb | mordred: looking. also you hinted at infra cloud on list, where are we on getting hardware straightend out? | 23:09 |
| clarkb | I think it would be a neat exercise to just use the d-g deploment while we do a real thing | 23:09 |
| mordred | clarkb: yeah - your idea came up over lunch today | 23:10 |
| mordred | clarkb: although I think we've got to get the network configured at some point ... | 23:10 |
| fungi | i might, you know, know someone who can configure network hardware, if that's the hold up | 23:11 |
| mordred | nah - that we've got | 23:11 |
| clarkb | right hardware and networking have to happen first, just wondering if that is moving at all | 23:11 |
| mordred | we've been holding off on that because infra-cloud is not on the priority effort list yet | 23:12 |
| * anteaya listens as she is also curious | 23:12 | |
| lifeless | mordred: new hardware? | 23:12 |
| mordred | nope. same old hardware | 23:12 |
| fungi | as in, is there out-of-band console access to the network gear and a diagram of how it's wired? the rest i'd be glad to help with | 23:12 |
| mordred | :) | 23:12 |
| mordred | fungi: the missing part right now is an agreed upon network design from us | 23:12 |
| *** ddieterly has quit IRC | 23:12 | |
| fungi | oh. got it | 23:13 |
| fungi | yes, that we will need | 23:13 |
| mordred | so maybe we'll come out of the summit with that | 23:13 |
| mordred | because once that's reconfigured, the ability for us to make interesting decisions and try things increases | 23:13 |
| anteaya | has it made it on the infra etherpad yet? | 23:13 |
| fungi | many snapchats of random whiteboard scribble | 23:14 |
| clarkb | /22 on a single l2 across all the nodes for VMs then like a /24 for the flat network? | 23:14 |
| mordred | clarkb: why two networks? | 23:14 |
| clarkb | hopefully we don't care much about it other than that? I suppose there may be lacp and other fun thigns to get into | 23:14 |
| clarkb | mordred: because nova net and neutron both expect that | 23:14 |
| anteaya | nova-net does too? | 23:14 |
| clarkb | anteaya: yes | 23:14 |
| * anteaya is trying to understand all the networking things | 23:15 | |
| mordred | clarkb: I'm assuming you mean for control plane communication? | 23:15 |
| fungi | clarkb: i was hoping for a /48 | 23:15 |
| mtreinish | clarkb: with a /24 be enough? | 23:15 |
| fungi | fat chance, i know | 23:15 |
| clarkb | anteaya: mordred: well nova network calls it the "flat" network | 23:15 |
| clarkb | fungi: I think one of the two regions can ipv6 but not both | 23:15 |
| mtreinish | I have no idea how big this will be | 23:15 |
| mordred | clarkb: both of the regions can ipv6 | 23:16 |
| clarkb | mordred: oh thats good news | 23:16 |
| mordred | clarkb: but we need to file the paperwork to get a range | 23:16 |
| fungi | mtreinish: /22 for the instances though, so that's "can haz 1k nodes" (i expect we'll be bottlenecked well below that for other resources anyway) | 23:16 |
| clarkb | but mostly my networking with openstack has been related to what we do in the gate which is pretty vanilla for both nova net and neutron | 23:16 |
| mordred | clarkb: assume in general that I'm not up on the nova-network/neutron terminology - you described two networks - can you expand on what they are in normal people words? | 23:16 |
| mtreinish | fungi: ah ok, I read that backwards | 23:17 |
| clarkb | mordred: yes, one is a public network where all your public addrs go | 23:17 |
| anteaya | clarkb: your notes have been helpful for me, thank you | 23:17 |
| mordred | clarkb: and the other is a second network that is used for? | 23:17 |
| fungi | mtreinish: and with a /24 for the metal, i seriously hope it's not so complicated we need more that that | 23:17 |
| clarkb | mordred: the other is a private side for VMs to talk to each other | 23:17 |
| mordred | clarkb: VMs or services? | 23:18 |
| clarkb | mordred: and now that I think about it mtreinish is right. for nova net you would need a /22 but neutron overlays so you wouldn't need any addrs assigned there | 23:18 |
| clarkb | mordred: VMs | 23:18 |
| clarkb | mordred: so think of it like how rax gives us two interfaces | 23:18 |
| mordred | clarkb: we don't want/need that | 23:18 |
| mordred | right | 23:18 |
| mordred | not needed | 23:18 |
| greghaynes | clarkb: you sure its not services also? I think ec2 api goes over internal | 23:18 |
| clarkb | mordred: sure I have no clue how to make it not happen though | 23:18 |
| thinrichs | fungi: sorry to bother again. Got that tag pushed, but now can't seem to push the branch. | 23:18 |
| openstackgerrit | Merged openstack-infra/project-config: Better shell. https://review.openstack.org/177482 | 23:18 |
| mordred | I have it both on good authority from markmcclain that we don't need 2 - and also vexxhost has one and only one network | 23:19 |
| clarkb | mordred: like we have a gazillion config options and the ones I understand end up with a rax like setup | 23:19 |
| thinrichs | fungi: followed all the instructions at the docs you pointed me to. | 23:19 |
| anteaya | clarkb: I think that is part of what I am trying to understand as well | 23:19 |
| mordred | clarkb: yah - apparently the private is not needed at all | 23:19 |
| greghaynes | mordred: I keep hearing you dont need two, its not very well documented how to do it without two | 23:19 |
| clarkb | mordred: I am sure we can configure it but I think it was kevin benton who said "do it this way except its terribly documented sorry" | 23:19 |
| mordred | greghaynes: sure - so maybe we can change that | 23:19 |
| thinrichs | fungi: trying to push directly from gerrit gave me the usual [remote rejected] | 23:19 |
| clarkb | greghaynes: I don't expect to run metadata service | 23:19 |
| mordred | o hell no | 23:19 |
| greghaynes | clarkb: awesome | 23:19 |
| greghaynes | hah | 23:19 |
| thinrichs | fungi: from review.openstack.org I couldn't seem to find anyway to create a new branch—but I could see the list of branches. | 23:19 |
| fungi | thinrichs: the branch creation works differently if you're not on the openstack release management team. you'll need have create permission on refs/heads/* in the acl and then use the web interface | 23:20 |
| mordred | I think we should design the thing we want - and then beat on PTLs to tell us how to configure it | 23:20 |
| greghaynes | ok, well then doing one isnt a problem I think, that was my only hiccup in my recent experiment on only using one interface | 23:20 |
| anteaya | mordred: oh perhaps ask them nicely? | 23:20 |
| fungi | thinrichs: hich repo again? | 23:20 |
| clarkb | mordred: thats fair, it will also be slower which is why the d-g option sprang to mind | 23:20 |
| mordred | anteaya: :) | 23:20 |
| fungi | er, whiwh | 23:20 |
| clarkb | mordred: we can have the bad thing that they want us to have | 23:20 |
| fungi | bleargh | 23:20 |
| thinrichs | fungi: openstack/congress | 23:20 |
| fungi | my fingers hate me tonight | 23:20 |
| clarkb | mordred: then point at X,Y,Z as problems and say "this is what we actually want" | 23:20 |
| mordred | clarkb: sure - but I'd like to make sure that whatever network reconfig we ask the HP NOC to do supports the thign we want | 23:20 |
| anteaya | fungi: ha ha ha | 23:20 |
| mordred | if there is also an extra 10. there, meh | 23:20 |
| mordred | don't care | 23:21 |
| clarkb | mordred: yup, I would go with two networks | 23:21 |
| clarkb | mordred: it will be most flexible | 23:21 |
| clarkb | and if we don't use one oh well | 23:21 |
| anteaya | clarkb mordred understanding why the make us have the bad thing would help me a lot | 23:21 |
| anteaya | clarkb mordred then the conversation about how to get the thing we want has a factual basis | 23:21 |
| mordred | anteaya: there is a "cloud-native" model that people like to talk about being the "good" way to do things | 23:21 |
| fungi | thinrichs: yes, in the refs/heads/* section of http://git.openstack.org/cgit/openstack-infra/project-config/tree/gerrit/acls/openstack/congress.config you need a line like "create = group congress-release" to be able to do that | 23:22 |
| mordred | anteaya: it involves giving VMs private ip addreses by default and then getting a public IP to them by use of floating ips | 23:22 |
| anteaya | mordred: who are these people and what documentation have they left in their wake? | 23:22 |
| mordred | anteaya: it's the "default" way things work | 23:22 |
| anteaya | mordred: except in openstack apparently | 23:22 |
| greghaynes | Another thing I just remembered - do we actually have two interfaces on all the hosts? I remember mention that some of the nics might have half their ports filled with infiniband instead of ethernet | 23:22 |
| mordred | anteaya: however - it breaks many things and is not advantageous to people actually using clouds | 23:22 |
| mordred | anteaya: nonononon | 23:22 |
| mordred | anteaya: sorry | 23:22 |
| mordred | anteaya: I radically misspoke | 23:22 |
| * anteaya listens again | 23:23 | |
| mordred | anteaya: there is an idea IN openstack that this is the right way tot od things | 23:23 |
| anteaya | ah | 23:23 |
| mordred | anteaya: we, in infra, disagree with it vehemently | 23:23 |
| anteaya | who is saying this in openstack? | 23:23 |
| mordred | because it makes things worse not better | 23:23 |
| mordred | anteaya: it's the default | 23:23 |
| mordred | anteaya: it's the way it works out of the box | 23:23 |
| anteaya | whose default? | 23:23 |
| mordred | anteaya: everythings | 23:23 |
| clarkb | anteaya: the documentations and devstack | 23:23 |
| lifeless | as much as openstack can be said to work out of the box | 23:23 |
| lifeless | at all | 23:23 |
| mordred | lifeless: well, yeah | 23:23 |
| fungi | bwahahaha | 23:23 |
| anteaya | nova-net's default or neutron's default? | 23:23 |
| clarkb | anteaya: both | 23:23 |
| * anteaya listens again | 23:24 | |
| clarkb | anteaya: this is how we test both neutron and nova net in the gate | 23:24 |
| mordred | it's a bonghits terrible design | 23:24 |
| anteaya | ah okay thank you | 23:24 |
| clarkb | anteaya: and we don't toggle any devstack bits to set it up that way | 23:24 |
| mordred | it's not a design to make somethign god | 23:24 |
| mordred | good | 23:24 |
| anteaya | clarkb: thank you I understand that part now | 23:24 |
| mordred | it's a design to conserve IPv4 addresses ina public cloud | 23:24 |
| anteaya | which with ipv6 we don't care about, yeah? | 23:24 |
| mordred | it makes applications harder to work with | 23:24 |
| mordred | well, we don't care about ipv4 conservation | 23:24 |
| mordred | we have plenty | 23:24 |
| anteaya | okay | 23:24 |
| anteaya | so what do we (infra) want? | 23:25 |
| fungi | also, any day now, ipv6. then we can just stop using ipv4 for things like job workers if we want | 23:25 |
| mordred | we want vms to get an ip directly allocated to them on the network | 23:25 |
| anteaya | fungi: yay | 23:25 |
| mordred | we don't want floating ips | 23:25 |
| anteaya | mordred: which from my reading is what dhcp does, yes? | 23:25 |
| mordred | they are a nightmare | 23:25 |
| mordred | anteaya: yes | 23:25 |
| clarkb | fungi: I am still keeping an eye on the turn on ipv6 by default changes. need one fix in nova first | 23:25 |
| clarkb | fungi: but that should happen early this cycle (liberty) | 23:25 |
| *** sigmavirus24 is now known as sigmavirus24_awa | 23:25 | |
| mordred | it's this great technology that was invented back in the 90s to dynamically configure hosts :) | 23:25 |
| fungi | for public clouds it'll likely be hard to convince them to give us instances with no global ipv4 but working ipv6. if we run our own though, possibility | 23:25 |
| clarkb | mordred: 80s even | 23:26 |
| mordred | clarkb: ++ | 23:26 |
| mordred | fungi: btw - vexxhost just gives a public ipv4 and no private network at all- I believe runabove is the same | 23:26 |
| anteaya | so does either networking solution give us what dhcp does? | 23:26 |
| clarkb | mordred: yes my runabove instance was that way | 23:26 |
| anteaya | or is close? | 23:26 |
| fungi | mordred: sure, but point being we can potentially also accomodate v6-only providers | 23:27 |
| mordred | anteaya: well, we can use dhcp with either neutron or nova-net | 23:27 |
| mordred | fungi: ++ | 23:27 |
| mordred | totally | 23:27 |
| mordred | anteaya: we will definitely run dhcp | 23:27 |
| fungi | mordred: and if wherever we're colocating the gear tells us we can't have more than a /24 of ipv4 for example, we could still spin up our own v6-only instances | 23:27 |
| mordred | yup | 23:27 |
| openstackgerrit | Merged openstack-infra/devstack-gate: Make multinode first class with ansible https://review.openstack.org/172614 | 23:28 |
| clarkb | funny story about the bug preventing ipv6 in devstack by default | 23:28 |
| fungi | and tell nodepool "your workers are over there, on the new internet, full of happy ipv6 elves and no ipv4 trolls" | 23:28 |
| mordred | clarkb: ^^ woot! | 23:28 |
| clarkb | it breaks floating IP attachment | 23:28 |
| clarkb | because it tries to nat ipv4 to ipv6 | 23:28 |
| clarkb | (more evidence that maybe this was a bad design) | 23:28 |
| mordred | hahahahaha | 23:28 |
| fungi | you just said nat. not map? | 23:28 |
| clarkb | fungi: NAT is how floating IPs work | 23:28 |
| mordred | yah | 23:29 |
| anteaya | so if we can run dhcp with both nova-net and neutron how am I not seeing that this is a solution that anyone can use? | 23:29 |
| mordred | it's amongst the many reasons they're bad | 23:29 |
| fungi | oh, got it. missed you said floating ip | 23:29 |
| anteaya | why all the hatred from users if they can do this themselves? | 23:29 |
| mordred | I mean, who in the world wants to run a real service on an internet connection that looks like their home DSL connection | 23:29 |
| * anteaya is still trying untangle and understand | 23:29 | |
| clarkb | anteaya: the default setups use dhcp, thats not the issue | 23:29 |
| anteaya | clarkb: oh okay | 23:29 |
| anteaya | what is the issue? | 23:29 |
| clarkb | anteaya: the issue is they force you to have two sets of networking and use NAT to map between them | 23:29 |
| clarkb | anteaya: this results in extra api calls as a user because you have to do that manuall | 23:30 |
| anteaya | can you configure that so you don't have to have that? | 23:30 |
| anteaya | or not configurable? | 23:30 |
| clarkb | anteaya: so your code is more complicated and error prone (we leak floating IPs) | 23:30 |
| openstackgerrit | Tim Hinrichs proposed openstack-infra/project-config: ACL changes for Congress create-reference https://review.openstack.org/177492 | 23:30 |
| mordred | anteaya: only when we run our own cloud | 23:30 |
| lifeless | mordred: well | 23:30 |
| mordred | anteaya: we cannot configure public clouds we consume to do this | 23:30 |
| lifeless | mordred: as a counterpoint to your rage | 23:30 |
| anteaya | mordred: oh | 23:30 |
| clarkb | anteaya: mordred however, the other options are basically undocumented | 23:30 |
| mordred | anteaya: which is why we have rage | 23:30 |
| mordred | clarkb: sure | 23:30 |
| anteaya | clarkb: ah ha | 23:30 |
| mordred | clarkb: I'll write docs | 23:30 |
| clarkb | anteaya: mordred so while you can strike it out on your own you are really on your own | 23:30 |
| lifeless | mordred: the largest most commercially successful clouds work with a private+public split | 23:30 |
| anteaya | mordred: yes I can feel the rage | 23:31 |
| mordred | lifeless: sure | 23:31 |
| *** dhritishikhar has quit IRC | 23:31 | |
| mordred | lifeless: for the reason that I stated above | 23:31 |
| clarkb | lifeless: that doesn't make it user friendly | 23:31 |
| mordred | it doesn't make it a good design | 23:31 |
| anteaya | clarkb: do you think better documentation might actually be part of the solution to finding less rage here? | 23:31 |
| mordred | it makes it the crappy design that people are forced to deal with because that's what bezos gives them | 23:31 |
| clarkb | anteaya: yes, I think it will help quite a bit so that deployers know that they can do things in different and potentially better ways for them | 23:31 |
| mordred | and so, stockholm syndrome kicks in, and people assume it's "right" | 23:31 |
| openstackgerrit | Tim Hinrichs proposed openstack-infra/project-config: ACL changes for Congress create-reference https://review.openstack.org/177492 | 23:31 |
| lifeless | mordred: I'm not arguing right | 23:32 |
| mordred | it's never been a feature a user has _wanted_ | 23:32 |
| anteaya | clarkb: okay thank you, this gives me some hope | 23:32 |
| mordred | it's a feature that can be dealt with | 23:32 |
| lifeless | mordred: but its clearly not such a burden that it makes the product undesirable | 23:32 |
| anteaya | mordred: yes stockholm syndrome, sigh | 23:32 |
| mordred | to maximize someone else's profits | 23:32 |
| mordred | lifeless: I didn't say it was | 23:32 |
| mordred | lifeless: I said that, in this case, I have a choice | 23:32 |
| lifeless | sure | 23:32 |
| mordred | because openstack gives me that freedom | 23:32 |
| mordred | so I'm going to use it | 23:32 |
| mordred | and I'm going to show that you do not have to be bound by what amazon deigns to give you | 23:32 |
| mordred | in fact | 23:33 |
| *** jerryz has joined #openstack-infra | 23:33 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Switch functools partials usage to kwargs https://review.openstack.org/177493 | 23:33 |
| mordred | it might be the most important feature of openstack | 23:33 |
| mordred | that it gives people the freedom to not be bound by the design choices that were made to maximize the profits of a bookseller | 23:33 |
| mordred | and instead make design choices taht are right for their workload(s) | 23:33 |
| mordred | that _might_ be a floating ip model | 23:33 |
| mordred | which is neat if it is | 23:33 |
| openstackgerrit | Tim Hinrichs proposed openstack-infra/project-config: ACL changes for Congress create-reference https://review.openstack.org/177492 | 23:34 |
| clarkb | also woot for better d-g multinode support | 23:34 |
| clarkb | pleia2: ^ thanks a bunch | 23:34 |
| mordred | but assuming that floating is the only or best way to go just beause AMZN did it that way is bonghits | 23:34 |
| mordred | clarkb: so excited! | 23:34 |
| *** akshai has joined #openstack-infra | 23:34 | |
| anteaya | clarkb pleia2 yay | 23:34 |
| anteaya | mordred: is that the reason floating ips are a thing? | 23:35 |
| anteaya | because amazon does it that way? | 23:35 |
| *** dannywilson has quit IRC | 23:35 | |
| fungi | well, in some cases also because of the same reasons that amazon did it | 23:35 |
| mordred | anteaya: yes | 23:35 |
| anteaya | fungi: ip conservation | 23:36 |
| mordred | yup | 23:36 |
| *** ChuckC has quit IRC | 23:36 | |
| fungi | e.g. to conserve global ipv4 assignments on the assumption that you may boot 20 instances in your tenant only 3 of which run webservers | 23:36 |
| anteaya | which we don't care about any more, if we ever did | 23:36 |
| fungi | and for most of the world, if it's not a webserver it's not part of "teh internets" | 23:36 |
| clarkb | mordred: your exception move failed some unittests so its not perfect yet | 23:36 |
| *** shashankhegde has quit IRC | 23:36 | |
| anteaya | fungi: yes, I'm hearing that | 23:36 |
| mordred | clarkb: I want to cry | 23:36 |
| clarkb | mordred: sorry, I did grep through init.py and that looked fine | 23:37 |
| *** shashankhegde has joined #openstack-infra | 23:37 | |
| clarkb | mordred: so I Think it must be in the tests themselves | 23:37 |
| anteaya | thank you for your patience in helping me to understand the issue | 23:37 |
| anteaya | not that I entirely do but this helps add | 23:37 |
| mordred | clarkb: I thought I'd run them locally ... | 23:37 |
| * mordred re-runs | 23:37 | |
| clarkb | going back to what infra cloud will need, it will need at least one network with publicly routable addresses (ipv6 only should be ok) | 23:37 |
| mordred | clarkb: well, it's actually easier/less work at first to just get a /22 | 23:38 |
| jroll | I'm missing context, but floating IPs are also a thing for HA and whatnot, no? (maybe this isn't inherently just the issue) | 23:38 |
| mordred | getting the IPv6 range will be more work, amusingly enough | 23:38 |
| mordred | jroll: they really aren't | 23:38 |
| fungi | yeah, i totally expect this one to start with v4 (or with our luck be v4 only for a while) | 23:38 |
| clarkb | jroll: you should not rely on floating Ips for HA | 23:38 |
| clarkb | jroll: your NAT tables are not replicated or anything, its basically a terrible ha model | 23:38 |
| mordred | jroll: they're not nearly good enough for that - if you want IP-based HA, you need to do a load-balancer or something | 23:38 |
| *** dhritishikhar has joined #openstack-infra | 23:38 | |
| clarkb | mordred: exactly | 23:38 |
| jroll | mordred: clarkb: I'm thinking like HA load balancers | 23:38 |
| fungi | jroll: ha needs actual load balancers with failure detection | 23:39 |
| mordred | right | 23:39 |
| mordred | jroll: they're not good enough for that | 23:39 |
| clarkb | jroll: floating IPs won't help there | 23:39 |
| mordred | there is one use case they are good for | 23:39 |
| jroll | then how do you do it? | 23:39 |
| * jroll is just being curious here | 23:39 | |
| * anteaya is also glad jroll is curious | 23:39 | |
| mordred | that is having a floating ip with DNS attached to it that yuo can use to minimize downtime across a server migration ofa pet-like service | 23:39 |
| lifeless | jroll: so there are three related things I think | 23:39 |
| mordred | which is the funny thing | 23:39 |
| *** pc_m has quit IRC | 23:39 | |
| mordred | the one main use case in which the implementation actually is useful | 23:39 |
| clarkb | jroll: anycast? I don't think openstack has a good answer there | 23:39 |
| lifeless | jroll: one is rapidly moving an IP from one host to another, used by *some* HA patterns | 23:39 |
| clarkb | jroll: but floating IPs are a bad answer | 23:40 |
| mordred | is the LEAST "cloud" design | 23:40 |
| fungi | jroll: in the openstack case, with a separate api service to instantiate a lb cluster of some sort | 23:40 |
| lifeless | jroll: another is making some hosts unreachable from the i/n | 23:40 |
| lifeless | jroll: a third is making hosts aware of all the IPs that they have | 23:40 |
| fungi | (also done at least as well with packet filters) | 23:40 |
| lifeless | NAT based floating ips give you the first two but not the third | 23:40 |
| *** shashankhegde has quit IRC | 23:41 | |
| lifeless | except via extras like the metadata service or config-drive | 23:41 |
| jroll | right, so for example I run haproxy in front of an API service - I use corosync and a vip/floating IP to do failover if one crashes and burns | 23:41 |
| clarkb | lifeless: and only metadata service will update | 23:41 |
| jroll | and I guess I'm wondering if/why that doesn't work | 23:41 |
| lifeless | the former is dynamic, except our impl is unperformant, and constrained to be little better than config drive | 23:41 |
| mordred | jroll: but a floating IP is not a vip | 23:41 |
| jroll | aha | 23:41 |
| lifeless | and config drive is static, but folk prefer it because it has less security ramifications | 23:41 |
| clarkb | lifeless: and its reliable | 23:42 |
| mordred | yes - it works much better | 23:42 |
| jroll | ok, that's where I was mistaken | 23:42 |
| mordred | of course, DHCP also informs a host of its ip ... :) | 23:42 |
| *** akshai has joined #openstack-infra | 23:42 | |
| jroll | vip vs floating ip (I sometimes hear of vips called floaters) | 23:42 |
| lifeless | so, vip's require appropriate acls in the network layer | 23:42 |
| mordred | jroll: yah - it's easy to think they're the same | 23:42 |
| lifeless | butthats doable | 23:42 |
| fungi | i have hope that _some_ day i can provision a network in openstack that will let me actually route additional ip addresses to use for things like real clustering/takeover/sharing protocols | 23:42 |
| jroll | ok, thanks for helping me get that :) | 23:43 |
| lifeless | there are other ways to deliver 1) | 23:43 |
| fungi | and to be able to control ip addresses by actually configuring them within the context of my virtual machines | 23:43 |
| clarkb | fungi: I think it would just be a matter of teaching neutron routers that some IPs are special and will be managed outside the neutron IPAM service | 23:43 |
| mordred | fungi: right? | 23:43 |
| lifeless | such as - for services that need it - running a vip | 23:43 |
| clarkb | fungi: a "leave this IP the hell a lone" api call | 23:43 |
| lifeless | or ip unicast | 23:43 |
| lifeless | export your hosts via BGP as service records | 23:44 |
| clarkb | then you could do VIPs within your service | 23:44 |
| lifeless | or a cloud provisions haproxy | 23:44 |
| lifeless | or <...> | 23:44 |
| lifeless | the second - filters please. | 23:44 |
| fungi | however, i'm probably a luddite who simply hasn't embraced the "cattle not pets" design philosophy | 23:44 |
| mordred | fungi: right - EXCEPT | 23:44 |
| lifeless | the third, having a vif per IP and dhcp on each vif, can get you it with no magic in the VM | 23:44 |
| mordred | fungi: floating IPs are only useful for pets | 23:44 |
| lifeless | no awareness of which cloud | 23:44 |
| mordred | fungi: they have no value at all for cattle | 23:45 |
| lifeless | mordred: why do you say that? | 23:45 |
| anteaya | fungi: yes I definitely picture you as a luddite | 23:45 |
| lifeless | mordred: they are AFAICT on different dimensions | 23:45 |
| jroll | devil's advocate: running mail servers on the cloud, you want to keep your IPs, floating IPs are useful | 23:45 |
| mordred | lifeless: beacuse the value is taht you can move the ip to another host - if you have cattle, that concept is not needed | 23:45 |
| jroll | maybe mail servers are a pet problem :P | 23:45 |
| mordred | lifeless: if an ip is not special, there is no need to migrate it | 23:45 |
| fungi | anteaya: sometimes i like to use "mendicant" instead | 23:45 |
| anteaya | ha ha ha | 23:45 |
| mordred | jroll: nope - you can scale out clusters of those | 23:45 |
| lifeless | mordred: so 'ip is special' and 'host is special' are IMO very different | 23:45 |
| lifeless | mordred: if the ip is the ip of one of your cattle haproxy | 23:46 |
| fungi | jroll: twitter is the new e-mail. see, you're a luddite too. embrace the cattle future | 23:46 |
| jroll | mordred: sure, but you need a clean external IP somehow, that may need to bounce somewhere else | 23:46 |
| mordred | jroll: not really | 23:46 |
| jroll | fungi: I bet twitter uses floating ips ;D | 23:46 |
| clarkb | twitter uses mesos iirc | 23:46 |
| mordred | jroll: you can TOTALLY have multiple non-related incoming | 23:46 |
| jroll | yeah, I kid about twitter | 23:46 |
| mordred | jroll: you do not need one IP | 23:46 |
| clarkb | which probably has its own crazy VIP implenetnation on top of it for routing traffic arbitrarily | 23:47 |
| jroll | mordred: sure, but you don't want to constantly recycle IPs either | 23:47 |
| lifeless | its largely about the clients | 23:47 |
| lifeless | most clients are not prepped for rapidly changing ip's on their endpoints | 23:47 |
| mordred | jroll: I don't think you need to constantly recycle - just because an IP isn't special doesn't mean it needs to change hourly | 23:47 |
| lifeless | nothing to do with how cattle-or-pet the endpoints are | 23:47 |
| * fungi wants to constantly recycle ips | 23:47 | |
| mordred | sure | 23:47 |
| mordred | totally agree with lifeless on that | 23:47 |
| mordred | it's not about changing them constnatly | 23:47 |
| jroll | mordred: recycle frequency depends how crappy your cloud is :P | 23:48 |
| jroll | but yeah, I get your point | 23:48 |
| fungi | amen | 23:48 |
| mordred | jroll: well, none of this is going to fix the cloud sucking :) | 23:48 |
| mordred | in fact - it turns out each of these things are complexity layers | 23:48 |
| * jroll is joyful it's friday and he can stop dealing with clouds for a couple days | 23:48 | |
| mordred | so if your cloud is crappy, NOT having floating ips means one less thing to break | 23:48 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/shade: Do not cache unstead state images https://review.openstack.org/177494 | 23:49 |
| mordred | lifeless: I do think there are some uses for them - but I think most of those are really things that should be more opt-in | 23:49 |
| clarkb | (if the gate testing is any evidence yourfloating IPs will break) | 23:49 |
| mordred | clarkb: ++ | 23:49 |
| lifeless | mordred: I can get behind that | 23:49 |
| clarkb | protip do not use nova shelve | 23:49 |
| SpamapS | clarkb: ^ my preferred answer to "shouldn't we invalidate outside the loop" :) | 23:49 |
| clarkb | SpamapS: looking | 23:49 |
| lifeless | mordred: I just don't think that an argument against them with significant conflation is a solid one | 23:49 |
| lifeless | because the people listening to the argument will be aware of this | 23:50 |
| jroll | clarkb: lol | 23:50 |
| lifeless | better to separate out the concerns | 23:50 |
| SpamapS | clarkb: actually let me re-state.. the answer to that question is _yes_, but we shouldn't have to invalidate _inside_ the loop. | 23:50 |
| mordred | lifeless: yup. I'm most appending "for me" or "for infra" on to most of my blanket assertions in here | 23:50 |
| clarkb | SpamapS: right looks like we are telling it to only cache once we have reached steady state | 23:50 |
| jroll | mordred: so is your actual grunt here that floating IPs are being forced onto you? | 23:50 |
| clarkb | SpamapS: which seems sane to me | 23:50 |
| jroll | I haven't read all the scrollback | 23:50 |
| lifeless | mordred: thats useful to know, but still can't copy-paste your argument to anyone else :) | 23:51 |
| clarkb | jroll: thats a major part of it at least for me | 23:51 |
| mordred | jroll: well, they have been for a while - HP uses them ... which means I have to deal with them | 23:51 |
| fungi | jroll: basically | 23:51 |
| clarkb | jroll: we have no other option in hpcloud | 23:51 |
| mordred | jroll: and it's much harder for no benefit | 23:51 |
| jroll | ok | 23:51 |
| mordred | so - for infra-cloud | 23:51 |
| clarkb | jroll: and we leak floating IPs and all kinds of nasty | 23:51 |
| mordred | we do not intend to use them | 23:51 |
| mordred | becase they do not benefit us | 23:51 |
| jroll | I haven't used HP cloud, so I wasn't aware you were required to use them | 23:51 |
| anteaya | mordred: I take every statement you make to be appended with 'for me' | 23:51 |
| mordred | anteaya: :) | 23:51 |
| SpamapS | clarkb: right, because caching things in 'saving' state is just going to result in a very out of date cache. | 23:51 |
| fungi | jroll: hpcloud gives you rfc-1918 "public" interfaces and then you make a separate api call to bind a globally routable fip | 23:51 |
| clarkb | SpamapS: yup | 23:51 |
| jroll | fungi: that's annoying | 23:51 |
| fungi | jroll: yeah. yeah it is | 23:52 |
| anteaya | jroll: the largest issue I see is that people are forced to use things they don't want to use | 23:52 |
| clarkb | jroll: but thats neutrons default setup | 23:52 |
| jroll | it's like legos | 23:52 |
| anteaya | jroll: that makes them unhappy | 23:52 |
| clarkb | jroll: if you follow the docs and/or use devstack this is what you get | 23:52 |
| SpamapS | That's neutron's default setup that happens to emulate EC2. ;) | 23:52 |
| *** sdake__ has quit IRC | 23:52 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Split exceptions into their own file https://review.openstack.org/177476 | 23:52 |
| mordred | clarkb: ^^ found it | 23:52 |
| SpamapS | Or does EC2 still automatically give you a routable IP? (I had heard that was going away) | 23:53 |
| fungi | also openstack conversations should have a godwin's law variant involving amazon ec2 | 23:53 |
| *** thinrichs has quit IRC | 23:53 | |
| clarkb | SpamapS: it doesn't matter we have the freedom to make something better | 23:53 |
| SpamapS | clarkb: ++ | 23:53 |
| mordred | ++ | 23:53 |
| *** achanda has joined #openstack-infra | 23:53 | |
| mordred | SpamapS: did you see I split the exceptions - just for you? | 23:54 |
| anteaya | clarkb: so again having better docs looks like a potential way forward? | 23:54 |
| mordred | SpamapS: you have suggestions on what exception handling tests should look like? | 23:55 |
| SpamapS | mordred: I did, and I promise to nuzzle you for that. | 23:55 |
| mordred | SpamapS: I can add that to my list next | 23:55 |
| anteaya | so deployers of public clouds are more likely to deploy them in a way that is useful to us? | 23:55 |
| mordred | anteaya: actually, we already have public clouds deploying the way we want | 23:55 |
| anteaya | we do | 23:55 |
| anteaya | runabout? | 23:55 |
| mordred | anteaya: vexxhost and runabove both have good configs | 23:55 |
| anteaya | runabove sorry | 23:56 |
| *** wenlock_ has quit IRC | 23:56 | |
| *** ChuckC has joined #openstack-infra | 23:56 | |
| anteaya | do we know what their configs are? | 23:56 |
| anteaya | can we get them? | 23:56 |
| SpamapS | mordred: mock_client.something.side_effect = Exception('whatever you want to emulate') | 23:56 |
| mordred | we've got project PTLs - we can figure it out I'm sure | 23:56 |
| mordred | anteaya: I mainly want to start fighting the theory | 23:56 |
| SpamapS | mordred: and you probably want to assert log contents | 23:56 |
| mordred | that "floating IPs are the 'right' way to do it" | 23:56 |
| anteaya | mordred: well you enjoy that | 23:56 |
| SpamapS | Ok, I believe weekend just started | 23:57 |
| anteaya | I'm interested in getting users something they are more likely to use | 23:57 |
| mordred | anteaya: and I want to make sure that we help get the documentation to show what you need to do to ge a cloud that behaves this way | 23:57 |
| mordred | anteaya: ya | 23:57 |
| anteaya | mordred: me too | 23:57 |
| * SpamapS also believes a slight fever is besetting him.. damnit | 23:57 | |
| mordred | anteaya: or - I want to make sure people know they have the freedom | 23:57 |
| anteaya | SpamapS: :( | 23:57 |
| mordred | SpamapS: gross! | 23:57 |
| mordred | anteaya: so that they dont' just hear people telling them that they MUST rewrite all of their apps to use cloud | 23:57 |
| anteaya | mordred: well helping them do it the way they want proves to them they have the freedom | 23:57 |
| SpamapS | Yeah, or its just the massive grey cloud asserting low pressure and general murkiness on LA | 23:57 |
| clarkb | mordred: fwiw I think they would be much more palatable if the implementation of the floating ip was to assign an IP to a real interface and not have NAT running | 23:57 |
| mordred | becaus that's just not true at all | 23:57 |
| anteaya | mordred: you are welcome to lead the parade | 23:57 |
| mordred | clarkb: ++ | 23:57 |
| mordred | clarkb: I agree | 23:57 |
| mordred | clarkb: I would like a floating-ip option that behaved more like a traditional vip | 23:58 |
| mordred | that would be neat | 23:58 |
| *** sabeen3 has quit IRC | 23:58 | |
| clarkb | because then the node is aware of its IP and can be smart and you don't have to work around the crazyness of NAT | 23:58 |
| clarkb | mordred: ya | 23:58 |
| mordred | ++ | 23:58 |
| SpamapS | clarkb: I believe that invites ARP storms from your users though. | 23:58 |
| SpamapS | clarkb: even if they won't work.. users will swear they do. | 23:58 |
| clarkb | SpamapS: not i you are controlling the assignment | 23:58 |
| anteaya | mordred: from what I'm reading they become unhappy any time anyone tells them they must do anything | 23:58 |
| clarkb | SpamapS: and otherwise your users can arp storm all day | 23:58 |
| anteaya | they want to do what they want to do | 23:58 |
| mordred | ok. I'm giong to go eat pork | 23:59 |
| clarkb | SpamapS: the same thing that prevents me from claiming I have an ip address today would be used in that case | 23:59 |
| *** achanda has quit IRC | 23:59 | |
| clarkb | mordred: I get to do that tomorrow | 23:59 |
| *** shardy_ has joined #openstack-infra | 23:59 | |
| clarkb | I bought >26lbs of meat yesterday | 23:59 |
| SpamapS | clarkb: yes, except that is now 100% always malicious. :) | 23:59 |
| SpamapS | clarkb: you are now inviting it to also be ignorant. | 23:59 |
| anteaya | mordred: enjoy pork | 23:59 |
| SpamapS | Just stating the psychological ramifications, not any actual reason it will break something. | 23:59 |
| clarkb | SpamapS: rgr | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
