diff options
author | Matthew Heon <matthew.heon@pm.me> | 2020-03-01 17:54:11 -0500 |
---|---|---|
committer | Matthew Heon <matthew.heon@pm.me> | 2020-03-02 10:58:11 -0500 |
commit | e45456223c4caa762be1a9b1f6b94006d5053c1a (patch) | |
tree | 79514017ed640ce8869818f9d68add27a5a4fbd0 /test | |
parent | 275e9b855dd0a384a283174912c08f3f097101b5 (diff) | |
download | podman-e45456223c4caa762be1a9b1f6b94006d5053c1a.tar.gz podman-e45456223c4caa762be1a9b1f6b94006d5053c1a.tar.bz2 podman-e45456223c4caa762be1a9b1f6b94006d5053c1a.zip |
Add validate() for containers
Until now, we've been validating every part of container
configuration through the With... functions that set the options.
This if fine when we are just validating the options to an
individual function, but things get complicated once we need to
validate conflicts between different options. We don't know the
order in which things were passed, so we need the validation on
both of the potential options that can conflict, resulting in
significant code duplication. To solve this, add a validate()
function for containers, and use this to check whether everything
is in a good state.
We can probably move more into this function (there are other
parts of container creation that also do validation of a sort)
but this is a good start to simplifying our options.
Signed-off-by: Matthew Heon <matthew.heon@pm.me>
Diffstat (limited to 'test')
0 files changed, 0 insertions, 0 deletions