Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-04
22:38:13 efried cdent Okay, I gotcha.
22:38:16 cdent If we don’t need to support that, the other chance is simply that the new placement hasn’t manage to get restarted yet
22:38:19 dansmith old compute needs to work with new placement
22:38:32 cdent dansmith: yes, but vice versa?
22:39:28 dansmith not vice versa
22:39:47 efried cdent "hasn't managed to get restarted yet" - so actually a recheck may clear it?
22:40:41 cdent efried: probably, but it may be that the process mgt in grenade is insufficiently robust and could be robustified
22:40:56 efried BTW, this was just exposed because of https://review.openstack.org/#/c/524263/ -- I think otherwise we would have blown right by this error and not noticed. So it may be a long-standing latent bug.
22:41:23 cdent yup, I dig
22:41:55 cdent efried: if you haven’t got time to think about grenade right now (or ever), I’m happy to go digging instead, but wouldn’t be until tomorrow
22:42:01 efried Well, I guess I'll try a recheck, but it seems like something that ought to be fixed *somewhere*
22:42:10 efried cdent I've never heard of grenade.
22:42:21 efried And it's jaypipes' patch anyway :P
22:43:01 cdent code ownership is, like, _so_ 1990, but yeah, my curiosity is piqued
22:43:16 cdent grenade is the sort of framework used to manage upgrade testing
22:43:32 cdent a devstack in the previous version is started, verified a bit
22:44:18 cdent then new code is installed, but old configruations kept
22:50:07 openstackgerrit Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854
22:50:07 openstackgerrit Ed Leafe proposed openstack/nova master: Move the to_dict() method to the Selection object https://review.openstack.org/523492
22:50:07 openstackgerrit Ed Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts https://review.openstack.org/510159
22:50:43 edleafe mriedem: ^^ addressed your comments
22:51:02 edleafe mriedem: will start on https://review.openstack.org/#/c/516707/ tomorrow
22:52:36 openstackgerrit Eric Fried proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
22:52:36 openstackgerrit Eric Fried proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
22:52:37 openstackgerrit Eric Fried proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
22:52:59 efried jaypipes ^ this is just a rebase on top of _Provider-private so I can start rebasing the other stuff in that series (e.g. _set_inventory doing nested stuff)
22:59:08 jaypipes efried: roger
23:24:06 openstackgerrit Eric Fried proposed openstack/nova master: Scheduler set_inventory_for_provider does nested https://review.openstack.org/520643
23:24:14 efried jaypipes Like that ^
23:24:37 efried jaypipes That _validate_provider thing really comes into its own now.
23:24:43 efried in the tests
23:28:21 efried jaypipes cdent et al: FYI I'm out tomorrow; working W, Th, and first half of Fr; then (officially) out til Jan. I'll probably be able to check in periodically to shepherd simple rebases/reviews, but not a lot of big stuff.
23:29:11 cdent efried: roger that. I’m fairly around throughout most of december, happy to mangle your stuff as required during that time
23:29:52 efried cdent I really hope to get things in a state where that would be remotely possible before I bugger off. Right now most of my patches are kind of a mess.
23:30:07 efried cdent And thank you for the offer.
23:52:45 openstackgerrit Hongbin Lu proposed openstack/nova master: Add support for listing hosts in cellv2 https://review.openstack.org/524755
#openstack-nova - 2017-12-05
00:43:00 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
00:43:58 Kevin_Zheng mriedem Hi, could you check https://review.openstack.org/#/c/515631/ again when you got time?
00:47:04 gmann stephenfin: zuulv3- i did not find any other user for 'tempest-dsvm-neutron-src-oslo.versionedobject', we should move this to nova then, any specific reason for not moving it
00:53:09 mriedem gmann: not nova,
00:53:13 mriedem the oslo.versionedobject repo
00:53:35 gmann mriedem: but they do not run this. its only needed by nova as of now
00:54:12 mriedem it should be run by oslo.versionedobjects
00:54:16 gmann but yea as its their lib testing then if they want to own
00:57:00 gmann mriedem: yea, as of now they do not run, they run only 'legacy-tempest-dsvm-neutron-src'
00:59:53 mriedem isn't it the same thing? run the dsvm neutron job with src oslo.versionedobjects for the change in question?
00:59:57 mriedem if the job is a duplicate, just drop it
01:01:41 openstackgerrit Matt Riedemann proposed openstack/nova master: Mention API behavior change when over quota limit https://review.openstack.org/515631
01:01:47 mriedem Kevin_Zheng: done
01:02:15 Kevin_Zheng mriedem thanks, updating
01:02:16 mriedem alex_xu: ^ want to get that?
01:02:21 mriedem Kevin_Zheng: i already updated it
01:02:47 Kevin_Zheng Ah, ok
01:06:42 gmann mriedem: yea. as oslo run neutron job, it make sure their master src does not break anything. that is enough i think.
01:17:43 openstackgerrit Merged openstack/nova-specs master: Remove the py27 tox target https://review.openstack.org/520680
01:28:41 mriedem interesting fact,
01:28:52 mriedem my kid sings the star spangled banner when in the bathroom now,
01:28:54 mriedem like all the time
01:37:00 mriedem edleafe: https://review.openstack.org/#/c/495854/ for your morning
01:37:04 mriedem missed something in my initial review
01:37:29 mriedem on the plus side, +2 on https://review.openstack.org/#/c/510159/
01:47:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
01:55:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811
01:55:55 mriedem Kevin_Zheng: some more comments in https://review.openstack.org/524047
01:56:37 Kevin_Zheng mriedem: thx
02:12:16 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Fix an error message in API validation https://review.openstack.org/524088
02:24:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix wrong argument order in functional test https://review.openstack.org/525363
02:36:31 alex_xu Kevin_Zheng: should we doc that in the api-ref also?
02:37:00 Kevin_Zheng alex_xu: which one?
02:37:15 alex_xu Kevin_Zheng: https://review.openstack.org/#/c/515631/5
02:37:43 alex_xu Kevin_Zheng: since you said, it is user-noticeable API change...so I guess you want to get attention from the API user, not only the deployer?
02:39:13 Kevin_Zheng I don't know, maybe it is too much details for normal users?>
02:39:48 alex_xu Kevin_Zheng: yea, it shouldn't be too much detail, just something like after specific release, if overquota, you will get a error instance...
02:40:25 alex_xu Kevin_Zheng: I guess that won't effect the API user too much
02:40:44 Kevin_Zheng But we have only have a Master branch doc
02:40:51 alex_xu Kevin_Zheng: actually more the end user, since they get an error instance, and they don't what happened.
02:41:09 Kevin_Zheng it might be different for different versions
02:41:36 Kevin_Zheng releases
02:41:42 alex_xu Kevin_Zheng: ah, got it, there is config option
02:42:30 alex_xu Kevin_Zheng: ok, just note in the release note, I think this is more important for the deployer/end user, I don't think that will break the API user app
02:42:46 alex_xu Kevin_Zheng: done
02:52:44 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/525006
02:52:52 openstackgerrit Merged openstack/nova master: Downgrade log for keystone verify client fail https://review.openstack.org/513243
03:13:44 openstackgerrit Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395
03:19:50 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/525386
03:33:41 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/525399
05:28:14 openstackgerrit Merged openstack/nova master: Add regression test for rebuilding a volume-backed server https://review.openstack.org/521200
06:12:07 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
06:32:42 openstackgerrit jichenjc proposed openstack/nova stable/pike: Downgrade log for keystone verify client fail https://review.openstack.org/525475
07:00:47 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
08:54:33 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386
08:59:45 openstackgerrit Zhenyu Zheng proposed openstack/nova master: [WIP] POC to use neutron port_list when filtering instance by ip https://review.openstack.org/525505
09:10:19 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance() https://review.openstack.org/524047
09:27:39 openstackgerrit Merged openstack/nova master: Mention API behavior change when over quota limit https://review.openstack.org/515631
09:28:38 openstackgerrit Zhenyu Zheng proposed openstack/nova stable/pike: Mention API behavior change when over quota limit https://review.openstack.org/525511
09:32:12 stephenfin gmann_afk, mriedem: I left a big comment about the 'tempest-dsvm-neutron-src-oslo.versionedobject' job here https://review.openstack.org/#/c/514310/3/zuul.d/zuul-legacy-jobs.yaml
09:33:00 stephenfin gmann_afk, mriedem: The tl;dr is that that job should be combined with a number of similar jobs and kept in a central location. There's nothing specific to o.v.o (or nova) about it
09:45:18 openstackgerrit Merged openstack/nova master: Remove incorrect comment about instance.locked https://review.openstack.org/515306
10:03:37 openstackgerrit Merged openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811
10:23:12 openstackgerrit Matthew Booth proposed openstack/nova master: Add uuid column to BlockDeviceMapping https://review.openstack.org/242602

Earlier   Later