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

Rename pkg/sdn to pkg/network, for consistency with its API #16057

Merged
merged 2 commits into from
Aug 31, 2017

Conversation

danwinship
Copy link
Contributor

@deads2k wanted this

This is on top of #15781, but that should be merged soon. Only the last two commits ("Rename pkg/sdn to pkg/network, for consistency with its API" and "Regenerate files") are for this PR.

@openshift-merge-robot openshift-merge-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. needs-api-review labels Aug 30, 2017
@deads2k
Copy link
Contributor

deads2k commented Aug 30, 2017

/lgtm

bumping queue priority because this one is going to suck to rebase.

@openshift-merge-robot openshift-merge-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 30, 2017
@deads2k
Copy link
Contributor

deads2k commented Aug 31, 2017

hrm, bot didn't obey and now you have to rebase again. sorry.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 31, 2017
@openshift-merge-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: danwinship, deads2k

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@danwinship
Copy link
Contributor Author

it was actually already needs-rebase when I pushed it, because I based it on top of #15781 rather than master (and apparently there was some change in master that this PR conflicted with that #15781 didn't).

and consistently refer to pkg/network/apis/network as "networkapi"
@openshift-merge-robot openshift-merge-robot removed lgtm Indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Aug 31, 2017
@deads2k
Copy link
Contributor

deads2k commented Aug 31, 2017

@danwinship for rebases, it's the same old rosie rules, but you have to re-apply your tags manually.

@deads2k deads2k added the lgtm Indicates that a PR is ready to be merged. label Aug 31, 2017
@danwinship
Copy link
Contributor Author

/test cmd

@danwinship
Copy link
Contributor Author

/test extended_conformance_install_update

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue (batch tested with PRs 16054, 16057, 15440, 16028, 16055)

@openshift-merge-robot openshift-merge-robot merged commit fbd5dd9 into openshift:master Aug 31, 2017
@danwinship danwinship deleted the rename-pkg-sdn branch October 19, 2017 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. component/networking lgtm Indicates that a PR is ready to be merged. needs-api-review queue/multiple-rebases size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants