| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-05-13 | |||
| 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 | ||
| 12:34:14 | sean-k-mooney | string | |
| 12:34:16 | sean-k-mooney | ||
| 12:34:18 | sean-k-mooney | Filter the server list result by the UUID of the server. | |
| 12:34:20 | sean-k-mooney | This parameter is only valid when specified by administrators. If non-admin users specify this parameter, it is ignored. | |
| 12:34:23 | sean-k-mooney | so based on the docs we dont supprot it | |
| 12:34:35 | jkulik | it would explicitly state that's it supports a list? | |
| 12:34:41 | sean-k-mooney | it might be possibel but its not intended to work | |
| 12:35:13 | jkulik | (because for neutron ports "id", it looked the same but works) | |
| 12:35:13 | sean-k-mooney | it filters the list of servers by the uuid | |
| 12:35:20 | sean-k-mooney | it does not say it supports a list of uuids | |
| 12:35:40 | jkulik | sure, but the code has extra ifs for a list ;) | |
| 12:36:07 | jkulik | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2838-L2841 | |
| 12:36:11 | sean-k-mooney | jkulik: do we have api test for the list case | |
| 12:36:20 | jkulik | haven't checked. will do. | |
| 12:48:07 | jkulik | can't find any in the unit-tests at least. for a lot of other query-params, there are tests, but uuid doesn't seem to be in there. | |
| 12:49:21 | sean-k-mooney | jkulik: looking at the code if ti ever worke i suspect it was a nova v2 or nova v1 capablity that was never fully supproted and is not supproted anymore | |
| 12:50:07 | jkulik | looking at the code, it seems totally supported, if we would just convert the MultiDict's multiple entries to a list instead | |
| 12:50:41 | sean-k-mooney | https://github.com/openstack/nova/commit/3f35fe6a88672ea2ab7e080a55235c5cca45dc2c | |
| 12:51:07 | sean-k-mooney | that code is related to listing instance by ip | |
| 12:51:27 | sean-k-mooney | jkulik: its not for listing servers by uuid | |
| 12:51:59 | francoisp | lyarwood, bauzas hello, when you get a chance: https://review.opendev.org/#/c/726508/ (this time around for Queens) - thanks! | |