jeantil commented on pull request #80:
URL: https://github.com/apache/pulsar-helm-chart/pull/80#issuecomment-726654784


   @sijie @wolfstudy I rebased on master and incremented the version to 2.6.2-2
   
   The lint task is green but I think it actually failed, the following comes 
up as a warning in the action
   >Unexpected input(s) 'command', valid inputs are ['version']
   
   The rest of the CI chain seemed stuck, after digging in a bit I realized it 
depended on the namespace being defined in the default values file (not 
overriden in the `.ci/cluster/` value files  and didn't pass the target 
namespace to helm.
   Because of my patch the deployed containers ended up in the  `default` 
namespace but the verification commands using kubectl  explicitely queried the 
`pulsar` namespace.\
   
   I have updated the ci script to explicitely pass the target namespace to 
helm. let me know if you prefer it another way ...


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to