Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test against ehelm's upgrade-rails-6.0.4.7 branch #678

Closed
wants to merge 1 commit into from
Closed

Conversation

evgeni
Copy link
Member

@evgeni evgeni commented Mar 21, 2022

No description provided.

@ezr-ondrej
Copy link
Member

Looks like passing

@evgeni
Copy link
Member Author

evgeni commented Mar 22, 2022

it didn't the first time, which is confusing…

https://github.com/theforeman/foreman-tasks/runs/5632286413?check_suite_focus=true

@ezr-ondrej
Copy link
Member

Looks like quite "innocent" transient failure, the fulltext search found task that it was not supposed to find, but that might have quite a lot of unrelated reasons (it matched the uuid, description, or whatever else we use the fullsearch on). Disclaimer: I did not find the attribute it could have matched, so I'm unclear about the exact cause.
Tho I do not get why we test on Ruby 2.6 anyway 😛

@ehelms
Copy link
Member

ehelms commented Mar 22, 2022

Tho I do not get why we test on Ruby 2.6 anyway

#677 would drop that aspect

@evgeni
Copy link
Member Author

evgeni commented Mar 22, 2022

Closing this PR ;)

@evgeni evgeni closed this Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants