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

New predefined labels are not installed for user when install and upgrade happen in less than 2hours. #302

Open
joshelser opened this issue Sep 13, 2023 · 1 comment

Comments

@joshelser
Copy link
Contributor

  1. Installed v1.19
  2. Upgraded to v1.21
  3. Observe that Reoccurring Query and Ad Hoc Query labels are missing

call internal.MIGRATE_PREDEFINED_LABELS(7200) called by admin.finalize_setup() does not appear to be properly detecting the new predef'n labels and creating them.

@joshelser
Copy link
Contributor Author

This is just be a symptom of upgrade testing where the 2hour window to differentiate between "old" and "new" predefined labels is insufficient.

The logic inside call INTERNAL.MIGRATED_PREDEFINED_LABELS(7200) fails when a developer is doing upgrade testing because the time difference between the "old" predefined labels and "new" predefined labels is shorter than the expected 2hour window (7200s).

It may be better to use a sequence number to differentiate "old" from "new" predefined labels and avoid this.

@joshelser joshelser changed the title Newly added predefined labels are not installed after upgrade New predefined labels are not installed for user when install and upgrade happen in less than 2hours. Sep 13, 2023
@rymurr rymurr modified the milestones: Sprint 25, Sprint 26 Sep 25, 2023
@joshelser joshelser removed this from the Sprint 26 milestone Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

2 participants