JeromeJu

Results 156 comments of JeromeJu

/assign Thanks for the discussion. Happy to get looped in for the feature flag case.

Thanks @pritidesai for the discussion at the WG 🙏 I understand the concern of dependencies of multiple flags used on a feature/enhancement. But if we move forward with the way...

Thanks for the listing out the options! Thinking out loud. Some context that might be helpful to taken into consideration when voting: - since the default `enable-api-fields` value will remain...

> Also, we are not introducing just one new feature for `matrix`, we have at least two new features apart from `matrix.strategy`: > Should we introduce an additional flags for...

[WG] Race condition for TEP# here spotted by @ericzzzzzzz thanks!

> I do think it make sense (if any Task fails due to Timeout, to mark the PipelineRun has failed due to Timeout). cc @tektoncd/core-maintainers Yes, this would be similar...

Looking into the codebase, it seems that the invalidation of COPY command needs to be added at: [pkg/commands/copy.go#L152](https://github.com/GoogleContainerTools/kaniko/blob/d088512c76593036d838e0ea7eaaebee16f1864e/pkg/commands/copy.go#L152) where it previously doesn't invalidate the COPY command unless [a cache key...

I had a chance to learn from @aaron-prindle that from previous design discussion and decisions made, there are currently only supports for `COPY --chown` and `ADD --chown` but not `RUN...

I will go ahead and close this issue for now and please redirect to #3018 Please feel free to reopen if necessary.

> I think this issue should not have been closed. > > Not invalidating the cache for the subsequent `RUN` command after a previous `COPY` or `ADD` command resulted in...