Skip to content
This repository was archived by the owner on May 4, 2018. It is now read-only.

Created plugin for getting artifacts from local dir - #159

Open
vbusch wants to merge 2 commits into
jboss-dockerfiles:developfrom
vbusch:master
Open

Created plugin for getting artifacts from local dir#159
vbusch wants to merge 2 commits into
jboss-dockerfiles:developfrom
vbusch:master

Conversation

@vbusch

@vbusch vbusch commented Jun 26, 2017

Copy link
Copy Markdown

Enable plugin with --artifact-local-enable
Set the director with --artifact-local-dir (default is artifacts)

Fixes: 114

Enable plugin with --artifact-local-enable
Set the director with --artifact-local-dir (default is artifacts)

Fixes: 114
Comment thread dogen/plugins/artifact.py Outdated

for f in sorted(self.artifact_files):
self.log.debug("Copying local artifact %s" % os.path.basename(f))
shutil.copy2(f, self.output)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how this handle target?
https://github.com/jboss-dockerfiles/dogen/blob/develop/dogen/generator.py#L289
artifacts in dogen can be copied anywhere into the subtree of output directory - this functionality is crucial to CCT plugin. This probably needs to be updated. @goldmann opinion?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The target name could be used in the artifacts directory. The sources would then find it and pick it up.
I'll need to update to handle subdirectories. If the artifacts directory contains the subdirectories and target names, they can be copied over.

@rwngwn rwngwn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also I'm thinking this plugin should not copy everything from local cached directory. It should probably copy only artifacts requested by sources section of image.yaml and place them to proper place specified by target attribute.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants