Wednesday, 2023-08-02

opendevreviewDr. Jens Harbott proposed openstack/devstack master: Add rocky job to gate  https://review.opendev.org/c/openstack/devstack/+/88546905:02
opendevreviewDr. Jens Harbott proposed openstack/devstack master: Add option to install everything in global venvs  https://review.opendev.org/c/openstack/devstack/+/55893005:09
opendevreviewDr. Jens Harbott proposed openstack/devstack master: Add debian-bookworm job  https://review.opendev.org/c/openstack/devstack/+/88754705:11
ykarelkopecmartin, the patches i proposed yesterday were for different issue06:50
ykarelfor current CentOS 9-stream issue reported https://bugs.launchpad.net/neutron/+bug/202933506:51
ykarellast night patches were just backport to fix C9-stream jobs in stable branches https://review.opendev.org/#/q/Icd99f467d47aaafaaf3ee8f2a3c4da08842cb67206:52
kopecmartinykarel: i see, thanks07:10
opendevreviewyatin proposed openstack/devstack master: Woraround systemd issue on CentOS 9-stream  https://review.opendev.org/c/openstack/devstack/+/89028007:43
kevkohi, second day i'm debugging tempest.scenario.test_server_volume_attachment.TestServerVolumeAttachmentScenario.test_server_detach_rules and still failing ... i have configured service_user on both sides cinder and nova ...but still getting unathorized 07:51
kevkohttps://paste.openstack.org/show/bYVAmLV2BQqoxE9LuRGa/07:52
kevkocan anyone help ? 07:59
fricklerykarel: did you consider switching neutron CI to rocky instead? devstack is gating on that platform now, seems much more stable10:13
opendevreviewJakub Skunda proposed openstack/devstack stable/2023.1: git: git checkout for a commit hash combinated with depth argument  https://review.opendev.org/c/openstack/devstack/+/89033213:41
opendevreviewJakub Skunda proposed openstack/devstack stable/zed: git: git checkout for a commit hash combinated with depth argument  https://review.opendev.org/c/openstack/devstack/+/89033313:43
opendevreviewJakub Skunda proposed openstack/devstack stable/xena: git: git checkout for a commit hash combinated with depth argument  https://review.opendev.org/c/openstack/devstack/+/89033413:44
opendevreviewJakub Skunda proposed openstack/devstack stable/wallaby: git: git checkout for a commit hash combinated with depth argument  https://review.opendev.org/c/openstack/devstack/+/89033713:53
opendevreviewJakub Skunda proposed openstack/devstack stable/victoria: git: git checkout for a commit hash combinated with depth argument  https://review.opendev.org/c/openstack/devstack/+/89033813:54
dansmithwow, the top thing in gate right now has been running jobs for 2 hours and one job is still in the middle of devstack, running incredibly slowly, like one line of output at a time16:01
dansmithso definitely some sort of very slow worker16:01
dansmithslaweq: When we see a test that has a server that gets no active interface, can't set an IP and can't be SSH'd to, what should we be looking for?16:09
dansmithhttps://2e881fa3fb67c342c77f-bc5fbdd8c1ab00cec2f92d8667e378c5.ssl.cf2.rackcdn.com/888470/1/check/nova-multi-cell/75ab1ec/testr_results.html16:10
dansmiththen: no interfaces have a carrier16:10
opendevreviewDan Smith proposed openstack/tempest master: Make stamp_pattern wait for SSHABLE  https://review.opendev.org/c/openstack/tempest/+/89034816:22
opendevreviewDan Smith proposed openstack/tempest master: Make stamp_pattern wait for SSHABLE  https://review.opendev.org/c/openstack/tempest/+/89034816:31
opendevreviewDan Smith proposed openstack/tempest master: Use vfat for timestamp  https://review.opendev.org/c/openstack/tempest/+/89035016:43
dansmithmelwitt: this ^ is the change that is potentially helpful for the mkfs problem16:43
melwittcool, it's worth a shot16:44
dansmithmelwitt: did you have a test you depends-on'd your discard change that you used to determine that the discard thing wasn't helping?16:44
melwittdansmith: no but the discard patch itself failed with the mke2fs timeout 🫤 16:45
melwittso I gave up on it immediately16:45
dansmithoh I didn't see because it's +1 now, but I guess because it was rebased16:45
melwittoh it looks ilke sean rebased it yeah16:47
dansmithunfortunately the giant nova-manila set has swamped the gate and nothing submitted in the last 3h is even running16:47
dansmithso won't get any data on that any time soon16:48
dansmithmelwitt: not sure if you saw my reply on the sshable one, but my link above about unable to be sshable is why I made that change16:57
dansmitheven though it may be futile, I think that console log means we decided about 1/4 of the way through it waiting for reachable metadata that it was un-ssh-able because the regular client call just tries once and bails16:58
melwittok, I guess I saw "2023-08-02 15:35:05,684 97347 ERROR    [tempest.lib.common.ssh] Failed to establish authenticated ssh connection to cirros@172.24.5.193 after 15 attempts. Proxy client: no proxy client" and the prior attempts. those are logged before the console log though17:01
dansmithum, okay maybe I'm missing something about where that's getting retried17:02
dansmithah, in _get_ssh_connection() I see17:03
melwittyeah.. there's at least a few layers but I saw the retry stuff here https://github.com/openstack/tempest/blob/4025fa3c8e5ba8de3f2cb60f4a225f1bc550b5e4/tempest/lib/common/ssh.py#L12917:03
dansmithaight, well, we can drop that then if you want17:05
melwittthere's still a chance in might change something bc it would be waiting for ssh before adding the floating ip. it would be interesting to see if it does17:12
dansmithit also just makes that part of the server create like all the other things17:14
dansmithnot sure that really matters much, but the consistency seems good17:14
dansmithwe were initially going to set everything to SSHABLE by default but couldn't for a few reasons, so I think this is at least in support of that consistency goal if nothing else17:15
melwittyeah, I had been thinking that but convinced myself it would be considered wasteful to add another ssh-able in, so I abandoned the idea. I think consistency is always a good thing17:17
opendevreviewMerged openstack/tempest master: create default network for few more server tests  https://review.opendev.org/c/openstack/tempest/+/89024218:09
opendevreviewmelanie witt proposed openstack/tempest master: DNM test rbd retype  https://review.opendev.org/c/openstack/tempest/+/89036019:58
dansmithgmann: really seems like the increased concurrency is pushing us to more OOMs21:11
dansmithwhen I look at dumps, mysql isn't even using the most memory, but the six qemus are bigger of course, but with more pinned memory21:11
melwittgmann: I'm trying to test something by adding an assert to a scenario test but I want it to run against wallaby.. I used "override-checkout: stable/wallaby" in my job definition and that checks out stuff for wallaby as expected, but the change I made to the tempest scenario test is not being used. is there a different way to do such a thing?21:32
dansmithmelwitt: hmm I just thought about something.. what if you add required-projects: tempest?21:37
dansmiththat normally says "get this from source" like for libraries21:38
dansmithalthough that might cause it to just checkout master, I dunno21:38
gmannmelwitt: dansmith: on wallaby we use old tempest so it would not pick up the master change21:38
dansmithgmann: yeah I figured.. so there's no way to arrange it?21:39
gmannmelwitt: dansmith: set these variable in job and it will use master tempest TEMPEST_BRANCH:master TEMPEST_VENV_UPPER_CONSTRAINTS:master21:40
gmannwhether it will raise any requirement constraints issue or not that is something difficult to solve but you can try this way and see if master tempest/constarints work against wallaby or not21:40
melwittgmann: sweet, thanks21:41
dansmithpresumably it might also fail a bunch of other tests as a result of being on master now too21:41
gmanndansmith: on concurrency, is oom issues everytime ? and which job you are seeing it more?21:42
dansmithgmann: not everytime, but a bunch today21:42
dansmithall jobs I think21:42
dansmithI mean, a bunch of different ones, not any specific ones21:42
dansmithnot on ceph though, but probably because it's limited to 321:42
gmanndansmith: more than test I think constraints will create issue as installing tempest with master constraints might conflict with what we have on wallaby but let's see if that work as wallaby is not so old21:42
opendevreviewmelanie witt proposed openstack/tempest master: DNM test rbd retype  https://review.opendev.org/c/openstack/tempest/+/89036021:46
opendevreviewGhanshyam proposed openstack/tempest master: DNM test rbd retype  https://review.opendev.org/c/openstack/tempest/+/89036023:18
gmannmelwitt: ^^ that was still using stable constraints due to those setting in role. it should use master constraints now23:19
melwittthank you gmann 🙏 23:20
gmannnp!23:22
opendevreviewGhanshyam proposed openstack/tempest master: DNM test rbd retype  https://review.opendev.org/c/openstack/tempest/+/89036023:57
gmannmelwitt: that worked for wallaby but not for train due to oslo.concurrency constraints. trying if ^^ this will work 23:57
melwittgmann: really cool, thank you :)23:59

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!