| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-06 | |||
| 06:04:18 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.42 - Fix tag attribute disappearing https://review.openstack.org/429512 | |
| 06:07:30 | Kevin_Zheng | alex_xu: thanks | |
| 06:07:38 | alex_xu | Kevin_Zheng: np | |
| 06:08:23 | Kevin_Zheng | https://review.openstack.org/#/c/403568/ could you review this one if you got time? seems a high priority bug | |
| 06:10:26 | alex_xu | Kevin_Zheng: will check that | |
| 06:14:26 | openstackgerrit | Zhenguo Niu proposed openstack/nova master: Fix breaking out of nested loops https://review.openstack.org/424931 | |
| 06:23:05 | openstackgerrit | LIU Yulong proposed openstack/nova-specs master: Reset the instance keypair while rebuilding (spec) https://review.openstack.org/375221 | |
| 06:23:18 | openstackgerrit | LIU Yulong proposed openstack/nova master: Enable reset keypair while rebuilding instance https://review.openstack.org/379128 | |
| 06:24:33 | liuyulong | alex_xu, ^^, can we move on these patch sets? | |
| 06:25:52 | alex_xu | liuyulong: yes, we can, as Pike is open, just need some reviews on the spec first | |
| 06:26:29 | liuyulong | alex_xu, thanks, :) | |
| 06:26:36 | alex_xu | liuyulong: np :) | |
| 06:32:32 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876 | |
| 06:34:32 | liuyulong | alex_xu, hi, could you please approve this bp https://blueprints.launchpad.net/nova/+spec/rebuild-keypair-reset on pike cycle? | |
| 06:35:15 | liuyulong | alex_xu, and remove the -2 here https://review.openstack.org/#/c/379128/, : ) | |
| 06:36:28 | alex_xu | liuyulong: I will remove -2 after the spec approved. The BP only can be approved after the spec approved | |
| 06:36:54 | liuyulong | alex_xu, got it, thanks. | |
| 06:37:16 | alex_xu | liuyulong: np | |
| 06:53:35 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.43 - Enable cold migration with target host https://review.openstack.org/406707 | |
| 06:57:44 | litao | alex_xu: I left some comments in my path | |
| 06:58:45 | alex_xu | litao: actually the updated_at you seen in the API isn't the heartbeat timestamp | |
| 06:59:30 | alex_xu | litao: for db driver, the 'last_seen_up' is the field for that https://github.com/openstack/nova/blob/master/nova/servicegroup/drivers/db.py#L60 | |
| 07:01:03 | alex_xu | litao: the updated_at's meaning actually changed by this patch https://review.openstack.org/#/c/163060/26 | |
| 07:03:58 | litao | alex_xu: I think you meaning is just to decide the status of a service using 'last_seen_up' | |
| 07:04:37 | litao | alex_xu: I meaning the display of 'nova service-list | |
| 07:04:44 | litao | ' | |
| 07:04:48 | alex_xu | litao: no, I mean the meanning of 'updated_at' is for that | |
| 07:05:23 | litao | alex_xu: but the output of 'nova service-list' will display the column | |
| 07:05:35 | litao | alex_xu: updated_at | |
| 07:06:11 | alex_xu | litao: yes, 'updated_at' means 'the timestamp when db record of service updated' | |
| 07:07:18 | litao | so i think the value should be updated regardless of db or mc driver | |
| 07:08:46 | litao | alex_xu: If it retains for a long time not to update, it will give us a doubt that is the service really up? | |
| 07:09:14 | alex_xu | litao: the patch https://review.openstack.org/#/c/163060/ separate the db record update time and the heartbeat update time. After that patch 'updated_at' means db record time. Maybe we can have another field for heartbeat timestamp, but i'm not sure | |
| 07:09:39 | alex_xu | litao: yes, but | |
| 07:09:51 | alex_xu | 'updated_at' isn't for determine whether the service is up | |
| 07:10:54 | alex_xu | 'last_seen_at' and your change for mc is for determine that, but that isn't expose in the API | |
| 07:11:27 | litao | alex_xu: but i think it should be same as db drvier to be updated since it presents to us | |
| 07:11:59 | litao | alex_xu: or remove it when using mc driver | |
| 07:12:01 | alex_xu | litao: db driver didn't update 'updated_at' also | |
| 07:13:03 | litao | alex_xu: it does | |
| 07:13:12 | alex_xu | litao: there is field 'State' for determine whether service is up, why you need to lookup another timestamp? | |
| 07:14:05 | litao | alex_xu: I mean that it just a display, not to determine the state of a service | |
| 07:15:12 | litao | alex_xu: I just want to display the same output of 'nova service-list' when using mc driver | |
| 07:20:10 | alex_xu | litao: emm...I see now, it will update. sorry, I missed that. But it still didn't equal to the your change for the mc driver I think | |
| 07:21:10 | alex_xu | litao: for db driver, 'updated_at' mixes db record update and heartbeat update. | |
| 07:24:36 | litao | alex_xu: yes, so what do you think to deal it? | |
| 07:24:46 | alex_xu | litao: I just comment on the patch | |
| 07:58:35 | openstackgerrit | Sergey Nikitin proposed openstack/nova-specs master: PCI NUMA Policies https://review.openstack.org/361140 | |
| 08:23:52 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Repropose tag-instance-when-boot https://review.openstack.org/415315 | |
| 08:44:32 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Send out notifications when instance tags chaned https://review.openstack.org/429569 | |
| 08:53:03 | openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Extend tag length limit to 80 https://review.openstack.org/423112 | |
| 09:01:34 | litao | alex_xu: thanks | |
| 09:12:54 | alex_xu | litao: np | |
| 10:24:21 | FransUrbo | I'm trying to figure out why my second primary servers services won't come up. They're listed as 'down' with OSC, but the 'service:updated_at' column have been updated very recently - https://gist.github.com/FransUrbo/a24e99a331c2a252ecdc76a5c6f12b20 (that's for Cinder, but Nova shows the same problem). | |
| 10:26:19 | FransUrbo | https://gist.github.com/FransUrbo/e11eb9d9311790c63729f16e108dbb1f (for nova) | |
| 10:29:38 | BobBall | johnthetubaguy: Are you happy with https://review.openstack.org/#/c/424428/ ? Clearly the change missed RC1 but I'd love to get it in RC2 if we can get reviews soon | |
| 10:30:10 | johnthetubaguy | BobBall: we spoke about it being eligible for RC2 on friday evening | |
| 10:30:35 | johnthetubaguy | BobBall: got my head in a spec right now, once my head is out of that, I will take a peak at it | |
| 10:30:40 | BobBall | Yeah - I spotted the change in the etherpad too. Awesome, thanks! | |
| 10:31:09 | johnthetubaguy | no worries, it would be a shame for that bug to stick around another release | |
| 10:32:18 | BobBall | Indeed. Getting the fix in is my priority ATM so let me know what I need to do. | |
| 10:50:39 | openstackgerrit | Bob Ball proposed openstack/nova master: XenAPI: fix memory usage miscalculation https://review.openstack.org/394857 | |
| 11:05:24 | openstackgerrit | sahid proposed openstack/nova master: libvirt: update the active migrations DS to support migration result https://review.openstack.org/409260 | |
| 11:05:25 | openstackgerrit | sahid proposed openstack/nova master: libvirt: ensure live-migration success https://review.openstack.org/409261 | |
| 11:39:32 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix network_label parameter type https://review.openstack.org/429659 | |
| 11:53:32 | artom_ | In the api samples tests, if the response template doesn't exit, it's created with the response data | |
| 12:45:16 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: WIP: Spec idea on the future of quotas for Nova https://review.openstack.org/429678 | |
| 12:58:32 | johnthetubaguy | BobBall: looks good to me | |
| 13:04:58 | BobBall | Thanks johnthetubaguy | |
| 13:45:18 | openstackgerrit | Timofey Durakov proposed openstack/nova master: abort_live_migration remove redundant check https://review.openstack.org/304621 | |
| 13:56:08 | mriedem | BobBall: i'm in a timewarp http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/76/429476/1/check/dsvm-tempest-neutron-network/00891c1//index.html | |
| 13:56:15 | mriedem | clicking logs takes me to a dir that just takes me back | |
| 13:56:46 | BobBall | mriedem: Did you take a jump to the left? And then a step to the right? | |
| 13:57:15 | mriedem | i forgot those steps | |
| 13:57:55 | bauzas | folks, FYI, will mostly be afk this week because I'm organizing a local conf named http://snowcamp.io/ | |
| 13:58:04 | bauzas | with ttx keynoting :p | |
| 13:58:07 | BobBall | mriedem: It looks like the rest of the logs didn't upload curiously | |
| 13:58:32 | BobBall | mriedem: Received disconnect from 192.168.33.2: 2: Packet corrupt <-- or there were no other logs. | |
| 13:58:52 | BobBall | mriedem: that test run didn't even get to the point of running devstack | |
| 13:59:04 | edleafe- | Scheduler subteam meeting in 1 minute over in #openstack-meeting-alt | |
| 13:59:27 | mriedem | BobBall: ok i just rechecked it | |
| 13:59:53 | BobBall | mriedem: We do put "run_tests.log" in both directories... which is perhaps unhelpful. | |
| 14:00:48 | ttx | \o/ and fresh snow | |
| 14:02:58 | bauzas | ttx: zactly, 30cm waiting for you ;) | |
| 14:05:30 | openstackgerrit | Timofey Durakov proposed openstack/nova master: nfs fix for xenial images https://review.openstack.org/329466 | |
| 14:06:09 | kashyap | sdague: Heya, random ping -- someone (a Kernel developer, actually) at FOSDEM asked me yesterday: Why DevStack, why not...just use 'Gentoo'. They were serious :-) | |
| 14:06:33 | sdague | kashyap: :) | |
| 14:06:45 | sdague | sounds like they never actually looked into the issue | |
| 14:07:03 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Move proxies to 'console/proxy' https://review.openstack.org/408192 | |
| 14:07:12 | kashyap | sdague: Yeah, right. But they were not really mean or anything, just thinking out loud | |
| 14:07:36 | kashyap | sdague: I explained them, also the reason why it's shell script (that a human / admin / operator should be able to read it, etc). | |
| 14:10:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate two more nova-net options https://review.openstack.org/412390 | |
| 14:23:34 | mriedem | superdan: first, where is dansmith | |
| 14:23:43 | mriedem | superdan: second, alex_xu and i have comments in https://review.openstack.org/#/c/428824/ | |
| 14:32:02 | dansmith | mriedem: okay coffee then that | |
| 14:34:22 | openstack | Launchpad bug 1659391 in OpenStack Compute (nova) "Server list API does not show scheduled servers that are not assigned to any cell" [Undecided,New] | |
| 14:34:22 | mriedem | manila also reported a bug that the change for multi-cell instance listing broke them https://bugs.launchpad.net/nova/+bug/1659391 but so far i'm not able to find a job that failed due to that to debug it, | |
| 14:34:59 | mriedem | plus the description is overly confusing with references to cells being disabled and default cells, which really throws me off | |
| 14:38:17 | dansmith | hmm, okay | |
| 14:39:01 | dansmith | mriedem: well the description says that after that change, nova only returns instances assigned to cells | |