Skip to content

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
### 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