| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-05-13 | |||
| 11:30:07 | sean-k-mooney | but if the filed does not exist we remove the filed | |
| 11:30:27 | sean-k-mooney | let me find an example quick | |
| 11:30:28 | huaqiang | yes... | |
| 11:30:57 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/objects/image_meta.py#L194-L200 | |
| 11:31:18 | sean-k-mooney | so in 1.22 we extended the hw_video_model filed | |
| 11:32:05 | sean-k-mooney | if an object was created using one of the values that was only supported in 1.22+ we raise an excption | |
| 11:32:05 | huaqiang | Yes. raised an exception | |
| 11:32:34 | sean-k-mooney | but for hw_time_hpet when it is backleveled it is dropped | |
| 11:32:40 | sean-k-mooney | so for your feature | |
| 11:32:56 | sean-k-mooney | we would raise an excpetion for the cpu policy | |
| 11:33:16 | sean-k-mooney | e.g. you would not be allowed to backevel an object with cpu_policy=mixed | |
| 11:33:39 | huaqiang | Yes. I agree | |
| 11:33:40 | sean-k-mooney | because that woudl result in incorrect behavior | |
| 11:33:52 | huaqiang | how about InstanceNUMACell | |
| 11:34:02 | huaqiang | the closest example is this:https://github.com/openstack/nova/blob/master/nova/objects/numa.py#L49 | |
| 11:34:25 | huaqiang | It dropped the host 'pcpuset' directly | |
| 11:35:19 | sean-k-mooney | yes | |
| 11:35:36 | huaqiang | Not translate the pcpuset to exsiting cpuset | |
| 11:35:53 | sean-k-mooney | so for mixed cpus you should not need to modify the OVOs out side fo adding the new value for the cpu policy correct | |
| 11:36:17 | sean-k-mooney | we dont need to modify the host cell object | |
| 11:36:27 | huaqiang | I don't | |
| 11:36:33 | brinzhang_ | gibi: can you open this link http://people.redhat.com/pbonzini/qemu-test-doc/_build/html/topics/vnc_005fsecurity.html | |
| 11:36:50 | sean-k-mooney | so the allowed values of https://github.com/openstack/nova/blob/master/nova/objects/instance_numa.py#L59 will change | |
| 11:37:12 | sean-k-mooney | but the cpu_pinning_raw value can be used in both the dedicated or mixed case | |
| 11:37:23 | sean-k-mooney | in the dedicated case it will have an entry for all cpus | |
| 11:37:44 | sean-k-mooney | in the mixed case it will have a mapping only for the pinned cpus and the floating cpus will not be listed | |
| 11:38:01 | gibi | brinzhang_: yes. Now I see where the limit is coming from. Maybe link this in the spec | |
| 11:38:27 | brinzhang_ | gibi: Ok, I will add this to spec | |
| 11:38:33 | gibi | brinzhang_: cool, thanks! | |
| 11:38:54 | huaqiang | sean-k-mooney: yes. cpu_pinning_raw for pinned cpus only | |
| 11:39:37 | sean-k-mooney | so what you will need to do is add a line here https://github.com/openstack/nova/blob/master/nova/objects/instance_numa.py#L42 | |
| 11:39:42 | huaqiang | sean-k-mooney: https://review.opendev.org/#/c/716267/5/nova/objects/instance_numa.py | |
| 11:40:01 | sean-k-mooney | which will raise an excpetion if you try to backlevel an object with the policy set to mixed and i think that is all | |
| 11:40:18 | sean-k-mooney | huaqiang: yep exactly | |
| 11:40:40 | huaqiang | For in 'obj_make_compatible' for version 1.6, we have choice to through a exception | |
| 11:41:27 | huaqiang | and we have another choice to ' primitive.pop('pcpuset', None)' in generating version prior to 1.5 | |
| 11:41:38 | huaqiang | I am thinking which one is better | |
| 11:41:57 | sean-k-mooney | that the thing we dont have a choice | |
| 11:42:08 | sean-k-mooney | for 1.6 we have to raise an excption | |
| 11:42:21 | sean-k-mooney | if we dropped the value then we would cahgne form mixed to shared | |
| 11:42:28 | sean-k-mooney | that is wrong | |
| 11:42:53 | huaqiang | Yes. I don't allow change from a mixed to shared | |
| 11:42:59 | huaqiang | sound reasonable | |
| 11:43:20 | sean-k-mooney | i think what is being done for 1.5 and 1.6 in those patches is correct | |
| 11:44:26 | huaqiang | sean-k-mooney: Thanks | |
| 11:44:39 | huaqiang | It is clear for me. | |
| 11:45:48 | huaqiang | sean-k-mooney: another thing | |
| 11:46:07 | huaqiang | since v1.6 is not compatible with v1.5 | |
| 11:46:09 | huaqiang | should be bump the major version number? | |
| 11:46:10 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Re-proposes 'Proposal for a safer remote console with password authentication' https://review.opendev.org/719720 | |
| 11:46:31 | sean-k-mooney | we dont really use the major version number | |
| 11:46:36 | brinzhang_ | gibi: done, thanks | |
| 11:46:37 | huaqiang | s/should be/ should we/ | |
| 11:46:39 | sean-k-mooney | but we could i guess | |
| 11:47:02 | sean-k-mooney | that wone i would want dansmith to weigh in on as i dont know why we dont bump the major version | |
| 11:47:25 | sean-k-mooney | proably because its only conditionally incompatiable | |
| 11:47:28 | huaqiang | I'll leave it for revewer for designe | |
| 11:47:36 | huaqiang | s/designe/decision/ | |
| 11:47:50 | sean-k-mooney | e.g. if you are using shraed or dedicated we can backlevel it | |
| 11:48:10 | sean-k-mooney | so since it not a hard incompatiablity we dont need to modify the major version | |
| 11:48:15 | huaqiang | it true. | |
| 11:48:40 | huaqiang | ok, thanks | |
| 11:49:15 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.PY2 and six.PY3 https://review.opendev.org/726512 | |
| 11:49:29 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.add_metaclass https://review.opendev.org/726565 | |
| 11:49:35 | gibi | brtknr: thanks, +2 | |
| 11:49:44 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.reraise https://review.opendev.org/726898 | |
| 11:50:00 | brtknr | gibi: wp | |
| 11:50:07 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224 | |
| 11:50:09 | gibi | brtknr: sorry | |
| 11:50:29 | brtknr | np :) | |
| 11:52:53 | openstackgerrit | sean mooney proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341 | |
| 11:52:54 | openstackgerrit | sean mooney proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | |
| 11:52:54 | openstackgerrit | sean mooney proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | |
| 11:52:55 | openstackgerrit | sean mooney proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 11:53:42 | sean-k-mooney | gibi: that is just a strait rebae with the merge conflicts and pep8 issues resolved by the way so its basically the same as the previous version | |
| 11:54:51 | gibi | sean-k-mooney: thanks. you and tony_su should sync on who will work on that patch series | |
| 11:55:25 | sean-k-mooney | gibi: yep | |
| 11:56:57 | sean-k-mooney | tony_su: if you want to take it over and have time to adress https://review.opendev.org/#/c/676029/34/nova/tests/unit/compute/test_provider_config.py@174 that was the only open item form last cycle i belive | |
| 11:59:28 | sean-k-mooney | ok i have been awaked since 3:30 and working more or less since 4:30 so im going to take a break for a few hours. ill be back after i have lunch and relax for a bit | |
| 12:09:10 | gibi | sean-k-mooney: have a nice lunch | |
| 12:22:10 | jkulik | anybody else experiencing problems with querying /servers for multiple uuids? nova seems to only get one uuid for filtering. I guess the MultiDict used by webob.Request gets botched here https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L178 see http://paste.openstack.org/show/793512/ for some python-console fuu showing the problem | |
| 12:23:01 | jkulik | docs for the MultiDict stuff https://docs.pylonsproject.org/projects/webob/en/stable/reference.html#query-post-variables | |
| 12:28:12 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224 | |
| 12:29:25 | jkulik | should I just open a bug report if I fear something is broken instead of asking on IRC for someone to confirm? | |
| 12:30:16 | sean-k-mooney | jkulik: im not sure if this is actully supported | |
| 12:30:28 | jkulik | oh, down the line it's checked against a list | |
| 12:30:33 | jkulik | let me find that part again | |
| 12:31:43 | sean-k-mooney | jkulik: the servers endpoint only supprot one uuid https://docs.openstack.org/api-ref/compute/?expanded=show-server-details-detail#show-server-details | |
| 12:31:48 | sean-k-mooney | not a list of them | |
| 12:32:03 | sean-k-mooney | at least the server detials endpoint | |
| 12:32:41 | jkulik | index does support more in theory, though. it ends up here in the end https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L1849-L1854 | |
| 12:33:11 | jkulik | oh, I mean query params to /servers or /servers/detail | |
| 12:33:13 | sean-k-mooney | well the docs text is | |
| 12:33:34 | sean-k-mooney | right so its not supported for /servers/detail | |
| 12:33:49 | sean-k-mooney | well i guess it might be | |
| 12:34:03 | sean-k-mooney | the docs text reads | |
| 12:34:06 | sean-k-mooney | uuid (Optional) | |
| 12:34:08 | sean-k-mooney | ||
| 12:34:10 | sean-k-mooney | query | |
| 12:34:12 | sean-k-mooney | ||