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

tsr: add command to sync users and repositories with Gandalf #1138

Closed
fsouza opened this issue Mar 14, 2015 · 1 comment
Closed

tsr: add command to sync users and repositories with Gandalf #1138

fsouza opened this issue Mar 14, 2015 · 1 comment
Assignees
Milestone

Comments

@fsouza
Copy link
Contributor

fsouza commented Mar 14, 2015

Currently, we always sync on start-up, making start-up sloooow. We should have a command for it.

It should not be a migration (see #1137), because users can turn Gandalf on and off all the time.

Some pending points:

  • are we going to really sync? (removing from gandalf whatever doesn't exist in tsuru)
  • if we are not going to really sync, how do we detect that a user existed in tsuru and doesn't exist anymore, and it stil exists on Gandalf?

Should we just don't care and always create on Gandalf what exists in tsuru and never care about removing anything from Gandalf in the sync process? Or maybe add a --dedicated flag to indicate that the Gandalf instance is dedicated to tsuru, which means that tsuru can mirror its data in Gandalf, removing any non-matching users and repositories?

And from a philosophical point of view: is tsuru supposed to support a shared Gandalf? How do we handle conflicts in user or repositories name?

It's easy, but there's room for a decision, so I'm marking it as brainstorm as well.

@fsouza
Copy link
Contributor Author

fsouza commented Mar 17, 2015

We have a direction: just create whatever exists in tsuru in Gandalf, just like the function in the start-up does today. Removing the brainstorm tag, it's now even easier.

@fsouza fsouza removed the brainstorm label Mar 17, 2015
@fsouza fsouza self-assigned this Mar 17, 2015
fsouza pushed a commit that referenced this issue Mar 17, 2015
We're going to use it in a tsr command.

Related to #1138.
fsouza pushed a commit that referenced this issue Mar 17, 2015
fsouza pushed a commit that referenced this issue Mar 17, 2015
Related to #1138. I will add some status reporting to context.Stdout
before closing the issue.
@fsouza fsouza closed this as completed in 36c9d9b Mar 17, 2015
fsouza pushed a commit that referenced this issue Mar 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant