You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@devlake.apache.org by "BrookeKatalon (via GitHub)" <gi...@apache.org> on 2023/05/22 18:27:51 UTC
[GitHub] [incubator-devlake-helm-chart] BrookeKatalon opened a new issue, #134: Helm install on AWS EKS fails due multiple containers being unable to mount /etc/localtime
BrookeKatalon opened a new issue, #134:
URL: https://github.com/apache/incubator-devlake-helm-chart/issues/134
Errors:
`MountVolume.SetUp failed for volume "devlake-mysql-localtime" : hostPath type check failed: /etc/localtime is not a file`
Fix:
Remove the mappings for /etc/localtime. The containers in the pod will inherit time from the cluster nodes.
affects: deployments.yaml & statefulsets.yaml.
--
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.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake-helm-chart] JorgeGar commented on issue #134: Helm install on AWS EKS fails due multiple containers being unable to mount /etc/localtime
Posted by "JorgeGar (via GitHub)" <gi...@apache.org>.
JorgeGar commented on issue #134:
URL: https://github.com/apache/incubator-devlake-helm-chart/issues/134#issuecomment-1558588520
I also had issues with the localtime being a hostPath but on the permission side. Kubernetes recommend avoiding the hostPath unless they are completely required: https://kubernetes.io/docs/concepts/storage/volumes/#hostpath.
@BrookeKatalon could you create a PR with the fix?
--
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.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake-helm-chart] JorgeGar commented on issue #134: Helm install on AWS EKS fails due multiple containers being unable to mount /etc/localtime
Posted by "JorgeGar (via GitHub)" <gi...@apache.org>.
JorgeGar commented on issue #134:
URL: https://github.com/apache/incubator-devlake-helm-chart/issues/134#issuecomment-1559418038
Are you trying to push from the very same repo?
If so, that's the issue.
The process should be to fork the repo, and then from the forked one, you create the branch, work on it and push it to your branch. Later you can create the PR from the forked repo branch to the main repo. https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork.
Having said that... If you're already doing, then I don't know what's going on 😄
--
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.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake-helm-chart] klesh commented on issue #134: Helm install on AWS EKS fails due multiple containers being unable to mount /etc/localtime
Posted by "klesh (via GitHub)" <gi...@apache.org>.
klesh commented on issue #134:
URL: https://github.com/apache/incubator-devlake-helm-chart/issues/134#issuecomment-1564142991
@IronCore864 @JorgeGar
The `localtime volume` makes sense for `docker-compose`, I believe the `helm chart localtime volume` was derived from it.
So, yes, we could remove `localtime volume` if it is not the best practice for `helm chart` which I'm not an expert by any means, I would concur with whatever you think is the best.😊
--
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.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake-helm-chart] IronCore864 commented on issue #134: Helm install on AWS EKS fails due multiple containers being unable to mount /etc/localtime
Posted by "IronCore864 (via GitHub)" <gi...@apache.org>.
IronCore864 commented on issue #134:
URL: https://github.com/apache/incubator-devlake-helm-chart/issues/134#issuecomment-1562874279
Is it possible to remove the localtime volume mount?
If not, is it possible to create a configmap containing the value and mount the configmap instead of a hostpath?
--
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.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake-helm-chart] JorgeGar commented on issue #134: Helm install on AWS EKS fails due multiple containers being unable to mount /etc/localtime
Posted by "JorgeGar (via GitHub)" <gi...@apache.org>.
JorgeGar commented on issue #134:
URL: https://github.com/apache/incubator-devlake-helm-chart/issues/134#issuecomment-1562931664
If we could pass the localtime as an env variable or configmap as @IronCore864 mentioned, that would be the best option.
--
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.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake-helm-chart] klesh commented on issue #134: Helm install on AWS EKS fails due multiple containers being unable to mount /etc/localtime
Posted by "klesh (via GitHub)" <gi...@apache.org>.
klesh commented on issue #134:
URL: https://github.com/apache/incubator-devlake-helm-chart/issues/134#issuecomment-1558573004
Calling experts @JorgeGar @IronCore864
--
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.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake-helm-chart] BrookeKatalon commented on issue #134: Helm install on AWS EKS fails due multiple containers being unable to mount /etc/localtime
Posted by "BrookeKatalon (via GitHub)" <gi...@apache.org>.
BrookeKatalon commented on issue #134:
URL: https://github.com/apache/incubator-devlake-helm-chart/issues/134#issuecomment-1559402092
I tried to push my local issues/134 with the workaround/fix but I get an error:
[issues/134 44b9c30] comment out volume mappings for TZ symlinks so AWS EKS doesnt fail
3 files changed, 75 insertions(+), 76 deletions(-)
❯ git push
ERROR: Permission to apache/incubator-devlake-helm-chart.git denied to BrookeKatalon.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
--
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.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-devlake-helm-chart] BrookeKatalon commented on issue #134: Helm install on AWS EKS fails due multiple containers being unable to mount /etc/localtime
Posted by "BrookeKatalon (via GitHub)" <gi...@apache.org>.
BrookeKatalon commented on issue #134:
URL: https://github.com/apache/incubator-devlake-helm-chart/issues/134#issuecomment-1559683331
D'oh! You can tell I work on internal repos only. Forked and PR created
--
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.
To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org