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

chore: optionally allow to use a separate collection repo #618

Merged
merged 1 commit into from
Nov 22, 2023

Conversation

mkanoor
Copy link
Contributor

@mkanoor mkanoor commented Nov 20, 2023

When developing new features which involves using newer collection we need access to the collection in the image. When doing a developer build of an image provide a build arg for the name of repo

e.g.

podman build --build-arg DEVEL_COLLECTION_LIBRARY=1 --build-arg DEVEL_COLLECTION_REPO=git+https://github.com/mkanoor/event-driven-ansible.git,pg_listener -t localhost/ansible-rulebook:latest .

hsong-rh
hsong-rh previously approved these changes Nov 20, 2023
Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
When developing new features which involves using newer collection
we need access to the collection in the image. When doing a developer
build of an image provide a build arg for the name of repo

e.g.
```
podman build --build-arg DEVEL_COLLECTION_LIBRARY=1 --build-arg DEVEL_COLLECTION_REPO=git+https://github.com/mkanoor/event-driven-ansible.git,pg_listener -t localhost/ansible-rulebook:latest .
```
@mkanoor mkanoor force-pushed the add_collection_repo branch from c9e9b85 to c9a67fc Compare November 21, 2023 21:53
@mkanoor
Copy link
Contributor Author

mkanoor commented Nov 22, 2023

@Alex-Izquierdo Please review updated it

@mkanoor mkanoor requested a review from ttuffin November 22, 2023 10:35
@Alex-Izquierdo Alex-Izquierdo merged commit 9eac9e1 into ansible:main Nov 22, 2023
8 checks passed
@Alex-Izquierdo
Copy link
Contributor

@ttuffin This should be ported to #575

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.

4 participants