This repository has been archived by the owner on Nov 29, 2022. It is now read-only.
[IMP] travis_install_nightly: Patch odoo to create unlogged DB tables #283
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
luisg123v
force-pushed
the
master-unlogged2-moy
branch
8 times, most recently
from
9579f66 to
0577c67
Compare
moylop260
commented
Sep 11, 2018
travis/test_server.py
Outdated
| @@ -355,6 +355,33 @@ def run_from_env_var(env_name_startswith, environ): | |||
| raise RuntimeWarning("Return different to zero") | |||
|
|
|||
|
|
|||
| def patch_odoo_unlogged_tables(server_path): | |||
| """ Patch Odoo to make it to create all DB tables as unlogged """ | |||
travis/test_server.py
Outdated
| if os.path.isdir(os.path.join(server_path, "odoo")): | ||
| odoo_subdir = "odoo" | ||
| else: | ||
| odoo_subdir = "openerp" |
travis/test_server.py
Outdated
| with open(filename) as f_to_patch: | ||
| if not re.search("create table", f_to_patch.read(), re.IGNORECASE): | ||
| continue | ||
| print("\tPatchin file %s" % filename) |
travis/test_server.py
Outdated
| @@ -405,6 +432,8 @@ def main(argv=None): | |||
| odoo_full = os.environ.get("ODOO_REPO", "odoo/odoo") | |||
| server_path = get_server_path( | |||
| odoo_full, odoo_branch or odoo_version, travis_home) | |||
| if is_runbot: | |||
There was a problem hiding this comment.
if it is runbot and it is gitlab
travis/travis_after_tests_success
Outdated
| # Stop PostgreSQL service. Even though the container is stopped once | ||
| # this script finishes, if changes are not saved before that, it | ||
| # could lead to inconsistencies on unlogged tables | ||
| subprocess.call(["/etc/init.d/postgresql", "stop"]) |
There was a problem hiding this comment.
remove it
it is not needed anymore
|
Create a dummy please |
|
Get last changes runbot-addons from oca to vx |
luisg123v
force-pushed
the
master-unlogged2-moy
branch
2 times, most recently
from
db4dbd3 to
e47535e
Compare
|
This commit causes Odoo to be patched so all database tables are created as unlogged. This is applied only for testing instances because, if a docker container is stopped without firstly saving the changes, all records will be deleted; so it's not suitable for working under other environments, e.g. locally. This is compatible with the versions 7.0 to master (12.0). Closes Vauxoo#282 Co-authored-by: Moisés López <[email protected]>
luisg123v
force-pushed
the
master-unlogged2-moy
branch
from
e47535e to
0f2ec01
Compare
|
Forget it, I have created the following PR: OCA#573 |
|
For the record, We missed travis-ci (It is patching just for runbot, gitlab-ci but not for travis-ci) |
luisg123v
added a commit
to vauxoo-dev/maintainer-quality-tools
that referenced
this pull request
Sep 12, 2018
This commit causes Odoo to be patched so all database tables are created as unlogged. This is applied only for testing instances because, if a docker container is stopped without firstly saving the changes, all records will be deleted; so it's not suitable for working under other environments, e.g. locally. This is compatible with the versions 7.0 to master (12.0). Co-authored-by: Moisés López <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit causes Odoo to be patched so all database tables are created
as unlogged, just after it's clonned.
It is applied just for runbot instance since that the records are
deleted after a docker stop if the postgreql service is started
This is compatible with the versions 7.0 to master (12.0).
Closes #279