Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-11
14:47:49 dansmith and it's still a ways off from working, so.. ignore it for now
14:47:50 bauzas mriedem: FWIW, looking at the BP whiteboards is somehow difficult
14:48:11 bauzas dansmith: okay, that was because today means I should help others
14:49:41 dansmith bauzas: yeah, you should do that instead of looking at my not-working patch :)
14:49:49 bauzas roger.
14:50:11 bauzas dansmith: today's cells meeting is at 2100UTC, right?
14:50:30 bauzas I *need* to attend that one after lot of times missing it
14:50:35 dansmith bauzas: are you asking me to be your calendar? today's meeting is in the later slot.
14:51:01 bauzas dansmith: no, just verifying my own calendar that I don't trust :)
14:52:12 openstackgerrit Matt Riedemann proposed openstack/nova: xenapi: support the hotplug of a neutron port https://review.openstack.org/365505
14:52:48 mriedem bauzas: how would you like me to custom tailor the etherpad to your delicate sensibilities? :)
14:53:01 mriedem bauzas: https://review.openstack.org/#/c/365505/ needs another core
14:53:02 bauzas mriedem: nothing, just a comment :)
14:53:26 bauzas mriedem: we should think about providing a dashboard for the next cycle
14:53:35 bauzas but meh
14:54:12 mriedem huanxie: got it
14:55:08 huanxie mriedem thanks!
14:55:25 mriedem huanxie: note that i already updated the release note on the vif hot plugging patch
14:55:27 mriedem so that's ready to go
15:01:04 huanxie mriedem, I was just trying to ssh to my linux and change this, you have already done, many many thanks:)
15:01:53 mriedem huanxie: in case you haven't done it yet, you can use the edit button in the gerrit UI, or the 'fix' button next to a comment and make changes in the gerrit UI directly
15:02:42 mriedem that will make it diverge from your local copy so i only do it for trivial things that are ready to get approved
15:04:24 huanxie mridem, I see, I haven't modified directly in gerrit UI, will have a try in my other patch, it's useful for very small change
15:07:08 cdent anywhere in particular I should put my eyes for sprinting, or just let the etherpad be my guide?
15:08:39 huanxie mriedem could you also help review this patch https://review.openstack.org/#/c/406059/, this patch is the first one when using os-xenapi, thanks a lot
15:09:51 mriedem huanxie: that's already approved
15:10:04 mriedem cdent: the diagnostics series could probably use a fresh set of eyes
15:10:11 mriedem as i think it was recently rebased
15:10:29 mriedem cdent: https://review.openstack.org/#/q/topic:bp/restore-vm-diagnostics,n,z
15:11:04 cdent thanks mriedem
15:12:25 huanxie mriedem, I see, remember each patch needs to get two core reviewers +2 or one +2 is enough?
15:13:00 mriedem huanxie: the bottom change was already approved, you just rebased it so sdague fast-approved it again
15:13:01 mriedem which is ok
15:16:41 huanxie mriedem, oh, I see, thanks for explaining this
15:17:48 mriedem raj_singh: some easy things to cleanup in https://review.openstack.org/#/c/397399/ - mostly remove the reset_globals() thing in service_auth.py
15:17:52 mriedem raj_singh: we don't want to repeat that pattern
15:19:03 mriedem raj_singh: and on the devstack change https://review.openstack.org/#/c/409329/ that needs to depend on the nova change which defines the config
15:19:37 raj_singh mriedem: ok, will respin it now. Thx for the review
15:20:32 artom bauzas, heya, back now. So yeah, I agree the method signature is not pretty, but I'm not sure I have a better idea
15:21:22 bauzas artom: yup, I understand
15:21:27 bauzas artom: me too honestly
15:21:39 bauzas fortunately, it's not a RPC API
15:21:56 bauzas so probably we could just add it now and discuss later on how to have a better argument
15:22:16 artom bauzas, I'll definitely update the docstring though
15:23:25 mriedem gibi: did you have a bug for the versioned notifications lazy-load issue?
15:24:33 gibi mriedem: yes
15:24:47 gibi mriedem: here it is https://review.openstack.org/#/c/415857/
15:25:07 mriedem thanks
15:29:10 cdent snikitin: I left these as a comment, but how come the new NovaObject based DiskDiagnostics doesn't get an id, when the old one did?
15:32:36 openstackgerrit John Garbutt proposed openstack/nova: Pass objects into _validate_requested_network_ids https://review.openstack.org/418027
15:34:48 openstackgerrit Artom Lifshitz proposed openstack/nova: Plumbing for tagged nic attachment https://review.openstack.org/415622
15:34:49 openstackgerrit Artom Lifshitz proposed openstack/nova: Plumbing for tagged volume attachment https://review.openstack.org/391941
15:34:49 openstackgerrit Artom Lifshitz proposed openstack/nova: Libvirt support for tagged volume attachment https://review.openstack.org/418100
15:34:50 openstackgerrit Artom Lifshitz proposed openstack/nova: Libvirt support for tagged nic attachment https://review.openstack.org/415520
15:34:50 openstackgerrit Artom Lifshitz proposed openstack/nova: API support for tagged device attachment https://review.openstack.org/376021
15:35:13 openstackgerrit Christopher Martin proposed openstack/nova: Stop using mox in test_compute_cells.py https://review.openstack.org/389750
15:47:24 artom mriedem, heya - you mentioned bumping the RPC version for the tagged attachment patches. I'm wondering whether I squish both volume and nic stuff into one patch and have a bigger patch but only one version bump, or keep it split for easier to digest patches but 2 version bumps (one for each, nic and volume)
15:47:41 artom s/squish/squash/ (though squish is funnier)
15:48:05 openstackgerrit Sivasathurappan Radhakrishnan proposed openstack/nova: Refactor _post_live_migratiion https://review.openstack.org/413021
15:49:25 openstackgerrit Sarafraj Singh proposed openstack/nova: Add service_token for nova-cinder interaction https://review.openstack.org/397399
15:50:08 mriedem cdent: that came up in the spec review, i asked the same thing, i think it was ultimately decided that the disk id wasn't useful, but we could add one in later if needed, i.e. via bdm uuid or something
15:50:36 cdent so how do you tell one disk from another?
15:50:58 mriedem cdent: https://review.openstack.org/#/c/357884/ if it helps
15:51:29 mriedem cdent: yeah i asked the same questions in the spec review
15:51:37 mriedem would have to go back through the conversation on that review to get the answer
15:51:38 cdent heh :)
15:52:40 cdent the gist appears to be: we never had good info there in the first place, and making it could be expensive, so we'll get rid of it for now
15:52:43 mriedem cdent: https://review.openstack.org/#/c/357884/9/specs/ocata/approved/restore-vm-diagnostics.rst@142
15:52:53 cdent sort-of-jinx
15:53:50 mriedem i'm pretty sure we also asked if/how people were using it in the ops list and didn't get much of a response
15:53:54 mriedem so we're kind of flying blind
15:54:53 mriedem there was some response i guess http://lists.openstack.org/pipermail/openstack-operators/2016-October/011793.html
15:55:02 openstackgerrit Sarafraj Singh proposed openstack/nova: WIP: Add service_token for nova-glance interaction https://review.openstack.org/412634
15:55:30 mriedem cdent: so i guess people use it as a poor mans way of bypassing ceilometer?
15:55:43 mriedem even though it's entire existence in the first place is for ceilometer i believe
15:55:44 cdent apparently ceilometer is now going to be using something to bypass nova ;)
15:55:47 mriedem dragon added it way back when
15:55:52 mriedem nice
15:56:13 cdent yeah, chaos everywhere, cats and dogs, etc
16:01:42 openstackgerrit Pawel Koniszewski proposed openstack/nova: Add new configuration option live_migration_scheme https://review.openstack.org/410817
16:12:11 openstackgerrit Dan Smith proposed openstack/nova: WIP: Test cellsv1 https://review.openstack.org/418141
16:15:57 mriedem wow, sudafed + 2 cups of coffee and i feel like a changed man
16:16:00 mriedem thanks drugs!
16:16:52 dansmith sudafed ftw
16:16:58 openstackgerrit Matt Riedemann proposed openstack/nova: Update notification for flavor https://review.openstack.org/413846
16:21:04 artom Breaking news: open source project leader secretly a drug addict.
16:21:49 cdent I think we'd be more surprised about an open source project leader that wasn't a drug addict
16:23:57 bauzas I need to be AFK for a few hours but will be back around 2000UTC
16:24:40 gibi mriedem: regarding https://review.openstack.org/#/c/413846/ not only flavor.project but flavor.extra_specs is also missing from the emitted payload
16:25:01 gibi mriedem: but it is expected to be solved by the same bug fix
16:25:19 bauzas in the meatime, I'd appreciate some reviews for https://review.openstack.org/#/c/392569/25
16:25:30 bauzas ^ placement API microversion for RP get
16:25:37 bauzas folks , \o
16:26:11 markus_z mriedem: Hand on heart, the newton backports for fixing live-migration with serial console won't make it, right? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton+topic:%22bug+1455252%22
16:26:49 markus_z I will only cry a little if you say "yes"
16:26:54 mriedem gibi: you forgot that flavor.extra_specs is loaded up by default when we pull the flavor from the database https://github.com/openstack/nova/blob/fd91a25324b7b7fed1f3f10c5b56668b20a170f4/nova/objects/flavor.py#L402
16:27:00 mriedem gibi: that one is the special snowflake
16:27:31 mriedem markus_z: hand on whose heart? mine or yours?
16:27:52 markus_z mine of course ;P (probably I got the idiom wrong)
16:28:57 gibi mriedem: but it is still not shows up in the flavor.create payload

Earlier   Later