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

enable customizations for podman-compose #421

Merged
merged 1 commit into from
Sep 14, 2022

Conversation

arewm
Copy link
Contributor

@arewm arewm commented Aug 26, 2022

Thanks to @yashvardhannanavati for the initial contents of podman.compose. I just wanted to help share it with others!

I haven't added any validation to this (namely ensuring that docker.compose is the content of docker-compose.yml), but this shouldn't be terribly hard to hook up to fail CI if false. I can also add content to the README if desired.

Signed-off-by: arewm arewm@users.noreply.github.com

@arewm arewm force-pushed the master branch 3 times, most recently from cc189e4 to 367ab43 Compare September 1, 2022 17:45
@arewm
Copy link
Contributor Author

arewm commented Sep 1, 2022

https://app.travis-ci.com/github/release-engineering/iib/builds/255102904 is an example of the failure when the docker-compose.yml file is not properly set.

@arewm arewm force-pushed the master branch 2 times, most recently from 17d90b6 to 8c00e33 Compare September 14, 2022 14:58
Signed-off-by: arewm <arewm@users.noreply.github.com>
@yashvardhannanavati yashvardhannanavati merged commit 18ce7a1 into release-engineering:master Sep 14, 2022
arewm added a commit to arewm/iib that referenced this pull request Sep 15, 2022
In release-engineering#421, I modified the
context and paths to both go up a directory. That was an error. I should
have just modified one.

This provides the context to be the root directory so all paths are
relative to that.

Signed-off-by: arewm <arewm@users.noreply.github.com>
arewm pushed a commit to arewm/iib that referenced this pull request Sep 16, 2022
In release-engineering#421, I modified the
context and paths to both go up a directory. That was an error. I should
have just modified one.

This provides the context to be the root directory so all paths are
relative to that.

Signed-off-by: arewm <arewm@users.noreply.github.com>
arewm pushed a commit to arewm/iib that referenced this pull request Sep 16, 2022
In release-engineering#421, I modified the
context and paths to both go up a directory. That was an error. I should
have just modified one.

This provides the context to be the root directory so all paths are
relative to that.

Signed-off-by: arewm <arewm@users.noreply.github.com>
arewm added a commit to arewm/iib that referenced this pull request Sep 16, 2022
In release-engineering#421, I modified the
context and paths to both go up a directory. That was an error. I should
have just modified one.

This provides the context to be the root directory so all paths are
relative to that.

Signed-off-by: arewm <arewm@users.noreply.github.com>
arewm added a commit to arewm/iib that referenced this pull request Sep 16, 2022
In release-engineering#421, I modified the
context and paths to both go up a directory. That was an error. I should
have just modified one.

This provides the context to be the root directory so all paths are
relative to that.

Signed-off-by: arewm <arewm@users.noreply.github.com>
arewm added a commit to arewm/iib that referenced this pull request Sep 16, 2022
In release-engineering#421, I modified the
context and paths to both go up a directory. That was an error. I should
have just modified one.

This provides the context to be the root directory so all paths are
relative to that.

Signed-off-by: arewm <arewm@users.noreply.github.com>
arewm added a commit to arewm/iib that referenced this pull request Sep 27, 2022
In release-engineering#421, I modified the
context and paths to both go up a directory. That was an error. I should
have just modified one.

This provides the context to be the root directory so all paths are
relative to that.

Signed-off-by: arewm <arewm@users.noreply.github.com>
arewm added a commit to arewm/iib that referenced this pull request Sep 27, 2022
In release-engineering#421, I modified the
context and paths to both go up a directory. That was an error. I should
have just modified one.

This provides the context to be the root directory so all paths are
relative to that.

Signed-off-by: arewm <arewm@users.noreply.github.com>
yashvardhannanavati pushed a commit that referenced this pull request Sep 27, 2022
In #421, I modified the
context and paths to both go up a directory. That was an error. I should
have just modified one.

This provides the context to be the root directory so all paths are
relative to that.

Signed-off-by: arewm <arewm@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants