Hey team! Long time no see! I have a doubt. We upgraded Flyte from

1.9

straight to

1.13.1

and we got an error: This seems to be something introduced by: https://github.com/flyteorg/flyte/pull/5453 So we upgraded to

1.12.0

and from there to

1.13.1

and that worked. Is this expected?

@thankful-tailor-28399, hello! Can you give us more details? What error were you seeing?

Sorry for the lack of context, my bad.

2024/08/28 072853 /go/src/github.com/flyteorg/flyteadmin/pkg/repositories/config/migrations.go:1256 ERROR: permission denied for table execution_admin_tags (SQLSTATE 42501)

[2.606ms] [rows:0] INSERT INTO execution_tags (execution_project, execution_domain, execution_name, created_at, updated_at, deleted_at, key, value) SELECT execution_project, execution_domain, execution_name, created_at, updated_at, deleted_at, name as key, ‘’ as value FROM execution_admin_tags INNER JOIN admin_tags a on execution_admin_tags.admin_tag_id = a.id;

Error: database migration failed: ERROR: permission denied for table execution_admin_tags (SQLSTATE 42501)

Usage:

flyteadmin migrate run [flags]

Flags:

-h, --help help for run

database migration failed: ERROR: permission denied for table execution_admin_tags (SQLSTATE 42501)

I don’t remember if this was the exact error message, but it was related to this table. The permissions granted were correct. With the same permissions, going from

1.9

to

1.13.1

did not work. Going from

1.9

to

1.12.0

to

1.13.1

worked My understanding is that between

1.9

and

1.13.1

we created this

execution_admin_tags

which we later deleted, so the latest migration could not run correctly

let me dig a little into this. I'm surprised that successive migrations didn't handle this.

Not a blocker at all for us as we could sort it out. Agree that migrations should handle this by default, so maybe we can wait and see if it happens to someone else and then investigate it