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 incorrect flag in nginx-ingress.md #245

Merged
merged 1 commit into from
Jun 20, 2017

Conversation

jordanjennings
Copy link
Contributor

Originally had dry-run=true as the dry run flag but that doesn't work and makes external-dns fail to start. Changed to --dry-run.

Originally had `dry-run=true` as the dry run flag but that doesn't work and makes external-dns fail to start. Changed to `--dry-run`.
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 20, 2017
@hjacobs
Copy link
Contributor

hjacobs commented Jun 20, 2017

Thanks for the PR!

@hjacobs hjacobs merged commit 1592ef0 into kubernetes-sigs:master Jun 20, 2017
@jordanjennings jordanjennings deleted the patch-1 branch June 20, 2017 22:22
ffledgling pushed a commit to ffledgling/external-dns that referenced this pull request Jan 18, 2018
Originally had `dry-run=true` as the dry run flag but that doesn't work and makes external-dns fail to start. Changed to `--dry-run`.
lou-lan pushed a commit to lou-lan/external-dns that referenced this pull request May 11, 2022
…t options (kubernetes-sigs#245)

* Add test cases for `krew install` with --archive and --manifest options

* Improve formatting

* ReplaceAll was added in go 1.12, but krew uses 1.11

* Rename test plugin from `test-foo` to `foo`

* Use wildcard platform selector in test data

* Add windows version of the test plugin for testing on windows
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants