Skip to content
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

Fix formatting issue. #524

Merged
merged 1 commit into from
Jul 10, 2019
Merged

Fix formatting issue. #524

merged 1 commit into from
Jul 10, 2019

Conversation

uthark
Copy link
Contributor

@uthark uthark commented Jul 10, 2019

Before

Watching crd.k8s.amazonaws.com/v1alpha1, ENIConfig, default, every %!d(float64=5) s

After

Watching crd.k8s.amazonaws.com/v1alpha1, ENIConfig, default, every 5 s

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

*Before*
```
Watching crd.k8s.amazonaws.com/v1alpha1, ENIConfig, default, every %!d(float64=5) s
```

*After*
```
Watching crd.k8s.amazonaws.com/v1alpha1, ENIConfig, default, every 5 s
```
Copy link
Contributor

@mogren mogren left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -157,7 +157,7 @@ func (eniCfg *ENIConfigController) Start() {
log.Errorf("failed to get watch namespace: %v", err)
}
resyncPeriod := time.Second * 5
log.Infof("Watching %s, %s, %s, every %d s", resource, kind, namespace, resyncPeriod.Seconds())
log.Infof("Watching %s, %s, %s, every %v s", resource, kind, namespace, resyncPeriod.Seconds())
Copy link
Contributor

Choose a reason for hiding this comment

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

I missed this one too! Thanks for fixing.

@mogren mogren merged commit e638b1b into aws:master Jul 10, 2019
@mogren mogren added this to the v1.6 milestone Jul 10, 2019
@uthark uthark deleted the patch-3 branch July 15, 2019 19:33
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