| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-08-18 | |||
| 20:23:07 | iurygregory | in this one https://review.opendev.org/741886 I'm testing the changes from tempest | |
| 20:51:22 | sean-k-mooney | iurygregory: what did the flavor look like | |
| 20:52:06 | sean-k-mooney | "cpu_arch"="$ironic_node_arch" | |
| 20:52:10 | sean-k-mooney | that is proaby the issue | |
| 20:52:26 | sean-k-mooney | that has no prifix so the compute capablity filter will match on it | |
| 20:52:43 | sean-k-mooney | using un namespaced extra specs is discuraged | |
| 20:53:11 | sean-k-mooney | it should not be done if usign either the compute capablity filter or aggreate instance extra specs filter | |
| 20:55:43 | sean-k-mooney | iurygregory: did you add the capablites to the host. can you point to them in any of the logs | |
| 20:56:06 | sean-k-mooney | it just looks like they dont match but there is very little to go on | |
| 21:06:35 | openstackgerrit | Douglas Mendizábal proposed openstack/nova stable/train: Allow TLS ciphers/protocols to be configurable for console proxies https://review.opendev.org/746798 | |
| 21:12:12 | iurygregory | shummm | |
| 21:12:19 | iurygregory | let me check here sean-k-mooney | |
| 21:12:52 | iurygregory | we are creating the flavors in https://review.opendev.org/#/c/741886/9/devstack/lib/ironic | |
| 21:13:21 | sean-k-mooney | yep but where are you setting the capablities on the compute nodes | |
| 21:13:28 | sean-k-mooney | i assume that is done via ironic | |
| 21:13:32 | iurygregory | the additional thing is that now our flavors will have the capabilities:boot_option=local/netboot | |
| 21:13:50 | iurygregory | we also set in the ironic node | |
| 21:13:52 | sean-k-mooney | right which will enable the computecablities filter | |
| 21:13:54 | iurygregory | so it would match | |
| 21:14:28 | sean-k-mooney | can you show in the logs that they are being set | |
| 21:16:05 | sean-k-mooney | https://zuul.opendev.org/t/openstack/build/65e4cb538b7149d48f0942e8639b9bb3/log/controller/logs/devstacklog.txt#30847 | |
| 21:16:17 | openstackgerrit | Douglas Mendizábal proposed openstack/nova stable/stein: Allow TLS ciphers/protocols to be configurable for console proxies https://review.opendev.org/746800 | |
| 21:16:25 | sean-k-mooney | iurygregory: uefi is beeing set | |
| 21:17:07 | iurygregory | sean-k-mooney, this job is with uefi | |
| 21:17:09 | sean-k-mooney | iurygregory: but i dont see the boot_option | |
| 21:17:16 | iurygregory | oh O.o | |
| 21:18:15 | sean-k-mooney | https://zuul.opendev.org/t/openstack/build/65e4cb538b7149d48f0942e8639b9bb3/log/controller/logs/devstacklog.txt#31101 | |
| 21:18:37 | sean-k-mooney | that is the only use of capabilities:boot_option=netboot | |
| 21:19:02 | iurygregory | oh it's because tempest was suposed to update ... | |
| 21:19:22 | iurygregory | with the patch https://review.opendev.org/#/c/741445/23/ironic_tempest_plugin/tests/scenario/baremetal_manager.py | |
| 21:19:36 | iurygregory | I thought this was working o.o | |
| 21:20:35 | sean-k-mooney | the ironic_tempest_plugin was installed form pip | |
| 21:20:47 | sean-k-mooney | https://zuul.opendev.org/t/openstack/build/65e4cb538b7149d48f0942e8639b9bb3/log/controller/logs/devstacklog.txt#32985-32988 | |
| 21:20:50 | sean-k-mooney | i think | |
| 21:21:24 | sean-k-mooney | actully maybe not | |
| 21:21:39 | sean-k-mooney | iurygregory: what repo is this job defined in | |
| 21:21:42 | sean-k-mooney | ironic? | |
| 21:21:53 | iurygregory | sean-k-mooney, yeah | |
| 21:22:11 | sean-k-mooney | im wondiering it it listed as a require_project | |
| 21:22:22 | sean-k-mooney | withiout that depends-on does not work | |
| 21:22:46 | iurygregory | it does have Depends-On | |
| 21:22:58 | iurygregory | ironic has Depends-On on the ironic-tempest-plugin change | |
| 21:23:12 | iurygregory | tempest https://review.opendev.org/#/c/741445 ironic https://review.opendev.org/#/c/741886 | |
| 21:23:18 | sean-k-mooney | yep but yyou need to add https://github.com/openstack/ironic/blob/master/zuul.d/ironic-jobs.yaml#L13 to the job or it does not work | |
| 21:23:26 | sean-k-mooney | in this case its there | |
| 21:25:36 | sean-k-mooney | so this plugin is being instealled in the tempet virtual env | |
| 21:25:39 | sean-k-mooney | https://zuul.opendev.org/t/openstack/build/65e4cb538b7149d48f0942e8639b9bb3/log/controller/logs/devstacklog.txt#32903-32924 | |
| 21:26:08 | iurygregory | yeah, seems like tempest is not updating the node with the boot_option... | |
| 21:26:16 | sean-k-mooney | i think its correctly installing it tox -evenv-tempest -- pip install -c /tmp/tempest_u_c_m.37WJbEB0sa /opt/stack/ironic-tempest-plugin | |
| 21:26:28 | sean-k-mooney | that using the version cloned by zuul | |
| 21:28:25 | sean-k-mooney | iurygregory: have you defiend the required tempest config options | |
| 21:28:26 | iurygregory | yeah the version installed is correct (at least the logs shows "23adec07f1d5998951fe841cd68989c6d85f7d98 Update basic_ops" | |
| 21:28:55 | iurygregory | yeah, I checked the config and it does look correct | |
| 21:30:04 | iurygregory | I double checked with the job-output.txt the flavors are correct | |
| 21:30:08 | sean-k-mooney | https://zuul.opendev.org/t/openstack/build/65e4cb538b7149d48f0942e8639b9bb3/log/controller/logs/tempest_conf.txt#141-144 | |
| 21:30:10 | iurygregory | uuid etc | |
| 21:30:24 | sean-k-mooney | so ya it looks like they are set correctly | |
| 21:31:18 | sean-k-mooney | so this is what is ment to set the caps https://review.opendev.org/#/c/741445/23/ironic_tempest_plugin/tests/scenario/baremetal_manager.py@205 | |
| 21:32:46 | sean-k-mooney | https://review.opendev.org/#/c/741445/23/ironic_tempest_plugin/tests/scenario/baremetal_manager.py@174 | |
| 21:32:54 | sean-k-mooney | by default it does not set teh caps | |
| 21:33:14 | iurygregory | yeah, but the test sends True | |
| 21:33:42 | iurygregory | https://review.opendev.org/#/c/741445/23/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py@208 | |
| 21:33:45 | sean-k-mooney | yes https://review.opendev.org/#/c/741445/23/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py@208 | |
| 21:34:08 | iurygregory | at least it should send =) | |
| 21:35:13 | sean-k-mooney | does the compute capablity filter requrie all cap to be requested | |
| 21:35:45 | iurygregory | this information I don't know =) | |
| 21:36:15 | sean-k-mooney | i was wonderinf it the default falvor was failing but no | |
| 21:36:26 | sean-k-mooney | the only failing tests are teh new ones | |
| 21:36:46 | sean-k-mooney | well actully | |
| 21:36:49 | sean-k-mooney | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_65e/741886/9/check/ironic-tempest-ipa-partition-uefi-pxe_ipmitool/65e4cb5/testr_results.html | |
| 21:36:56 | sean-k-mooney | only 2 tests failed | |
| 21:37:03 | iurygregory | yeah | |
| 21:37:25 | iurygregory | the ones I did the changes and require the new flavor etc =) | |
| 21:38:23 | sean-k-mooney | well you added https://review.opendev.org/#/c/741445/23/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py@320 | |
| 21:38:25 | sean-k-mooney | whic passed | |
| 21:38:35 | sean-k-mooney | that looks like it can use the new code | |
| 21:39:39 | sean-k-mooney | oh that was skipped | |
| 21:39:45 | iurygregory | I'm not sure where did you see that test_baremetal_server_ops_wholedisk_image passed o.o | |
| 21:40:04 | iurygregory | oh =) | |
| 21:40:24 | sean-k-mooney | im guessing rescure is not enabled or something like that | |
| 21:40:24 | iurygregory | because rescue wasn't enable so it will skip | |
| 21:40:30 | sean-k-mooney | yep | |
| 21:41:12 | iurygregory | we are trying to make our jobs test partition and wholedisk images at the same time, so we can remove some jobs | |
| 21:43:07 | sean-k-mooney | so this is wehre the caps are currently configured | |
| 21:43:09 | sean-k-mooney | https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L2293-L2307 | |
| 21:43:15 | sean-k-mooney | staticly in devstack | |
| 21:43:46 | sean-k-mooney | well it goes on longer | |
| 21:43:48 | sean-k-mooney | https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L2293-L2344 | |
| 21:45:14 | iurygregory | yeah | |
| 21:45:29 | iurygregory | but tempest should be able to just go and update | |
| 21:45:37 | iurygregory | .-. | |
| 21:45:44 | sean-k-mooney | well | |
| 21:45:50 | iurygregory | or nope? | |
| 21:45:55 | sean-k-mooney | its updating them after it creates the instance | |
| 21:46:17 | sean-k-mooney | https://review.opendev.org/#/c/741445/23/ironic_tempest_plugin/tests/scenario/baremetal_manager.py@190 | |
| 21:46:23 | sean-k-mooney | https://review.opendev.org/#/c/741445/23/ironic_tempest_plugin/tests/scenario/baremetal_manager.py@197 | |
| 21:46:32 | sean-k-mooney | the node is not update until 205 | |
| 21:47:10 | sean-k-mooney | so you need to do it before creating the nova instance | |
| 21:47:17 | sean-k-mooney | so move it to 183 | |
| 21:47:20 | iurygregory | wow | |