Compare commits
11 Commits
acd4b464a7
...
main
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
81e036cd4a | ||
|
|
aa48543a12 | ||
|
|
9cbb5eaff2 | ||
|
|
ef75d4a5b1 | ||
|
|
d2c4ec6de5 | ||
|
|
e27c309d06 | ||
|
|
8ea746a4d1 | ||
|
|
778730e396 | ||
|
|
969414e649 | ||
|
|
f9d678508b | ||
|
|
55218946c5 |
1
.gitignore
vendored
1
.gitignore
vendored
@@ -1 +1,2 @@
|
||||
final_404_addons
|
||||
migration.log
|
||||
|
||||
74
README.md
74
README.md
@@ -9,6 +9,7 @@ A tool for migrating Odoo databases between major versions, using [OpenUpgrade](
|
||||
- [Project Structure](#project-structure)
|
||||
- [How It Works](#how-it-works)
|
||||
- [Usage](#usage)
|
||||
- [Resuming a Failed Migration](#resuming-a-failed-migration)
|
||||
- [Customization](#customization)
|
||||
- [Troubleshooting](#troubleshooting)
|
||||
|
||||
@@ -158,7 +159,7 @@ The script performs a **step-by-step migration** between each major version. For
|
||||
### Running the Migration
|
||||
|
||||
```bash
|
||||
./upgrade.sh <source_version> <target_version> <database_name> <source_service>
|
||||
./upgrade.sh <source_version> <target_version> <database_name> <source_service> [--resume-from|-r <version>]
|
||||
```
|
||||
|
||||
**Parameters:**
|
||||
@@ -169,6 +170,11 @@ The script performs a **step-by-step migration** between each major version. For
|
||||
| `database_name` | Database name | `my_prod_db` |
|
||||
| `source_service` | Source Docker Compose service | `odoo14` |
|
||||
|
||||
**Options:**
|
||||
| Option | Description |
|
||||
|--------|-------------|
|
||||
| `--resume-from <version>`, `-r <version>` | Resume from an intermediate checkpoint (see [Resuming a Failed Migration](#resuming-a-failed-migration)) |
|
||||
|
||||
**Example:**
|
||||
```bash
|
||||
./upgrade.sh 14 17 elabore_20241208 odoo14
|
||||
@@ -239,6 +245,48 @@ compose run odoo17 shell -d ou17 --no-http --stop-after-init < lib/python/valida
|
||||
- **JSON report** written to `/tmp/validation_views_<db>_<timestamp>.json`
|
||||
- **Exit code**: `0` = success, `1` = errors found
|
||||
|
||||
## Resuming a Failed Migration
|
||||
|
||||
Each version hop copies the database before modifying it (`ou14` → `ou15` → `ou16` → …). If a migration crashes mid-way, the intermediate databases from completed hops are still intact and can be used as a restart point.
|
||||
|
||||
### How It Works
|
||||
|
||||
Use `--resume-from <version>` (or `-r <version>`) to restart from an intermediate checkpoint:
|
||||
|
||||
```bash
|
||||
./upgrade.sh <source_version> <target_version> <database_name> <source_service> --resume-from <checkpoint_version>
|
||||
```
|
||||
|
||||
When a checkpoint is specified, the script:
|
||||
- **Skips** the initial database and filestore copy
|
||||
- **Skips** `prepare_db.sh` (already done before the crash)
|
||||
- **Starts the migration loop** from `checkpoint_version + 1`
|
||||
|
||||
### Example
|
||||
|
||||
A migration from 14 to 18 crashes during the 16→17 hop. The database `ou15` was successfully created. Resume from there:
|
||||
|
||||
```bash
|
||||
./upgrade.sh 14 18 my_database odoo14 --resume-from 15
|
||||
```
|
||||
|
||||
This runs: `16.0 → 17.0 → 18.0`, starting from `ou15`.
|
||||
|
||||
### Constraints
|
||||
|
||||
- The checkpoint version must be strictly between the source and target versions.
|
||||
- The intermediate database (`ou<version>`) and its filestore must exist before resuming.
|
||||
|
||||
### Restarting From Scratch
|
||||
|
||||
To restart a full migration from the beginning (ignoring all intermediate databases):
|
||||
|
||||
```bash
|
||||
./upgrade.sh 14 18 my_database odoo14
|
||||
```
|
||||
|
||||
The script automatically drops and recreates the final target database (`ou18`) if it already exists.
|
||||
|
||||
## Customization
|
||||
|
||||
### Version Scripts
|
||||
@@ -365,10 +413,30 @@ The script works on a **copy** of the original database. You can restart as many
|
||||
|
||||
### Viewing Detailed Logs
|
||||
|
||||
Odoo/OpenUpgrade logs are displayed in real-time. For a problematic migration:
|
||||
All output (scripts, Docker containers, OpenUpgrade logs) is automatically saved to `migration.log` at the project root. The file is overwritten at the start of each run (full migration or resume).
|
||||
|
||||
```bash
|
||||
# Follow live output while also saving to file (already done automatically)
|
||||
tail -f migration.log
|
||||
|
||||
# Search for errors after the fact
|
||||
grep -i error migration.log
|
||||
|
||||
# View with ANSI colors preserved (less handles CRLF line endings natively)
|
||||
less -R migration.log
|
||||
|
||||
# Plain text output without CRLF artifacts
|
||||
cat migration.log | col -b
|
||||
```
|
||||
|
||||
> **Note:** `migration.log` is recorded via a pseudo-TTY (`script`), which preserves
|
||||
> color output in the terminal. As a side effect, the file uses `\r\n` line endings.
|
||||
> Use `less -R` or `col -b` for clean viewing.
|
||||
|
||||
For a problematic migration:
|
||||
|
||||
1. Note the version where the error occurs
|
||||
2. Check the logs to identify the problematic module/table
|
||||
2. Check `migration.log` to identify the problematic module/table
|
||||
3. Add a fix in the `pre_upgrade.sh` for that version
|
||||
4. Restart the migration
|
||||
|
||||
|
||||
@@ -14,7 +14,7 @@ readonly FILESTORE_SUBPATH="var/lib/odoo/filestore"
|
||||
|
||||
check_required_commands() {
|
||||
local missing=()
|
||||
for cmd in docker compose sudo rsync; do
|
||||
for cmd in docker compose sudo rsync yq; do
|
||||
if ! command -v "$cmd" &>/dev/null; then
|
||||
missing+=("$cmd")
|
||||
fi
|
||||
@@ -100,7 +100,83 @@ exec_python_script_in_odoo_shell() {
|
||||
run_compose --debug run "$service_name" shell -d "$db_name" --no-http --stop-after-init < "$python_script"
|
||||
}
|
||||
|
||||
# Classifies missing modules into 4 categories based on the known_changes.yaml
|
||||
# files from each traversed version (from ORIGIN_VERSION+1 to FINAL_VERSION).
|
||||
# The following global arrays are populated:
|
||||
# addons_obsolete : modules that became obsolete
|
||||
# addons_core : modules merged into Odoo Core
|
||||
# addons_renamed : renamed modules (format "old_name -> new_name")
|
||||
# addons_truly_missing : modules that are genuinely missing
|
||||
#
|
||||
# Prerequisites: ORIGIN_VERSION and FINAL_VERSION must be exported.
|
||||
classify_missing_addons() {
|
||||
local missing_addons_raw="$1"
|
||||
|
||||
addons_obsolete=()
|
||||
addons_core=()
|
||||
addons_renamed=()
|
||||
addons_truly_missing=()
|
||||
|
||||
# Convert the string into an array (one entry per line)
|
||||
local -a missing=()
|
||||
while IFS= read -r line; do
|
||||
[[ -n "$line" ]] && missing+=("$line")
|
||||
done <<< "$missing_addons_raw"
|
||||
|
||||
if [[ ${#missing[@]} -eq 0 ]]; then
|
||||
return 0
|
||||
fi
|
||||
|
||||
# Build lookup tables from all known_changes.yaml files in traversed versions
|
||||
local -A known_obsolete=()
|
||||
local -A known_core=()
|
||||
local -A known_renamed=()
|
||||
|
||||
local versions_path="${PROJECT_ROOT}/versions"
|
||||
local v
|
||||
for v in $(seq $((ORIGIN_VERSION + 1)) "$FINAL_VERSION"); do
|
||||
local yaml_file="${versions_path}/${v}.0/known_changes.yaml"
|
||||
[[ -f "$yaml_file" ]] || continue
|
||||
|
||||
local mod
|
||||
while IFS= read -r mod; do
|
||||
[[ -n "$mod" && "$mod" != "null" ]] && known_obsolete["$mod"]=1
|
||||
done < <(yq '.obsolete[]?' "$yaml_file" 2>/dev/null)
|
||||
|
||||
while IFS= read -r mod; do
|
||||
[[ -n "$mod" && "$mod" != "null" ]] && known_core["$mod"]=1
|
||||
done < <(yq '.merged_in_core[]?' "$yaml_file" 2>/dev/null)
|
||||
|
||||
local count
|
||||
count=$(yq '.renamed | length' "$yaml_file" 2>/dev/null)
|
||||
if [[ "$count" =~ ^[0-9]+$ && "$count" -gt 0 ]]; then
|
||||
local i
|
||||
for ((i = 0; i < count; i++)); do
|
||||
local old new
|
||||
old=$(yq ".renamed[$i].old" "$yaml_file" 2>/dev/null)
|
||||
new=$(yq ".renamed[$i].new" "$yaml_file" 2>/dev/null)
|
||||
[[ -n "$old" && "$old" != "null" ]] && known_renamed["$old"]="$new"
|
||||
done
|
||||
fi
|
||||
done
|
||||
|
||||
# Classify each missing module
|
||||
local mod
|
||||
for mod in "${missing[@]}"; do
|
||||
if [[ -n "${known_obsolete[$mod]:-}" ]]; then
|
||||
addons_obsolete+=("$mod")
|
||||
elif [[ -n "${known_core[$mod]:-}" ]]; then
|
||||
addons_core+=("$mod")
|
||||
elif [[ -n "${known_renamed[$mod]:-}" ]]; then
|
||||
addons_renamed+=("${mod} -> ${known_renamed[$mod]}")
|
||||
else
|
||||
addons_truly_missing+=("$mod")
|
||||
fi
|
||||
done
|
||||
}
|
||||
|
||||
export PROJECT_ROOT DATASTORE_PATH FILESTORE_SUBPATH
|
||||
export -f log_info log_warn log_error log_step confirm_or_exit
|
||||
export -f check_required_commands
|
||||
export -f query_postgres_container copy_database copy_filestore run_compose exec_python_script_in_odoo_shell
|
||||
export -f classify_missing_addons
|
||||
|
||||
141
lib/python/regenerate_pos_hashes.py
Normal file
141
lib/python/regenerate_pos_hashes.py
Normal file
@@ -0,0 +1,141 @@
|
||||
#!/usr/bin/env python3
|
||||
"""Regenerate all POS order inalterability hashes and sequence numbers.
|
||||
|
||||
This script should only be called when the SQL pre-check in finalize_db.sh
|
||||
has confirmed that some pos_orders are missing l10n_fr_hash or
|
||||
l10n_fr_secure_sequence_number.
|
||||
|
||||
It resets ALL sequence numbers and hashes from scratch, in chronological order
|
||||
(date_order ASC, id ASC), and re-establishes the full hash chain.
|
||||
|
||||
Usage (via Odoo shell):
|
||||
odoo-shell -d <db_name> < regenerate_pos_hashes.py
|
||||
"""
|
||||
import logging
|
||||
from hashlib import sha256
|
||||
|
||||
_logger = logging.getLogger(__name__)
|
||||
|
||||
|
||||
def regenerate_all():
|
||||
PosOrder = env['pos.order']
|
||||
Company = env['res.company']
|
||||
|
||||
for company in Company.search([]):
|
||||
if not company._is_accounting_unalterable():
|
||||
continue
|
||||
|
||||
print(f"\n{'='*60}")
|
||||
print(f" Company: {company.name}")
|
||||
print(f"{'='*60}")
|
||||
|
||||
orders = PosOrder.search([
|
||||
('state', 'in', ['paid', 'done', 'invoiced']),
|
||||
('company_id', '=', company.id),
|
||||
], order='date_order ASC, id ASC')
|
||||
|
||||
if not orders:
|
||||
print(" No orders to process.")
|
||||
continue
|
||||
|
||||
n = len(orders)
|
||||
|
||||
# ── Step 1: Reset all fields ─────────────────────────────
|
||||
print("\n--- Resetting fields ---")
|
||||
env.cr.execute("""
|
||||
UPDATE pos_order
|
||||
SET l10n_fr_hash = NULL,
|
||||
l10n_fr_secure_sequence_number = NULL,
|
||||
previous_order_id = NULL
|
||||
WHERE id IN %s
|
||||
""", (tuple(orders.ids),))
|
||||
env.invalidate_all()
|
||||
env.cr.commit()
|
||||
print(f" ✓ {n} orders reset")
|
||||
|
||||
# ── Step 2: Assign sequence numbers ──────────────────────
|
||||
print("\n--- Assigning sequence numbers ---")
|
||||
for i, order in enumerate(orders, start=1):
|
||||
env.cr.execute("""
|
||||
UPDATE pos_order
|
||||
SET l10n_fr_secure_sequence_number = %s
|
||||
WHERE id = %s
|
||||
""", (i, order.id))
|
||||
env.invalidate_all()
|
||||
env.cr.commit()
|
||||
print(f" ✓ Sequence numbers 1 → {n} assigned")
|
||||
|
||||
# ── Step 3: Compute previous_order_id ────────────────────
|
||||
print("\n--- Computing previous_order_id ---")
|
||||
orders = PosOrder.search([
|
||||
('state', 'in', ['paid', 'done', 'invoiced']),
|
||||
('company_id', '=', company.id),
|
||||
], order='l10n_fr_secure_sequence_number ASC')
|
||||
orders._compute_previous_order()
|
||||
env.cr.commit()
|
||||
print(" ✓ OK")
|
||||
|
||||
# ── Step 4: Compute hashes (with cache invalidation after each write) ─
|
||||
print("\n--- Computing hashes ---")
|
||||
success = errors = 0
|
||||
|
||||
for idx in range(n):
|
||||
order = PosOrder.search([
|
||||
('company_id', '=', company.id),
|
||||
('l10n_fr_secure_sequence_number', '=', idx + 1),
|
||||
], limit=1)
|
||||
|
||||
if not order:
|
||||
continue
|
||||
|
||||
try:
|
||||
order._compute_string_to_hash()
|
||||
|
||||
prev = order.previous_order_id
|
||||
prev_hash = prev.l10n_fr_hash if prev else ''
|
||||
if not prev_hash:
|
||||
prev_hash = ''
|
||||
|
||||
computed_hash = sha256(
|
||||
(prev_hash + order.l10n_fr_string_to_hash).encode('utf-8')
|
||||
).hexdigest()
|
||||
|
||||
env.cr.execute(
|
||||
"UPDATE pos_order SET l10n_fr_hash = %s WHERE id = %s",
|
||||
(computed_hash, order.id)
|
||||
)
|
||||
|
||||
env.invalidate_all()
|
||||
print(f" ✓ {order.name} (seq {idx+1})")
|
||||
success += 1
|
||||
|
||||
except Exception as e:
|
||||
print(f" ✗ {order.name} (seq {idx+1}) : {e}")
|
||||
errors += 1
|
||||
import traceback
|
||||
traceback.print_exc()
|
||||
|
||||
env.cr.commit()
|
||||
|
||||
remaining = PosOrder.search_count([
|
||||
('state', 'in', ['paid', 'done', 'invoiced']),
|
||||
('company_id', '=', company.id),
|
||||
'|', ('l10n_fr_hash', '=', False),
|
||||
('l10n_fr_hash', '=', None),
|
||||
])
|
||||
print(f"\n Final result: {success} hashes written, {errors} errors, "
|
||||
f"{remaining} remaining")
|
||||
|
||||
# Reset sequence for future orders
|
||||
seq = company.l10n_fr_pos_cert_sequence_id
|
||||
if seq:
|
||||
env.cr.execute(
|
||||
"UPDATE ir_sequence SET number_next = %s WHERE id = %s",
|
||||
(n + 1, seq.id)
|
||||
)
|
||||
print(f" ✓ ir_sequence number_next set to {n + 1}")
|
||||
|
||||
print("\n✓ Done.")
|
||||
|
||||
|
||||
regenerate_all()
|
||||
@@ -12,17 +12,6 @@ CLEANUP_SQL=$(cat <<'EOF'
|
||||
DROP SEQUENCE IF EXISTS base_registry_signaling;
|
||||
DROP SEQUENCE IF EXISTS base_cache_signaling;
|
||||
|
||||
-- Reset website templates to their original state.
|
||||
-- Views with arch_fs (file source) that have been customized (arch_db not null)
|
||||
-- are reset to use the file version, EXCEPT for actual website pages which
|
||||
-- contain user content that must be preserved.
|
||||
UPDATE ir_ui_view
|
||||
SET arch_db = NULL
|
||||
WHERE arch_fs IS NOT NULL
|
||||
AND arch_fs LIKE 'website/%'
|
||||
AND arch_db IS NOT NULL
|
||||
AND id NOT IN (SELECT view_id FROM website_page);
|
||||
|
||||
-- Purge compiled frontend assets (CSS/JS bundles).
|
||||
-- These cached files reference old asset versions and must be regenerated
|
||||
-- by Odoo after migration to avoid broken stylesheets and scripts.
|
||||
@@ -34,6 +23,24 @@ EOF
|
||||
)
|
||||
query_postgres_container "$CLEANUP_SQL" "$DB_NAME"
|
||||
|
||||
# Reset modules still marked as 'to upgrade' before launching the Odoo shell
|
||||
# scripts below. Loading the registry in `odoo shell` re-triggers the upgrade
|
||||
# of these modules, which can fail on broken views (e.g. website_sale
|
||||
# TypeError). The controlled `-u all` at the end of this script performs the
|
||||
# real update afterwards. We deliberately do NOT touch 'to install' modules:
|
||||
# forcing them to 'installed' would skip their install scripts entirely.
|
||||
#
|
||||
# Uncomment the SELECT below to trace which modules were pending upgrade
|
||||
# before we neutralize their state (useful if the `-u all` above is removed).
|
||||
# query_postgres_container "
|
||||
# SELECT name FROM ir_module_module WHERE state = 'to upgrade' ORDER BY name;
|
||||
# " "$DB_NAME" || true
|
||||
query_postgres_container "
|
||||
UPDATE ir_module_module
|
||||
SET state = 'installed'
|
||||
WHERE state = 'to upgrade';
|
||||
" "$DB_NAME" || true
|
||||
|
||||
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
|
||||
|
||||
PYTHON_SCRIPT="${SCRIPT_DIR}/lib/python/fix_duplicated_views.py"
|
||||
@@ -44,12 +51,40 @@ PYTHON_SCRIPT="${SCRIPT_DIR}/lib/python/cleanup_modules.py"
|
||||
echo "Uninstall obsolete add-ons with script $PYTHON_SCRIPT ..."
|
||||
exec_python_script_in_odoo_shell "$DB_NAME" "$DB_NAME" "$PYTHON_SCRIPT"
|
||||
|
||||
# ────────────────────────────────────────────────────────────
|
||||
# Regenerate POS inalterability hashes if needed
|
||||
# ────────────────────────────────────────────────────────────
|
||||
HASHES_NEEDED=$(query_postgres_container "
|
||||
SELECT COUNT(*) FROM information_schema.tables WHERE table_name = 'pos_order'
|
||||
" "$DB_NAME")
|
||||
if [[ "$HASHES_NEEDED" =~ ^[0-9]+$ && "$HASHES_NEEDED" -gt 0 ]]; then
|
||||
HASHES_NEEDED=$(query_postgres_container "
|
||||
SELECT COUNT(*)
|
||||
FROM pos_order po
|
||||
JOIN res_company rc ON rc.id = po.company_id
|
||||
WHERE po.state IN ('paid', 'done', 'invoiced')
|
||||
AND rc.l10n_fr_pos_cert_sequence_id IS NOT NULL
|
||||
AND (po.l10n_fr_hash IS NULL OR po.l10n_fr_secure_sequence_number IS NULL)
|
||||
" "$DB_NAME")
|
||||
fi
|
||||
|
||||
if [[ "$HASHES_NEEDED" =~ ^[0-9]+$ && "$HASHES_NEEDED" -gt 0 ]]; then
|
||||
echo ""
|
||||
echo "Found $HASHES_NEEDED pos.order(s) with missing inalterability hash or sequence number."
|
||||
echo "Regenerating all POS hashes..."
|
||||
PYTHON_SCRIPT="${SCRIPT_DIR}/lib/python/regenerate_pos_hashes.py"
|
||||
exec_python_script_in_odoo_shell "$DB_NAME" "$DB_NAME" "$PYTHON_SCRIPT"
|
||||
echo "POS hash regeneration completed."
|
||||
else
|
||||
echo "No missing POS hashes detected."
|
||||
fi
|
||||
|
||||
# Give back the right to user to access to the tables
|
||||
# docker exec -u 70 "$DB_CONTAINER_NAME" pgm chown "$FINALE_SERVICE_NAME" "$DB_NAME"
|
||||
|
||||
|
||||
# Launch Odoo with database in finale version to run all updates
|
||||
run_compose --debug run "$ODOO_SERVICE" -u all --log-level=debug --stop-after-init --no-http
|
||||
run_compose --debug run "$ODOO_SERVICE" -u all --log-level=debug --stop-after-init --no-http --load=base,web,openupgrade_framework
|
||||
|
||||
echo ""
|
||||
echo "Running post-migration view validation..."
|
||||
|
||||
@@ -60,9 +60,31 @@ echo "Retrieve missing addons..."
|
||||
missing_addons=$(query_postgres_container "$SQL_MISSING_ADDONS" "$DB_NAME")
|
||||
|
||||
log_step "ADD-ONS CHECK"
|
||||
echo "Installed add-ons not available in final Odoo version:"
|
||||
echo "$missing_addons"
|
||||
confirm_or_exit "Do you accept to migrate with these add-ons still installed?"
|
||||
classify_missing_addons "$missing_addons"
|
||||
|
||||
if [[ ${#addons_obsolete[@]} -gt 0 ]]; then
|
||||
log_info "Obsolete modules (${#addons_obsolete[@]}):"
|
||||
printf "%s\n" "${addons_obsolete[@]}"
|
||||
echo ""
|
||||
fi
|
||||
if [[ ${#addons_core[@]} -gt 0 ]]; then
|
||||
log_info "Merged into Odoo Core (${#addons_core[@]}):"
|
||||
printf "%s\n" "${addons_core[@]}"
|
||||
echo ""
|
||||
fi
|
||||
if [[ ${#addons_renamed[@]} -gt 0 ]]; then
|
||||
log_info "Renamed modules (${#addons_renamed[@]}):"
|
||||
printf "%s\n" "${addons_renamed[@]}"
|
||||
echo ""
|
||||
fi
|
||||
if [[ ${#addons_truly_missing[@]} -gt 0 ]]; then
|
||||
log_warn "Truly missing modules (${#addons_truly_missing[@]}):"
|
||||
printf "%s\n" "${addons_truly_missing[@]}"
|
||||
echo ""
|
||||
confirm_or_exit "Do you accept to migrate with these add-ons truly missing?"
|
||||
else
|
||||
log_info "No truly missing modules — all accounted for."
|
||||
fi
|
||||
|
||||
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
|
||||
PYTHON_SCRIPT="${SCRIPT_DIR}/lib/python/check_views.py"
|
||||
|
||||
154
upgrade.sh
154
upgrade.sh
@@ -4,13 +4,20 @@ set -euo pipefail
|
||||
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
|
||||
source "${SCRIPT_DIR}/lib/common.sh"
|
||||
|
||||
readonly LOG_FILE="${SCRIPT_DIR}/migration.log"
|
||||
if [[ -z "${_MIGRATION_LOGGING:-}" ]]; then
|
||||
rm -f "$LOG_FILE"
|
||||
export _MIGRATION_LOGGING=1
|
||||
exec script -q -c "$(printf '%q ' "$0" "$@")" "$LOG_FILE"
|
||||
fi
|
||||
|
||||
####################
|
||||
# USAGE & ARGUMENTS
|
||||
####################
|
||||
|
||||
usage() {
|
||||
cat <<EOF >&2
|
||||
Usage: $0 <origin_version> <final_version> <db_name> <service_name>
|
||||
Usage: $0 <origin_version> <final_version> <db_name> <service_name> [--resume-from|-r <version>]
|
||||
|
||||
Arguments:
|
||||
origin_version Origin Odoo version number (e.g., 12 for version 12.0)
|
||||
@@ -18,29 +25,78 @@ Arguments:
|
||||
db_name Name of the database to migrate
|
||||
service_name Name of the origin Odoo service (docker compose service)
|
||||
|
||||
Example:
|
||||
$0 14 16 elabore_20241208 odoo14
|
||||
Options:
|
||||
--resume-from, -r <version>
|
||||
Resume migration from an already-migrated intermediate
|
||||
database (e.g., ou15). Skips the initial DB copy and
|
||||
prepare_db.sh phases. The intermediate DB must exist.
|
||||
|
||||
Examples:
|
||||
$0 14 18 elabore_20241208 odoo14
|
||||
$0 14 18 elabore_20241208 odoo14 --resume-from 15
|
||||
$0 14 18 elabore_20241208 odoo14 -r 15
|
||||
EOF
|
||||
exit 1
|
||||
}
|
||||
|
||||
if [[ $# -lt 4 ]]; then
|
||||
log_error "Missing arguments. Expected 4, got $#."
|
||||
log_error "Missing arguments. Expected at least 4, got $#."
|
||||
usage
|
||||
fi
|
||||
|
||||
check_required_commands
|
||||
|
||||
readonly ORIGIN_VERSION="$1"
|
||||
readonly FINAL_VERSION="$2"
|
||||
export ORIGIN_VERSION="$1"
|
||||
readonly ORIGIN_VERSION
|
||||
export FINAL_VERSION="$2"
|
||||
readonly FINAL_VERSION
|
||||
readonly ORIGIN_DB_NAME="$3"
|
||||
readonly ORIGIN_SERVICE_NAME="$4"
|
||||
|
||||
# Parse optional --resume-from / -r flag
|
||||
RESUME_FROM_VERSION=""
|
||||
shift 4
|
||||
while [[ $# -gt 0 ]]; do
|
||||
case "$1" in
|
||||
--resume-from|-r)
|
||||
if [[ $# -lt 2 ]]; then
|
||||
log_error "Option '$1' requires a version number argument."
|
||||
usage
|
||||
fi
|
||||
RESUME_FROM_VERSION="$2"
|
||||
shift 2
|
||||
;;
|
||||
*)
|
||||
log_error "Unknown option: '$1'"
|
||||
usage
|
||||
;;
|
||||
esac
|
||||
done
|
||||
readonly RESUME_FROM_VERSION
|
||||
|
||||
readonly COPY_DB_NAME="ou${ORIGIN_VERSION}"
|
||||
export FINALE_DB_NAME="ou${FINAL_VERSION}"
|
||||
readonly FINALE_DB_NAME
|
||||
readonly FINALE_SERVICE_NAME="${FINALE_DB_NAME}"
|
||||
|
||||
# Validate --resume-from value if provided
|
||||
if [[ -n "$RESUME_FROM_VERSION" ]]; then
|
||||
if ! [[ "$RESUME_FROM_VERSION" =~ ^[0-9]+$ ]]; then
|
||||
log_error "--resume-from value must be a numeric version number (got: '$RESUME_FROM_VERSION')."
|
||||
exit 1
|
||||
fi
|
||||
if [[ "$RESUME_FROM_VERSION" -le "$ORIGIN_VERSION" ]]; then
|
||||
log_error "--resume-from ($RESUME_FROM_VERSION) must be strictly greater than origin version ($ORIGIN_VERSION)."
|
||||
log_error "To start a fresh migration, run without --resume-from."
|
||||
exit 1
|
||||
fi
|
||||
if [[ "$RESUME_FROM_VERSION" -ge "$FINAL_VERSION" ]]; then
|
||||
log_error "--resume-from ($RESUME_FROM_VERSION) must be strictly less than final version ($FINAL_VERSION)."
|
||||
log_error "Nothing to migrate: checkpoint is at or beyond the target version."
|
||||
exit 1
|
||||
fi
|
||||
fi
|
||||
|
||||
readarray -t postgres_containers < <(docker ps --format '{{.Names}}' | grep postgres || true)
|
||||
|
||||
if [[ ${#postgres_containers[@]} -eq 0 ]]; then
|
||||
@@ -59,8 +115,13 @@ readonly POSTGRES_SERVICE_NAME
|
||||
log_step "INPUT PARAMETERS"
|
||||
log_info "Origin version .......... $ORIGIN_VERSION"
|
||||
log_info "Final version ........... $FINAL_VERSION"
|
||||
log_info "Origin DB name ........... $ORIGIN_DB_NAME"
|
||||
log_info "Origin DB name .......... $ORIGIN_DB_NAME"
|
||||
log_info "Origin service name ..... $ORIGIN_SERVICE_NAME"
|
||||
if [[ -n "$RESUME_FROM_VERSION" ]]; then
|
||||
log_info "Resume from version ..... $RESUME_FROM_VERSION (ou${RESUME_FROM_VERSION})"
|
||||
else
|
||||
log_info "Resume from version ..... none (full migration)"
|
||||
fi
|
||||
|
||||
log_step "COMPUTED GLOBAL VARIABLES"
|
||||
log_info "Copy DB name ............. $COPY_DB_NAME"
|
||||
@@ -72,20 +133,42 @@ log_info "Postgres service name .... $POSTGRES_SERVICE_NAME"
|
||||
|
||||
log_step "CHECKS ALL NEEDED COMPONENTS ARE AVAILABLE"
|
||||
|
||||
db_exists=$(docker exec -it -u 70 "$POSTGRES_SERVICE_NAME" psql -tc "SELECT 1 FROM pg_database WHERE datname = '$ORIGIN_DB_NAME'" | tr -d '[:space:]')
|
||||
if [[ "$db_exists" ]]; then
|
||||
log_info "Database '$ORIGIN_DB_NAME' found."
|
||||
else
|
||||
log_error "Database '$ORIGIN_DB_NAME' not found in the local postgres service. Please add it and restart the upgrade process."
|
||||
exit 1
|
||||
fi
|
||||
if [[ -n "$RESUME_FROM_VERSION" ]]; then
|
||||
readonly RESUME_DB_NAME="ou${RESUME_FROM_VERSION}"
|
||||
|
||||
filestore_path="${DATASTORE_PATH}/${ORIGIN_SERVICE_NAME}/${FILESTORE_SUBPATH}/${ORIGIN_DB_NAME}"
|
||||
if [[ -d "$filestore_path" ]]; then
|
||||
log_info "Filestore '$filestore_path' found."
|
||||
resume_db_exists=$(docker exec -u 70 "$POSTGRES_SERVICE_NAME" psql -tc "SELECT 1 FROM pg_database WHERE datname = '${RESUME_DB_NAME}'" | tr -d '[:space:]')
|
||||
if [[ "$resume_db_exists" ]]; then
|
||||
log_info "Checkpoint database '${RESUME_DB_NAME}' found."
|
||||
else
|
||||
log_error "Checkpoint database '${RESUME_DB_NAME}' not found in the local postgres service."
|
||||
log_error "Ensure the migration up to version ${RESUME_FROM_VERSION} completed successfully before resuming."
|
||||
exit 1
|
||||
fi
|
||||
|
||||
resume_filestore_path="${DATASTORE_PATH}/${RESUME_DB_NAME}/${FILESTORE_SUBPATH}/${RESUME_DB_NAME}"
|
||||
if [[ -d "$resume_filestore_path" ]]; then
|
||||
log_info "Checkpoint filestore '${resume_filestore_path}' found."
|
||||
else
|
||||
log_error "Checkpoint filestore '${resume_filestore_path}' not found."
|
||||
log_error "Ensure the filestore for '${RESUME_DB_NAME}' is intact before resuming."
|
||||
exit 1
|
||||
fi
|
||||
else
|
||||
log_error "Filestore '$filestore_path' not found, please add it and restart the upgrade process."
|
||||
exit 1
|
||||
db_exists=$(docker exec -u 70 "$POSTGRES_SERVICE_NAME" psql -tc "SELECT 1 FROM pg_database WHERE datname = '$ORIGIN_DB_NAME'" | tr -d '[:space:]')
|
||||
if [[ "$db_exists" ]]; then
|
||||
log_info "Database '$ORIGIN_DB_NAME' found."
|
||||
else
|
||||
log_error "Database '$ORIGIN_DB_NAME' not found in the local postgres service. Please add it and restart the upgrade process."
|
||||
exit 1
|
||||
fi
|
||||
|
||||
filestore_path="${DATASTORE_PATH}/${ORIGIN_SERVICE_NAME}/${FILESTORE_SUBPATH}/${ORIGIN_DB_NAME}"
|
||||
if [[ -d "$filestore_path" ]]; then
|
||||
log_info "Filestore '$filestore_path' found."
|
||||
else
|
||||
log_error "Filestore '$filestore_path' not found, please add it and restart the upgrade process."
|
||||
exit 1
|
||||
fi
|
||||
fi
|
||||
|
||||
log_step "LAUNCH VIRGIN ODOO IN FINAL VERSION"
|
||||
@@ -100,24 +183,36 @@ run_compose --debug run "$FINALE_SERVICE_NAME" -i base --stop-after-init --no-ht
|
||||
|
||||
log_info "Model database in final Odoo version created."
|
||||
|
||||
log_step "COPY ORIGINAL COMPONENTS"
|
||||
if [[ -z "$RESUME_FROM_VERSION" ]]; then
|
||||
log_step "COPY ORIGINAL COMPONENTS"
|
||||
|
||||
copy_database "$ORIGIN_DB_NAME" "$COPY_DB_NAME" "$COPY_DB_NAME"
|
||||
log_info "Original database copied to ${COPY_DB_NAME}@${COPY_DB_NAME}."
|
||||
copy_database "$ORIGIN_DB_NAME" "$COPY_DB_NAME" "$COPY_DB_NAME"
|
||||
log_info "Original database copied to ${COPY_DB_NAME}@${COPY_DB_NAME}."
|
||||
|
||||
copy_filestore "$ORIGIN_SERVICE_NAME" "$ORIGIN_DB_NAME" "$COPY_DB_NAME" "$COPY_DB_NAME"
|
||||
log_info "Original filestore copied."
|
||||
copy_filestore "$ORIGIN_SERVICE_NAME" "$ORIGIN_DB_NAME" "$COPY_DB_NAME" "$COPY_DB_NAME"
|
||||
log_info "Original filestore copied."
|
||||
else
|
||||
log_step "COPY ORIGINAL COMPONENTS — SKIPPED (resuming from checkpoint ou${RESUME_FROM_VERSION})"
|
||||
fi
|
||||
|
||||
|
||||
log_step "PATH OF MIGRATION"
|
||||
|
||||
readarray -t versions < <(seq $((ORIGIN_VERSION + 1)) "$FINAL_VERSION")
|
||||
log_info "Migration path is ${versions[*]}"
|
||||
if [[ -n "$RESUME_FROM_VERSION" ]]; then
|
||||
readarray -t versions < <(seq $((RESUME_FROM_VERSION + 1)) "$FINAL_VERSION")
|
||||
log_info "Resuming migration from ou${RESUME_FROM_VERSION} — path is ${versions[*]}"
|
||||
else
|
||||
readarray -t versions < <(seq $((ORIGIN_VERSION + 1)) "$FINAL_VERSION")
|
||||
log_info "Migration path is ${versions[*]}"
|
||||
fi
|
||||
|
||||
|
||||
log_step "DATABASE PREPARATION"
|
||||
|
||||
"${SCRIPT_DIR}/scripts/prepare_db.sh" "$COPY_DB_NAME" "$COPY_DB_NAME" "$FINALE_DB_NAME" "$FINALE_SERVICE_NAME"
|
||||
if [[ -z "$RESUME_FROM_VERSION" ]]; then
|
||||
log_step "DATABASE PREPARATION"
|
||||
"${SCRIPT_DIR}/scripts/prepare_db.sh" "$COPY_DB_NAME" "$COPY_DB_NAME" "$FINALE_DB_NAME" "$FINALE_SERVICE_NAME"
|
||||
else
|
||||
log_step "DATABASE PREPARATION — SKIPPED (resuming from checkpoint ou${RESUME_FROM_VERSION})"
|
||||
fi
|
||||
|
||||
|
||||
log_step "UPGRADE PROCESS"
|
||||
@@ -137,3 +232,4 @@ log_step "POST-UPGRADE PROCESSES"
|
||||
"${SCRIPT_DIR}/scripts/finalize_db.sh" "$FINALE_DB_NAME" "$FINALE_SERVICE_NAME"
|
||||
|
||||
log_step "UPGRADE PROCESS ENDED WITH SUCCESS"
|
||||
log_info "Full logs available at: ${LOG_FILE}"
|
||||
|
||||
10
versions/13.0/known_changes.yaml
Normal file
10
versions/13.0/known_changes.yaml
Normal file
@@ -0,0 +1,10 @@
|
||||
# Modules that became obsolete in version 13.0
|
||||
obsolete: []
|
||||
|
||||
# Modules merged into Odoo Core in version 13.0
|
||||
merged_in_core: []
|
||||
|
||||
# Modules renamed in version 13.0
|
||||
renamed: []
|
||||
# - old: old_name
|
||||
# new: new_name
|
||||
10
versions/14.0/known_changes.yaml
Normal file
10
versions/14.0/known_changes.yaml
Normal file
@@ -0,0 +1,10 @@
|
||||
# Modules that became obsolete in version 14.0
|
||||
obsolete: []
|
||||
|
||||
# Modules merged into Odoo Core in version 14.0
|
||||
merged_in_core: []
|
||||
|
||||
# Modules renamed in version 14.0
|
||||
renamed: []
|
||||
# - old: old_name
|
||||
# new: new_name
|
||||
16
versions/15.0/known_changes.yaml
Normal file
16
versions/15.0/known_changes.yaml
Normal file
@@ -0,0 +1,16 @@
|
||||
# Modules that became obsolete in version 15.0
|
||||
obsolete:
|
||||
- l10n_ch_base_bank
|
||||
- l10n_ch_isrb
|
||||
- project_timeline_task_dependency
|
||||
- account_reconcile_reconciliation_date
|
||||
|
||||
# Modules merged into Odoo Core in version 15.0
|
||||
merged_in_core:
|
||||
- project_category
|
||||
- project_deadline
|
||||
|
||||
# Modules renamed in version 15.0
|
||||
renamed:
|
||||
- old: crm_phone
|
||||
new: crm_phonecall
|
||||
23
versions/16.0/known_changes.yaml
Normal file
23
versions/16.0/known_changes.yaml
Normal file
@@ -0,0 +1,23 @@
|
||||
# Modules that became obsolete in version 16.0
|
||||
obsolete:
|
||||
- account_reconciliation_widget
|
||||
- account_statement_import
|
||||
- account_statement_import_file_reconciliation_widget
|
||||
|
||||
# Modules merged into Odoo Core in version 16.0
|
||||
merged_in_core:
|
||||
- account_balance_line
|
||||
- l10n_ch_states
|
||||
- project_task_dependency
|
||||
- web_ir_actions_act_view_reload
|
||||
- mail_activity_creator
|
||||
- website_sale_require_login
|
||||
|
||||
# Modules renamed in version 16.0
|
||||
renamed:
|
||||
- old: account_statement_import_file_reconcile_oca
|
||||
new: account_statement_import_file_reconcile_oca
|
||||
- old: mass_editing
|
||||
new: server_action_mass_edit
|
||||
- old: crm_project
|
||||
new: crm_lead_to_task
|
||||
13
versions/17.0/known_changes.yaml
Normal file
13
versions/17.0/known_changes.yaml
Normal file
@@ -0,0 +1,13 @@
|
||||
# Modules that became obsolete in version 17.0
|
||||
obsolete: []
|
||||
|
||||
# Modules merged into Odoo Core in version 17.0
|
||||
merged_in_core:
|
||||
- project_list
|
||||
- web_advanced_search
|
||||
- web_listview_range_select
|
||||
|
||||
# Modules renamed in version 17.0
|
||||
renamed: []
|
||||
# - old: old_name
|
||||
# new: new_name
|
||||
36
versions/18.0/known_changes.yaml
Normal file
36
versions/18.0/known_changes.yaml
Normal file
@@ -0,0 +1,36 @@
|
||||
# Modules that became obsolete in version 18.0
|
||||
obsolete:
|
||||
- account_payment_paired_internal_transfer
|
||||
- account_reconciliation_widget
|
||||
|
||||
# Modules merged into Odoo Core in version 18.0
|
||||
merged_in_core:
|
||||
- account_payment_partner
|
||||
- hr_expense_report_merge_attachment
|
||||
|
||||
# Modules renamed in version 18.0
|
||||
renamed:
|
||||
# Refactor bank-payment-alternative
|
||||
- old: account_payment_mode
|
||||
new: account_payment_base_oca
|
||||
- old: account_payment_sale
|
||||
new: account_payment_base_oca_sale
|
||||
- old: account_payment_order
|
||||
new: account_payment_batch_oca
|
||||
- old: account_payment_order_tier_validation
|
||||
new: account_payment_batch_oca_tier_validation
|
||||
- old: account_banking_pain_base
|
||||
new: account_payment_sepa_base
|
||||
- old: account_banking_sepa_credit_transfer
|
||||
new: account_payment_sepa_credit_transfer
|
||||
- old: account_banking_mandate
|
||||
new: account_payment_mandate
|
||||
- old: account_banking_mandate_sale
|
||||
new: account_payment_mandate_sale
|
||||
- old: account_banking_sepa_direct_debit
|
||||
new: account_payment_sepa_direct_debit
|
||||
# End refactor
|
||||
- old: base_delivery_carrier_label
|
||||
new: delivery_shipping_label_default
|
||||
- old: l10n_fr_oca
|
||||
new: l10n_fr_account_oca
|
||||
@@ -116,7 +116,7 @@ BEGIN
|
||||
|
||||
RAISE NOTICE 'Starting account_payment_batch_oca migration...';
|
||||
|
||||
IF EXISTS (SELECT FROM information_schema.columns
|
||||
IF EXISTS (SELECT FROM information_schema.columns
|
||||
WHERE table_name = 'account_payment_method' AND column_name = 'payment_order_only') THEN
|
||||
UPDATE account_payment_method
|
||||
SET payment_order_ok = payment_order_only
|
||||
@@ -136,7 +136,7 @@ BEGIN
|
||||
WHERE apml.old_payment_mode_id IS NOT NULL
|
||||
AND apm.id = apml.old_payment_mode_id;
|
||||
|
||||
IF EXISTS (SELECT FROM information_schema.tables
|
||||
IF EXISTS (SELECT FROM information_schema.tables
|
||||
WHERE table_name = 'account_journal_account_payment_method_line_rel') THEN
|
||||
DELETE FROM account_journal_account_payment_method_line_rel
|
||||
WHERE account_payment_method_line_id IN (
|
||||
@@ -173,4 +173,236 @@ else
|
||||
echo "Table account_payment_mode not found, skipping bank-payment migration."
|
||||
fi
|
||||
|
||||
# ============================================================================
|
||||
# FIX: stock_picking name='/' and missing POS picking type sequences
|
||||
#
|
||||
# Two issues can occur after migration:
|
||||
#
|
||||
# 1. stock_picking records with name='/' violate the new V18 unique constraint
|
||||
# stock_picking_name_uniq (name, company_id). In previous versions this
|
||||
# constraint did not exist, so multiple pickings could share name='/'.
|
||||
#
|
||||
# 2. POS picking types (pos_type_id on stock_warehouse) may lack a sequence_id.
|
||||
# In V18, stock.picking.create() assigns the name from picking_type.sequence_id,
|
||||
# but if sequence_id is NULL the name stays '/' and the unique constraint is
|
||||
# violated on every new POS payment.
|
||||
#
|
||||
# The standard _create_missing_pos_picking_types() only fixes warehouses where
|
||||
# pos_type_id is NULL — it does NOT fix existing pos_type_id records that are
|
||||
# missing their sequence_id.
|
||||
#
|
||||
# Only executed if stock module tables exist.
|
||||
# ============================================================================
|
||||
|
||||
STOCK_PICKING_TABLE_EXISTS=$(query_postgres_container "SELECT COUNT(*) FROM information_schema.tables WHERE table_name = 'stock_picking';" ou18 2>/dev/null | grep -E '^\s*[0-9]+' | tr -d ' ' || echo "0")
|
||||
|
||||
if [ "$STOCK_PICKING_TABLE_EXISTS" -gt 0 ]; then
|
||||
echo "stock_picking table exists, proceeding with POS picking fixes..."
|
||||
|
||||
# --- Step 1: Rename stock_picking records with name='/' ---
|
||||
FIX_SLASH_PICKING_NAMES_SQL=$(cat <<'EOF'
|
||||
DO $$
|
||||
DECLARE
|
||||
slash_count INTEGER;
|
||||
BEGIN
|
||||
IF NOT EXISTS (SELECT FROM information_schema.tables WHERE table_name = 'stock_picking') THEN
|
||||
RAISE NOTICE 'stock_picking table not found, skipping slash name fix';
|
||||
RETURN;
|
||||
END IF;
|
||||
|
||||
SELECT COUNT(*) INTO slash_count FROM stock_picking WHERE name = '/';
|
||||
|
||||
IF slash_count > 0 THEN
|
||||
UPDATE stock_picking SET name = 'MIGRATED-' || id WHERE name = '/';
|
||||
RAISE NOTICE 'Renamed % stock_picking record(s) with name=/ to MIGRATED-<id>', slash_count;
|
||||
ELSE
|
||||
RAISE NOTICE 'No stock_picking with name=/ found, nothing to rename';
|
||||
END IF;
|
||||
END $$;
|
||||
EOF
|
||||
)
|
||||
echo "Fixing stock_picking names with '/'..."
|
||||
query_postgres_container "$FIX_SLASH_PICKING_NAMES_SQL" ou18 || exit 1
|
||||
|
||||
# --- Step 2: Create missing ir.sequence for POS picking types without sequence ---
|
||||
FIX_POS_PICKING_TYPE_SEQUENCES_SQL=$(cat <<'EOF'
|
||||
DO $$
|
||||
DECLARE
|
||||
pt_rec RECORD;
|
||||
seq_id INTEGER;
|
||||
seq_name TEXT;
|
||||
seq_prefix TEXT;
|
||||
BEGIN
|
||||
IF NOT EXISTS (SELECT FROM information_schema.tables WHERE table_name = 'stock_picking_type') THEN
|
||||
RAISE NOTICE 'stock_picking_type table not found, skipping POS sequence fix';
|
||||
RETURN;
|
||||
END IF;
|
||||
|
||||
IF NOT EXISTS (SELECT FROM information_schema.columns
|
||||
WHERE table_name = 'stock_warehouse' AND column_name = 'pos_type_id') THEN
|
||||
RAISE NOTICE 'stock_warehouse.pos_type_id column not found (point_of_sale not installed), skipping POS sequence fix';
|
||||
RETURN;
|
||||
END IF;
|
||||
|
||||
FOR pt_rec IN
|
||||
SELECT spt.id AS picking_type_id,
|
||||
spt.name AS picking_type_name,
|
||||
COALESCE(spt.sequence_code, 'POS') AS sequence_code,
|
||||
sw.code AS warehouse_code,
|
||||
sw.name AS warehouse_name,
|
||||
sw.company_id
|
||||
FROM stock_picking_type spt
|
||||
JOIN stock_warehouse sw ON sw.pos_type_id = spt.id
|
||||
WHERE spt.sequence_id IS NULL OR spt.sequence_id = 0
|
||||
LOOP
|
||||
seq_prefix := pt_rec.warehouse_code || '/' || pt_rec.sequence_code || '/';
|
||||
seq_name := pt_rec.warehouse_name || ' Picking POS';
|
||||
|
||||
INSERT INTO ir_sequence (
|
||||
name, prefix, padding, company_id, implementation,
|
||||
number_increment, number_next, active
|
||||
) VALUES (
|
||||
seq_name, seq_prefix, 5, pt_rec.company_id, 'standard',
|
||||
1, 1, true
|
||||
) RETURNING id INTO seq_id;
|
||||
|
||||
UPDATE stock_picking_type
|
||||
SET sequence_id = seq_id
|
||||
WHERE id = pt_rec.picking_type_id;
|
||||
|
||||
RAISE NOTICE 'Created ir.sequence % (%) for POS picking type % (ID:%) on warehouse %',
|
||||
seq_name, seq_id, pt_rec.picking_type_name, pt_rec.picking_type_id, pt_rec.warehouse_name;
|
||||
END LOOP;
|
||||
END $$;
|
||||
EOF
|
||||
)
|
||||
echo "Creating missing POS picking type sequences..."
|
||||
query_postgres_container "$FIX_POS_PICKING_TYPE_SEQUENCES_SQL" ou18 || exit 1
|
||||
|
||||
# --- Step 3: Create missing PostgreSQL sequences for ir.sequence records ---
|
||||
# When ir.sequence records are created via SQL INSERT (not ORM), the underlying
|
||||
# PostgreSQL sequence (ir_sequence_NNN) is NOT created. This causes errors when
|
||||
# Odoo tries to read or use the sequence.
|
||||
FIX_MISSING_PG_SEQUENCES_SQL=$(cat <<'EOF'
|
||||
DO $$
|
||||
DECLARE
|
||||
seq_rec RECORD;
|
||||
seq_pg_name TEXT;
|
||||
seq_count INTEGER;
|
||||
BEGIN
|
||||
seq_count := 0;
|
||||
|
||||
FOR seq_rec IN
|
||||
SELECT id, number_increment, number_next
|
||||
FROM ir_sequence
|
||||
WHERE implementation = 'standard'
|
||||
LOOP
|
||||
seq_pg_name := 'ir_sequence_' || lpad(seq_rec.id::text, 3, '0');
|
||||
|
||||
IF NOT EXISTS (
|
||||
SELECT 1 FROM pg_class
|
||||
WHERE relkind = 'S' AND relname = seq_pg_name
|
||||
) THEN
|
||||
EXECUTE format('CREATE SEQUENCE %I INCREMENT BY %s START WITH %s',
|
||||
seq_pg_name, seq_rec.number_increment, seq_rec.number_next);
|
||||
seq_count := seq_count + 1;
|
||||
RAISE NOTICE 'Created PostgreSQL sequence %', seq_pg_name;
|
||||
END IF;
|
||||
END LOOP;
|
||||
|
||||
IF seq_count > 0 THEN
|
||||
RAISE NOTICE 'Created % missing PostgreSQL sequence(s)', seq_count;
|
||||
ELSE
|
||||
RAISE NOTICE 'All PostgreSQL sequences already exist, nothing to create';
|
||||
END IF;
|
||||
END $$;
|
||||
EOF
|
||||
)
|
||||
echo "Creating missing PostgreSQL sequences for ir.sequence records..."
|
||||
query_postgres_container "$FIX_MISSING_PG_SEQUENCES_SQL" ou18 || exit 1
|
||||
|
||||
# --- Step 4: Grant permissions on newly created PostgreSQL sequences ---
|
||||
# Sequences created above are owned by the current DB user, but we ensure
|
||||
# the Odoo DB user has proper access (needed when created by a different superuser).
|
||||
GRANT_PG_SEQUENCES_SQL=$(cat <<'EOF'
|
||||
DO $$
|
||||
DECLARE
|
||||
seq_rec RECORD;
|
||||
seq_pg_name TEXT;
|
||||
db_user TEXT;
|
||||
grant_count INTEGER;
|
||||
BEGIN
|
||||
-- Determine the Odoo database user from the current connection
|
||||
SELECT current_user INTO db_user;
|
||||
grant_count := 0;
|
||||
|
||||
FOR seq_rec IN
|
||||
SELECT id
|
||||
FROM ir_sequence
|
||||
WHERE implementation = 'standard'
|
||||
LOOP
|
||||
seq_pg_name := 'ir_sequence_' || lpad(seq_rec.id::text, 3, '0');
|
||||
|
||||
IF EXISTS (
|
||||
SELECT 1 FROM pg_class
|
||||
WHERE relkind = 'S' AND relname = seq_pg_name
|
||||
) THEN
|
||||
BEGIN
|
||||
EXECUTE format('GRANT ALL ON SEQUENCE %I TO %I', seq_pg_name, db_user);
|
||||
grant_count := grant_count + 1;
|
||||
EXCEPTION WHEN insufficient_privilege THEN
|
||||
RAISE NOTICE 'Cannot GRANT on % (not owner), skipping', seq_pg_name;
|
||||
END;
|
||||
END IF;
|
||||
END LOOP;
|
||||
|
||||
RAISE NOTICE 'Granted permissions on % PostgreSQL sequence(s) to %', grant_count, db_user;
|
||||
END $$;
|
||||
EOF
|
||||
)
|
||||
echo "Granting permissions on PostgreSQL sequences..."
|
||||
query_postgres_container "$GRANT_PG_SEQUENCES_SQL" ou18 || exit 1
|
||||
|
||||
# --- Step 5: Verification ---
|
||||
VERIFY_POS_FIXES_SQL=$(cat <<'EOF'
|
||||
DO $$
|
||||
DECLARE
|
||||
slash_count INTEGER;
|
||||
missing_seq_count INTEGER;
|
||||
BEGIN
|
||||
-- Check remaining pickings with name='/'
|
||||
IF EXISTS (SELECT FROM information_schema.tables WHERE table_name = 'stock_picking') THEN
|
||||
SELECT COUNT(*) INTO slash_count FROM stock_picking WHERE name = '/';
|
||||
IF slash_count > 0 THEN
|
||||
RAISE WARNING 'Still % stock_picking record(s) with name=/', slash_count;
|
||||
ELSE
|
||||
RAISE NOTICE 'OK: No stock_picking with name=/ remaining';
|
||||
END IF;
|
||||
END IF;
|
||||
|
||||
-- Check POS picking types without sequence
|
||||
IF EXISTS (SELECT FROM information_schema.tables WHERE table_name = 'stock_picking_type')
|
||||
AND EXISTS (SELECT FROM information_schema.columns
|
||||
WHERE table_name = 'stock_warehouse' AND column_name = 'pos_type_id') THEN
|
||||
SELECT COUNT(*) INTO missing_seq_count
|
||||
FROM stock_picking_type spt
|
||||
JOIN stock_warehouse sw ON sw.pos_type_id = spt.id
|
||||
WHERE spt.sequence_id IS NULL OR spt.sequence_id = 0;
|
||||
|
||||
IF missing_seq_count > 0 THEN
|
||||
RAISE WARNING 'Still % POS picking type(s) without sequence_id', missing_seq_count;
|
||||
ELSE
|
||||
RAISE NOTICE 'OK: All POS picking types have a sequence_id';
|
||||
END IF;
|
||||
END IF;
|
||||
END $$;
|
||||
EOF
|
||||
)
|
||||
echo "Verifying POS picking fixes..."
|
||||
query_postgres_container "$VERIFY_POS_FIXES_SQL" ou18 || exit 1
|
||||
|
||||
else
|
||||
echo "stock_picking table not found, skipping POS picking fixes."
|
||||
fi
|
||||
|
||||
echo "Post migration to 18.0 completed!"
|
||||
|
||||
@@ -127,6 +127,8 @@ EOF
|
||||
# Execute SQL pre-migration commands
|
||||
PRE_MIGRATE_SQL=$(cat <<'EOF'
|
||||
UPDATE account_analytic_plan SET default_applicability=NULL WHERE default_applicability='optional';
|
||||
DELETE FROM ir_ui_view WHERE model = 'res.config.settings';
|
||||
|
||||
EOF
|
||||
)
|
||||
echo "SQL command = $PRE_MIGRATE_SQL"
|
||||
|
||||
Reference in New Issue
Block a user