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

migration error in stderr when downgrading from v1.16.1 to v1.15.1 #3629

Open
webern opened this issue Nov 30, 2023 · 1 comment
Open

migration error in stderr when downgrading from v1.16.1 to v1.15.1 #3629

webern opened this issue Nov 30, 2023 · 1 comment
Assignees
Labels
type/bug Something isn't working

Comments

@webern
Copy link
Contributor

webern commented Nov 30, 2023

I saw these errors when I downgraded from develop:b375585ce32e61a2d796a1f9eff9d374964fa813 (version in Release.toml is v1.16.1) to an image built at the v1.15.1 tag.

Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [INFO] Running migration 'migrate_v1.16.1_updog-network-affected.lz4'
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [INFO] Running migration 'migrate_v1.16.0_schnauzer-v2-generators.lz4'
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [INFO] Running migration 'migrate_v1.16.0_public-control-container-v0-7-5.lz4'
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [INFO] Running migration 'migrate_v1.16.0_aws-control-container-v0-7-5.lz4'
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [ERROR] Migration stderr: 'settings.host-containers.control.source' has no 'template' key in metadata
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [INFO] Running migration 'migrate_v1.16.0_public-admin-container-v0-11-1.lz4'
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [INFO] Running migration 'migrate_v1.16.0_aws-admin-container-v0-11-1.lz4'
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [ERROR] Migration stderr: 'settings.host-containers.admin.source' has no 'template' key in metadata
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [INFO] Running migration 'migrate_v1.16.0_kernel-modules-autoload-settings.lz4'
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [INFO] Running migration 'migrate_v1.16.0_kernel-modules-autoload-restart.lz4'
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [INFO] Running migration 'migrate_v1.16.0_kernel-modules-autoload-files.lz4'
Nov 29 19:33:13 localhost migrator[941]: 19:33:13 [INFO] Running migration 'migrate_v1.16.0_kernel-modules-autoload-configs.lz4'
@webern webern added type/bug Something isn't working status/needs-triage Pending triage or re-evaluation labels Nov 30, 2023
@webern webern mentioned this issue Nov 30, 2023
5 tasks
@cbgbt cbgbt self-assigned this Dec 6, 2023
@cbgbt
Copy link
Contributor

cbgbt commented Dec 6, 2023

This is a bug in this migration.

Prior to schnauzer-v2, template-based settings generators had two critical pieces of datastore-based metadata:

  • The template, which told the generator what template string to render
  • The setting-generator, which told the generator how to invoke schnauzer to render the template.

Moving to schnauzer-v2, the template became irrelevant. The migration "up" preserved that metadata in case of a downwards migration; however, if you downgrade from 1.16.0 to 1.15.1 without a previous upgrade first, you will not have this metadata. The migration needs to be modified to explicitly insert this metadata.

I'll submit a fix.

@yeazelm yeazelm removed the status/needs-triage Pending triage or re-evaluation label Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants