Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Conversation

@zrhoffman
Copy link
Member

Regressed by #7614. This PR explicitly seeds the database after running migrations, since db/admin upgrade no longer does.


Which Traffic Control components are affected by this PR?

  • Documentation
  • CDN in a Box for Developers

What is the best way to verify this PR?

Check the result of the TR Ultimate Test Harness GHA workflow on this PR, which failed after #7825 was merged.

PR submission checklist

@zrhoffman zrhoffman added regression bug a bug in existing functionality introduced by a new version database relating to setup/installation/structure of the Traffic Ops database dev-ciab Related to CDN in a Box for Developers labels Oct 3, 2023
@zrhoffman zrhoffman mentioned this pull request Oct 3, 2023
4 tasks
@srijeet0406 srijeet0406 merged commit 1954ac2 into apache:master Oct 3, 2023
@zrhoffman zrhoffman deleted the seed-dev-ciab branch October 3, 2023 20:19
@zrhoffman zrhoffman mentioned this pull request Oct 3, 2023
4 tasks
@ocket8888
Copy link
Contributor

This is using the incorrect seed data - should be using the dev-env specific seeds.psql file

@zrhoffman
Copy link
Member Author

zrhoffman commented Oct 4, 2023

This is using the incorrect seed data - should be using the dev-env specific seeds.psql file

Thanks, readded in #7828 in 3039f63 (though running db/admin seed is still necessary afaict)

@ocket8888
Copy link
Contributor

ocket8888 commented Oct 4, 2023

running db/admin seed is still necessary

more or less. I had just been using psql directly, which works just as well.

rimashah25 pushed a commit that referenced this pull request Oct 23, 2023
* Seed database in CDN in a Box for developers

* Remove psql command that does nothing, since seed.psql does not exist

(cherry picked from commit 1954ac2)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

database relating to setup/installation/structure of the Traffic Ops database dev-ciab Related to CDN in a Box for Developers regression bug a bug in existing functionality introduced by a new version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants