Closed dacleyra closed 4 years ago
@dacleyra - Hi Dan is this bug related to release 0.3.0?
yes, adding tag
@aadeshpa can you fill out your GitHub profile so that I can find your id when assigning defects please? :) Since your ID is aadeshpa typing ashish does not bring up your id. :) . Same for @kvijai82 ... please add your name to your profile.
I've assigned @teddyjtorres to diagnose but may need help from the pipeline team.
for now we added tls-verify=false like we do in other places to get us past this issue. have another issue to deal with documenting cert setup for users who would prefer doing that. the fix for this is part of the larger scan task fix I will be creating a PR for shortly.
@kvijai82 Where would the fix go? In collections? Do we expect new collections release?
@aadeshpa can you fill out your GitHub profile so that I can find your id when assigning defects please? :) Since your ID is aadeshpa typing ashish does not bring up your id. :) . Same for @kvijai82 ... please add your name to your profile.
@stephenkinder : updated the name in the profile. Now I think you would be able to search us with name.
@mtamboli yes I am in the process of pushing the fix to collections right now. we will need a respin of the collections to pick it up.
@kvijai82 @marikaj123 Please let me know when the PR needs review and I can review then I can cut a 0.3.0.-rc.3 release (if that is what is required).
Fix should be in the new collections release. @mtamboli / @dacleyra could you please verify & close this issue when you get a chance? Thanks!
@kvijai82 It seems to be working fine. Do you expect image scanning to happen after deployment?
@mtamboli In this pipeline, the deploy task & scan task run in parallel. The thinking right now is that the scan results would be reviewed by a human as there are some false positive results that can happen.
When run via the eventing framework, this task is called in a pipeline that is driven only when a PR is merged. The results can then be reviewed by the team and when they create a release in the git repo, the deploy pipeline will run.
Closing this issue as @mtamboli validated the task passes now.