Skip to content

Conversation

@madhanrm
Copy link
Contributor

Expose DSR Settings in Load Balancer
Fix Hcn tests to continue on failure.

@madhanrm
Copy link
Contributor Author

FYI @erfrimod

}
if foundEndpoint == nil {
t.Errorf("No Endpoint found")
t.Fatalf("No Endpoint found")
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: Fatal

}
if foundEndpoint == nil {
t.Errorf("No Endpoint found")
t.Fatalf("No Endpoint found")
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: Fatal

}
if len(foundEndpoints) == 0 {
t.Errorf("No Endpoint found")
t.Fatalf("No Endpoint found")
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: Fatal

}
if len(foundEndpoints) == 0 {
t.Errorf("No Endpoint found")
t.Fatalf("No Endpoint found")
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: Fatal

}
if Endpoint.HostComputeNamespace == "" {
t.Errorf("No Namespace detected.")
t.Fatalf("No Namespace detected.")
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: Fatal

}
if len(foundEndpoint.Policies) == 0 {
t.Errorf("No Endpoint Policies found")
t.Fatalf("No Endpoint Policies found")
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: Fatal

}
if len(foundEndpoint.Policies) == 0 {
t.Errorf("No Endpoint Policies found")
t.Fatalf("No Endpoint Policies found")
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: Fatal

_, err := GetNetworkByName("Not found name")
if err == nil {
t.Errorf("Error was not thrown.")
t.Fatalf("Error was not thrown.")
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: Fatal and others

Expose ELB DSR flag
@madhanrm
Copy link
Contributor Author

@jterry75 can you get this merged?

@jterry75
Copy link
Contributor

@madhanrm - Done

@madhanrm
Copy link
Contributor Author

@jterry75 Thanks!

@madhanrm madhanrm deleted the elbdsr branch October 19, 2018 17:54
dcantah pushed a commit to dcantah/hcsshim that referenced this pull request Mar 17, 2021
Fix issue causing SCSI hot add to find no matching device
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 28, 2024
Expose DSR Settings in Load Balancer
princepereira pushed a commit to princepereira/hcsshim that referenced this pull request Aug 29, 2024
Expose DSR Settings in Load Balancer
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