| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-25 | |||
| 21:20:22 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM: update_provider_tree() (compatible) https://review.openstack.org/468560 | |
| 21:21:15 | efried | jaypipes: trivial update on rebase -^ | |
| 21:22:53 | aspiers | jaypipes, sean-k-mooney, mriedem, artom: I'm (perhaps naively) hoping that this can be the final patchset of https://review.openstack.org/#/c/609779/ ;-) | |
| 21:23:13 | artom | aspiers, heh, I don't have +2 powers | |
| 21:23:25 | aspiers | artom: that's OK, a +1 is welcome too ;-) | |
| 21:23:26 | artom | I've been seeing all the activity and been meaning to look it over | |
| 21:23:45 | artom | I'll try *real hard this time* to actually get to it ;) | |
| 21:24:49 | aspiers | Thanks :) No rush. Just wanted to point out my hope that I've addressed enough of the feedback that it should be stabilising now and the flurry of activity should slow right down | |
| 21:46:25 | efried | mriedem: https://blueprints.launchpad.net/nova/+spec/run-meta-api-per-cell lgtm, except I would like to see the actual config option section/name in that text. | |
| 21:46:39 | mriedem | i don't have it yet... | |
| 21:46:41 | efried | since it's taking the place of a spec, which would surely have that. | |
| 21:47:02 | mriedem | CONF.api.local_meta_per_cell? | |
| 21:51:38 | openstackgerrit | Merged openstack/nova master: Make CellDatabases fixture reentrant https://review.openstack.org/611665 | |
| 21:51:54 | efried | metadata_host and metadata_port are in the [DEFAULT] section. (Not arguing anything yet, just collating data) | |
| 21:52:38 | efried | other metadata_* options too ^ | |
| 21:53:11 | efried | but metadata_cache_expiration is in [api] | |
| 21:55:53 | efried | mriedem: [api] section works; [cells] is another possibility. | |
| 21:56:36 | efried | mriedem: I would spell out 'metadata' (local_metadata_per_cell). | |
| 21:56:59 | efried | and that's about all the opinion I've got on the matter. I defer to someone who actually understands the feature. | |
| 21:57:26 | melwitt | fwiw, I agree to use 'metadata' instead of just meta | |
| 21:58:11 | melwitt | I notice we have other cells related options in [api] so I think that's fine | |
| 21:58:26 | mriedem | efried: we don't use [cells] anymore since that's cells v1 | |
| 21:58:32 | efried | okey | |
| 21:58:40 | mriedem | https://docs.openstack.org/nova/latest/configuration/config.html#cells | |
| 21:58:42 | efried | The [api] section says it's for the nova API | |
| 21:58:44 | mriedem | DEPRECATED: Cells options allow you to use cells v1 functionality in an OpenStack deployment. | |
| 21:58:53 | mriedem | metadata api can be run from the nova api service | |
| 21:58:54 | efried | but that may implicitly incorporate the metadata API. | |
| 21:58:59 | efried | Yeah, that. | |
| 21:59:40 | melwitt | yeah, the all-in-one metadata API does from nova api service | |
| 22:00:15 | efried | so an aside, should we be moving [DEFAULT]metadata_* into [api]? | |
| 22:00:31 | efried | or... creating a [metadata_api] section? | |
| 22:00:49 | mriedem | probably the former, but moving 100 options every release gets tiresome | |
| 22:00:53 | mriedem | both for us and i'm sure for deployers | |
| 22:01:28 | efried | k, shrug, I've got no skin in the game. | |
| #openstack-nova - 2018-10-26 | |||
| 00:30:22 | openstackgerrit | Merged openstack/osc-placement stable/queens: Do not depend on jenkins user in devstack gate https://review.openstack.org/556666 | |
| 01:02:38 | openstack | bug 1799727 in OpenStack Compute (nova) "CPU_Allocation_Ratio from nova.conf doesn't update exisiting providers" [High,In progress] https://launchpad.net/bugs/1799727 - Assigned to Matt Riedemann (mriedem) | |
| 01:02:38 | openstackgerrit | Merged openstack/nova master: Add functional recreate test for bug 1799727 https://review.openstack.org/613115 | |
| 01:05:16 | sean-k-mooney | melwitt: sorry i needed to go lie down for a bit as i was feeling off (just a headache) | |
| 01:05:30 | sean-k-mooney | melwitt: i commented on the os-vif release patch | |
| 01:06:19 | sean-k-mooney | i think its fine but there are two patches i would like to incude in the release if we have time or do a 1.12.1 release before the end of the year if they need future work before november 1st | |
| 01:08:23 | sean-k-mooney | efried: ill review your provider config yaml spec https://review.openstack.org/#/c/612497 tomorow if i can. i have abandon mine in favor of your for this release | |
| 01:08:58 | sean-k-mooney | but with that siad im going to go back to sleep talk to everyone in the morning o/ | |
| 01:38:29 | cervigni | hello everyone, we have a very odd problem in our environment. lauching an instance stucks on scheduling / building. But if I launch more than one from web interface, the first instance always fails, while the other instances (other than the first one) always succeed | |
| 01:38:42 | cervigni | any idea where I should start looking? I though it was a placement problem but no errors whatsoever | |
| 01:38:42 | openstackgerrit | Yongli He proposed openstack/nova-specs master: add 'show-server-group' spec https://review.openstack.org/612255 | |
| 01:38:43 | openstackgerrit | Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256 | |
| 01:51:56 | openstackgerrit | Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256 | |
| 02:06:47 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Fixes Python3 issue in decoding password https://review.openstack.org/604870 | |
| 02:13:05 | openstackgerrit | Yongli He proposed openstack/nova-specs master: add 'show-server-group' spec https://review.openstack.org/612255 | |
| 02:13:06 | openstackgerrit | Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.openstack.org/612256 | |
| 02:34:24 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add compute_node ratio online data migration https://review.openstack.org/609995 | |
| 03:52:00 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499 | |
| 04:34:56 | openstackgerrit | melanie witt proposed openstack/nova master: Add nova-status upgrade check for consoles https://review.openstack.org/611214 | |
| 07:00:58 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Use new ``initial_xxx_allocation_ratio`` CONF https://review.openstack.org/602804 | |
| 07:00:59 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499 | |
| 07:06:03 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add compute_node ratio online data migration https://review.openstack.org/609995 | |
| 07:42:20 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Consider root id is None in the database case https://review.openstack.org/613305 | |
| 07:42:20 | openstack | bug 1799892 in OpenStack Compute (nova) "Placement API crashes with 500s in Rocky upgrade with downed compute nodes" [Undecided,New] https://launchpad.net/bugs/1799892 | |
| 07:42:20 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add recreate test for bug 1799892 https://review.openstack.org/613304 | |
| 07:46:38 | openstackgerrit | mashuting proposed openstack/nova master: VMware: Support volumes backed by VStorageObject https://review.openstack.org/521690 | |
| 08:52:04 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Integration tests using gabbi-tempest https://review.openstack.org/613386 | |
| 09:19:50 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499 | |
| 10:49:14 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Integration tests using gabbi-tempest https://review.openstack.org/613386 | |
| 10:52:43 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Integration tests using gabbi-tempest https://review.openstack.org/613386 | |
| 10:53:59 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add compute_node ratio online data migration script https://review.openstack.org/609995 | |
| 10:53:59 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node https://review.openstack.org/613499 | |
| 12:39:10 | jaypipes | aspiers: +2 from me. | |
| 13:02:48 | jaypipes | efried: done | |
| 13:18:51 | mpasserini | Hi, what is the service which has the task of creating the ephemeral volumes on the compute nodes? Nova-compute? | |
| 13:34:56 | mriedem | dansmith: i'd ping tssurya, but she's not around and it seems you're updating the down cell series. i've gone up through https://review.openstack.org/#/c/584829/ | |
| 13:35:14 | dansmith | mriedem: just going through my gerrit mail right now | |
| 13:35:37 | mriedem | i've got comments/questions in the changes with +1s too, so you could look at those as well. like wondering about applying limits when listing from down cells. | |
| 13:35:43 | mriedem | seems we could do that, but we don't. | |
| 13:36:11 | dansmith | roger | |
| 13:48:39 | mriedem | cdent: requests.GET question for you, if i have a call like GET /servers?all_tenants=1&project_id=1&project_id=2, does that result in {'project_id': [1,2]} ? | |
| 13:49:00 | mriedem | with the all_tenants key in the dict as well, but i care about the double project_id | |
| 13:49:37 | cdent | mriedem: I think it depends on how you are making the call. let me refresh my memory | |
| 13:50:23 | cdent | mriedem: looks like you need getall to get what you want: https://docs.pylonsproject.org/projects/webob/en/stable/reference.html#query-post-variables | |
| 13:51:03 | cdent | GET() will get you both, in tuples | |
| 13:51:44 | cdent | GET[key] gets the second, getall(key) gets both | |
| 13:51:50 | mriedem | ok i forgot about getall(), i think i found the list of tuples while digging through requests code yesterday | |
| 13:53:20 | mriedem | ok and looking at the list servers code, the only param we do getall on is 'status' so you can filter the list of servers on multiple status values, anything else would just be whatever the last entry is in the query string | |
| 13:53:30 | mriedem | we don't document that of course, but that's what i'm seeing | |
| 13:53:52 | mriedem | i just wasn't sure how search_opts.update(req.GET) would work | |
| 13:55:54 | cdent | there are lots of things about the nova-api that are "unique" | |
| 14:09:37 | openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Standardize CPU resource tracking https://review.openstack.org/555081 | |
| 14:10:21 | jaypipes | fried_rice, stephenfin, artom, alex_xu: thx for the reviews. I think I got all of your comments. please check it again. thanks in advance! mriedem, I added a bunch more to the upgrade impact section. ^^ | |
| 14:10:41 | fried_rice | ack | |
| 14:25:57 | finucannot | mriedem: This is another one of those changes that I thought would require a microversion but apparently does not. The output is wrong but it doesn't raise a HTTP 5xx https://review.openstack.org/#/c/608685/ | |
| 14:26:48 | finucannot | What distinguishes a fix from this from, say, modifying the output of os-hypervisor to respect (cpu|ram)_allocation_ratio. Both are currently giving "wrong" output | |
| 14:29:03 | mriedem | i don't have a good answer for that except the os-hypervisors stuff has been around forever and i don't know if including allocation ratios in that calculation would be considered by all to be "right" | |
| 14:30:33 | finucannot | Yeah, probably not the best example | |
| 14:30:51 | finucannot | mriedem: Would it be fair to ask for 'bug'-type reno for these kind of changes, in that case? | |
| 14:30:59 | finucannot | I'm thinking of the case where different clouds would give different results for what's supposed to be the same version | |
| 14:31:33 | mriedem | i wouldn't say it's unfair, although this simple tenant usage thing lpetrut is fixing is admin-only by default | |
| 14:31:39 | mriedem | non-admins can't list usage across all tenants | |
| 14:31:51 | mriedem | so that's not really an interop concern | |
| 14:32:11 | finucannot | Ohh, I didn't know that. In that case, not an issue here | |
| 14:32:55 | mriedem | well, | |