| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-08 | |||
| 13:23:35 | openstackgerrit | Alexandru Muresan proposed openstack/nova master: Fix driver tests on Windows https://review.openstack.org/518525 | |
| 13:38:04 | jaypipes | ralonsoh: you in Sydney? | |
| 13:38:27 | ralonsoh | jaypipes: no, in Ireland | |
| 13:38:41 | jaypipes | ralonsoh: k. (I'm at home in Florida) | |
| 13:38:54 | jaypipes | ralonsoh: I'll review your netlink patches today. | |
| 13:39:01 | ralonsoh | jaypipes: thanks! | |
| 13:39:03 | jaypipes | ralonsoh: will try to get some forward progress on those | |
| 13:39:12 | jaypipes | ralonsoh: apologies of the long delay :( | |
| 13:39:47 | ralonsoh | jaypipes: no problem. Now I'm just taking care of my uploaded patches, but I'm leaving OpenStack development | |
| 13:39:58 | jaypipes | ralonsoh: seriously? :( | |
| 13:40:18 | ralonsoh | jaypipes: yes, I'm moving to other team/department | |
| 13:40:30 | ralonsoh | jaypipes: sorry about that | |
| 13:41:05 | jaypipes | ralonsoh: that sucks man :( | |
| 13:48:39 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | |
| 14:28:44 | efried | jaypipes Perhaps you can save me some digging. | |
| 14:29:09 | efried | jaypipes When I print an AllocationRequestResource, it prints amount & RC just fine | |
| 14:29:26 | efried | jaypipes Then it prints the RP as ResourceProvider({uuid}) | |
| 14:29:38 | efried | Any idea where that __str__/__repr__ method is coming from? I'd like to print the name as well. | |
| 14:29:49 | efried | For test purposes, at least. | |
| 14:29:55 | jaypipes | efried: it'll print the name if the name attr is set. | |
| 14:30:27 | jaypipes | efried: https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L309-L319 | |
| 14:31:28 | efried | That's what I thought too, but I don't think we're hitting that method. | |
| 14:31:44 | efried | jaypipes Cause the key isn't being printed. | |
| 14:31:49 | efried | Looks like this: AllocationRequestResource(amount=1,resource_class='VCPU',resource_provider=ResourceProvider(25d6392d-73e3-40a8-a415-f25a4e25ef00)) | |
| 14:32:04 | efried | If we were hitting that method, I would have expected ResourceProvider(uuid=....) | |
| 14:32:34 | jaypipes | efried: how are you printing things? | |
| 14:32:49 | efried | for ar in p_alts.allocation_requests: | |
| 14:32:50 | efried | rp_obj.LOG.error("AR: %s", ar) | |
| 14:32:50 | efried | for rr in ar.resource_requests: | |
| 14:32:50 | efried | rp_obj.LOG.error("RR: %s", rr) | |
| 14:33:32 | jaypipes | efried: try "%r" | |
| 14:33:37 | efried | ... | |
| 14:34:18 | efried | jaypipes Same | |
| 14:34:25 | jaypipes | hmm | |
| 14:35:02 | jaypipes | efried: gimme a sec | |
| 14:36:29 | jaypipes | efried: and if you print out the resource provider object itself, what shows up? | |
| 14:37:48 | efried | jaypipes That comes out good: ResourceProvider(generation=2,id=23,name='twelve',uuid=1c6fc5a8-4d2c-4b50-ac9b-5c003a0b4c35) | |
| 14:38:30 | jaypipes | efried: weird... | |
| 14:38:35 | efried | agreed | |
| 14:39:21 | openstackgerrit | Tyler Blakeslee proposed openstack/nova master: TMB TEST https://review.openstack.org/518547 | |
| 14:42:14 | jaypipes | efried: https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L814-L824 | |
| 14:42:32 | jaypipes | efried: apparently, this is by design. | |
| 14:43:12 | efried | jaypipes Cool, that's fine, I can just stuff it in there for testing purposes. Thanks. | |
| 14:43:23 | jaypipes | np | |
| 15:04:08 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: nova-manage update_cell allows more than once cell to have the same db/transport urls https://review.openstack.org/518556 | |
| 15:05:53 | efried | jaypipes I couldn't even get that to work. Whenever I try to get the 'name' it complains about not being able to load it on the base object. Something about it being funneled through an ObjectField makes all those other fields blank out. | |
| 15:06:26 | efried | jaypipes I wound up writing a cache of uuid:name when I create the RPs. It's not a permanent thing - just to debug the test case. | |
| 15:06:35 | jaypipes | ok | |
| 15:06:43 | efried | jaypipes Though debugging in the future is going to be a real bear without *something* like this. | |
| 15:07:08 | jaypipes | efried: ack. note that the ProviderTree stores that name -> uuid and reverse mapping. | |
| 15:07:25 | jaypipes | efried: but of course, that's used on the compute host, not the placement service side. | |
| 15:07:32 | efried | Mm. | |
| 15:28:30 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | |
| 16:03:05 | efried | jaypipes I'm uncovering all sorts of interesting bugs. | |
| 16:03:14 | jaypipes | efried: good :) | |
| 16:03:36 | efried | jaypipes At the moment, I'm at the top of your stack: https://review.openstack.org/#/c/517119/6 | |
| 16:03:55 | efried | I put together a test case with 14 different aggregates | |
| 16:04:17 | efried | jaypipes Did a request with no traits, and I'm getting candidates that cross aggregates. Which is bad. | |
| 16:05:51 | efried | jaypipes Does one write a bug against a patch in progress? Or better for me to put this test up with its actual expected assertions, which will fail and block the patch until they're fixed? | |
| 16:06:00 | jaypipes | the latter | |
| 16:06:43 | efried | ack. Though I suspect some of this behavior may be applicable to current master. I can check that out. | |
| 16:09:08 | efried | jaypipes If you could look at the test refactor patches and we can get those in, I can start rebasing this series accordingly while I'm doing this. | |
| 16:09:19 | efried | please and thank you :) | |
| 16:09:43 | efried | (gibi did a first pass) | |
| 16:13:44 | jaypipes | efried: l | |
| 16:13:48 | jaypipes | efried: k | |
| 16:13:54 | jaypipes | ugh, typing sucks today | |
| 16:14:06 | efried | jaypipes Still getting used to that Mac keyboard? | |
| 16:14:14 | jaypipes | efried: lol, no. | |
| 16:14:24 | jaypipes | efried: I don't use the Mac keyboard at all unless I'm traveling. | |
| 16:15:01 | jaypipes | efried: I use Synergy to share my regular Deck Francium keyboard across all my puters while at home. | |
| 16:15:39 | jaypipes | efried: just wrapped up a new article: https://github.com/jaypipes/articles/blob/master/openstack/walkthrough-launch-instance-request.md | |
| 16:17:23 | efried | jaypipes looking... | |
| 16:19:56 | efried | jaypipes How does a guy mark up one of these things? | |
| 16:20:07 | jaypipes | efried: pull request :) | |
| 16:20:37 | efried | jaypipes I don't know how to do that. I'll just take notes on the side. If it gets extensive, I'll look into this "pull request" thing. | |
| 16:21:38 | jaypipes | efried: ok dokey :) | |
| 16:30:37 | openstackgerrit | Marco Chiappero proposed openstack/nova master: Add a communication channel for power management support https://review.openstack.org/512815 | |
| 16:57:28 | openstackgerrit | Marco Chiappero proposed openstack/nova-specs master: Enable VM communication channel for power management https://review.openstack.org/511453 | |
| 18:33:35 | openstackgerrit | Mike Lowe proposed openstack/nova master: live-mig: keep disk device address same https://review.openstack.org/518022 | |
| 20:48:51 | openstackgerrit | Mike Lowe proposed openstack/nova master: live-mig: keep disk device address same https://review.openstack.org/518022 | |
| 23:59:17 | openstackgerrit | Eric Fried proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | |
| 23:59:19 | efried | jaypipes Check this puppy out ^ (See https://bugs.launchpad.net/nova/+bug/1731072) | |
| 23:59:20 | openstack | Launchpad bug 1731072 in OpenStack Compute (nova) "AllocationCandidates.get_by_filters returns garbage with multiple aggregates" [Undecided,New] | |
| #openstack-nova - 2017-11-09 | |||
| 00:18:32 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | |
| 01:31:37 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Remove incorrect comment about instance.locked https://review.openstack.org/515306 | |
| 01:37:30 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: block_device_mapping_v2.bus_type is missing from api-ref https://review.openstack.org/499488 | |
| 01:39:31 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842 | |
| 01:43:40 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Clean up exception caught in _validate_and_build_base_options https://review.openstack.org/463253 | |
| 02:00:51 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | |
| 03:29:56 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644 | |
| 07:10:21 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644 | |
| 07:27:56 | openstackgerrit | Lajos Katona proposed openstack/nova master: Change live_migrate tests to use fakedriver https://review.openstack.org/505202 | |
| 08:02:56 | Tengu | hello there! | |
| 08:03:43 | openstackgerrit | Marco Chiappero proposed openstack/nova master: Add a communication channel for power management support https://review.openstack.org/512815 | |
| 08:04:50 | Tengu | small question: what can prevent a nova instance to be resized with a "no valid host found" while we have plenty of resources, and another instance CAN actually be resized (same original flavor, same destination flavor)? | |
| 08:06:30 | Tengu | it looks like some weird flag on the instance that prevents the resize. We had some issues with the infra yesterday that prevented the resize (i.e. it crashed due to 1730949) that might have trigger some safe-guard? | |
| 08:09:46 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Implement query param schema for migration index https://review.openstack.org/518644 | |
| 09:54:18 | openstackgerrit | Merged openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | |
| 10:14:40 | openstackgerrit | Forest Romain proposed openstack/nova master: Fix list servers by hypervisor https://review.openstack.org/518699 | |