Earlier  
Posted Nick Remark
#openstack-nova - 2017-09-22
09:33:38 tssurya dansmith: mriedem: I was able to reproduce this bug - https://bugs.launchpad.net/nova/+bug/1687362 ; so will start my contributions with this.
09:33:39 openstack Launchpad bug 1687362 in OpenStack Compute (nova) "Instance not mapped after running command "nova-manage cell_v2 map_instances --cell_uuid <cell_uuid>"." [Undecided,Incomplete] - Assigned to Surya Seetharaman (tssurya)
10:00:59 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
10:01:00 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
10:04:10 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from ``ip`` commands to ``pyroute2`` https://review.openstack.org/484386
10:08:31 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Virtual Ethernet device pair https://review.openstack.org/484726
10:09:41 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451
10:13:36 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Open vSwitch patch port https://review.openstack.org/485228
10:19:46 artom lyarwood, heya, now that https://review.openstack.org/#/c/468514/ has merged I think the -1 can come off https://review.openstack.org/#/c/468514/
10:28:13 openstackgerrit Dinesh Bhor proposed openstack/nova master: Fix 500 if list servers called with empty regex pattern https://review.openstack.org/506585
10:28:21 Sara_ Hello..
10:29:10 Sara_ Whiel adding a node to cluster one of VM went to ERROR state..Could someone help me out ?
10:29:15 openstackgerrit Artom Lifshitz proposed openstack/nova stable/newton: Catch InstanceNotFound exception https://review.openstack.org/467321
10:30:03 Sara_ Error:2017-09-22 08:36:40.046 18679 ERROR oslo_messaging.rpc.dispatcher [req-63c7b5d2-83a4-4531-b120-afbab733fb5a a31283a931844c4a88309b0a6d1ece85 26cc9b81f0b14ae0856eac6d635da676 - - -] Exception during message handling: Instance <instance_id> in power_state 1. Cannot get_diagnostics while the instance is in this state.
11:37:00 openstackgerrit Jesse Pretorius (odyssey4me) proposed openstack/nova master: Add default configuration files to data_files https://review.openstack.org/506188
11:52:45 openstackgerrit Sean Dague proposed openstack/nova master: Change livesnapshot to true by default https://review.openstack.org/454323
11:56:11 liuyulong 1
11:57:36 liuyulong ping sdague
11:58:51 sdague liuyulong: what's up?
12:00:57 liuyulong sdague, hello, https://review.openstack.org/#/c/375221/, can we move this spec forward?
12:01:38 liuyulong sdague, you've one given a +2.
12:02:07 sdague at the PTG it was discussed, let me get notes up
12:02:54 liuyulong sdague, nice, so what's the next step?
12:03:59 sdague https://etherpad.openstack.org/p/nova-ptg-queens L498
12:04:30 liuyulong sdague, I am now an individual contributor, : )
12:04:35 sdague so, I think the concensus was that Kevin_Zheng was going to help on the spec, and that we'd actually just add it as one of the mutable fields on the server object
12:04:48 sdague so that it applies to reboot as well as rebuild
12:05:48 Kevin_Zheng sdague: yeah, I pinged tiling
12:05:56 liuyulong looking...
12:06:00 sdague Kevin_Zheng: ok, great
12:06:29 Kevin_Zheng I pinged yulong today and I think we can work together if he wish
12:08:21 sdague Kevin_Zheng: great
12:08:46 liuyulong sdague, Kevin_Zheng, I'm glad to. so here is the patch for that spec, https://review.openstack.org/#/c/379128/. I don't think there has any conflicts.
12:09:55 liuyulong correct me, if i missing something.
12:10:58 liuyulong Idividual contributor has a small chance to attend ptgs. haha
12:12:58 liuyulong Kevin_Zheng, feel free to directly update the spec: https://review.openstack.org/#/c/375221/, since the PTG etherpad linked this.
12:15:35 Kevin_Zheng I already had one up, but sure I can do that
12:17:47 liuyulong https://review.openstack.org/#/c/506552/, this one? nice work.
12:19:14 Kevin_Zheng Yeah I can merge these two
12:24:20 liuyulong I don't think this two spec can be merged. Your spec is going to add a new API, mine updating rebuild API. Basically they don't affect each other.
12:24:48 Kevin_Zheng Ok
12:38:01 openstackgerrit Dinesh Bhor proposed openstack/nova master: Fix 500 if list servers called with empty regex pattern https://review.openstack.org/506585
12:46:01 liuyulong mriedem
12:47:10 openstackgerrit Elod Illes proposed openstack/nova master: Add instance.interface_detach notification https://review.openstack.org/506284
12:57:33 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/506640
12:59:46 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646
13:02:40 openstackgerrit Elod Illes proposed openstack/nova master: Add error notification for instance.interface_attach https://review.openstack.org/506643
13:13:09 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove allocations when unshelve fails on host https://review.openstack.org/506458
13:23:14 openstackgerrit Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
13:42:01 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612
13:42:25 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226
13:43:19 openstackgerrit Matt Riedemann proposed openstack/nova master: Add _wait_for_action_fail_completion to InstanceHelperMixin https://review.openstack.org/506660
13:43:28 mriedem gibi: ^ is where i was going with the refactor
13:54:25 mriedem bauzas: you've got a race in the test here https://review.openstack.org/#/c/506093/
13:54:30 mriedem need to wait for the migration to be done
13:54:55 bauzas mriedem: yeah I know
13:55:01 bauzas mriedem: I saw that this morning
13:55:07 bauzas mriedem: I have a f*** agenda today
13:55:16 bauzas but I'll try to look at it soon
13:55:16 mriedem agenda?!
13:55:22 bauzas schedule
13:55:35 mriedem oh right, red hat friday meetings and bug smash day
13:55:36 bauzas meetings day
13:55:50 mriedem i love how you guys load all of the worst stuff to the end of the week
13:55:51 bauzas even more than that - spoiling some very specific French stuff
13:56:16 mriedem very specific french stuff? like a smoking mime eating a croissant?
13:58:20 stephenfin mriedem, bauzas: Can one of you merge this so it stops flooding my inbox with Gerrit mails? https://review.openstack.org/#/c/503646/
13:59:14 bauzas stephenfin: Wipped
13:59:20 openstackgerrit Evgeny Antyshev proposed openstack/nova master: Vzstorage: synchronize volume connect https://review.openstack.org/505708
14:00:58 mriedem alex_xu: you might like to see this one https://review.openstack.org/#/c/506585/
14:01:15 mriedem stephenfin: your gerrit emails go to your inbox?
14:01:21 mriedem that's your mistake
14:01:47 mriedem gotta setup a filter rule, especially for the CI emails
14:02:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix 500 if list servers called with empty regex pattern https://review.openstack.org/506585
14:02:20 openstackgerrit Chris Dent proposed openstack/nova master: DNM: Don't monkey patch eventlet in functional tests https://review.openstack.org/506668
14:05:07 liuyulong hi mriedem, https://review.openstack.org/#/c/375221/, I've updated the spec, could you please review it again?
14:05:39 openstackgerrit Merged openstack/os-traits master: Updated from global requirements https://review.openstack.org/503646
14:07:08 mriedem liuyulong: Kevin_Zheng was going to update that spec since we thought it was abandoned,
14:07:13 liuyulong mriedem, there are some things that need to be clarified is that this spec can be under parallel development with Kenvin_zheng's os-keypair API.
14:07:19 mriedem liuyulong: we discussed this at the PTG last week, and want a different direction
14:07:27 mriedem right
14:07:49 mriedem we said we'd provide a way to update the keypair for a specific server, and then the user can reboot or rebuild the instance
14:08:00 liuyulong it's not the same. One adding API, one updating rebuild API.
14:08:01 mriedem because they might not need to rebuild the guest depending on how cloud-init is configured
14:08:10 mriedem the goal is the same
14:08:26 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Deprecate 'keymap' options https://review.openstack.org/483994
14:10:24 liuyulong One API can done for updating keypair. But the new API may need two steps. It's waste.
14:10:55 liuyulong but the new API can be used for a running injection.
14:11:27 liuyulong So I'd like to say, both can be added to nova.
14:12:06 mriedem liuyulong: by that logic, why don't we also add keypair to the reboot API so you can specify a new keypair on reboot?
14:12:13 mriedem it starts to become a slippery slope
14:12:33 liuyulong New API can result an inconsistent situtation when the vm is not reboot.
14:12:48 mriedem does Kevin_Zheng have a spec up for review yet?
14:12:54 liuyulong yes
14:13:00 mriedem inconsistent how?
14:13:06 liuyulong https://review.openstack.org/#/c/506552
14:13:17 mriedem yes the keypair is different, because the user changed it
14:13:28 mriedem and it's different from the guest until the guest is rebooted or rebuilt
14:14:11 mriedem mikal: cburgess: keypair update API spec https://review.openstack.org/#/c/506552
14:14:13 mriedem since you wanted to review

Earlier   Later