| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-12 | |||
| 13:52:33 | sahid | something like fields.IntegerField(version=2.0, nullable=True) | |
| 13:52:55 | sahid | so we could remove that obj_make_compatible method from all of our objects | |
| 13:52:58 | mriedem | sahid: there is already a version on the object | |
| 13:53:00 | sahid | to have it only in the base class | |
| 13:53:08 | mriedem | oh version per field.... | |
| 13:53:12 | sahid | yes | |
| 13:53:33 | dansmith | sahid: removing the field isn't the only action we need to take during a backport, | |
| 13:53:40 | dansmith | it's just a common one | |
| 13:54:08 | sahid | but that seems a bit hacky to have that obj_make_compatible() method everywhere, no? | |
| 13:54:29 | sahid | we could probably provide something smart to avoid that change each time we add new field | |
| 13:55:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove network info stubbing in functional test https://review.openstack.org/570018 | |
| 13:55:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840 | |
| 13:55:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267 | |
| 13:55:08 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847 | |
| 13:55:09 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | |
| 13:55:10 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | |
| 13:55:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| 13:55:33 | dansmith | I dunno, it's two lines of code per thing we add, and it makes people think about what needs to happen to backport the thing | |
| 13:56:18 | dansmith | we used to have a mapping for backporting nested objects, and maintaining that was not free either | |
| 13:56:22 | sahid | yes i don't know eother, it's just a suggestion | |
| 13:57:03 | dansmith | version per field kinda adds noise to the definition too, which isn't great | |
| 13:58:08 | sahid | yep true, but beside of that we could avoid to have the obj_make_compatible() method + unit test related | |
| 13:58:32 | dansmith | sahid: you could add to our base object an implementation of make compatible that, if field in self.obj_field_history: delete if < self.obj_field_history[field] | |
| 13:58:45 | dansmith | it'd still be one line per change though | |
| 14:00:35 | sahid | what this obj_field_history is supposed to carry? | |
| 14:00:44 | dansmith | sahid: hang on I'll show you what I mean | |
| 14:01:09 | mriedem | bauzas: http://lists.openstack.org/pipermail/openstack-dev/2018-June/131410.html | |
| 14:01:40 | mriedem | guess i can go back and read the giant spec on this | |
| 14:02:10 | openstackgerrit | Simon Dodsley proposed openstack/nova master: Add enhanced KVM storage QoS quotas https://review.openstack.org/558530 | |
| 14:02:13 | dansmith | sahid: something like this: https://pastebin.com/waWNUM3U | |
| 14:02:27 | dansmith | sahid: (that is in base.NovaObject btw) | |
| 14:03:10 | sahid | dansmith: oh i see | |
| 14:03:38 | sahid | not a big fan :) but yes it's a way to achieve the same thing | |
| 14:04:44 | mriedem | bauzas: relevant parts of the spec https://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/nova-validate-certificates.html#proposed-change | |
| 14:04:51 | mriedem | https://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/nova-validate-certificates.html#other-deployer-impact | |
| 14:05:30 | mriedem | i think it just means, enable_certificate_validation=True means enforce cert validation and default_trusted_certificate_ids is if you enforce and don't want to kill your users if they don't provide trusted certs in the server create/rebuild requests | |
| 14:06:04 | mriedem | both get used in the API changes, and you wouldn't want to set enable_certificate_validation=True UNTIL the api changes are in place to use default_trusted_certificate_ids | |
| 14:06:09 | mriedem | at least as things are written today | |
| 14:07:03 | mriedem | since enable_certificate_validation is deprecated, i'm not sure at what point it is removed, | |
| 14:07:09 | mriedem | nor do i understand how it's a transition to anything | |
| 14:07:41 | mriedem | i guess both options together are meant to enable the transition to cert validation of images | |
| 14:07:48 | mriedem | before forcing it on users | |
| 14:11:38 | bauzas | mriedem: sorry was afk | |
| 14:13:56 | bauzas | mriedem: " all prior usage of the server create/rebuild API when booting signed images will now fail if trusted certificates cannot be located." | |
| 14:15:21 | bauzas | mriedem: okay, so when the API change is merged, I understand that things will continue to work as defaults will be seen as user-provided | |
| 14:15:49 | simondodsley_ | mreidem: referencing the Cinder QoS change... the QoS values you see are the capacity based QoS options. These are for things like Ceph and ScaleIO (I think). The external disk QoS spec is separate to these per_gb values. The spec for external disk QoS is here: https://github.com/openstack/cinder/blob/master/cinder/volume/qos_specs.py. This doesn't appear to have any documentation, so I will create these once the | |
| 14:15:49 | simondodsley_ | Nova piece has been merged there are no Cinder changes required to add new QoS values | |
| 14:15:54 | bauzas | mriedem: later, when they want to drop the defaults and force the user to ask, then it would pop up an exception | |
| 14:16:39 | simondodsley_ | mriedem: ^ | |
| 14:17:33 | bauzas | mriedem: unrelated how can we know that vdi_remote_stream from https://review.openstack.org/#/c/486475/ is going to be CI-tested by Xen ? | |
| 14:17:49 | bauzas | because the 3rd-party CI says cool, but I don't know if it checked it | |
| 14:18:21 | mriedem | bauzas: that patch isn't using vdi_remote_stream | |
| 14:18:26 | mriedem | that's why i have https://review.openstack.org/#/c/574318/ | |
| 14:18:32 | mriedem | which hasn't had xenserver ci run on it | |
| 14:18:54 | mriedem | "Matt, thanks for this DNM patch. But it looks like the Citrix CI is out of service now. Naichuan and Bob will help to fix it." | |
| 14:19:37 | mriedem | looks like https://review.openstack.org/#/c/486446/ does it though... | |
| 14:19:50 | bauzas | mriedem: but AFAICU, your DNM change only checks UTs | |
| 14:19:56 | mriedem | no | |
| 14:20:04 | mriedem | it's meant to trigger the xenserver 3rd party CI | |
| 14:20:06 | bauzas | mriedem: how can we sure that the Xen CI is using those opts ? | |
| 14:20:08 | mriedem | to test this new image handler | |
| 14:20:20 | bauzas | aaaaaah | |
| 14:20:26 | bauzas | I see, hence the default opt value change | |
| 14:20:29 | mriedem | yes | |
| 14:20:43 | bauzas | ok | |
| 14:20:43 | mriedem | how do you ensure we test the libvirt driver with the raw image backend? | |
| 14:20:52 | mriedem | since qcow2 is the default | |
| 14:21:07 | bauzas | lemme be political | |
| 14:21:16 | bauzas | that's an excellent question, I'll come to it later | |
| 14:21:18 | mriedem | so the xen image handler defaults to direct_vhd, so that's what was getting tested, but it looks like they have changed the xenserver CI to use vdi_remote_stream https://review.openstack.org/#/c/486446/ | |
| 14:21:38 | mriedem | so now the 3rd party CI is testing the non-default image handler....which seems not great | |
| 14:21:51 | garyk | johnthetubaguy: mriedem: can you please remove the -2 on https://review.openstack.org/#/c/270116/. the spec has been approved | |
| 14:22:02 | garyk | we are still working on some kinks with the CI | |
| 14:22:20 | bauzas | mriedem: I see, thanks for the heads up | |
| 14:23:07 | kashyap | Some of you here might be interested in this 'fun' info: upstream QEMU there is "fierce" and KM-long discussion whether to put some configuration info directly into qcow2 image format (e.g. the min amount of RAM, and importantly the machine type of 'pc' vs. the more advanced 'q35'). | |
| 14:23:24 | openstackgerrit | Merged openstack/nova master: Move db using provider unit tests to functional https://review.openstack.org/572280 | |
| 14:23:24 | kashyap | But no one has seriously committed to it, though. | |
| 14:23:32 | openstackgerrit | Merged openstack/nova master: Adjust db using allocation unit tests https://review.openstack.org/572281 | |
| 14:23:40 | openstackgerrit | Merged openstack/nova master: Remove nova dependencies from test_resource_provider https://review.openstack.org/572282 | |
| 14:23:58 | mriedem | garyk: i can't remove johnthetubaguy's -2 | |
| 14:24:02 | mriedem | i asked him last week to drop it | |
| 14:24:13 | garyk | mriedem: thanks! | |
| 14:24:21 | garyk | johnthetubaguy: ^ | |
| 14:25:16 | kashyap | (It's primarily because, disk images are shipped as qcow2 images. And QEMU folks don't want to break Nova et al, if they switch the default machine type) | |
| 14:25:47 | kashyap | Nova-related discussion: https://bugzilla.redhat.com/show_bug.cgi?id=1581414 | |
| 14:25:48 | openstack | bugzilla.redhat.com bug 1581414 in openstack-nova "OpenStack shouldn't break if the default machine-type in QEMU is "q35"" [Medium,New] - Assigned to kchamart | |
| 14:26:17 | mriedem | simondodsley_: i'm not sure why the cinder documentation for the new qos specs needs to wait for the nova change to merge, | |
| 14:26:29 | mriedem | simondodsley_: as i said in the nova patch, i won't merge the nova change until i at least see the cinder docs change for those new qos specs | |
| 14:26:38 | mriedem | otherwise you have no incentive to actually write the docs | |
| 14:27:42 | mriedem | dansmith: is there anything logged from oslo.messaging when it does a heartbeat while we're running pre_live_migration with the long timeout? | |
| 14:27:54 | mriedem | i was looking for something in the n-cpu logs from this https://review.openstack.org/#/c/574482/ | |
| 14:28:07 | dansmith | mriedem: at debug, I said on your patch | |
| 14:28:23 | dansmith | but that causes a bunch of noise if enabled | |
| 14:28:39 | simondodsley_ | mreidem: I'm actually writing the docs now. If I say I will do something then I will do it... As there is no documentation for these external disk QoS parameters including the originals, that would mean adding a page without the new parameters, and then updating it once the Nova patch merged. | |
| 14:29:04 | mriedem | on *your* patch https://review.openstack.org/#/c/566696/9/nova/compute/rpcapi.py@699 | |
| 14:29:18 | mriedem | but ok, i can make that happen too | |
| 14:31:06 | dansmith | mriedem: oh right I thought it was on one of yours above, but whatever :) | |
| 14:33:20 | mriedem | simondodsley_: replies inline https://review.openstack.org/#/c/558530/3 | |
| 14:34:10 | mriedem | bauzas: yeah, once they remove that stuff it becomes an error - i don't think anyone should probably do that, though, until we have the compute capabilities traits filtering during scheduling | |
| 14:34:25 | mriedem | so at least scheduling of new instances is smarter wrt where trusted certs instances can be built | |