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

cmd/operator-sdk: fix run local flags #2478

Merged
merged 2 commits into from
Jan 29, 2020
Merged

cmd/operator-sdk: fix run local flags #2478

merged 2 commits into from
Jan 29, 2020

Conversation

liranp
Copy link
Contributor

@liranp liranp commented Jan 28, 2020

Closes #2477.

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 28, 2020
@joelanford
Copy link
Member

@liranp Thanks! Could you add a line to the CHANGELOG for this? Something like:

Fixes regression that causes operator-sdk run --local flags to be ignored (#2478)

/cc @estroz
We should do a sweep of the recent CLI changes to make sure we don't have this problem in other places.

@joelanford joelanford added the kind/bug Categorizes issue or PR as related to a bug. label Jan 28, 2020
@liranp
Copy link
Contributor Author

liranp commented Jan 28, 2020

@joelanford Done. Thank you for reviewing this.

CHANGELOG.md Outdated
@@ -11,6 +11,7 @@
### Bug Fixes

- Fixed issue with Go dependencies caused by removed tag in `openshift/api` repository ([#2466](https://github.com/operator-framework/operator-sdk/issues/2466))
- Fixed a regression in the `operator-sdk run` command that caused `--local` flags to be ignored (#2478)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Fixed a regression in the `operator-sdk run` command that caused `--local` flags to be ignored (#2478)
- Fixed a regression in the `operator-sdk run` command that caused `--local` flags to be ignored ([#2478](https://github.com/operator-framework/operator-sdk/issues/2478))

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, my bad. I've missed that. Fixed.

Copy link
Member

Choose a reason for hiding this comment

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

No worries. I should have been more clear the first time :)

@joelanford
Copy link
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 28, 2020
Copy link
Member

@estroz estroz left a comment

Choose a reason for hiding this comment

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

/lgtm

@estroz estroz merged commit 87787d1 into operator-framework:master Jan 29, 2020
@liranp liranp deleted the fix-run-local branch January 29, 2020 09:40
camilamacedo86 added a commit that referenced this pull request Feb 11, 2020
* CHANGELOG.md: update for next release

Co-authored-by: Liran Polak <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

operator run --local: --go-ldflags not effective
4 participants