Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-07
22:51:01 dansmith melwitt: this isn't for a lazy-load though right?
22:51:25 dansmith oh, we're actually hitting it via lazy load I see from the comment
22:51:29 dansmith er, commit message
22:51:35 melwitt dansmith: I *think* the way this is happening during scheduling is that this line causes a lazy-load and that's how we get the host list for group members https://github.com/openstack/nova/blob/master/nova/scheduler/utils.py#L700
22:52:00 dansmith so yeah, we probably need to not let that happen (i.e. make sure we pull the hosts early, or manually fill the group objects from the list with some cells-aware helper
22:52:05 melwitt so I was thinking we need to replace that with a proper scatter-gather calling get_hosts for all cells except cell0 or something?
22:52:10 melwitt yeah
22:52:15 dansmith melwitt: there should be a debug log about it if there is
22:52:48 dansmith melwitt: yeah, a check before that and a call to a dedicated fill routine would be easiest and cleanest I think
22:52:56 melwitt there actually isn't because InstanceGroup override obj_load_attr and doesn't log a message about it, I think. /re-checks
22:53:00 dansmith alternatively a non-remotable helper, but that really blurs the line more which isn't great
22:53:05 dansmith ah
22:53:08 dansmith well that sucks :)
22:53:15 melwitt yeah, srsly
22:53:16 dansmith we should fix that too
22:53:35 melwitt https://github.com/openstack/nova/blob/master/nova/objects/instance_group.py#L294
22:53:36 melwitt yeah
22:54:01 melwitt it was fun times chasing that down
22:54:26 dansmith we probably have more of those I'd guess, but yeah
22:54:29 dansmith anyway,
22:54:48 dansmith you know the path now yes?
22:54:54 melwitt yeah
22:55:14 dansmith suh-weet
23:02:11 cfriesen when instances query the metadata server, what code adds the webob.headers.EnvironHeaders to the request? (I'm trying to figure out how the mapping to instance_id is done)
23:04:49 openstackgerrit melanie witt proposed openstack/nova master: Refine waiting for vif plug events during _hard_reboot https://review.openstack.org/541442
#openstack-nova - 2018-02-08
00:24:12 jroll melwitt: congrats and condolences
00:24:14 jroll :)
00:24:30 melwitt heh, thanks jroll
00:27:51 mriedem gibi: i likely can't run the nova meeting tomorrow, can you do that? i'll update the agenda tonight.
00:30:26 mnaser melwitt: early unofficial congrats too :)
00:30:40 melwitt thanks mnaser :)
00:43:23 mriedem efried: apparently we lookup the image service endpoint url from the catalog 410 times in n-api alone in a successful tempest run
00:43:28 mriedem which seems....excessive
00:48:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Workaround glanceclient bug when CONF.glance.api_servers not set https://review.openstack.org/541008
00:49:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells https://review.openstack.org/521400
01:15:59 efried mriedem: Accessing an image 410 times in a tempest run doesn't seem excessive to me. And if things are set up correctly, that lookup ought to be really really cheap.
01:16:28 mriedem we just reconstruct the glance client wrapper every time
01:16:36 efried Now if we were doing endpoint discovery every one of those times (or even a tenth of them) that would be excessive.
01:16:37 mriedem i'm pretty sure we do the same for cinder client though too
01:16:45 efried I don't doubt it.
01:17:05 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove unnecessary variables https://review.openstack.org/541964
01:17:07 efried Are you suggesting we could cache the client? Maybe so.
01:17:29 efried FWIW, constructing the client is also pretty cheap.
01:17:45 mriedem i was more worried about the endpoint lookup from ksa every time,
01:17:50 mriedem but is that cached somewhere?
01:18:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Workaround glanceclient bug when CONF.glance.api_servers not set https://review.openstack.org/541008
01:18:52 mriedem btw, this fixes the nova-next job ^, despite using some regex parsing and stripping
01:18:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove unnecessary arguments in notification methods https://review.openstack.org/535599
01:20:46 efried mriedem: Yes, the endpoint is cached, if things are working right.
01:21:30 efried oh, you did that, thanks.
01:27:06 openstackgerrit melanie witt proposed openstack/nova master: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/540258
01:29:32 openstackgerrit Merged openstack/nova master: Update links in documents https://review.openstack.org/533134
02:04:33 mriedem mordred: would you like it if the GET /flavors/detail response included the extra specs with each flavor also so you don't have to make a 2nd request per flavor to get the specs (as a subresource GET) ?
02:42:44 mriedem sgordon: is red hat still interested in this? https://blueprints.launchpad.net/nova/+spec/complex-soft-anti-affinity-policies
02:42:51 mriedem because i think our product team wants that also
02:58:30 mriedem Kevin_Zheng: this is the bp for the error code thing I was thinking of for placement https://review.openstack.org/#/c/418393/
03:01:01 mriedem Kevin_Zheng: https://blueprints.launchpad.net/nova/+spec/add-extra-specs-to-flavor-list
03:01:04 mriedem yikun: ^
03:02:02 mriedem https://review.openstack.org/#/c/39920/
03:02:08 mriedem apparently people weren't against it 4 years ago
03:02:09 mriedem :)
03:05:34 Kevin_Zheng yeah
03:05:36 Kevin_Zheng more about v2 v3 thing
03:07:07 mriedem this was the other bp i was looking for from ibm years ago https://blueprints.launchpad.net/nova/+spec/formal-message-ids
03:08:02 Kevin_Zheng Hmm got approved :)
03:08:13 mriedem ?
03:08:17 mriedem it's marked obsolete
03:08:39 Kevin_Zheng Oh ok
03:08:59 mriedem it's complete in that it's dead
03:27:12 openstackgerrit Fan Zhang proposed openstack/osc-placement master: Update doc link in README.rst https://review.openstack.org/542002
04:18:43 openstackgerrit Fan Zhang proposed openstack/osc-placement master: Update doc link in README.rst https://review.openstack.org/542002
04:30:12 openstackgerrit Ghanshyam Mann proposed openstack/nova master: trivial: Fix microversion number in test comment https://review.openstack.org/542019
06:13:39 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/541561
07:01:17 openstackgerrit Merged openstack/nova master: fix misspelling of 'projectUser' https://review.openstack.org/541108
07:52:08 openstackgerrit Ameed Ashour proposed openstack/osc-placement master: Update doc link in README.rst https://review.openstack.org/542067
08:11:28 hrw morning
08:11:50 hrw https://review.openstack.org/#/c/541728/ - can someone take a look at aarch64 support matrix stuff and help get it merged?
08:23:26 openstackgerrit Thomas Goirand proposed openstack/python-novaclient master: Fix crashing console-log https://review.openstack.org/541610
08:52:40 openstackgerrit jichenjc proposed openstack/nova master: TrivialFix: Add a blankline https://review.openstack.org/542094
08:58:26 openstackgerrit jichenjc proposed openstack/nova master: TrivialFix: rename cinder to volume https://review.openstack.org/542095
09:03:01 bauzas good morning Nova
09:03:05 hrw hi bauzas
09:17:32 openstackgerrit jichenjc proposed openstack/nova master: Add log for snapshot an instance https://review.openstack.org/542101
09:24:52 openstackgerrit jichenjc proposed openstack/nova master: Use with method to consistent oslo timeutils usage https://review.openstack.org/542105
09:36:14 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove a duplicate colon https://review.openstack.org/542109
10:13:25 openstackgerrit Marcin Juszkiewicz proposed openstack/nova master: Provide basic data for AArch64 support matrix/functionality https://review.openstack.org/541728
10:13:39 hrw bauzas: marked live migration as missing
10:15:48 bauzas hrw: you tramplet my revision
10:15:57 bauzas trampled
10:16:06 hrw sorry, will refix
10:16:13 bauzas hrw: https://review.openstack.org/#/c/541728/4..5/releasenotes/notes/aarch64-minimum-libvirt-version-86331e5282effbf0.yaml
10:16:27 bauzas hrw: np, just FYI because Zuul will NACK
10:17:43 openstackgerrit Marcin Juszkiewicz proposed openstack/nova master: Provide basic data for AArch64 support matrix/functionality https://review.openstack.org/541728
10:17:46 hrw restored
10:18:37 openstackgerrit Marcin Juszkiewicz proposed openstack/nova master: Provide basic data for AArch64 support matrix/functionality https://review.openstack.org/541728
10:18:39 hrw typo ;(
10:24:51 gibi mriedem: sure, I will do the weekly meeting
10:27:58 bauzas hrw: again, you will have a problem https://review.openstack.org/#/c/541728/4..7/releasenotes/notes/aarch64-minimum-libvirt-version-86331e5282effbf0.yaml
10:28:15 hrw s...
10:28:23 bauzas hrw: the simpliest thing for passing a new rev is just to do git review -d <change_number>

Earlier   Later