Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-07
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>
10:28:38 bauzas then, commit -a --amend and git review -y
10:28:53 hrw thx
10:28:58 bauzas hrw: so, here, just do git review -d 541728,4
10:29:06 bauzas it will get the PS4
10:29:28 bauzas and you'll just have to modify the support-matrix.ini file
10:30:35 bauzas (git review -d does the same than git fetch https://git.openstack.org/openstack/nova refs/changes// && git checkout FETCH_HEAD
10:31:06 openstackgerrit Marcin Juszkiewicz proposed openstack/nova master: Provide basic data for AArch64 support matrix/functionality https://review.openstack.org/541728
10:31:14 hrw everyday learns something new :)
10:31:17 hrw thx bauzas
10:31:23 bauzas np
10:32:02 bauzas stephenfin: look at https://bugs.launchpad.net/nova/+bug/1746188
10:32:03 openstack Launchpad bug 1746188 in OpenStack Compute (nova) "Virtlogd recreates console.log file as root:root after live migration" [Undecided,New]
10:34:59 stephenfin bauzas: I thought we'd fixed that
10:35:20 bauzas yeah me too
10:35:37 bauzas but the reporter used a Pike version
10:42:47 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4) https://review.openstack.org/511183
10:42:47 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642
10:42:48 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644
10:42:48 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643
10:42:49 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
10:42:49 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
10:42:50 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: [WIP] Get resource provider by uuid or name https://review.openstack.org/527791
10:47:35 sticker bauzas: that reporter is me if you need any more info

Earlier   Later