Pack detect #2132
Pack detect #2132
Conversation
github-actions
bot
added
the
type/enhancement
|
@natalieparellano @jjbustamante can you have a look |
rashadism
force-pushed
the
Pack-detect
branch
5 times, most recently
from
3fe2e35 to
841279f
Compare
natalieparellano
left a comment
natalieparellano
left a comment
There was a problem hiding this comment.
@rashadism thank you for taking the lead on this one. The implementation looks like it's progressing in the right direction. I left a few comments. I think we will want to add more test coverage (and at least one acceptance test although we can help you with that as the acceptance suite is notoriously tricky) but in general this looks good.
internal/commands/detect.go
Outdated
| Args: cobra.ExactArgs(0), | ||
| Short: "Run the detect phase of buildpacks against your source code", | ||
| Example: "pack detect --path apps/test-app --builder cnbs/sample-builder:bionic", | ||
| Long: "Pack Detect uses Cloud Native Buildpacks to run the detect phase of buildpack groups against the source code.\n" + |
There was a problem hiding this comment.
Maybe this part could describe how/where to expect the output of running pack detect.
There was a problem hiding this comment.
where the output is shown wasn't touched on specifically in the RFC. We can either output to stdout or write it to disk. What do you think?
There was a problem hiding this comment.
That's a good question - I think since we'd want both the log output from each buildpack's ./bin/detect as well as the resulting group.toml and plan.toml, writing to disk makes the most sense. pack build has --report-output-dir and --sbom-output-dir configurable options, maybe we make a --detect-output-dir?
There was a problem hiding this comment.
Yes, I included the --detect-output-dir in the corresponding RFC. So that will be the way to go
rashadism
force-pushed
the
Pack-detect
branch
4 times, most recently
from
2d0b90b to
d2ccb49
Compare
| mountPaths mountPaths | ||
| opts LifecycleOptions | ||
| tmpDir string | ||
| // DetectOnly bool |
There was a problem hiding this comment.
I don't think so. There are some conflicts as well. WIll tidy it up and and go through everything again and push
Signed-off-by: Rashad Sirajudeen <rashadsirajudeen@gmail.com>
Signed-off-by: Rashad Sirajudeen <rashadsirajudeen@gmail.com>
Signed-off-by: Rashad Sirajudeen <rashadsirajudeen@gmail.com>
Signed-off-by: Rashad Sirajudeen <rashadsirajudeen@gmail.com>
|
Hi @rashadism I am so sorry we have not moved forward with this implementation. Are you willing to try to get it into the last mile? |
|
Hi @jjbustamante |
3 participants
Summary
Implementation for
pack detectcommand. Up for feedbackOutput
Before
After
Documentation
Runs only the analyze and detect phases and gives the selected group as output.
Related
Corresponding RFC
Resolves #___