| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-11 | |||
| 20:57:35 | dansmith | well, | |
| 20:57:44 | dansmith | I don't think that refreshing it all the time is bad, | |
| 20:58:00 | dansmith | because it gives us a chance to update things like the ceph monitor ip address and such | |
| 20:58:07 | dansmith | but we don't have to do it blindly | |
| 20:58:14 | bauzas | yeah I can understand that reasoning | |
| 20:58:43 | bauzas | but yeah, instead of just amending what's needed, we're just wiping out the whole config | |
| 20:58:49 | bauzas | anyway | |
| 20:58:53 | bauzas | this is what it is | |
| 20:58:56 | dansmith | you could probably write a clean utility method that is "get me the mdev(s) from $guest" and just call that everywhere | |
| 20:59:18 | dansmith | everywhere you need it that is.. don't call it EVERYWHERE :P | |
| 20:59:28 | dansmith | and don't call it everywhere() because that won't be very obvious | |
| 21:00:07 | bauzas | I need to look at the driver code | |
| 21:00:08 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Fix being able to create a reno using tox -e venv https://review.openstack.org/532971 | |
| 21:00:08 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated MigrationManager.list cell_name kwarg https://review.openstack.org/532972 | |
| 21:02:25 | cfriesen | Is it documented anywhere that deleting a quota-set for a tenant will also delete it for any users within that tenant? (This seems to be how it behaves, but I don't see it in the API ref.) | |
| 21:03:41 | melwitt | cfriesen: user quota is a subset of tenant quota so I think it would behave that way. and I doubt it's documented | |
| 21:04:36 | cfriesen | melwitt: I can see the rationale, was just a bit unexpected. | |
| 21:05:05 | bauzas | dansmith: mmm, your last approach could work, except for migrations (but I think it's a silverlining) | |
| 21:05:27 | dansmith | bauzas: we should have allocations passed for migrations I imagine | |
| 21:05:30 | bauzas | dansmith: I mean, we can get the guest, hence its mdevs, before we regenerate it | |
| 21:05:31 | dansmith | we have to look them up anyway | |
| 21:05:40 | melwitt | cfriesen: yeah. user quota was an afterthought. originally it was tenant quota only (and I think it might go back that way eventually). so to tack on user quota, all they did was add a user column and filter on project AND user if user quota | |
| 21:06:04 | bauzas | dansmith: right, I just feel migrations are off the scope for queens | |
| 21:06:44 | dansmith | yeah, but it's good to think about how it'd be handled | |
| 21:06:49 | stvnoyes | mriedem: how exactly do I remove the test from the top patch? I don't want to make things worse. (git is not my strong point). | |
| 21:06:49 | dansmith | we don't even know if libvirt could do that I guess | |
| 21:07:25 | mriedem | stvnoyes: you just undo your change | |
| 21:07:33 | mriedem | stvnoyes: want me to do it? | |
| 21:07:56 | stvnoyes | that might be easier, just let me know what you did, for future reference. | |
| 21:08:27 | stvnoyes | once you do that I'll submit a new change | |
| 21:09:23 | edmondsw | mriedem sdague the PowerVM vSCSI patch is also ready for review now, which is the last one for that bp | |
| 21:09:36 | sean-k-mooney | dansmith: bauzas just reading back im not that surprised to seee the xml being regenerated on hard reboots its requried for some config changes | |
| 21:09:49 | dansmith | sean-k-mooney: yup | |
| 21:10:15 | dansmith | I've argued for doing it more often than we need even so we can freshen it, since some instances run for years without us getting to tweak things | |
| 21:10:19 | sean-k-mooney | bauzas: you will need to be careful not to miss livemigration too as we regenerate part of the xml there also + resize | |
| 21:10:22 | dansmith | we just don't need to do it blindly | |
| 21:14:36 | cfriesen | sean-k-mooney: and for "proper" resource tracking on live migration with we need to update the XML before the live-migration call and update it with the resources to use on the new compute node. | |
| 21:14:50 | mriedem | stvnoyes: done | |
| 21:15:15 | sean-k-mooney | cfriesen: yep we discussed that in some depth in the multiple port binding spec | |
| 21:15:33 | mriedem | stvnoyes: i did this basically: | |
| 21:15:41 | mriedem | 1. git review -d 266605 | |
| 21:15:49 | mriedem | to pull down the series from gerrit since it's no longer my local changes | |
| 21:15:56 | mriedem | 2. removed your test and committed that | |
| 21:16:00 | sean-k-mooney | cfriesen: in that specific case we were talking about nics but its the same problem statement just in a different domain | |
| 21:16:02 | mriedem | 3. added your test as a new commit | |
| 21:16:15 | mriedem | 4. changed the author to you: git commit --amend --author <steve-o> | |
| 21:16:23 | mriedem | 5. push it up: git review -R -y | |
| 21:18:30 | stvnoyes | mriedem: thanks | |
| 21:25:43 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecate certs CLIs and python bindings https://review.openstack.org/532974 | |
| 21:25:56 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated certs CLIs and python bindings https://review.openstack.org/532974 | |
| 21:31:59 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated cloudpipe CLIs and python API bindings https://review.openstack.org/532976 | |
| 21:42:59 | bauzas | sean-k-mooney: dansmith: understood, will consider migrations in general by looking at allocations | |
| 21:43:40 | bauzas | sean-k-mooney: dansmith: that said, my change is working for just spawning new instances https://review.openstack.org/#/c/528832/ | |
| 21:43:44 | bauzas | so you can review it | |
| 21:45:22 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated MigrationManager.list cell_name kwarg https://review.openstack.org/532972 | |
| 21:45:23 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated certs CLIs and python bindings https://review.openstack.org/532974 | |
| 21:45:23 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated cloudpipe CLIs and python API bindings https://review.openstack.org/532976 | |
| 21:59:27 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecate os-hosts CLIs and python API bindings https://review.openstack.org/532989 | |
| 21:59:33 | mriedem | let's fing remove everything | |
| 22:03:12 | ildikov | mriedem: made my day; 'fing' in Hungarian means 'fart' :) | |
| 22:03:39 | mriedem | excellent | |
| 22:03:58 | mriedem | mtreinish: how do you say "staying alive" in czech again? | |
| 22:04:01 | mriedem | hodina ho? | |
| 22:14:59 | openstackgerrit | Eric Fried proposed openstack/nova master: Raise conflict exception on RP create 409 https://review.openstack.org/532563 | |
| 22:15:00 | openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_traits_for_provider https://review.openstack.org/532564 | |
| 22:15:00 | openstackgerrit | Eric Fried proposed openstack/nova master: Track associated sharing RPs in report client https://review.openstack.org/526539 | |
| 22:15:01 | openstackgerrit | Eric Fried proposed openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540 | |
| 22:15:01 | openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | |
| 22:15:02 | openstackgerrit | Eric Fried proposed openstack/nova master: Track tree-associated providers in report client https://review.openstack.org/526541 | |
| 22:15:02 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | |
| 22:15:03 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | |
| 22:15:03 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | |
| 22:15:04 | openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | |
| 22:15:27 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: SchedulerReportClient.set_aggregates_for_provider https://review.openstack.org/532995 | |
| 22:21:23 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated fixedip/floatingip/virtual interface CLIs/APIs https://review.openstack.org/532999 | |
| 22:24:10 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated os-hosts CLIs and python API bindings https://review.openstack.org/532989 | |
| 22:24:10 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated fixedip/floatingip/virtual interface CLIs/APIs https://review.openstack.org/532999 | |
| 22:25:40 | mtreinish | mriedem: "Hodina H" is the title | |
| 22:25:56 | mriedem | yeah but the song is called staying alive | |
| 22:26:17 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated fixedip/floatingip/virtual interface CLIs/APIs https://review.openstack.org/532999 | |
| 22:52:02 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated fixedip/floatingip/virtual interface CLIs/APIs https://review.openstack.org/532999 | |
| 22:52:02 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated services binary CLI arg https://review.openstack.org/533005 | |
| 22:52:38 | mriedem | Vek is earning his salt today | |
| 22:53:57 | mriedem | melwitt: i'm going to try and get https://review.openstack.org/#/q/topic:bug/1668743+status:open into novaclient and then we'll cut the major 10.0.0 release | |
| 22:54:16 | mriedem | it's always nice to flush the toilet once every 6 months | |
| 22:54:45 | melwitt | lol | |
| 23:17:16 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958 | |
| 23:41:37 | mriedem | edleafe: left comments on the resize patch with what i think needs to happen wrt allocations getting moved from the primary selected host to the alternate | |
| 23:53:38 | gmann | mriedem: re: https://review.openstack.org/#/c/330406/85/nova/api/openstack/compute/migrations.py | |
| 23:54:09 | gmann | mriedem: it will stop schema validation for legacy v2 compatibility mode | |
| 23:54:39 | gmann | mriedem: https://github.com/openstack/nova/blob/90a92d33edaea2b7411a5fd528f3159a486e1fd0/nova/api/validation/__init__.py#L74 | |
| 23:57:22 | rybridges | Good afternoon! Might have found a bug with user_storage_quota in glance-api.conf. It looks like when we are out of quota and try to directly upload an image we get the error on the command line "Image exceeds the storage quota". However, when we attempt to make a snapshot with "openstack server image create", we do not see the error on the command line. The snapshot gets created and is in queued | |
| 23:57:24 | rybridges | state, but then just disappears. We then see a stack trace on the hypervisor with the error in the log, but the user is never directly informed | |
| 23:57:33 | rybridges | Is this the expected behavior? | |
| 23:59:59 | gmann | mriedem: 1 query on this. why we did not add paging on GET /servers/{server_id}/migrations -https://review.openstack.org/#/c/532904/1 | |
| #openstack-nova - 2018-01-12 | |||
| 00:02:08 | mriedem | gmann: replied on the spec | |
| 00:02:22 | mriedem | as for the 2.0 thing, like i said, i think there are several API schema checks like that with 2.1 as the min version, | |
| 00:02:36 | mriedem | so if they should all be 2.0, we should fix them at the same time and have something in the docs or tests for making sure 2.0 is used rather than 2.1 | |
| 00:07:40 | gmann | mriedem: got your point on spec. not fully convinced but ok as per previous spec and patch proposal | |
| 00:08:25 | gmann | mriedem: on schema, i agree there might be few. for query it is just server index and all other are validating schema for legacy comp mode also | |