Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-04
22:37:53 cdent efried: hmmm. yeah, that code doesn’t have a fall back, but I’m not sure we want that or not. jaypipes, mriedem or dansmith will be able to say more on whether we need to be able to work with new compute/old placement
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: Modify select_destinations() to return objects and alts https://review.openstack.org/510159
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: Return Selection objects from the scheduler driver https://review.openstack.org/495854
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: adds REST API for nested providers https://review.openstack.org/384807
22:52:36 openstackgerrit Eric Fried proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
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

Earlier   Later