Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-04-03
20:08:41 mriedem ok so we're a go for removing location header and return image_id in a response body dict in a microversion, yes?
20:08:45 mriedem if so, i'll start writing up a spec
20:08:54 raj_singh mriedem: sdague We are working up a spec to use catalog lookup for Glance. This spec will have details on making consistent use of keystone_auth in Neutron and Cinder client as well.
20:09:05 mordred mriedem: ++ I think that's a GREAT microversion
20:09:07 raj_singh and we will have it up by today
20:09:30 mordred raj_singh: hey - will you point me to it as well?
20:09:37 raj_singh pumaranikar: is working on it
20:09:49 mordred pumaranikar, raj_singh: I have some thoughts and a half-done patch for improving that area as well :)
20:09:53 raj_singh mriedem: Sure I will ping you once ready
20:10:09 mordred sweet
20:10:16 sdague mriedem: sounds great
20:31:08 melwitt mriedem: I thought you might want to look at this docs change, makes a new page for release notes how-to and links from the main page and the code review page https://review.openstack.org/#/c/451870
20:33:06 mriedem sdague: mordred: funny, but we emit notifications for snapshot start/end, but don't actually include the snapshot image id in those
20:33:11 mriedem kind of useless
20:33:30 mriedem gibi: ^
20:33:35 sdague so, question, in log messages, sometimes we use instance= and sometimes we use instance_uuid=
20:33:43 mriedem sdague: no different
20:33:45 mriedem *difference
20:33:56 sdague is there a reason we couldn't abolish instance= and do the uuid dereference thre?
20:33:56 mriedem oslo.log checks to see if it's 'instance' and pulls the uuid from it
20:34:13 sdague mriedem: yes, I realize it, but it actually adds a reasonable amount of gorp in oslo.log
20:34:15 mriedem it would be a bunch of churn
20:35:51 sdague yeh, though the dual fields do make doing some additional stuff with exposing variables to systemd journal a big gorpier
20:36:53 mriedem melwitt: can i have no opinion?
20:37:07 melwitt mriedem: of course
20:49:42 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP encryptors: Do not overwrite file volumes on attach https://review.openstack.org/452938
20:49:54 lyarwood melwitt: ^ hackaround for the encrypted nfs volume corruption bug if you get time today
20:50:41 melwitt lyarwood: cool, thanks for the heads up
20:50:52 lyarwood mriedem: thanks for the heads up re the stable bugfixes btw, I'll queue them up for the morning
20:51:27 mriedem mordred: createBackup API has the same issue :)
20:54:33 dansmith sdague: instance_uuid would suck measurably more, IMHO
21:02:05 openstackgerrit Anusha Unnam proposed openstack/nova master: Add description to policies in servers.py https://review.openstack.org/451966
21:04:25 mriedem wow, nova backup does not print out the actual image id for the created snapshot image
21:04:31 mriedem so i guess that's just an exercise for the user
21:04:53 mriedem melwitt: ^ have you ever seen anyone propose a change for that in novaclient?
21:05:36 melwitt mriedem: I haven't but to be honest I haven't been paying attention to novaclient changes in awhile
21:07:40 dansmith mriedem: backup is broken for certain things, right? like bfv instances?
21:07:47 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Remove Location header from createImage and createBackup response (spec) https://review.openstack.org/452951
21:07:55 dansmith tbh I kinda relegated it into an old usage scenario
21:08:12 mriedem flwang tried fixing bfv with backup years ago, but we blocked it
21:08:27 dansmith yeah
21:10:09 openstackgerrit Sarafraj Singh proposed openstack/nova-specs master: WIP: Replace live migration steps and delays with memory iteration logic https://review.openstack.org/443339
21:10:58 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Remove Location header from createImage and createBackup response (spec) https://review.openstack.org/452951
21:11:08 mriedem mordred: sdague: ^ the spec
21:11:49 mriedem dansmith: i bet we don't mention that limitation in the api ref for createBackup either
21:12:21 dansmith I kinda wish we didn't mention it at all :)
21:12:30 mriedem like the os-pci API?
21:12:34 mriedem which reminds me...
21:13:44 mriedem nice, this is the message you get back with the 400 in that case, "The request is invalid."
21:14:00 mriedem it should also kick you in the balls while it's being helpful
21:14:34 dansmith it logs something relevant, but yeah, no help for the user
21:15:20 mriedem logs something for the admin to check when he gets the support ticket
21:19:44 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove unused migrate_data kwarg from virt driver destroy() method https://review.openstack.org/452958
22:02:36 mordred mriedem: awesome
22:03:25 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: Separate the releasenotes guide from the code-review section https://review.openstack.org/451870
22:04:25 mordred mriedem: the impacts are awesome
22:04:33 mriedem None?
22:06:41 openstackgerrit Matt Riedemann proposed openstack/nova master: Document and provide useful error message for volume-backed backup https://review.openstack.org/452970
22:08:31 openstackgerrit Matt Riedemann proposed openstack/nova master: api-ref: move createBackup to server-actions https://review.openstack.org/452971
22:16:19 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: List instances using Searchlight (spec) https://review.openstack.org/441692
22:18:22 dtp nova/objects/service.py/Service.obj_make_compatible_from_manifest doesn't seem to be called from anywhere within nova; is it still used?
22:19:16 mriedem dtp: i believe that's used in the oslo.versionedobjects library
22:19:21 mriedem so it's probably an override in nova
22:19:42 dansmith mriedem: aye
22:19:42 mriedem https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L527
22:19:56 dtp roger; thanks
#openstack-nova - 2017-04-04
04:48:29 vks1 ConfigFileValueError: Value for option instances is not valid: invalid literal for int() with base 10 . Getting this error. I have set nova.conf like this:
04:48:58 vks1 [quota]
04:48:59 vks1 instances=-1
04:48:59 vks1 #cores=50
04:48:59 vks1 ram=-1
04:48:59 vks1 gigabytes=-1
06:25:57 openstackgerrit Ghanshyam Mann proposed openstack/nova master: api-ref: fix os-extended-volumes:volumes_attached in servers responses https://review.openstack.org/449809
06:25:57 openstackgerrit Ghanshyam Mann proposed openstack/nova master: ix api-ref for create servers response https://review.openstack.org/451673
06:27:18 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix api-ref for create servers response https://review.openstack.org/451673
06:27:30 gmann oomichi: updated https://review.openstack.org/#/c/451673/
06:39:09 openstackgerrit Dan Peschman proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025
06:42:38 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add a removeFloatingIp example https://review.openstack.org/453026
06:45:10 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237
06:45:20 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Specify keymap on server boot https://review.openstack.org/448830
06:45:35 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
06:58:41 openstackgerrit melanie witt proposed openstack/nova-specs master: Add project/user association to placement https://review.openstack.org/453030
07:09:45 openstackgerrit Sergey Nikitin proposed openstack/nova-specs master: TinyFix: Changed type of CPU ID from 'String' to 'Integer' in diagnostics spec https://review.openstack.org/445855
07:13:52 snikitin_ johnthetubaguy, jaypipes: Hi guys! You approved my little fix in diagnostics spec but I had to reupload it because of pep8 error. Could you please fast reapprove it? https://review.openstack.org/#/c/445855
07:17:25 openstackgerrit Sihan Wang proposed openstack/nova master: Show quota detail when inject file quota exceeds https://review.openstack.org/453040
07:23:30 openstackgerrit melanie witt proposed openstack/nova-specs master: Convert consoles code to use objects framework https://review.openstack.org/453041
07:27:24 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api sample tests for security group rules https://review.openstack.org/452055
07:28:46 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api sample tests for security group rules https://review.openstack.org/452055
08:22:17 lyarwood mdbooth: https://review.openstack.org/#/c/452938/ - hackaround for an encrypted nfs volume corruption bug if you get time today
08:30:02 johnthetubaguy sfinucan: do you have context on this, I have some dumb questions: https://review.openstack.org/#/c/361140
08:33:34 sfinucan johnthetubaguy: looking
08:33:56 sfinucan johnthetubaguy: Sure do. What's up?
08:34:32 johnthetubaguy sfinucan: so when I read the problem, I assumed I would see the reverse solution, let me try sketch that out
08:35:05 johnthetubaguy sfinucan: PCI whitelist tells Nova to only prefer NUMA affinity, because all the PCI devices are only on one NUMA node
08:35:22 johnthetubaguy that way the user doesn't need to worry themselves
08:36:27 johnthetubaguy sfinucan: I am guessing there is a use case that doesn't work for, but I am not seeing it myself
08:36:37 sfinucan johnthetubaguy: Is that a statement or a question? :)
08:36:59 sfinucan the PCI whitelist bit, that is
08:37:24 johnthetubaguy sfinucan: bit of both really, lol
08:37:39 johnthetubaguy snikitin_ I think this is your spec I am look at, if you are about ^

Earlier   Later