Earlier  
Posted Nick Remark
#openstack-nova - 2017-11-07
23:34:38 mriedem dansmith: https://review.openstack.org/#/q/I29fac80d08baf64bf69e54cf673e55123174de2a
23:34:42 melwitt openstack_newb_: the config option is if you want to require specific namespace(s) for the keys related to aggregate image properties. you don't have to set it
23:35:02 jaypipes oh, look, it's Sydney nova folks :)
23:35:09 melwitt hi jaypipes :)
23:35:23 jaypipes melwitt: hey Melanie! having a good time in Roo-country? :)
23:35:26 melwitt openstack_newb_: you also have to set the keys on your images, did you do that?
23:35:35 mriedem mtreinish: https://review.openstack.org/#/q/I29fac80d08baf64bf69e54cf673e55123174de2a
23:35:43 melwitt jaypipes: yeah, it's very nice here :)
23:35:49 jaypipes good to hear!
23:36:35 mriedem mtreinish: this is the tempest test for ^ but it's hit a niggle as johnthetubaguy would say https://review.openstack.org/#/c/516396/5/tempest/api/compute/servers/test_server_actions.py@322
23:36:40 mriedem i would never say that word, except just now
23:38:44 melwitt jaypipes: wish you and julie were here too
23:39:07 jaypipes melwitt: well, hopefully I'll see you in Dublin eh? :)
23:39:26 melwitt yeah :)
23:39:48 openstack_newb_ Thank you! So having "aggregate_image_properties_isolation_namespace = None" is fine
23:40:59 openstack_newb_ I haven't set the keys on my images, I set os=cirros and os=linux ....
23:41:18 openstack_newb_ isn't that the same thing?
23:41:53 openstack_newb_ For example : glance image-update --property os=linux a51553ae-7b85-4b79-92ec-33a20221e611
23:42:39 openstack_newb_ And then in my aggregates I did for example: nova aggregate-set-metadata linux os=linux
23:43:04 openstack_newb_ But I'm not seeing it work at all .... the filter still returns two hosts in the log
23:45:59 melwitt sounds like you're having this same issue https://ask.openstack.org/en/question/30934/aggregateimagepropertiesisolation-returns-hosts-that-shouldnt-be-applicable/
23:46:28 melwitt you only have two hosts though and each one is in an aggregate, one tagged os=cirros and one tagged os=linux, right?
23:46:39 openstack_newb_ Yes... exactly
23:46:49 openstack_newb_ one tagged linux and one cirros
23:47:26 openstack_newb_ and each aggregate has one of the hosts ... should work - right?
23:48:28 melwitt yeah. did you set the aggregate_image_properties_isolation_namespace option? if you don't want a namespace, then don't set it. I think if you set it to "None" it might expect you do prefix your tags with "None" or something
23:48:45 openstack_newb_ OK... will disable that
23:48:46 melwitt s/do/to/
23:49:03 openstack_newb_ Might be the problem
23:49:12 openstack_newb_ Thank you!
23:49:32 melwitt okay, hope that works
23:54:27 openstack_newb_ No luck but I'm seeing a warning in the scheduler log ... Host 'hv142.ostk.sc1.corp.gq1.yahoo.com' has a metadata key 'os' that is not present in the image metadata.
23:55:01 openstack_newb_ Something wrong with the way I added the os value to the image?
23:55:58 melwitt yeah, must be I would think
23:56:07 openstack_newb_ https://www.irccloud.com/pastebin/uU7SN7w5/
23:56:43 openstack_newb_ I assume os and metadata are the same in this context ?
23:57:56 melwitt I wonder if you need to do like glance image-update --property os=cirros <image>
23:58:24 openstack_newb_ I notice that tags are empty - is this the missing "metadata" perhaps?
23:58:49 openstack_newb_ Yes, I did that command already
#openstack-nova - 2017-11-08
00:04:00 melwitt openstack_newb_: from what I can tell, the tags shouldn't be related
00:04:06 openstackgerrit Tristan Cacqueray proposed openstack/nova master: Mask the token used to allow access to consoles https://review.openstack.org/220622
00:05:26 openstack_newb_ yes... I think you are correct. Just looking at this - he has the same issue: https://bugzilla.redhat.com/show_bug.cgi?id=1435597
00:05:27 openstack bugzilla.redhat.com bug 1435597 in openstack-nova "AggegateImagePropertiesIsolatefiler not working" [High,New] - Assigned to sbauza
00:07:02 openstack_newb_ So... it's a bug then in Ocata too - right?
00:08:16 melwitt openstack_newb_: yeah, must be if that's what you're seeing. we'll need to open an upstream bug for it if we don't already have one and get it fixed
00:10:50 openstack_newb_ Well.. ... I guess I'm going to have to fix it here for our use if that's the case ;)
00:11:55 openstack_newb_ Thank you for all your help!
00:13:08 melwitt yeah, it helps when people report bugs and post patches to fix bugs
00:14:27 openstack_newb_ Yes... if I can fix it I'll eitehr try to send it to Sylvain Bauza? or push it upstream myself
00:14:48 openstack_newb_ It's probably something silly...
00:15:30 melwitt best to push it upstream yourself if you can and just ping me when it's posted
00:16:10 openstack_newb_ OK ... will do
00:16:16 melwitt cool, thank you
00:16:18 openstack_newb_ Thank you!
03:41:36 med_ mikal, we're in 4.11
03:41:39 med_ for privsep
03:41:43 med_ in nova updates
04:04:02 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Add cross cell sort support for get_migrations https://review.openstack.org/517273
06:04:32 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Add cross cell sort support for get_migrations https://review.openstack.org/517273
06:58:10 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965
07:16:16 openstackgerrit Merged openstack/nova master: Fix return type in FilterScheduler._legacy_find_hosts https://review.openstack.org/517134
07:16:35 openstackgerrit Merged openstack/nova stable/pike: libvirt: do not remove inst_base when volume-backed during resize https://review.openstack.org/517388
07:16:40 openstackgerrit Merged openstack/nova stable/ocata: libvirt: do not remove inst_base when volume-backed during resize https://review.openstack.org/517390
07:20:00 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217
07:20:00 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899
07:28:45 openstackgerrit edan david proposed openstack/nova master: Don't overwrite binding-profile https://review.openstack.org/505613
09:03:12 openstackgerrit Marco Chiappero proposed openstack/nova master: Add a communication channel for power management support https://review.openstack.org/512815
09:12:34 openstackgerrit Lajos Katona proposed openstack/nova master: Extend ServerMovingTests with custom resources https://review.openstack.org/497399
09:20:38 openstackgerrit Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
09:48:09 openstackgerrit Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
10:59:25 openstackgerrit sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457
12:26:02 openstackgerrit Edward Hope-Morley proposed openstack/nova master: Ensure deleted computes aren't counted in stats https://review.openstack.org/518520
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)

Earlier   Later