Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: izturn <44051386+izturn@users.noreply.github.com>
  • Loading branch information
mvgmb and izturn authored Sep 6, 2023
1 parent 3a6f412 commit 7bff54a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions pkg/mocks/plugin.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ const (

HTTPProxyName = "argo-rollouts"
ValidHTTPProxyName = "argo-rollouts-valid"
OutdatedHTTPProxy = "argo-rollouts-outdated"
OutdatedHTTPProxyName = "argo-rollouts-outdated"
InvalidHTTPProxyName = "argo-rollouts-invalid"
FalseConditionHTTPProxyName = "argo-rollouts-false-condition"

Expand Down Expand Up @@ -65,7 +65,7 @@ func MakeObjects() []runtime.Object {
return objs
}

func newValidHttpProxy(name string) *contourv1.HTTPProxy {
func newHTTPProxy(name string) *contourv1.HTTPProxy {
return &contourv1.HTTPProxy{
ObjectMeta: metav1.ObjectMeta{
Name: name,
Expand Down
2 changes: 1 addition & 1 deletion pkg/plugin/plugin.go
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ func (r *RpcPlugin) Type() string {
return Type
}

func (r *RpcPlugin) getHttpProxy(ctx context.Context, namespace string, name string) (*contourv1.HTTPProxy, error) {
func (r *RpcPlugin) getHTTPProxy(ctx context.Context, namespace string, name string) (*contourv1.HTTPProxy, error) {
unstr, err := r.dynamicClient.Resource(contourv1.HTTPProxyGVR).Namespace(namespace).Get(ctx, name, metav1.GetOptions{})
if err != nil {
return nil, err
Expand Down

0 comments on commit 7bff54a

Please sign in to comment.