| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-12-04 | |||
| 22:07:33 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add aggregate link note in API ref https://review.openstack.org/521502 | |
| 22:07:57 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094 | |
| 22:08:20 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref https://review.openstack.org/521541 | |
| 22:08:38 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells https://review.openstack.org/521400 | |
| 22:08:57 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell' https://review.openstack.org/513771 | |
| 22:09:39 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506 | |
| 22:10:37 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Fix foreign key constraint error https://review.openstack.org/516880 | |
| 22:11:09 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Fix getting placement request ID https://review.openstack.org/523606 | |
| 22:11:26 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579 | |
| 22:12:56 | openstackgerrit | Hongbin Lu proposed openstack/nova master: [WIP] Handle RPC timeout on get_console_output https://review.openstack.org/525335 | |
| 22:21:35 | efried | jaypipes Been staring at this for a bit, can't figure out where 1.10 is coming from: http://logs.openstack.org/93/385693/83/check/legacy-grenade-dsvm-neutron/b513223/logs/screen-n-cpu.txt.gz#_Dec_04_15_47_11_988730 | |
| 22:31:31 | gryf | efried, looks like you run pike version of placement against latest source | |
| 22:31:50 | efried | gryf Wow, that would be... bad. How can you tell? | |
| 22:32:12 | gryf | 1.10 was the max microversion in pike | |
| 22:33:22 | efried | cdent If you're still around -- how does legacy-grenade-dsvm-neutron spin up the placement API? Is there some chance we could be using the pike version in latest master? | |
| 22:33:33 | efried | cdent And wouldn't lots of other stuff have broken by now if that were the case? | |
| 22:33:57 | cdent | efried: I assume you’re talking about the thing that’s claiming 1.10 is the max version? | |
| 22:34:05 | efried | cdent yeah | |
| 22:34:12 | cdent | was just reading that in my email | |
| 22:35:27 | cdent | it could be that a new compute is talking to an older placement, which I think is something we _may_ do intentionally in some tests | |
| 22:35:48 | efried | oy vay, that could be... complicated to fix. | |
| 22:36:03 | cdent | and for the most part we’ve tried to make the report client be smart about that | |
| 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: 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 | |