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

[ADD] keep 0 overtime records #67

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

hbrunn
Copy link
Contributor

@hbrunn hbrunn commented Nov 27, 2023

das ist jetzt die erste Anwendung von #66, hier sorge ich im Migrationsscript dafür dass die noupdate-Records von base/hr_attendance_missing_days mit der provisionierten Konfiguration überschrieben wird.

Weiterhin generiert die Migration fehlende Überstunden-Records, da dort die erwartete Zeit lebt, und bevor der Codeänderung unten gab es fur den Fall von #65 kein Überstunden-Record.

PS: Das Update wird abhängig von bestehenden Daten etwas (=Minuten) dauern

@albig
Copy link
Member

albig commented Nov 29, 2023

Bei mir läuft das Update lokal nicht durch.

Kann das an einem Timeout liegen? Ist das limit_time_real?

2023-11-29 09:15:19,027 1 WARNING lv-bb odoo.service.server: Thread <Thread(odoo.service.http.request.140429176555264, started 140429176555264)> virtual real time limit (175/120s) reached.
2023-11-29 09:15:19,027 1 WARNING lv-bb odoo.service.server: Thread <Thread(odoo.service.http.request.140429154531072, started 140429154531072)> virtual real time limit (142/120s) reached.

Nach dem Eintrag kommen noch mehr Stack-Traces. Fängt damit an:

2023-11-29 09:15:19,483 1 ERROR lv-bb odoo.sql_db: bad query: SELECT "hr_leave_allocation".id FROM "hr_leave_allocation" WHERE ("hr_leave_allocation"."employee_id" in (41)) ORDER BY  "hr_leave_allocation"."id"
ERROR: cursor already closed
2023-11-29 09:15:19,483 1 ERROR lv-bb OpenUpgrade: verdigado_attendance: error in migration script /mnt/extra-addons/verdigado_attendance/migrations/15.0.1.0.1/post-fix-hr_attendance_missing_days-config.py: cursor already closed
2023-11-29 09:15:19,483 1 ERROR lv-bb OpenUpgrade: cursor already closed

Ich erhöh erst mal den Timeout und probier weiter.

@hbrunn
Copy link
Contributor Author

hbrunn commented Nov 29, 2023

ja, das ist das time-limit. Aber updates solltest Du ohnehin nicht via UI ausfuhren, sondern mit -u $module

@albig
Copy link
Member

albig commented Nov 29, 2023

Und dann greift das Limit nicht? Ich probier's mal. Auch mit 720 Sekunden hat er abgebrochen.

@albig
Copy link
Member

albig commented Nov 29, 2023

Das ändert nix.

2023-11-29 09:50:46,051 1 INFO lv-bb odoo.modules.migration: module verdigado_attendance: Running migration [15.0.1.0.1>] post-fix-hr_attendance_missing_days-config
2023-11-29 09:50:46,131 1 INFO lv-bb OpenUpgrade: verdigado_attendance: post-migration script called with version 15.0.0.0.1
2023-11-29 09:50:46,131 1 INFO lv-bb OpenUpgrade: verdigado_attendance: loading data/ir_cron.xml
2023-11-29 09:50:46,135 1 INFO lv-bb OpenUpgrade: verdigado_attendance: loading data/res_company.xml


2023-11-29 09:51:03,136 1 INFO ? odoo.http: HTTP Configuring static files
2023-11-29 09:52:10,703 1 INFO lv-bb odoo.models.unlink: User #1 deleted hr.attendance.overtime records with IDs: [10545, 10332, 10448, 10676, 10300, 10643, 10356, 10516, 10481, 10672, 10637, 10590]
2023-11-29 10:06:14,104 1 INFO lv-bb odoo.addons.base.models.ir_translation: module verdigado_attendance: loading translation file (de) for language de_DE
2023-11-29 10:06:14,104 1 INFO lv-bb odoo.tools.translate: loading /mnt/extra-addons/verdigado_attendance/i18n/de.po
2023-11-29 10:06:14,130 1 INFO lv-bb odoo.modules.loading: Module verdigado_attendance loaded in 929.91s, 233177 queries
2023-11-29 10:06:14,130 1 INFO lv-bb odoo.modules.loading: 53 modules loaded in 930.13s, 233177 queries (+0 extra)
2023-11-29 10:06:14,522 1 INFO lv-bb odoo.modules.loading: Modules loaded.
2023-11-29 10:06:14,527 1 INFO lv-bb odoo.modules.registry: Registry loaded in 931.829s
2023-11-29 10:06:14,530 1 WARNING lv-bb odoo.service.server: Thread <Thread(odoo.service.http.request.140379438966528, started 140379438966528)> virtual real time limit (911/720s) reached.
2023-11-29 10:06:14,530 1 INFO lv-bb odoo.service.server: Dumping stacktrace of limit exceeding threads before reloading
2023-11-29 10:06:14,532 1 INFO lv-bb odoo.tools.misc:

Ich setz den Timeout noch höher. Aber das scheint mir sehr ungünstig. So schwach ist meine lokale Umgebung nun auch wieder nicht.

@hbrunn
Copy link
Contributor Author

hbrunn commented Nov 29, 2023

has Du sowohl time_real und time_cpu angepasst? Wenn das grosse Probleme macht kann ich das Migrationsscript auch komplexer und schneller machen, langsam war halt einfacher

@hbrunn
Copy link
Contributor Author

hbrunn commented Nov 29, 2023

...und lasst Du den Server threaded laufen? Das ist in Production nicht so gut, siehe https://www.odoo.com/documentation/16.0/administration/install/deploy.html#builtin-server

@albig
Copy link
Member

albig commented Nov 29, 2023

Geschafft.

2023-11-29 10:27:39,515 1 INFO lv-bb odoo.modules.loading: Module verdigado_attendance loaded in 1031.09s, 232412 queries

Wie gesagt, das ist meine lokale Docker-Umgebung und schon eine komplexere Datenbank.

Da das Update ja nur ein Mal läuft, wenn man auf die 15.0.1.0.1 geht, ist das ok, denke ich.

@albig
Copy link
Member

albig commented Nov 29, 2023

Lokal im Docker läuft das multi-threaded. Das ist ja ok.

Auf dem Server ist es auch multi-threaded. Da hatte ich mit multi-processing mal experimentiert, es aber wieder sein lassen. Ich kann das ja noch mal einstellen und dann dort probieren, wie es läuft.

@albig albig merged commit c8db819 into verdigado:15.0 Nov 29, 2023
4 checks passed
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