| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-06 | |||
| 09:15:03 | EugenMayer | With openstack, are there only web-based consoles? I would like to have a terminal based serial console so c&p works. Any hints on how to get this working? | |
| 10:33:05 | lyarwood | EugenMayer: https://docs.openstack.org/nova/queens/admin/remote-console-access.html#serial-console I thought this produced a connection URI you could use from a cli ? | |
| 10:55:38 | EugenMayer | lyarwood asking myself, which client can be used on the cli | |
| 10:56:13 | lyarwood | it's actually a websocket so something like this should work | |
| 10:56:15 | EugenMayer | lyarwood it is not like stty or similar would work | |
| 10:56:16 | lyarwood | $ sudo dnf copr enable atim/websocat -y && sudo dnf install websocat | |
| 10:56:24 | lyarwood | $ websocat $(openstack console url show --serial test -c url -f value) | |
| 10:56:39 | lyarwood | ^ that's on fedora 35 | |
| 10:56:45 | lyarwood | https://github.com/vi/websocat is the tool | |
| 10:56:53 | lyarwood | there are likely others | |
| 10:57:08 | EugenMayer | interesting, will try that! Nuked the entire cluster just right now and bootstrapping with terraform. Both lab and DC are nuke, so takes some time now | |
| 10:57:21 | lyarwood | cool, gl | |
| 10:57:43 | EugenMayer | lyarwood have a nice weekend! | |
| 18:34:34 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/776250 | |
| 18:46:40 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/776250 | |
| 18:59:33 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/776250 | |
| #openstack-nova - 2021-11-07 | |||
| 10:03:48 | EugenMayer | lyarwood tried websocat with ./websocat_linux64 'wss://fqdn:6083?token=xx' it connected but i cannot see any terminal or somewhat | |
| 23:11:37 | opendevreview | David Hill proposed openstack/nova stable/xena: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/816882 | |
| 23:11:58 | opendevreview | David Hill proposed openstack/nova stable/wallaby: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/816883 | |
| #openstack-nova - 2021-11-08 | |||
| 00:12:48 | opendevreview | David Hill proposed openstack/nova stable/victoria: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/816927 | |
| 05:02:21 | opendevreview | Rajat Dhasmana proposed openstack/nova-specs master: Add spec for volume backed server rebuild https://review.opendev.org/c/openstack/nova-specs/+/809621 | |
| 07:52:00 | EugenMayer | good morning. the default_availability_zone setting, where does it belong, on the nova-schedular service or nova-compute on the compute? Kind of make sense on the former | |
| 07:55:15 | EugenMayer | maybe in common, when reading https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.default_availability_zone - how to understand, for each of those values, which service use them? Since i use kolla, i have a lot of nova.conf files and default_schedule_zone could go on potentially several of those. How would i find those | |
| 07:55:15 | EugenMayer | informations in the docs? | |
| 08:55:20 | gibi | EugenMayer: for some of the config the docs states which service uses them, but some it does not. In the latter case you need to look into the code | |
| 09:01:45 | gibi | EugenMayer: based on a quick grep for CONF.default_availability_zone it is used by the scheduler and the api service | |
| 09:12:39 | bauzas | good morning | |
| 09:13:07 | bauzas | gibi: EugenMayer: correct, we use the default AZ value in the API | |
| 09:13:44 | bauzas | if none AZ is provided, then we look at the default one, which is 'None' | |
| 09:14:12 | gibi | bauzas: o/ | |
| 09:14:33 | bauzas | if an operator sets the default_az value, then any instance which would be created will have an AZ | |
| 09:14:55 | bauzas | either the default AZ if the user doesn't ask for it, or then the AZ that the user provides with --az | |
| 09:21:25 | EugenMayer | bauzas / gibi thank you. The question is in which group? | |
| 09:27:42 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to use NUMA cell with free resources first https://review.opendev.org/c/openstack/nova/+/805649 | |
| 10:06:41 | bauzas | EugenMayer: sorry, missed your ping | |
| 10:06:51 | bauzas | EugenMayer: you ask about the config group ? | |
| 10:07:58 | EugenMayer | yes since beside one does not know which service, one does not know the group bauzas - kind of heavily lifting expected to look this up in the source code but well, i guess i could just contribute the docs and stop complaining :) | |
| 10:12:09 | opendevreview | Martin Kopec proposed openstack/nova master: Update Interop doc https://review.opendev.org/c/openstack/nova/+/816980 | |
| 10:48:44 | opendevreview | Jan Hartkopf proposed openstack/nova master: add support for updating server's user_data https://review.opendev.org/c/openstack/nova/+/816157 | |
| 11:20:11 | opendevreview | wujian proposed openstack/nova master: Fix instance action event result inconsistent https://review.opendev.org/c/openstack/nova/+/816990 | |
| 11:48:10 | opendevreview | David Hill proposed openstack/nova stable/victoria: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/816927 | |
| 12:14:30 | gibi | lyarwood: hi! could please look at the stable/victoria backport of https://review.opendev.org/q/topic:bug/1944759 elodilles already has +2 on it | |
| 12:15:41 | lyarwood | gibi: ACK'd | |
| 12:15:48 | gibi | thanks! | |
| 12:22:13 | gibi | bauzas: replied to your question in https://review.opendev.org/c/openstack/nova/+/815690/comment/764e7a47_1cdc2d2d/ | |
| 12:29:45 | sean-k-mooney | stephenfin: im going to respin this shortly can you remove your -2 on https://review.opendev.org/c/openstack/nova/+/804292 | |
| 12:31:19 | sean-k-mooney | lyarwood: ^ that is what we agreed to do at the ptg by the way so that will allow you to avoid issue with 3.10 | |
| 12:31:53 | lyarwood | ah cool sorry missed that | |
| 12:31:55 | sean-k-mooney | although we proably need to start testing with 3.10 this cycle at least non voting | |
| 12:32:23 | sean-k-mooney | im going to quickly repin my autopep8 patch and then that | |
| 12:37:47 | opendevreview | sean mooney proposed openstack/nova master: Add autopep8 to tox and pre-commit https://review.opendev.org/c/openstack/nova/+/806182 | |
| 12:38:15 | opendevreview | Takashi Kajinami proposed openstack/nova stable/xena: Clean up allocations left by evacuation when deleting service https://review.opendev.org/c/openstack/nova/+/816954 | |
| 12:38:30 | sean-k-mooney | lyarwood: gibi if we can merge ^ relitivily quickly that woudl be great as its enforcing that all patch pass autopep8 without needing file changes | |
| 12:44:34 | opendevreview | sean mooney proposed openstack/nova master: This change replaces all hardcoded tox enve with generative envs https://review.opendev.org/c/openstack/nova/+/804292 | |
| 12:46:22 | lyarwood | sean-k-mooney: is that mostly picking up a rule we skip with flake8? | |
| 12:46:50 | sean-k-mooney | autopep8 actully read the tox file for what we skip | |
| 12:46:52 | sean-k-mooney | but yes | |
| 12:47:07 | sean-k-mooney | i dont know how to ignore that 1 new line for doc stings | |
| 12:47:21 | sean-k-mooney | that basicly the only change it really makes today | |
| 12:47:50 | lyarwood | ah right that's why I was asking, I couldn't tell if it was picking up our skip list or not | |
| 12:48:29 | sean-k-mooney | ya it reads the flake8 section | |
| 12:49:26 | sean-k-mooney | so it should respect ignore = E121,E122,E123,E124,E125,E126,E127,E128,E129,E131,E251,H405,W504,E731,H238 | |
| 12:49:48 | sean-k-mooney | if we know what rule was adding the new line we could add it to the list | |
| 12:50:26 | lyarwood | tbh I'd rather do that and avoid the huge diff | |
| 12:50:51 | lyarwood | brb | |
| 12:51:31 | sean-k-mooney | well its much smaller then black ecta but sure | |
| 12:51:55 | sean-k-mooney | looking at https://github.com/hhatto/autopep8#features there are seveeral that are fixing indentaiton or addign missing blank lines | |
| 12:58:14 | sean-k-mooney | it might be comeing form https://www.python.org/dev/peps/pep-0257/ | |
| 13:01:01 | sean-k-mooney | """Insert a blank line after all docstrings (one-line or multi-line) that document a class -- generally speaking, the class's methods are separated from each other by a single blank line, and the docstring needs to be offset from the first method by a blank line.""" | |
| 13:01:14 | sean-k-mooney | https://www.python.org/dev/peps/pep-0257/#multi-line-docstrings | |
| 13:01:23 | sean-k-mooney | lyarwood: ^ that is where that si cominng form | |
| 13:02:38 | sean-k-mooney | ah and its called out in the autopep8 | |
| 13:02:40 | sean-k-mooney | Put a blank line between a class docstring and its first method declaration. (Enabled with E301.) | |
| 13:03:27 | sean-k-mooney | E301 is inconsitent tabs and spaces | |
| 13:03:29 | sean-k-mooney | https://www.flake8rules.com/rules/E101.html | |
| 13:08:12 | lyarwood | and that's enabled so why isn't flake8 picking it up already? | |
| 13:08:29 | sean-k-mooney | its not part of pep8 | |
| 13:08:41 | lyarwood | ah | |
| 13:08:46 | sean-k-mooney | E301 does not specify the docstring behavior | |
| 13:09:00 | sean-k-mooney | auto pep8 impleted as part of the E301 fixing | |
| 13:09:04 | lyarwood | sorry I thought you said it did, I see what you're sayingnow | |
| 13:13:14 | sean-k-mooney | lyarwood: summerised it in the review https://review.opendev.org/c/openstack/nova/+/806182/3#message-f37506ed0411c71d9ce66de251196048da171b34 | |
| 13:15:31 | gibi | sean-k-mooney: I have one question in autopep8 https://review.opendev.org/c/openstack/nova/+/806182/comment/e15200f9_a50f0219/ | |
| 13:16:08 | sean-k-mooney | we can add it to test-requiremetns yes | |
| 13:16:21 | sean-k-mooney | i just did not to avoid installing it in enves that did not need it | |
| 13:16:36 | sean-k-mooney | but if you prefer that i can update it shortly | |
| 13:16:41 | gibi | hm, will we ever need to pin the version of autopep8? | |
| 13:17:07 | sean-k-mooney | this can be done with upperconstraitns | |
| 13:17:11 | sean-k-mooney | or inline | |
| 13:17:36 | sean-k-mooney | i dont think so but its a vaild question | |
| 13:17:50 | gibi | currently it can only be inline afaik. but anyhow we don't pin it now, and we can move it to test-requirements when we need to pin it | |
| 13:17:59 | gibi | so I'm +2 | |
| 13:19:04 | sean-k-mooney | for line 69 yes since that does not include the base test env dep for the pep8 env the upper constratis form the base env still apply | |
| 13:22:56 | lyarwood | okay LGTM | |
| 13:23:27 | lyarwood | oh wait, downstream meeting :| | |
| 13:23:41 | lyarwood | helps if I look at the correct day in my cal | |
| 13:26:13 | opendevreview | Merged openstack/nova master: db: Remove legacy placement models https://review.opendev.org/c/openstack/nova/+/812146 | |
| 13:26:23 | opendevreview | Merged openstack/nova master: objects: Stop querying the main DB for keypairs https://review.opendev.org/c/openstack/nova/+/812147 | |
| 13:50:21 | bauzas | gibi: +w'd your series | |
| 13:50:27 | gibi | bauzas: thanks | |