sh-edraft.de/sh_discord_bot
2
1
Fork 1
Code Issues 26 Pull Requests Actions Packages Projects 3 Releases 27 Wiki Activity

#428 #436

Merged
edraft merged 19 commits from #428 into dev 2023-11-15 16:56:32 +01:00
Conversation 1 Commits 19 Files Changed 114 +2717 -2610
edraft commented 2023-11-13 22:56:08 +01:00
Owner
Copy Link

Ticket Referenz:

#428

Gibt es etwas beim Review zu beachten?

Nein

#### Ticket Referenz: #428 #### Gibt es etwas beim Review zu beachten? Nein
edraft added 15 commits 2023-11-13 22:56:09 +01:00
Added mock db base #428 666b20d3a9
Improved sql migration to script migration tool #428 1f2fbc362f
Added logic to handle down and upgrades #428 4628f31993
Added 0.1 migration scripts #428 407ec08463
Added 0.2.2 migration scripts #428 aec7dac4c7
Added 0.3 migration scripts #428 d93c3ad6c7
Added 0.3.1 migration scripts #428 5461a6d8dc
Fixed migrations #428 2801c617f6
Fixed versions #428 39b9def76c
Updated and added script order #428 e018fdcbdf
Readded known scripts & added formatting #428 5c8feed8aa
Added 1.0.0 scripts #428 35a8b8f592
Added other scripts #428 4e12ba5ffe
Finished script migration #428 dd3bfa68c6
Removed old scripts #428 d2d59bdad7
edraft added 2 commits 2023-11-13 23:38:09 +01:00
Removed old logic #428 e01c738cf0
Added version logic & fixed downgrade scripts #428 fe5b0207c0
edraft changed title from WIP: #428 to #428 2023-11-13 23:38:26 +01:00
edraft added 1 commit 2023-11-14 23:50:08 +01:00
Added migration to old naming safety #428 06a0eba5c5
Ebola-Chan approved these changes 2023-11-15 02:21:05 +01:00
Ebola-Chan left a comment
Contributor
Copy Link

Es gibt noch Stellen an denen die Tabelle geändert werden aber im Downgrade werden diese Änderungen nicht mehr Rückgängig gemacht.

Das sind Sachen die mir Aufgefallen sind, ohne dass ich mich groß mit Migration auskenne.

Es gibt noch Stellen an denen die Tabelle geändert werden aber im Downgrade werden diese Änderungen nicht mehr Rückgängig gemacht. Das sind Sachen die mir Aufgefallen sind, ohne dass ich mich groß mit Migration auskenne.
bot/src/bot_data/scripts/1.1.7/2_FixUpdates_up.sql
@@ -0,0 +1,195 @@
ALTER TABLE CFG_ServerHistory
Ebola-Chan commented 2023-11-15 01:41:02 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/scripts/1.1.9/1_ShortRoleNameOnlyHighest_up.sql
@@ -0,0 +1,134 @@
ALTER TABLE CFG_Server
Ebola-Chan commented 2023-11-15 01:41:28 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/scripts/1.2.0/1_FixUserHistory_up.sql
@@ -0,0 +1,45 @@
ALTER TABLE `Users`
Ebola-Chan commented 2023-11-15 01:41:42 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/scripts/1.2.0/2_Birthday_up.sql
@@ -0,0 +1,190 @@
ALTER TABLE Users
Ebola-Chan commented 2023-11-15 01:41:58 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/scripts/1.2.0/4_MaxSteamOfferCount_up.sql
@@ -0,0 +1,79 @@
ALTER TABLE CFG_Technician
Ebola-Chan commented 2023-11-15 01:42:14 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/scripts/1.2.0/5_MaintenanceMode_up.sql
@@ -0,0 +1,84 @@
ALTER TABLE CFG_Technician
Ebola-Chan commented 2023-11-15 01:44:28 +01:00
Contributor
Copy Link

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden

Hier werden Tabellen angelegt, welche im Downgrade nicht gedroppt werden
bot/src/bot_data/service/migration_service.py Outdated
@@ -31,0 +41,4 @@
if migration.migration_id.endswith("Migration"):
return
self._logger.debug(__name__, f"Migrate old migration {migration.migration_id} to new method")
Ebola-Chan commented 2023-11-15 01:48:11 +01:00
Contributor
Copy Link

Sollte hier nicht vielleicht die Nachricht "Migrate new migration {migration.migration_id} to old method" geloggt werden?

Jedenfalls steht in Zeile 52 die selbe Nachricht drinne.

Sollte hier nicht vielleicht die Nachricht `"Migrate new migration {migration.migration_id} to old method"` geloggt werden? Jedenfalls steht in Zeile 52 die selbe Nachricht drinne.
bot/src/bot_data/service/migration_service.py
@@ -31,0 +86,4 @@
if upgrade and version.Version(version_str) > version.Version(bot.__version__):
break
# downgrade run migrations from higher versions
if not upgrade and version.Version(version_str) <= version.Version(bot.__version__):
Ebola-Chan commented 2023-11-15 02:05:15 +01:00
Contributor
Copy Link

Wie ich es verstehe, sammelst du Downgrades von der höchsten Version bis zum Botversion und führst sie an einer anderen Stelle aus. Sollte es da nicht zu einem Fehler führen, da du im Downgrade Tabellen Droppst, die evtl. nicht existieren?

Wie ich es verstehe, sammelst du Downgrades von der höchsten Version bis zum Botversion und führst sie an einer anderen Stelle aus. Sollte es da nicht zu einem Fehler führen, da du im Downgrade Tabellen Droppst, die evtl. nicht existieren?
bot/src/bot_data/service/migration_service.py
@@ -31,0 +87,4 @@
break
# downgrade run migrations from higher versions
if not upgrade and version.Version(version_str) <= version.Version(bot.__version__):
continue
Ebola-Chan commented 2023-11-15 02:08:18 +01:00
Contributor
Copy Link

Bei einem Downgrade macht du ein continue wenn die Bedingung zutrifft, während du beim Upgrade ein break abrufst. Gibt es hier einen Fall, in dem die Ordner evtl. nicht in richtiger Reihenfolge stehen und due damit versuchst relevante Ordner abzufangen?

Bei einem Downgrade macht du ein `continue` wenn die Bedingung zutrifft, während du beim Upgrade ein `break` abrufst. Gibt es hier einen Fall, in dem die Ordner evtl. nicht in richtiger Reihenfolge stehen und due damit versuchst relevante Ordner abzufangen?
edraft added 1 commit 2023-11-15 16:56:17 +01:00
Fixed scripts #428 4ba40b826a
edraft merged commit 25df0e4876 into dev 2023-11-15 16:56:32 +01:00
edraft deleted branch #428 2023-11-15 16:56:32 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Ebola-Chan
Labels
Clear labels
MVP
web-interface
bug
Something is not working
bugfix
Solution of a problem
cli
Changes for command line interface
discussion
Something needs to be discussed
documentation
Indicates a need for improvements or additions to documentation
duplicate
This issue or pull request already exists
expense
high
expense
low
expense
medium
expense
very-high
help wanted
Need some help
improvement
Improvement of a feature
invalid
Something is wrong
priority
blocker
priority
high
priority
low
priority
medium
question
More information is needed
refactoring
Code need some refactoring
release
Create a release
story
New feature
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ci edraft edraft-dev
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sh-edraft.de/sh_discord_bot#436
No description provided.
Delete Branch "#428"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?