| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-06-22 | |||
| 14:07:16 | gibi | but this is way less important that make the unshelve patch land :D | |
| 14:07:24 | gibi | s/that/than/ | |
| 14:07:42 | bauzas | that's why I explained why the pattern and also why I said in my last comment that docstrings help | |
| 14:08:12 | bauzas | I'd rather see a docstring saying (optional) my param | |
| 14:08:43 | bauzas | rather than asking to import a specific instance of a global class object | |
| 14:10:46 | gibi | the function itself is a global on the class an you import that to be able to call it | |
| 14:11:19 | gibi | None is an interpreter global :) | |
| 14:11:33 | gibi | (we are getting philosophycal ) | |
| 14:12:33 | gibi | we are using enums from the fields module those are class level fields too | |
| 14:26:49 | Uggla | gibi, bauzas, last stuff, ok with FIELD_SENTINEL wording ? | |
| 14:29:48 | gibi | go with what bauzas asked for I won't block on it | |
| 14:30:29 | bauzas | Uggla: gibi: you can even name it privatly | |
| 14:30:36 | bauzas | like _unset_field | |
| 14:30:45 | gibi | sure | |
| 14:30:48 | bauzas | or _unsel_field_sentinel | |
| 14:31:01 | bauzas | I don't wanna nitpick on the naming :) | |
| 14:31:22 | Uggla | naming one of the hardest stuff in computing. | |
| 14:31:30 | bauzas | but yeah, actually, since this is an internal object, make it private | |
| 14:31:43 | bauzas | (by convention of course) | |
| 14:32:27 | bauzas | https://twitter.com/codinghorror/status/506010907021828096?lang=fr | |
| 14:32:31 | bauzas | :) | |
| 14:32:44 | bauzas | (I like this tweet :p ) | |
| 14:33:38 | bauzas | https://www.karlton.org/2017/12/naming-things-hard/ for the wider context | |
| 17:11:44 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds validation for hw machine type in host caps https://review.opendev.org/c/openstack/nova/+/847126 | |
| 19:45:42 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds validation for hw machine type in host caps https://review.opendev.org/c/openstack/nova/+/847126 | |
| 20:38:09 | ade_lee | sean-k-mooney, sean-k-mooney[m] slaweq stephenfin hey - any idea what might be causing failures here? https://8767bac9cdd8c58da256-ee4c5d809145a8a3246cc4d26d65fbe0.ssl.cf5.rackcdn.com/831844/10/experimental/tempest-centos9-stream-fips/45be779/testr_results.html | |
| 20:41:16 | ade_lee | oh -- maybe thats this -- https://bugs.launchpad.net/neutron/+bug/1979047 | |
| #openstack-nova - 2022-06-23 | |||
| 05:28:12 | frickler | ade_lee: yes, that's the same failure pattern that caused us to make c9s non-voting in devstack | |
| 08:27:14 | bauzas | gibi: others, fwiw, I'll be on PTO tomorrow | |
| 08:27:24 | gibi | bauzas: o/ ack | |
| 09:15:51 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds validation for hw machine type in host caps https://review.opendev.org/c/openstack/nova/+/847126 | |
| 10:06:59 | sean-k-mooney | ade_lee: assuming this is related to fips and centos 9 stream ya interface attach is broken due to an upstream libvirt bug which has a fix internally but is not relased yet | |
| 11:06:03 | opendevreview | Merged openstack/nova master: Change TooOldComputeService upgrade check code to failure https://review.opendev.org/c/openstack/nova/+/845262 | |
| 14:08:12 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds validation for hw machine type in host caps https://review.opendev.org/c/openstack/nova/+/847126 | |
| 14:31:20 | amorin | hello nova! | |
| 14:34:55 | amorin | I was looking for an idea on how to schedule instances on computes based on some metrics that are not cpu related | |
| 14:35:04 | amorin | like, the kernel version, the operating system version | |
| 14:40:35 | amorin | i was thinking that I could tweak the compute monitors (that collect metrics) | |
| 14:40:40 | amorin | but I dont know if this is the best idea? | |
| 14:45:05 | mnaser | amorin: maybe you could use traits out of bands? | |
| 14:45:21 | mnaser | like have something that ships the traits of those boxes to placement | |
| 14:45:23 | amorin | out of bands? | |
| 14:45:32 | amorin | like a custom traits | |
| 14:45:44 | mnaser | yeah i meant out of band as in you wont be in the nova code path to determine this | |
| 14:45:46 | amorin | yes, I did try that, but it seems not available in the nova weighter | |
| 14:45:59 | mnaser | ah so you need this info in the weigher and not in the filter stage | |
| 14:46:20 | amorin | yes, because I dont want to filter out the hosts | |
| 14:46:28 | amorin | I just want them to be lower in the priority | |
| 14:47:30 | mnaser | ah okay so my suggestion wont help then | |
| 14:47:42 | amorin | unfortunately :( but thanks! | |
| 14:47:44 | gibi | placement can only be used today to do filtering decision, there is no weighing support there | |
| 14:47:54 | gibi | so you need a now weigher code for that | |
| 14:48:01 | gibi | so you need nova weigher code for that | |
| 14:48:02 | amorin | yup, and doing placement call from weigher seems a bad idea | |
| 14:50:46 | gibi | I dont think we have code for that in nova yet, you can try to abuse the MetricsWeigher | |
| 14:50:53 | gibi | but it feels hackish | |
| 14:51:50 | amorin | thats exactly what I am going to do, but I just wanted to know if something else, less hacky would exist | |
| 14:52:41 | gibi | I'm not aware of any less hackish way now | |
| 14:55:13 | amorin | ok, thanks gibi ! | |
| 15:00:36 | bauzas | amorin: I was trying to reply to you about your question :) | |
| 15:01:02 | bauzas | yeah, you should use weights | |
| 15:01:10 | bauzas | placement doesn't support them | |
| 15:10:21 | amorin | bauzas yeah! thanks | |
| 15:10:32 | amorin | you can still answer, maybe that would be nice for other people :) | |
| 15:10:44 | bauzas | amorin: sorry, I didn't have time to reply | |
| 15:10:59 | bauzas | amorin: I'll try, but I'm off tomorrow :( | |
| 15:11:08 | amorin | no worries :) | |
| 17:49:39 | sean-k-mooney | bauzas: i will be off tomorrow to recover but if you can take a look at https://review.opendev.org/c/openstack/nova/+/847001 i think i adress all your issues | |
| #openstack-nova - 2022-06-24 | |||
| 02:29:30 | opendevreview | Merged openstack/python-novaclient master: Imported Translations from Zanata https://review.opendev.org/c/openstack/python-novaclient/+/846880 | |
| 08:38:31 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Replace old URLs with new ones https://review.opendev.org/c/openstack/python-novaclient/+/840693 | |
| 09:50:53 | opendevreview | Stephen Finucane proposed openstack/os-resource-classes master: Update Python testing as oer zed cycle testing runtime https://review.opendev.org/c/openstack/os-resource-classes/+/847562 | |
| 09:51:11 | stephenfin | gibi: Easy review there ^ | |
| 09:57:24 | stephenfin | never mind, we had similar patches already that I hadn't spotted | |
| 10:00:26 | opendevreview | Merged openstack/os-resource-classes master: Update CI to use unversioned jobs template https://review.opendev.org/c/openstack/os-resource-classes/+/834537 | |
| 10:02:05 | opendevreview | Merged openstack/os-resource-classes master: setup: Update Python testing classifiers https://review.opendev.org/c/openstack/os-resource-classes/+/834643 | |
| 10:02:30 | opendevreview | Merged openstack/os-resource-classes master: setup: Replace dashes with underscores, add links https://review.opendev.org/c/openstack/os-resource-classes/+/834644 | |
| 10:42:54 | gibi | stephenfin: :) | |
| 11:11:22 | opendevreview | Merged openstack/placement master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/placement/+/791971 | |
| 11:35:13 | opendevreview | Merged openstack/python-novaclient master: Replace old URLs with new ones https://review.opendev.org/c/openstack/python-novaclient/+/840693 | |
| 11:51:01 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds validation for hw machine type in host caps https://review.opendev.org/c/openstack/nova/+/847126 | |
| 13:47:42 | opendevreview | Takashi Natsume proposed openstack/os-resource-classes master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/os-resource-classes/+/791974 | |
| 14:03:54 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host (Compute API part) https://review.opendev.org/c/openstack/nova/+/831507 | |
| 14:03:55 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host (REST API part) https://review.opendev.org/c/openstack/nova/+/845897 | |
| 14:26:55 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds validation for hw machine type in host caps https://review.opendev.org/c/openstack/nova/+/847126 | |
| 14:44:59 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host (REST API part) https://review.opendev.org/c/openstack/nova/+/845897 | |
| 14:44:59 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host (Compute API part) https://review.opendev.org/c/openstack/nova/+/831507 | |
| 17:49:43 | colby_ | Hello everyone. we are experienceing a strange but with vGPU. We are using Virtoria on Centos 8 Stream with A40 gpus in SRIOV setup. Everything works correctly except when we delete an instance. It does not seem to release the virtualfunction mdev device. I can manually release them by echoing to the remove method of the mdev. So its like nova is not doing that. | |
| 17:50:49 | colby_ | Are there any bugs about this? Can someone point me to the code that handles this so I can see if I can tell why its failing. Thanks. | |
| #openstack-nova - 2022-06-25 | |||
| 09:40:18 | frickler | https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1013410 looks like a unit test that fails under load. expected: timed out after 0.00 seconds, actual: timed out after 0.02 seconds | |
| 15:09:52 | opendevreview | Artom Lifshitz proposed openstack/nova master: Broken WIP: Update libvirt enlightenments for Windows https://review.opendev.org/c/openstack/nova/+/847641 | |
| #openstack-nova - 2022-06-26 | |||
| 15:27:44 | opendevreview | Takashi Kajinami proposed openstack/osc-placement master: Replace deprecated assertRaisesRegexp https://review.opendev.org/c/openstack/osc-placement/+/847689 | |
| 15:28:44 | opendevreview | Takashi Kajinami proposed openstack/osc-placement master: Replace deprecated assertRaisesRegexp https://review.opendev.org/c/openstack/osc-placement/+/847689 | |
| 15:43:15 | tobias-urdin | colby_: iirc the mdev is the actual resource so it'll be reused | |
| 17:46:39 | colby_ | tobias-urdin: It does not seem to be the case. Nova gives the error that there are no resources available. If I manually remove the mdev then we can create more instances | |
| 17:57:01 | tobias-urdin | colby_: do you get any errors or similar? do you have some more info about what you are running, the only issue we have currently with vgpu part is https://review.opendev.org/c/openstack/nova/+/838976 | |
| 18:03:34 | colby_ | tobias-urdin: unfortunately no errors other than no resources. I have looked all through system logs for any sort of errors. mdevctl shows all the devices still used and nvidia-smi vgpu -c show no vgpu available. | |
| 18:04:54 | colby_ | if I do the following command: echo "1" > nvidia-563/devices/8f5be0ec-3a4b-4e72-b613-4e93a27951a0/remove | |
| 18:05:24 | colby_ | then nvidia-smi -c shows available gpu and mdevctl shows it not being used any more. I am then able to spin up another vgpu instance | |
| 18:06:06 | colby_ | that is being done in the /sys/bus/pci/devices/<virtual function cpi address>/mdev_supported_types directory | |
| 18:06:32 | colby_ | we are on Centos8 stream and using Victoria version of openstack currently | |