Fixes #37150 - Race condition in edit smart proxy test

[dosas]

No description provided.

[theforeman-bot]

Can one of the admins verify this patch?

[theforeman-bot]

Can one of the admins verify this patch?

[theforeman-bot]

Can one of the admins verify this patch?

[ekohl]

Ok to test

[stejskalleos]

[test katello]

[stejskalleos]

Integration tests are failing on:

    test/integration_test_helper.rb:302:in `login_admin'
    test/integration_test_helper.rb:331:in `before_setup',Selenium::WebDriver::Error::InvalidSessionIdError: invalid session id

Guess it's not related to this changes, let's if they'll fail on rerun again

[dosas]

@stejskalleos Looks to me like the integration tests are failing all over the place for some time now.
https://github.com/theforeman/foreman/actions/runs/7934552063
I would be surprised it rerun works.

[stejskalleos]

Yeah without green CI I can't be sure that this actually fixes the issue.
cc @ekohl @evgeni

[evgeni]

you can temporarily pull in the changes from #10034 and see if things pass.

chrome 122 is supposed to be released today and fix the issue (fingers crossed)

[dosas]

@evgeni Are there any news on the chromedriver? The PR you mentioned above is still a draft.
Should I rebase?

[evgeni]

Yes, rebase please, there is now a fixed chrome deployed.

[evgeni]

Thanks @dosas and sorry it took way too long for a one-liner.