Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-05
17:32:44 melwitt dansmith: I have a crazy idea of maybe counting BuildRequests as part of the instances count. not sure if there's any gotchas there
17:33:01 melwitt to make it so we dont return an instance if we go over quota during a race in the first place
17:33:07 dansmith melwitt: meaning counting both and taking the sum?
17:33:31 melwitt yeah
17:33:38 dansmith I don't think you can do that naively unless you filter out the dupes, because we do have dupes for a bit right?
17:33:49 sean-k-mooney stephenfin: if i boot a new instace after designate is there it correct populates the server name from the create command into the hostname and fqdn fields http://paste.openstack.org/show/614486/
17:34:07 dansmith melwitt: we create the instance and then delete the buildreq so the api code starts looking at the actual instance, IIRC
17:34:19 melwitt dansmith: hm, I didn't realize it dupes
17:34:25 melwitt yeah
17:34:29 melwitt dangit
17:34:31 dansmith melwitt: two different databases, so it can't be atomic
17:52:31 jaypipes bauzas, cdent, dansmith: sorry for being absent today, guys :( been struggling with allergies all day. can't seem to shake em.
17:53:17 cdent jaypipes: hot baths and spicy foods
17:53:35 jaypipes cdent: yes, but not together.
17:53:46 cdent spicy foods in the hot tub!
17:54:03 dansmith jaypipes: s'okay, I'm allergic to florida too
17:54:12 jaypipes dansmith: ack
17:55:40 dims LOL
18:26:37 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
18:49:40 ildikov mriedem: hi
18:50:13 ildikov mriedem: patches are nice and green if you would happen to have time to review: https://review.openstack.org/#/q/topic:bp/cinder-new-attach-apis :)
19:18:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix error message when support matrix entry is missing a driver https://review.openstack.org/480708
19:22:22 openstackgerrit Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API https://review.openstack.org/449288
19:46:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
19:47:19 efried mordred https://review.openstack.org/#/c/479818 looks like exactly a case where your new stuff needs to be used.
19:47:42 mordred efried: yup. if you don't beat me to it - I'll write a patch
19:48:04 efried mordred And all the supporting code is merged at this point?
19:48:12 efried And in a ksa release that's percolated through g-r?
19:48:36 mordred efried: yes - merged. no - not released yet - need to respond to some reviews on the docs update patch
19:48:55 mordred efried: and I was hoping I could get the service-types support in
19:49:13 efried mordred Then the novaclient patch has to wait for that, neh?
19:49:50 mordred efried: yah - although if it's urgent we could likely talk to folks about cutting a release once the docs patch lands, then doing another one once service-types is in
20:33:51 openstackgerrit Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
22:57:04 openstackgerrit Michael Still proposed openstack/nova master: Only setup iptables for metadata if using nova-net https://review.openstack.org/480765
#openstack-nova - 2017-07-06
00:37:25 openstackgerrit Matt Riedemann proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
01:27:00 Kevin_Zheng ping mriedem
01:27:21 mriedem pong
01:29:50 Kevin_Zheng Hi, about tag creation
01:29:54 Kevin_Zheng http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/tag.py#n70
01:30:33 Kevin_Zheng it is a class method and takes ['tag1','tag2'] as param
01:30:44 Kevin_Zheng so I think it cannot be used directly
01:30:54 mriedem because we have a TagList,
01:31:01 mriedem so you'd have to convert the TagList to a list of simple tags
01:31:15 mriedem tags = [tag.tag for tag in tags]
01:31:18 mriedem something like that right?
01:31:26 Kevin_Zheng besides, build_request contains TagList.obj so I build taglist in API layer
01:31:27 Kevin_Zheng yes
01:32:32 mriedem i wonder if people would be ok with checking the type for the tags parameter in TagList.create
01:32:42 mriedem and if it's a TagList, convert it to a simple form
01:33:27 mriedem it's just a lot more efficient to do it that way in a single call if we have multiple tags for the server
01:34:00 mriedem because every separate tag create call does this extra _check_instance_exists_in_project db query
01:34:02 mriedem and context switch
01:34:28 Kevin_Zheng you mean do it in objects/tag.py?
01:34:53 mriedem well that would abstract the check which would be nice and clean, but people might not like that, like dansmith
01:35:25 mriedem otherwise we could just convert the TagList to a simple list in the conductor method
01:35:37 mriedem i think passing the TagList object from the compute api to conductor is still ok and what we want
01:36:24 Kevin_Zheng yeah, and we also need it in build_request
01:36:33 mriedem sure i think that's fine
01:36:50 mriedem i wish TagList just had a non-classmethod create() but that would conflict with the classmethod
01:37:13 mriedem we could add a TagList.create_all(context) or something
01:37:22 mriedem TagList.create_all(context, resource_id)
01:37:35 mriedem which would use the tags already stored in the TagList
01:37:48 mriedem create_all() is a confusing name when there is already a create() method
01:37:54 Kevin_Zheng yeah
01:38:07 mriedem i'm just kicking out ideas, i could talk to dan in the morning,
01:38:24 mriedem but i'd really like to do the create in the single call fashion rather than iterate in conductor manager
01:38:44 Kevin_Zheng OK, I will have a try and have all the other comments addressed
01:38:46 Kevin_Zheng first
01:38:53 mriedem the simplest way to do that today is just doing tags = [tag.tag for tag in tags] in conductor manager
01:39:13 mriedem we could always move ^ into TagList later if we wanted
01:39:27 Kevin_Zheng OK
01:42:51 openstackgerrit Matt Riedemann proposed openstack/nova master: api-ref: mark instance action events parameter as optional https://review.openstack.org/480792
02:03:05 openstackgerrit Merged openstack/nova master: Fix arguments in calling _delete_nic_metadata https://review.openstack.org/480369
02:08:20 openstackgerrit Merged openstack/nova master: Delete all inventory has its own method DELETE https://review.openstack.org/460147
02:10:23 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
03:15:46 openstackgerrit Ghanshyam Mann proposed openstack/nova-specs master: Update quota class spec to filter out networks quota https://review.openstack.org/480815
03:27:21 gmann alex_xu, mriedem: updated spec for filter out network quotas in os-quota-class - https://review.openstack.org/#/c/480815/1
03:27:33 alex_xu gmann: thanks
03:45:56 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
04:06:59 openstackgerrit zhangdaolong proposed openstack/python-novaclient master: Fix the inappropriate parameter name https://review.openstack.org/480826
04:33:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add 'updated_at' field to InstancePayload in notifications https://review.openstack.org/475276
05:04:52 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.49 - Virt device tagged attach https://review.openstack.org/480392
05:19:44 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
05:20:07 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
05:20:34 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
05:21:07 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
05:34:18 openstackgerrit zhangdaolong proposed openstack/nova master: Fix parameters and description for os-volume_attachments https://review.openstack.org/480832
05:35:28 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.50 - Enable cold migration with target host https://review.openstack.org/406707
05:53:17 openstackgerrit Moshe Levi proposed openstack/os-vif master: fix read the representor phys_port_name https://review.openstack.org/480213
05:54:43 reedip Hi guys, With Ceph as backend, and using cinder, when we attach a volume to an instance, the following error is visible in the compute's nova-compute.log
05:54:57 reedip libvirtError: Secret not found: rbd no secret matches uuid '------------------------------------'
05:55:07 reedip any ideas how to resolve it ?
05:55:20 reedip we have one ceph, one compute and one controller+network node
06:01:38 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix to return server groups quota in quota class set https://review.openstack.org/467999
06:08:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
06:09:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
06:41:13 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform keypair.import notification https://review.openstack.org/467514
06:45:54 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.50 - List/Show all server migration types https://review.openstack.org/430839
06:50:17 gaurangt Hi, I'm facing an issue where quota is not getting updated when the failed instance is deleted. Is there any work around for setting the right values for the quotas?
06:55:39 openstackgerrit Merged openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938

Earlier   Later