-
Notifications
You must be signed in to change notification settings - Fork 287
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
Some test fixes #5085
Some test fixes #5085
Conversation
This was missed in #5071
Hosts contain IP addresses in string form, whose order is not defined.
@@ -95,7 +95,7 @@ func TestSessionConnectionCleanupJob(t *testing.T) { | |||
|
|||
// Create the job. | |||
job, err := newSessionConnectionCleanupJob(ctx, rw, gracePeriod) | |||
job.gracePeriod = gracePeriod // by-pass factory assert so we dont have to wait so long | |||
job.workerStatusGracePeriod = gracePeriod // by-pass factory assert so we dont have to wait so long |
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 is also needed on the main
branch
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.
I would think the other change (not swagger) is as well. Maybe make this against main
and then backport?
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.
Looks like those changes are part of this PR to main
: #5084.
internal/session: update variable name in test
This was missed in #5071
internal/gen: regenerate files
internal/host/plugin: fix flaky test
Hosts contain IP addresses in string form, whose
order is not defined.