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

Remove integration test which references issue 1222 and close #1222

Open
mhagen-vmware opened this issue Jun 27, 2016 · 8 comments
Open

Remove integration test which references issue 1222 and close #1222

mhagen-vmware opened this issue Jun 27, 2016 · 8 comments
Assignees
Labels
impact/test/integration/enable The test is associated with a disabled integration test kind/feature New functionality you could include in marketing material priority/p4 resolution/incomplete Insufficint information is available to address this issue

Comments

@mhagen-vmware
Copy link
Contributor

I can't find an epic or implementation issue to work on the above support, so creating this. Close if we have a feature implementation issue already for this.

@mhagen-vmware mhagen-vmware added the kind/feature New functionality you could include in marketing material label Jun 27, 2016
@mdubya66 mdubya66 added this to the VIC GA Release milestone Jun 29, 2016
@mdubya66
Copy link
Contributor

@mhagen-vmware is there any context for this?

Searching the code found this.

/lib/apiservers/portlayer/models/scope_config.go: /* scope type
./lib/portlayer/network/context.go: return nil, fmt.Errorf("scope type not supported")

@mhagen-vmware
Copy link
Contributor Author

Sorry, I talked to @hmahmood when I made this and didn't translate our conversation into this issue:
Yes, this is network related. You get this error when you run:
docker ${params} network create -d overlay test-network2

This is mostly just a place holder for my test scripts until we decide we want to support this feature.

@corrieb
Copy link
Contributor

corrieb commented Jul 7, 2016

NOTE: CLOSING THIS ISSUE MAY CAUSE AN INTEGRATION TEST FAILURE. Please re-enable the test in 1-15-Docker-Network-Create.robot as part of a PR to fix this issue. Please also ensure that the test passes cleanly. If the test needs improving or updating, please address it. Thx

@hmahmood
Copy link
Contributor

hmahmood commented Aug 8, 2016

We are not going to support overlay networks for GA. Is this just a wording issue? i.e. do we need to change the error message?

@mhagen-vmware
Copy link
Contributor Author

I am expecting something like the following error message, when I haven't setup an overlay network yet:
#Should Contain ${output} Error response from daemon: failed to parse pool request for address space "GlobalDefault" pool "" subpool "": cannot find address space GlobalDefault (most likely the backing datastore is not configured)

@mdubya66 mdubya66 removed this from the VIC GA Release milestone Sep 20, 2016
@mdubya66 mdubya66 added the impact/doc/note Requires creation of or changes to an official release note label Sep 20, 2016
@mdubya66
Copy link
Contributor

Moving to Icebox per 9/20 Triage. Needs release note.

@anchal-agrawal anchal-agrawal added the impact/test/integration/enable The test is associated with a disabled integration test label Oct 26, 2016
@stuclem
Copy link
Contributor

stuclem commented Nov 7, 2016

Already in the release notes, so removing the kind/note tag.

@mhagen-vmware mhagen-vmware removed the impact/doc/note Requires creation of or changes to an official release note label Nov 7, 2016
@hmahmood hmahmood removed their assignment Jan 17, 2017
@renmaosheng
Copy link
Contributor

closing this impacts tests, Wending is trying a new pull request:
#8396

@renmaosheng renmaosheng reopened this Nov 23, 2018
@zjs zjs added the resolution/incomplete Insufficint information is available to address this issue label Mar 6, 2019
@zjs zjs changed the title Error response from daemon: scope type not supported Remove integration test which references issue 1222 and close Mar 6, 2019
@renmaosheng renmaosheng reopened this Mar 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/test/integration/enable The test is associated with a disabled integration test kind/feature New functionality you could include in marketing material priority/p4 resolution/incomplete Insufficint information is available to address this issue
Projects
None yet
Development

No branches or pull requests

9 participants