-
Notifications
You must be signed in to change notification settings - Fork 7.4k
Persist parameters during deployment (Sync) #39
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
Merged
Merged
Changes from 20 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
a61db98
Add basic skeleton for improvements
94176ba
Update protobuf config for repository API
5b625d4
Update skeleton of deploy params code
fe66571
Add DeploymentInfo struct
606fd13
Add comment, comment out code
18dbd9f
Update repository.pb.go
bdc7a2e
Update comment in repository.proto
253e65d
Update reposerver subpackage
94348a6
Comment out for now
0339674
Start getting function into shape
a100fb7
Flesh out logic further
15da5e1
Get unit test started for TestListEnvParams
3ec6340
Add missing interface declaration
a4491be
Finish unit test for ListEnvParams
3a9c380
Use string literals for regexp.MustCompile
08a4e82
Clean up RPC response code
428a56d
Use map[string]interface{} for now, thanks @alexmt
3a90924
First draft attempt to integrate deployment info retrieval
f4e0b84
Install more wiring
eba2122
DRY
b15ebb3
Move CRD update to after successful deployment, thanks @alexmt
0895d06
Move app params update to after successful deployment, too
094a3c8
Don't replace application status, thanks @alexmt
a0451ad
Convert to string types
9a7b714
Don't create unused var
9445dba
Log received values
09bd414
Unquote quoted params
08e3871
Use string as value type
14e1b43
Use named returns, no magic number
7e8d2c4
Use map for gRPC data transfer
015e197
Update generated code
73c4a21
Log actual map, not wrapper object
a41dec1
Fix unit test
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please set recent deployment only if actual deployment is successful?