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

Make dump method independent from update method #42

Merged
merged 1 commit into from
Nov 19, 2018

Conversation

keigohtr
Copy link
Member

@keigohtr keigohtr commented Nov 1, 2018

What is this PR for?

Dumping Kubernetes Service info is too slow. We need to separate it from update_dbs_kubernetes method.

Note

  • Batch execution for dumping is out of scope on this PR.
  • Exception rule of app/auth/__init__.py is a temporary. We will use access_token of the future expansion.

This PR includes

  • Create an endpoint for dump

What type of PR is it?

Minor

What is the issue?

#33

How should this be tested?

Sync on WebUI

Copy link
Member

@yoquankara yoquankara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@keigohtr keigohtr merged commit 5f3697c into master Nov 19, 2018
@keigohtr keigohtr deleted the fix/make-dump-independent branch November 19, 2018 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants