[GitHub] [incubator-heron] simingweng edited a comment on issue #3219: upgrade to Apache Bookkeeper 4.7.3 to fix DistributedLog based stateful storage

[GitBox <gi...@apache.org>]

simingweng edited a comment on issue #3219: upgrade to Apache Bookkeeper 4.7.3 to fix DistributedLog based stateful storage
URL: https://github.com/apache/incubator-heron/pull/3219#issuecomment-501003361
 
 
   I did adapt the relevant Helm templates in this PR. So, theoretically, you should be able to install the chart from locally. But, the tricky part is that there's no post-0.17.8 official release available in public, which means no heron post-0.17.8 docker image in public.
   
   The reason you will need post-0.17.8 Heron docker image is because there're other changes across the code base to migrate to newer Bookkeeper Client dependencies.
   
   So, what we did in our deployment is to build heron RC release, push the docker image built to our own Artifactory repository, also push the generated maven artifacts there. Then, in our k8s cluster (we're on-promise), we configure the cluster to include our own Artifactory docker image repo.
   
   Then we maintain a slightly different version of Helm chart based on the bare metal flavor of the templates available in this code base, because we're on-promise, and we don't need to care about GKE, AWS. So, our in house helm chart is simplified, but fairly close to the one in the source.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services