-
Notifications
You must be signed in to change notification settings - Fork 78
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
Add locality lb settings support on destination rules #41
Add locality lb settings support on destination rules #41
Conversation
Signed-off-by: aattuluri <anil_attuluri@intuit.com>
Signed-off-by: aattuluri <anil_attuluri@intuit.com>
Signed-off-by: aattuluri <anil_attuluri@intuit.com>
Signed-off-by: aattuluri <anil_attuluri@intuit.com>
This reverts commit a9fa409.
373c5bc
to
408c9c8
Compare
@@ -19,6 +19,7 @@ import ( | |||
"errors" | |||
"fmt" | |||
"github.com/istio-ecosystem/admiral/admiral/pkg/controller/secret/resolver" | |||
"github.com/sirupsen/logrus" |
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.
Nothing I think we need to change, but for future reference you can save yourself the trouble of renaming every reference from log -> logrus by importing this as log: log "github.com/sirupsen/logrus"
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.
Changed this to log.
@@ -30,7 +30,7 @@ spec: | |||
- $(secret_resolver) | |||
- --secret_resolver_config_path | |||
- /etc/admiral/config.yaml | |||
image: docker.io/admiralproj/admiral:v0.1-alpha | |||
image: docker.io/admiralproj/admiral:kubecon |
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.
This should be a version or sha, right?
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.
Changed it to the current stable release.
@@ -2,16 +2,12 @@ package clusters | |||
|
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.
Seems like you deleted these tests but moved the methods they are testing - let's make sure they are re-added in the right places
* Bootstrapping admiral * Add circle ci config * Fix the working directory for builds * Bootstrapping admiral Signed-off-by: aattuluri <anil_attuluri@intuit.com> * Add circle ci config Signed-off-by: aattuluri <anil_attuluri@intuit.com> * Fix the working directory for builds Signed-off-by: aattuluri <anil_attuluri@intuit.com> * Add build status badge. Signed-off-by: aattuluri <anil_attuluri@intuit.com> * Revert "Add build status badge." This reverts commit a9fa409. * Working version after major refactor. * Fix test files. * Use global traffic policy to update destination rule. * Minor fixes. * Update image for admiral * Add outlier detection for locality load balancing to work * Undo tag change * Sample files for demo * Remove/comment some tests * Add the needed method * Commenting out few more tests. * Remove unwanted files. * Adding back missing tests. * Create an alias for logrus. * Fixing tests. Signed-off-by: Joe Peacock <joe_peacock@intuit.com>
* Bootstrapping admiral * Add circle ci config * Fix the working directory for builds * Bootstrapping admiral Signed-off-by: aattuluri <anil_attuluri@intuit.com> * Add circle ci config Signed-off-by: aattuluri <anil_attuluri@intuit.com> * Fix the working directory for builds Signed-off-by: aattuluri <anil_attuluri@intuit.com> * Add build status badge. Signed-off-by: aattuluri <anil_attuluri@intuit.com> * Revert "Add build status badge." This reverts commit a9fa409. * Working version after major refactor. * Fix test files. * Use global traffic policy to update destination rule. * Minor fixes. * Update image for admiral * Add outlier detection for locality load balancing to work * Undo tag change * Sample files for demo * Remove/comment some tests * Add the needed method * Commenting out few more tests. * Remove unwanted files. * Adding back missing tests. * Create an alias for logrus. * Fixing tests. Signed-off-by: Madeline <madeline_li@intuit.com>
No description provided.