-
Notifications
You must be signed in to change notification settings - Fork 49
test/*: de-duplicate value timeout and retryInterval #1049
Conversation
eff647f
to
584e827
Compare
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.
Hm, it seems there is more occurrences, even with 5sec/5min values.
I also think 9 minute tests could also get a const, like TimeoutSlow
or something.
There are 2 different values for timeout, 1 is 7min and other is 9 min. What should be the naming convention? TimeoutSlow and TimeoutSlower? |
7 minutes can either be left as-is (as there is no duplication) or just increased to 9 minutes. |
584e827
to
85fd4b8
Compare
453f775
to
625abe5
Compare
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.
Nice work @knrt10
625abe5
to
c6434ff
Compare
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.
LGTM
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.
Rebase needed.
timeout and retryInterval were same for most of the components and was duplicated. closes: #502 Signed-off-by: knrt10 <kautilya@kinvolk.io>
c6434ff
to
d39fc22
Compare
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.
LGTM
timeout and retryInterval were the same for most of the components and was
duplicated.
closes: #502
Signed-off-by: knrt10 kautilya@kinvolk.io