tests/kola/experimental: fix LastRefreshTime method name#1260
tests/kola/experimental: fix LastRefreshTime method name#1260jlebon wants to merge 1 commit intocoreos:mainfrom
Conversation
We were using the wrong method name here. Likely fixes coreos#1176. (E.g. maybe what happened there is that previously zbus would return "Access denied" for unauthorized users even for non-existent methods and maybe the logic changed so the existence check was done first.)
|
I tried manually with the build and failed, it is wired that |
Hmm, I couldn't reproduce it locally. Oh! I bet the test is racing with Zincati actually setting up D-Bus. Can you try adding a e.g. a loop that waits until |
I am able to reproduce this issue, even with this fix. Did you update zbus ? |
|
OK yes, I can reproduce it now. But actually, Zincati isn't exposing a D-Bus service at all (which explains the error message), so there's something more fundamental going wrong here. I think possibly related to the way we use zbus+actix. |
Thanks! It is out of my knowledge now. |
|
I added the commit from this PR into #1309 |
4 participants
We were using the wrong method name here.
Likely fixes #1176.
(E.g. maybe what happened there is that previously zbus would return "Access denied" for unauthorized users even for non-existent methods and maybe the logic changed so the existence check was done first.)