| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-22 | |||
| 14:07:26 | kashyap | For Stein: 28d337b -- Pick next minimum libvirt / QEMU versions for "Stein" | |
| 14:07:31 | sean-k-mooney | 3.2.0 is needed to resovle a specific sriov issue but i can avoid the version bump by not cleaning up some code and allowing it to fail | |
| 14:08:11 | kashyap | sean-k-mooney: Okay, can do the version compat check across, and let you know in a bit | |
| 14:08:35 | kashyap | But it's normally not nice to "deviate from the plan", as we announced it to the mailing list as well | |
| 14:09:22 | sean-k-mooney | true but the wiki still states it as a draft and i had no idea it was tracked anywhere else | |
| 14:10:29 | kashyap | sean-k-mooney: So, actually I should bump it right now to: | |
| 14:10:30 | kashyap | +NEXT_MIN_QEMU_VERSION = (2, 8, 0) | |
| 14:10:30 | kashyap | +NEXT_MIN_LIBVIRT_VERSION = (3, 0, 0) | |
| 14:10:33 | sean-k-mooney | kashyap: anyway as i said in my reply to jay here https://review.openstack.org/#/c/624842/4/nova/network/linux_utils.py@a74 its not needed | |
| 14:11:29 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Share snapshot image membership with instance owner https://review.openstack.org/630769 | |
| 14:11:34 | kashyap | Thanks for the reminder! I lost this TODO item from my Org-Mode file, damn | |
| 14:12:24 | tosky | hi all, did anyone notice an issue when attaching volumes on a devstack/python3/bionic environment? (yes, it may be a cinder issue, but the relevant code is in lib/nova) | |
| 14:12:31 | tosky | I proposed a fix in https://review.openstack.org/#/c/632452/ | |
| 14:14:24 | sean-k-mooney | tosky: we have been haveing some issue yes but not sure if its related to that | |
| 14:21:59 | mriedem | stephenfin: can you push through this simple change https://review.openstack.org/#/c/628205/ | |
| 14:22:08 | mriedem | that will get the approved bp change on top merged | |
| 14:30:44 | stephenfin | mriedem: I can | |
| 14:44:02 | openstackgerrit | Elod Illes proposed openstack/nova stable/pike: De-dupe subnet IDs when calling neutron /subnets API https://review.openstack.org/632480 | |
| 15:19:27 | openstackgerrit | sean mooney proposed openstack/os-vif master: make kuryr-kubernetes-tempest-daemon-octavia non voting https://review.openstack.org/632484 | |
| 15:21:34 | openstackgerrit | sean mooney proposed openstack/os-vif master: make kuryr-kubernetes-tempest-daemon-octavia non voting https://review.openstack.org/632484 | |
| 15:23:41 | openstackgerrit | sean mooney proposed openstack/os-vif master: make kuryr-kubernetes-tempest-daemon-octavia non voting https://review.openstack.org/632484 | |
| 15:26:18 | sean-k-mooney | stephenfin: jaypipes ^ can ye take a look at that. 2 in every 3 ci runs seam to be failing for the k8s job and that is not really a new occurance | |
| 15:27:12 | sean-k-mooney | i have treatened to do this many times in the past and its finally at the point where it has annoyed me enought to right the patch. | |
| 15:29:52 | jaypipes | sean-k-mooney: is "ye" a reference to the fact that I am old(e) English? | |
| 15:30:43 | sean-k-mooney | no its the pluar of you | |
| 15:31:20 | jangutter | sean-k-mooney: y'all is singular, all y'all is plural | |
| 15:31:50 | jaypipes | sean-k-mooney: I'm not entirely sure what pluar is, but I have left a comment on said patch. :P | |
| 15:32:10 | jaypipes | sean-k-mooney: thou are most welcome. | |
| 15:32:26 | sean-k-mooney | y'all presuablly is a contraction of you all then | |
| 15:33:25 | jaypipes | sean-k-mooney: indeed it is. thine thoughts on the matter be correct. | |
| 15:34:50 | sean-k-mooney | ye is till common in ireland partly because in irish we do have distinct words for singualr and plural you so we still use the older english word "ye" for the plural form commonly | |
| 15:35:36 | jaypipes | sean-k-mooney: that's interesting, considering there are not distinct words for "three" and "tree". ;) | |
| 15:36:43 | jaypipes | I will never forget ildikov and I trying to find that town of Houth and all the Dubliners looking at us like we were morons for saying "hooth" like "tooth". | |
| 15:37:49 | jaypipes | sean-k-mooney: in any case, I blame stephenfin for all of it. | |
| 15:38:23 | sean-k-mooney | jaypipes: this might help if you are interested https://en.wikipedia.org/wiki/Hiberno-English | |
| 15:38:26 | jaypipes | (apparently stephenfin doesn't have a proper Irish accent, much to my surprise) :) | |
| 15:38:58 | ildikov | jaypipes: Ireland is a fun place :) | |
| 15:39:21 | jaypipes | ildikov: it is! especially when you miss your train stop! | |
| 15:39:48 | ildikov | jaypipes: yeah, I o | |
| 15:40:53 | ildikov | wouldn't recommend it :) | |
| 15:41:11 | jaypipes | sean-k-mooney, stephenfin: ooooh! on that wikipedia page I've officially found my new favorite Irish word derived from old English: "Eejit". | |
| 15:42:51 | sean-k-mooney | eejit is a great word | |
| 15:45:40 | sean-k-mooney | usually used to refer a harmless idoit or bafoon or equavlent to the irish Amadán pernounced "om a dawn" | |
| 15:45:50 | jaypipes | Also, I have gained some new knowledge: Irish lacks words that directly translate as "yes" or "no" | |
| 15:46:38 | jangutter | jaypipes: That's probably why Nancy Reagan's anti-drug campaign failed there. | |
| 15:47:17 | mriedem | jaypipes: did you sort out that rebuild with double allocation NoValidHost thing? like i said, i'm pretty sure that's been fixed but maybe you don't have the backport | |
| 15:49:19 | jaypipes | mriedem: yup, between yourself and melwitt, found the right patch and am applying it to our ocata downstream this morning. thank you! | |
| 15:49:25 | jaypipes | jangutter: lol! | |
| 15:52:13 | jangutter | sean-k-mooney: my favourite word there is "banjaxed", it's just under my all-time favourite word, "defenestration". | |
| 15:52:34 | stephenfin | `<jaypipes> sean-k-mooney: that's interesting, considering there are not distinct words for "three" and "tree"` :D | |
| 15:53:14 | jangutter | sean-k-mooney: The kuryr-kubernetes-tempest-daemon-octavia job is banjaxed, it should be defenestrated. | |
| 15:53:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert port to str when validate console port https://review.openstack.org/626853 | |
| 15:56:21 | sean-k-mooney | jangutter: stephenfin jangutter im adding some examples of the k8s jobs issue by the way. | |
| 15:56:52 | jangutter | sean-k-mooney: yeah, I've seen a number of them that really should not have been triggered by the os-vif changes. | |
| 15:59:36 | jaypipes | stephenfin: I see at least one person appreciated my joke :) | |
| 15:59:50 | sean-k-mooney | jaypipes: i did too :) | |
| 16:01:46 | jaypipes | sean-k-mooney: :) | |
| 16:01:54 | jaypipes | sean-k-mooney: well, that makes tree of us, then. | |
| 16:05:53 | sean-k-mooney | updated with 5 differnet issue that have required a recheck in the last week https://review.openstack.org/#/c/632484/3 | |
| 16:09:50 | sean-k-mooney | huh the k8s schduler service failed to start on the patch to make it non voting... http://logs.openstack.org/84/632484/3/check/kuryr-kubernetes-tempest-daemon-octavia/4bcca98/job-output.txt.gz#_2019-01-22_15_55_22_393153 | |
| 16:11:27 | mriedem | zzzeek: this should unblock the sqla-migrate release https://review.openstack.org/#/c/632470/ | |
| 16:12:26 | zzzeek | +2ed | |
| 16:15:04 | sean-k-mooney | ralonsoh: jangutter by the way if the current recheck of https://review.openstack.org/#/c/629025/ and https://review.openstack.org/#/c/632077/2 fail can you leave them be until we decide what to do with the k8s job | |
| 16:15:34 | jangutter | sean-k-mooney: yah, not touching 'em. | |
| 16:16:09 | sean-k-mooney | ralonsoh: jangutter im watching them on the zuul status page si ill eiterh rebase or recheck depending on what we decide | |
| 16:21:28 | jangutter | sean-k-mooney: etcd is also not happy: http://logs.openstack.org/84/632484/3/check/kuryr-kubernetes-tempest-daemon-octavia/4bcca98/controller/logs/screen-etcd.txt.gz | |
| 16:22:04 | sean-k-mooney | jangutter: ya ectd proably is why the scheduler did not start | |
| 16:23:18 | sean-k-mooney | i check the logs everytime the job failes but i always seam to root cause it to a setup issue or something unrelated to the patch. | |
| 16:23:57 | sean-k-mooney | dont get me wrong we could break kuryr kubernetese with some change to the ovo but not unit test chagnes | |
| 16:24:50 | sean-k-mooney | i have a todo to add more non-voting jobs to os-vif which i hope to make voting after stien | |
| 16:33:57 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: [WIP] libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Stein" https://review.openstack.org/632507 | |
| 16:35:46 | kashyap | (If anyone wonders why "WIP" (Work In Progress) and why not Workflow -1 — WIP spares people from clicking on the URL when they see it on IRC.) | |
| 16:36:48 | kashyap | (And putting the Workflow-1 will make the change not appear for those who use dashboards. So using both is good.) | |
| 16:36:54 | stephenfin | mriedem: That has to be a bug with PyPI | |
| 16:36:58 | stephenfin | mriedem: Wait, no. Commented | |
| 16:39:06 | stephenfin | mriedem: Is moving from sqlalchemy-migrate to alembic a thing we'd want to do at some point in the future. Have discussed roughly how to do it with zzzeek and can attempt it at some point in the next few months | |
| 16:39:10 | stephenfin | *? | |
| 16:39:26 | mriedem | stephenfin: not in nova probably not | |
| 16:39:29 | mriedem | thanks for the docs thing | |
| 16:39:39 | mriedem | back in a few hours (funeral) | |
| 16:39:45 | stephenfin | o/ | |
| 16:41:09 | sean-k-mooney | stephenfin: maybe add it as a train ptg topic. it has been mentioned before i think but i dont think it was ever seen as a high priority in the past | |
| 16:45:38 | stephenfin | sean-k-mooney: That's why I asked. It came up due to bnemec noting that it was effectively dead and zzzeek saying that alembic was a far more capable replacement | |
| 16:46:05 | stephenfin | sean-k-mooney: Sounds similar to the mox-mock thing though, and I don't want to get involved in another one of those /o\ | |
| 16:47:23 | kashyap | Cleaning up these unit test excrements after version libvirt / QEMU bump is just a bottomless pit... | |
| 16:47:26 | kashyap | "Endless fun" | |
| 16:48:29 | zzzeek | mriedem_away: you wouldn't accept the effort by a volunteer even ? | |
| 16:48:42 | sean-k-mooney | stephenfin: we discussed that neutron uses alembic i think and should nova also move to it but it was a non trivial amount of work and madde ffu even harder | |
| 16:49:15 | zzzeek | sean-k-mooney: the placement service was broken off from nova and uses alembic. this was already a mini-migration of former parts of nova from migrate to alembic | |
| 16:49:19 | sean-k-mooney | so i think we jsut said maybe in the futre but since its not causeing us issue not done "fix" it | |
| 16:50:08 | sean-k-mooney | zzzeek: i remember cdent and i spoke about that brefily in denver actully | |
| 16:50:38 | zzzeek | sean-k-mooney: nova cant be on sqlalhcemy-migrate forever because I can't keep supporting migrate forever from my side | |
| 16:50:49 | sean-k-mooney | zzzeek: i think in stien it just was not high enough on our priority list | |
| 16:51:29 | sean-k-mooney | zzzeek: sure and as a maintainer if that is deperacted we shoudl come up with a transition plan so you dont have too but it would need a spec and need to be a train thing i think | |
| 16:51:33 | zzzeek | sean-k-mooney: im all for it being bottom priority but that made me nervous just now when mriedem_away said it was a "never" thing | |
| 16:51:58 | zzzeek | sean-k-mooney: if you look at what placement did, it's that | |
| 16:52:51 | sean-k-mooney | nova has a lot more legacy stuff to deal with. | |
| 16:54:02 | sean-k-mooney | zzzeek: as i said its something we should talk about for train. | |
| 16:54:31 | sean-k-mooney | zzzeek: is sqlalcamy-migrate offially deprecated/unsupproted | |
| 16:54:42 | zzzeek | sean-k-mooney: if nova got off it, it sure would be | |