Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-06
03:45:01 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle InstanceInvalidState for os-assisted-volume-snapshots https://review.openstack.org/429476
03:47:15 openstackgerrit Nam Nguyen Hoai proposed openstack/nova master: Use oslo_serialization.base64 to follow OpenStack Python3 https://review.openstack.org/413885
05:46:14 alex_xu Kevin_Zheng: fyi, the reno part of this patch https://review.openstack.org/#/c/421760/ is separated to another patch https://review.openstack.org/#/c/428285/, you just need update the part of api-ref.
05:54:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix keypair API parameters https://review.openstack.org/420626
05:54:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882
05:54:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters and examples in aggregate API https://review.openstack.org/425512
05:54:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: Use ListOfIntegersField in oslo.versionedobjects https://review.openstack.org/423912
05:55:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
05:55:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436
05:55:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738
05:55:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix doc generation warnings https://review.openstack.org/425549
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

Earlier   Later