| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-09 | |||
| 15:34:35 | ianychoi | mriedem, both: stable version craeation on Zanata & translation sync job for stablr branch | |
| 15:34:47 | mriedem | ok | |
| 15:34:50 | EmilienM | dansmith: we can't make this change at this stage of this cycle, we'll only do nova compute for now and if things work let's ship it in ocata and improve it in pike | |
| 15:34:56 | sdague | dansmith: ++ | |
| 15:35:01 | kashyap | mriedem: I'll assume that fix wasn't backported to Mitaka, as it's security-only. Hence you must've down a downstream backport | |
| 15:35:20 | mriedem | kashyap: i think the L3 guys are working with the customer on the backport to their systems on mitaka | |
| 15:35:27 | mriedem | which....they could have screwed up for all i know | |
| 15:35:31 | mriedem | i've been trying to stay out of it | |
| 15:35:39 | EmilienM | dansmith: what you're asking has too much impact on different areas and we need to address it without stress & pressure (outside release week) | |
| 15:35:51 | ianychoi | mriedem, I am responsible for submitting a patch like https://review.openstack.org/#/c/429409/ to support translation sync with stable branches also the cration for stable versions in Zanata | |
| 15:35:55 | sdague | mordred: did you figure out what cloud provided the keystone / az bug? | |
| 15:35:58 | ianychoi | (as I18n PTL) | |
| 15:36:04 | sdague | mordred: I mean keypair | |
| 15:36:20 | mriedem | ianychoi: ok | |
| 15:36:22 | dansmith | EmilienM: your call, but just seems like a recipe for other issues | |
| 15:36:31 | ianychoi | mriedem, by the way, auggy is still I18n - nova cross project liaison? https://wiki.openstack.org/wiki/CrossProjectLiaisons#I18n | |
| 15:37:14 | EmilienM | dansmith: in other words: "yes you're right but I don't have the time to do it now, the list of items to address your requests is to big if we want to release Ocata before the deadline" | |
| 15:37:29 | ianychoi | I know that some information on https://wiki.openstack.org/wiki/CrossProjectLiaisons#I18n seems old. I would like to update it as much as possible now | |
| 15:38:08 | mriedem | auggy: are you still i18n liaison? | |
| 15:38:25 | mordred | sdague: just responded to the bug asking that question | |
| 15:38:48 | openstackgerrit | John Garbutt proposed openstack/nova master: Use admin context to create neutron ports https://review.openstack.org/357540 | |
| 15:40:20 | ianychoi | Hmm.. no response now but so fine, since we are living with different timezone :) | |
| 15:41:11 | ianychoi | mriedem, then.. would it be fine if I create stable-ocata version in Zanata and support translation sync job with stable/ocata branch around release date? | |
| 15:41:44 | ianychoi | If there would be more concern (e.g., more early period), please tell me. | |
| 15:44:33 | cdent | macsz: looks like Eric is fixing that himself, yeah? Ought to be a one line fix (not counting tests). Thanks for finding that. | |
| 15:45:38 | macsz | macsz: yeah, just wanted to keep you posted, as the timeline for fixing anything crucial (if it has any chance to land for RC2) is rather short | |
| 15:46:43 | cdent | macsz: Do you think that should warrant a backport? It's not breaking anything too badly, is it? | |
| 15:48:18 | mordred | sdague: I have an unconfirmed hunch that it's fujitsu - the bug reporter has some other github repos related to the fujitsu-k5 cloud | |
| 15:48:27 | mordred | sdague: https://github.com/JonTheNiceGuy/k5-ansible-modules "As K5 is Availability Zone centric new modules were created to communicate with K5 which then bypass limitations in the Ansible Openstack os modules." | |
| 15:48:35 | mriedem | ianychoi: i'm fine with that, we just won't hold up the nova ocata rc2 on translations | |
| 15:48:36 | macsz | cdent: it isn't breaking anything, to be frank, rather just the error is not stated correctly, but it is also a really small change with impact close to 0 | |
| 15:48:54 | sdague | mordred: ok, so crazy upstream fork that does something different with az | |
| 15:49:00 | mordred | yup | |
| 15:49:05 | sdague | I bet they are doing one of those fake federated models | |
| 15:49:15 | sdague | where the az is really a whole other openstack deploy | |
| 15:49:22 | cdent | macsz: I'm not the expert on these things (/me looks at mriedem ) but I reckon that breaking something is probably a reasonab bar at this point? | |
| 15:49:33 | macsz | cdent: i guess | |
| 15:49:45 | macsz | cdent: well, let's just leave it :) | |
| 15:49:50 | mordred | sdague: yah. there are people who can't get past the fact that in openstack the word "region" means what the word "az" means in amazon | |
| 15:50:02 | mordred | like it REALLY drives them completely batty | |
| 15:50:29 | ianychoi | mriedem, I see thanks! I will create stable-ocata version for nova and support translation sync around your rc3 or final release, but it would be much better if nova-i18n liaison will notify rc3 and final release to me :) | |
| 15:51:03 | sdague | mordred: yeh, well, we do have a lot of non overlapping grouping constructs that are weird | |
| 15:51:10 | mordred | we do for sure | |
| 15:51:18 | mordred | but making a fork which redefines them doesn't really help anybody | |
| 15:51:24 | sdague | mordred: ++ | |
| 15:53:54 | mriedem | ianychoi: to be clear, we have rc1 now, and are working to rc2 by 2/16 | |
| 15:54:02 | mriedem | ianychoi: so i don't plan on an rc3 | |
| 15:55:07 | ianychoi | mriedem, thanks than I will consider with nova rc2 rather than rc3 | |
| 15:56:12 | openstackgerrit | Andrey Volkov proposed openstack/nova master: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | |
| 15:56:15 | openstackgerrit | Andrey Volkov proposed openstack/nova master: libvirt: pass libvirt reference methods to migration https://review.openstack.org/328997 | |
| 15:56:17 | openstackgerrit | Andrey Volkov proposed openstack/nova master: libvirt: live-migrate updates NUMA and cpus in the XML https://review.openstack.org/286744 | |
| 15:56:58 | mordred | sdague: yup. verified. I have responded, hopefully in such a way as to not make the dude hate me too much | |
| 16:00:48 | sfinucan | johnthetubaguy: I forget - are we killing the feature that uses these opts? https://review.openstack.org/#/c/323313/13/nova/conf/libvirt.py | |
| 16:01:11 | sfinucan | ...or was that something else | |
| 16:01:45 | pkoniszewski | avolkov: hey, are you around? | |
| 16:04:12 | mriedem | sfinucan: this? https://review.openstack.org/#/c/429798 | |
| 16:04:31 | sfinucan | mriedem: Aye, that's the one | |
| 16:04:59 | avolkov | pkoniszewski hi | |
| 16:05:35 | pkoniszewski | avolkov: could you briefly describe what you have changed in the lm claims patch? | |
| 16:06:19 | avolkov | I just rebased, main change is service version from 16 to 17, because of placement api | |
| 16:06:28 | pkoniszewski | ah, okay | |
| 16:06:49 | pkoniszewski | asking whether i need to change something in my local changes | |
| 16:06:54 | pkoniszewski | still working on rollback issue | |
| 16:11:46 | pkoniszewski | avolkov: thanks a lot of this! will save me some time :) | |
| 16:11:54 | pkoniszewski | for this | |
| 16:12:36 | mriedem | sdague: i've got all of those backports up https://review.openstack.org/#/q/Ice78a0982bcce491f0c9690903ed2c6b6aaab1be,n,z | |
| 16:15:46 | mriedem | ildikov: smcginnis: for the nova/cinder meeting today i mostly wanted to just talk about PTG planning | |
| 16:16:00 | smcginnis | mriedem: +1 | |
| 16:16:04 | mriedem | we know the local.conf passthrough in infra is already on the infra agenda | |
| 16:16:10 | mriedem | for testing with cinder v3 | |
| 16:16:26 | ildikov | mriedem: sounds good | |
| 16:16:47 | ildikov | mriedem: I talked to the infra guys briefly about that patch, they were very positive about the progress | |
| 16:18:01 | mriedem | johnthetubaguy: on https://review.openstack.org/#/c/429798/ is the live migration progress timeout mostly just an issue for live block migrations? | |
| 16:18:14 | pkoniszewski | mriedem: for all of them | |
| 16:19:02 | ildikov | mriedem: can we at some point look into the remove check_attach patch too? | |
| 16:19:21 | pkoniszewski | its just wrong, doesnt matter whether it is block live migration or live migration | |
| 16:19:25 | openstackgerrit | John Garbutt proposed openstack/nova master: Placement api: set custom json_error_formatter in allocations https://review.openstack.org/395194 | |
| 16:19:52 | ildikov | mriedem: I need to figure out one notification test failure, but it would be great to have the other parts figured out too so we can merge it and finally move on | |
| 16:20:02 | sdague | mriedem: bam! | |
| 16:20:13 | mriedem | ildikov: yeah i was just looking it up | |
| 16:20:14 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform aggregate.remove_host notification https://review.openstack.org/396211 | |
| 16:20:14 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210 | |
| 16:20:33 | ildikov | mriedem: cool, tnx | |
| 16:20:59 | mriedem | sdague: thanks | |
| 16:25:25 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | |
| 16:26:47 | johnthetubaguy | mriedem: I think its technically a problem for all live-migrations | |
| 16:26:55 | johnthetubaguy | mriedem: but the block migrations have it worse | |
| 16:27:10 | johnthetubaguy | mriedem: heh, what pkoniszewski said | |
| 16:29:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489 | |
| 16:29:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor InstancePayload creation https://review.openstack.org/418488 | |
| 16:29:16 | mriedem | johnthetubaguy: ok, +W | |
| 16:29:25 | mriedem | johnthetubaguy: i've also sent a heads up to the operators list | |
| 16:29:46 | johnthetubaguy | mriedem: awesome, thanks | |
| 16:29:58 | johnthetubaguy | mriedem: I can get the backport in the queue to speed that up | |
| 16:30:47 | mriedem | johnthetubaguy: already done | |
| 16:31:06 | johnthetubaguy | oh. cool, thanks | |
| 16:39:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Rework get_number_of_serial_ports https://review.openstack.org/407174 | |
| 16:39:42 | openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Rework '_get_cpu_topology_constraints' https://review.openstack.org/407173 | |
| 16:39:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Log when conflicting image/flavor metadata found https://review.openstack.org/407175 | |
| 16:48:12 | auggy | mriedem: if someone else wanted to do the i18n liaison they could. There really wasn't anything to do. | |