[MIG][15.0] mail: migration script

[sang250399]

This PR

Migration done

[StefanRijnhart]

Thanks for the quick fixes! This is really great work especially for you as a new contributor.

I see that due to a small error in our scripting the mail module is not upgraded in this PRs github action. I'm hoping to have it fixed in #3278. Would you like to try and merge that branch in here?

[sang250399]

Thanks for the quick fixes! This is really great work especially for you as a new contributor.

I see that due to a small error in our scripting the mail module is not upgraded in this PRs github action. I'm hoping to have it fixed in #3278. Would you like to try and merge that branch in here?

okay, i will try

[StefanRijnhart]

Thanks, scripts are now run!

Testing modules base,base,mail
...
2022-06-14 09:47:58,233 5347 INFO openupgrade odoo.modules.migration: module mail: Running migration [>15.0.1.5] pre-migration 
...
2022-06-14 09:48:00,643 5347 INFO openupgrade odoo.modules.migration: module mail: Running migration [15.0.1.5>] post-migration

[legalsylvain]

/ocabot migration mail

[pedrobaeza]

Incorrect syntax

Suggested change

	| mail |Done | |
	| mail | Done | |

[pedrobaeza]

My comment is pending, and also on the test logs, there is the following line:

2022-06-15 06:18:28,759 5443 INFO openupgrade odoo.modules.graph: module mail: Unmet dependencies: base_setup, bus, web_tour 

so this is not correct and can't be merged. Please check it.

[royle-vietnam]

My comment is pending, and also on the test logs, there is the following line:

Done

2022-06-15 06:18:28,759 5443 INFO openupgrade odoo.modules.graph: module mail: Unmet dependencies: base_setup, bus, web_tour

Maybe the docsource file is not standard, |Nothing to do -> | Nothing to do, Can you do mass editing PR? @pedrobaeza

[pedrobaeza]

Yes, that syntax is not correct and it's not correctly parsed. Please fix it in the rest of your PRs. I'll fix the current main one.

[pedrobaeza]

Are you the one that have put No DB layout changes in the docs? That's not a valid text. It should be Nothing to do.

[royle-vietnam]

Yes, that syntax is not correct and it's not correctly parsed. Please fix it in the rest of your PRs. I'll fix the current main one.

Okey

[royle-vietnam]

Are you the one that have put No DB layout changes in the docs? That's not a valid text. It should be Nothing to do.

No, not me, looks like @legalsylvain

[legalsylvain]

Are you the one that have put No DB layout changes in the docs? That's not a valid text. It should be Nothing to do.

Hum.... @pedrobaeza :
OCA/server-tools#2206 (comment)

[pedrobaeza]

I didn't remember that this text is parsed by the tests:

OpenUpgrade/.github/workflows/test.yml

Line 103 in edb3b9c

| grep "Done\|Partial\|Nothing" \

so this has to be changed.

[pedrobaeza]

Sorry for the noise. @legalsylvain it was correct, as such column is an extra one. Digging more about why base_setup is not being considered.

[pedrobaeza]

This is weird, as now I have re-run the tests and they are working, so let's continue. I'm worried about these situations being masked on the CI and only searching for the green status. I will continue checking the logs of other PRs to see if this happens again.