| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-31 | |||
| 12:51:32 | openstackgerrit | Merged openstack/nova stable/ocata: Handle exception on adding secgroup https://review.openstack.org/515421 | |
| 12:51:40 | openstackgerrit | Merged openstack/nova stable/pike: Don't persist could-be-stale InstanceGroup fields in RequestSpec https://review.openstack.org/528225 | |
| 12:51:48 | openstackgerrit | Merged openstack/nova stable/ocata: Don't persist could-be-stale InstanceGroup fields in RequestSpec https://review.openstack.org/528226 | |
| 13:02:22 | openstackgerrit | Lajos Katona proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728 | |
| 13:03:18 | openstackgerrit | Chris Dent proposed openstack/nova master: Remove microversion fallback code from report client https://review.openstack.org/528794 | |
| 13:07:01 | openstackgerrit | Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385 | |
| 13:30:37 | openstackgerrit | Merged openstack/nova stable/pike: Bumping functional test job timeouts https://review.openstack.org/539319 | |
| 13:30:45 | openstackgerrit | Merged openstack/nova master: Don't launch guestfs in a thread pool if guestfs.debug is enabled https://review.openstack.org/526748 | |
| 13:30:56 | openstackgerrit | Merged openstack/nova master: Don't log a warning for InstanceNotFound in detach_interface https://review.openstack.org/530969 | |
| 13:31:07 | openstackgerrit | Merged openstack/nova stable/ocata: Make sure that functional test triggered on sample changes https://review.openstack.org/536936 | |
| 13:41:16 | openstackgerrit | Merged openstack/nova stable/ocata: Bumping functional test job timeouts https://review.openstack.org/539320 | |
| 13:42:36 | openstackgerrit | Dan Smith proposed openstack/nova master: Pass limit to /allocation_requests https://review.openstack.org/531517 | |
| 14:01:08 | zigo | I got issue with Pike's console-log: | |
| 14:01:11 | zigo | # nova console-log 92847293-3b4d-44cc-8cdd-6cb57e23b0cf | |
| 14:01:11 | zigo | ERROR (UnicodeEncodeError): 'ascii' codec can't encode characters in position 22096-22098: ordinal not in range(128) | |
| 14:01:14 | zigo | WTF ? :) | |
| 14:01:18 | zigo | bauzas: ^ | |
| 14:04:49 | efried | zigo: Is there a stack trace? | |
| 14:04:58 | zigo | edleafe: Let me check. | |
| 14:06:02 | zigo | edleafe: No stack trace in the logs. | |
| 14:07:09 | lyarwood | zigo: can you run the nova command again with --debug and pastebin the output? | |
| 14:07:27 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Don't launch guestfs in a thread pool if guestfs.debug is enabled https://review.openstack.org/539541 | |
| 14:07:58 | openstackgerrit | Merged openstack/nova master: Fix the order of target host checks https://review.openstack.org/526225 | |
| 14:08:18 | zigo | lyarwood: http://paste.openstack.org/show/658151/ | |
| 14:08:19 | openstackgerrit | Merged openstack/nova master: Fix missing 'if_notifications_enabled' decorator https://review.openstack.org/537745 | |
| 14:09:19 | jroll | zigo: definitely a client issue, py2 or py3? | |
| 14:09:35 | zigo | jroll: py2 | |
| 14:09:45 | zigo | I got the same issue with openstackclient | |
| 14:10:01 | jroll | probably just need a .decode('utf8') or something there | |
| 14:10:24 | zigo | Note that this is Debian Unstable ... | |
| 14:10:31 | zigo | Because I can ... (tm) | |
| 14:10:32 | zigo | :) | |
| 14:10:52 | zigo | So, something bleeding edge may be the cause. | |
| 14:10:58 | zigo | To be fixed never the less. | |
| 14:11:12 | jroll | regardless, it's a unicode handling issue in the client, not a library issue | |
| 14:11:12 | lyarwood | yup, looks valid, can you create a bug? | |
| 14:11:25 | bauzas | zigo: strange thing | |
| 14:11:56 | zigo | same stuff with openstackclient: http://paste.openstack.org/show/658154/ | |
| 14:12:01 | zigo | Will do. | |
| 14:12:37 | jaypipes | efried: hey, sorry, was afk yesterday evening... you were asking me something about PUT vs DELETE? | |
| 14:13:14 | bauzas | zigo: any trace we could look at ? | |
| 14:13:21 | bauzas | looks a novaclient issue to me | |
| 14:13:25 | zigo | bauzas: Yeah, just pasted 2 of them. | |
| 14:13:47 | bauzas | since openstackclient uses novaclient, I guess it's from here | |
| 14:13:51 | efried | jaypipes: Yeah, basically is there any reason to prefer a {DELETE <URI> => 204 no content} API over its corresponding {PUT <URI> => 200 <payload>} | |
| 14:13:53 | zigo | jaypipes: Good hey mate ! | |
| 14:15:10 | efried | jaypipes: Specifically for RP inventory, traits, etc. | |
| 14:15:40 | bauzas | zigo: python 2 or 3 ? | |
| 14:15:42 | openstackgerrit | Merged openstack/nova master: Add get_traits() method to ComputeDriver https://review.openstack.org/532290 | |
| 14:15:57 | openstackgerrit | Merged openstack/nova master: libvirt: test to make sure volume_use_multipath is properly used https://review.openstack.org/532270 | |
| 14:16:08 | openstackgerrit | Merged openstack/nova master: libvirt: add tests to check multipath in iscsi/fc volume connectors https://review.openstack.org/532272 | |
| 14:18:34 | zigo | bauzas: py2 | |
| 14:20:50 | ameeda | hello, I am try to find new bugs to resolve, I found this bug review https://review.openstack.org/#/c/212908/ . I think if I create new table with unique constraints then copy the data from old table to new one , so we clear the dirty data. | |
| 14:23:46 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: [dnm] Make sure that we have USB Host controller for tablet https://review.openstack.org/538003 | |
| 14:24:00 | hrw | morning | |
| 14:25:07 | mriedem | ameeda: nova hasn't used quota usages since pike | |
| 14:25:35 | mriedem | ameeda: https://specs.openstack.org/openstack/nova-specs/specs/pike/implemented/cells-count-resources-to-check-quota-in-api.html | |
| 14:25:53 | hrw | mriedem: can you take a look at https://review.openstack.org/#/c/538003/ patch and tell is it acceptable? | |
| 14:25:53 | mriedem | i marked the bug as won't fix | |
| 14:26:10 | ameeda | mriedem: Thank you! | |
| 14:26:26 | mriedem | hrw: should probably have sahid look at that, he worked on the pointer model stuff in the libvirt driver | |
| 14:26:30 | ameeda | mriedem: can you please recheck this https://review.openstack.org/#/c/528385/ | |
| 14:26:55 | hrw | mriedem: will add him, thanks | |
| 14:27:57 | mriedem | ameeda: can you please decouple that patch from the one below it in the series? | |
| 14:28:24 | mriedem | because melwitt's patch https://review.openstack.org/#/c/340614/ is already dealing with the api change (the first patch in your series) | |
| 14:29:00 | ameeda | how can I do that ? | |
| 14:29:45 | ccamacho | hey guys! quick question, Im working with the tripleo upgrades squad and Im trying to analyze the Nova database status prior an upgrade, mostly to run cleanup operation before running an upgrade in nova services. | |
| 14:29:45 | mriedem | put them in separate branches | |
| 14:29:56 | ccamacho | do you know briefly where can I find information about it? | |
| 14:30:16 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: remove unnecessary conf imports https://review.openstack.org/539314 | |
| 14:30:24 | mriedem | ccamacho: find information about what specifically? | |
| 14:30:38 | mriedem | ccamacho: there are a few nova dbs | |
| 14:30:48 | mriedem | nova_api, nova_cell0, nova_cell1 (or just "nova") | |
| 14:31:16 | ameeda | mriedem: I should remove this from my branch https://review.openstack.org/#/c/528069 ? or create new branch for this https://review.openstack.org/#/c/528385/ and resubmit the review with new change id ? | |
| 14:31:17 | ccamacho | mriedan hi, basically about what to check to clean the databases | |
| 14:31:41 | mriedem | ccamacho: like how to archive anything that's already in it but deleted? | |
| 14:31:48 | mriedem | nova-manage db archive_deleted_rows | |
| 14:31:55 | mriedem | will move soft deleted records into the shadow tables | |
| 14:32:09 | mriedem | soft deleted in this case means records where the deleted column != 0 | |
| 14:32:33 | mriedem | we do'nt have a straight up db purge command | |
| 14:32:42 | openstackgerrit | Merged openstack/nova master: trivial: use cn instead of rp https://review.openstack.org/529786 | |
| 14:32:53 | openstackgerrit | Merged openstack/nova master: Remove the inherits parameter for the Resource object https://review.openstack.org/529218 | |
| 14:32:58 | mriedem | ccamacho: https://docs.openstack.org/nova/latest/cli/nova-manage.html#nova-database | |
| 14:32:59 | ccamacho | mrieden oki but that is the only cmd available to do it in a more or less automated way | |
| 14:33:07 | ccamacho | mrieden thanks for the link | |
| 14:33:37 | ccamacho | mrieden are the shadow tables there for ever? | |
| 14:33:39 | mriedem | there has been a spec and code up before for a command to do similar to archive_deleted_rows but actually hard delete the records rather than move them | |
| 14:33:45 | mriedem | ccamacho: yeah | |
| 14:33:52 | mriedem | so there is still bloat in the db for deleted stuff | |
| 14:34:49 | mriedem | https://blueprints.launchpad.net/nova/+spec/purge-deleted-instances-cmd | |
| 14:35:36 | mriedem | https://blueprints.launchpad.net/nova/+spec/purge-deleted-instances-cmd | |
| 14:35:38 | mriedem | oops | |
| 14:35:42 | mriedem | this is the spec https://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/purge-deleted-instances-cmd.html | |
| 14:35:57 | mriedem | and a poc from someone at red hat https://review.openstack.org/#/c/330791/ | |
| 14:36:15 | ccamacho | mrieden thanks, the idea is to write +- a procedure to verify and clean the DBs before the upgrade and be sure they are cleaned.. hopefully we can try to implement it after | |
| 14:36:18 | ccamacho | thanks man | |
| 14:36:42 | ameeda | mriedem: I should remove this from my branch https://review.openstack.org/#/c/528069 ? or create new branch for this https://review.openstack.org/#/c/528385/ and resubmit the review with new change id ? | |
| 14:36:56 | mriedem | ameeda: i think you should abandon https://review.openstack.org/#/c/528069 | |
| 14:37:14 | mriedem | ameeda: to do that locally, rebase your series and drop that change from the local branch, | |
| 14:37:18 | mriedem | git rebase -i master | |