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

(dperny) there could be a further case where we check if #474

Open
piascikj opened this issue Dec 15, 2017 · 0 comments
Open

(dperny) there could be a further case where we check if #474

piascikj opened this issue Dec 15, 2017 · 0 comments
Labels

Comments

@piascikj
Copy link
Member

there are host ports in the config that aren't in the spec, but
that's only possible if there's a mismatch in the number of
ports, which is handled by a length check earlier in the code

opened via imdone.io from a code comment on efae8db785 by Abhinandan Prativadi

NOTE (dperny) there could be a further case where we check if id:473


// NOTE(dperny) there could be a further case where we check if
// there are host ports in the config that aren't in the spec, but
// that's only possible if there's a mismatch in the number of
// ports, which is handled by a length check earlier in the code
}
}
return true
}

@piascikj piascikj added the NOTE label Jan 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant