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

[TMP][DO NOT MERGE] Test for seeing CI state #1388

Closed
wants to merge 1 commit into from

Conversation

pedrobaeza
Copy link
Member

No description provided.

@pedrobaeza
Copy link
Member Author

@LoisRForgeFlow there's a failing test in mrp_multi_level. Can you please check?

@pedrobaeza pedrobaeza added this to the 15.0 milestone Nov 11, 2024
@LoisRForgeFlow
Copy link
Contributor

I think it might be the same thing reported here #1307. It is probably a timing issue, I suspect there is a range of time in which that tests fails. In my local currently is fine, I restarted the CI here to see if it gets green now.

@pedrobaeza
Copy link
Member Author

I don't think so. I have just opened the PR (so not around midnight), and it's consistent failing in all the attempts.

@LoisRForgeFlow
Copy link
Contributor

I'm checking in local with freeze_time and I am not succeeding making that test fail. It can also be an interference with another module. Checking.

@LoisRForgeFlow
Copy link
Contributor

An update, the conflict is with mrp_production_date_planned_finished, I will check in more details when I have time later.

@LoisRForgeFlow
Copy link
Contributor

@pedrobaeza Today tests passed, which I suspected. So the issue is only happening on Mondays. I will try to get care of it during this week and propose a fix for next Monday. In the meanwhile your other PR should be no longer blocked.

@pedrobaeza
Copy link
Member Author

Wow, it's the first time I see that tests are blocked depending on the week day. It would be good to have that fixed. Launching the merge on the other PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants