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.

Sign up for GitHub

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

Jump to bottom

Add database migration command #23

Closed
ehelms opened this issue Dec 6, 2019 · 2 comments
Closed

Add database migration command #23

ehelms opened this issue Dec 6, 2019 · 2 comments

Comments

@ehelms
Copy link
Member

ehelms commented Dec 6, 2019

No description provided.

@ekohl
Copy link
Member

ekohl commented Dec 6, 2019

There is one, but it's only when it's refreshed. This is only when the package changes or the config changes. We need a way to figure out how to determine if it needs to run:

puppet-pulpcore/manifests/database.pp

Line 28 in f96b33b

refreshonly => true,

@ehelms
Copy link
Member Author

ehelms commented Dec 6, 2019

Looks like this should do it:

[root@centos7-pulpcore pulp]# django-admin migrate --plan
Planned operations:
  No planned migration operations.

ehelms added a commit to ehelms/puppet-pulpcore that referenced this issue Dec 6, 2019
@ehelms
fixes theforeman#23: Only run database migrations if they are any pen…
6c8a513 
…ding
ehelms added a commit to ehelms/puppet-pulpcore that referenced this issue Dec 6, 2019
@ehelms
fixes theforeman#23: Only run database migrations if they are any pen…
eb5d5ff 
…ding
@ekohl ekohl closed this as completed in ee06c85 Dec 6, 2019
cegeka-jenkins pushed a commit to cegeka/puppet-pulpcore that referenced this issue Aug 9, 2022
@ehelms @ekohl
fixes theforeman#23: Only run database migrations if they are any pen…
cd12486 
…ding
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ekohl @ehelms