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

Fix/depconf ref #83

Merged
merged 6 commits into from
Apr 22, 2021
Merged

Fix/depconf ref #83

merged 6 commits into from
Apr 22, 2021

Conversation

KineticCookie
Copy link
Member

@KineticCookie KineticCookie commented Apr 5, 2021 •
edited
Loading

  • Make default deployment configuration explicit
  • Fix name for default deployment configuration as hydrosphere_manager_default
  • Remove reflection util for configuration logging
  • Add secret datatype to hide passwords from logging
  • Remove obsolete configuration case classes

@KineticCookie KineticCookie marked this pull request as ready for review April 20, 2021 21:07
@KineticCookie KineticCookie requested a review from Vixtir April 21, 2021 10:03
@KineticCookie KineticCookie mentioned this pull request Apr 21, 2021
Merged
@KineticCookie KineticCookie merged commit b017e6b into master Apr 22, 2021
@KineticCookie KineticCookie deleted the fix/depconf-ref branch April 22, 2021 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Vixtir Vixtir Vixtir approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@KineticCookie @Vixtir