openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: Implement Barbican cache for quick secret payload/ref data https://review.openstack.org/585118 | 00:29 |
---|---|---|
*** aagate has quit IRC | 00:51 | |
openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: Implement Barbican cache for quick secret payload/ref data https://review.openstack.org/585118 | 01:15 |
openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: Clear rendered documents cache when deleting all revisions https://review.openstack.org/592697 | 01:55 |
openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: Clear rendered documents cache when deleting all revisions https://review.openstack.org/592697 | 02:01 |
openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: Clear rendered documents cache when deleting all revisions https://review.openstack.org/592697 | 02:04 |
*** roman_g has quit IRC | 03:06 | |
openstackgerrit | Ahmad Mahmoudi proposed openstack/airship-shipyard master: Block site_update if there is no host in MaaS/Dryd https://review.openstack.org/583665 | 06:01 |
*** hemanth_n has joined #airshipit | 06:48 | |
*** sekharvajjula has joined #airshipit | 07:05 | |
sekharvajjula | Hi all, I am Chandra Shekar from Nokia. I am new to Airship, trying to install it on one of our Nokia Hardware. But I am getting "Lint" errors in the very initial phase! | 07:14 |
sekharvajjula | I created a ticket https://github.com/att-comdev/treasuremap/issues/11 on this already. Can someone help me here? | 07:14 |
*** roman_g has joined #airshipit | 09:10 | |
*** aaronsheffield has joined #airshipit | 12:18 | |
openstackgerrit | Sangeet Gupta proposed openstack/airship-promenade master: [WIP] Authentication via keystone webhook https://review.openstack.org/592157 | 12:21 |
*** hemanth_n has quit IRC | 12:42 | |
*** vrv_ has joined #airshipit | 13:20 | |
*** hemanth_n has joined #airshipit | 13:24 | |
seaneagan | cfriesen: here's some background on `recreate_pods` and `force` flags: | 13:30 |
seaneagan | https://gist.github.com/seaneagan/74b0cac6622d09206a3d94c9c07bdb4a | 13:30 |
seaneagan | yes, helm (and armada) will recreate the pods of a controller when you update it's template, but with `recreate_pods` it will do that even if there weren't changes to the controller's template, e.g. to re-run init containers or re-intialize with a new config map. | 13:30 |
*** felipemonteiro has joined #airshipit | 13:51 | |
openstackgerrit | Felipe Monteiro proposed openstack/airship-pegleg master: Add git and branch revision support to Pegleg CLI https://review.openstack.org/577886 | 14:11 |
*** felipemonteiro has quit IRC | 14:47 | |
*** felipemonteiro has joined #airshipit | 14:50 | |
openstackgerrit | Ahmad Mahmoudi proposed openstack/airship-shipyard master: Block site_update if there is no host in MaaS/Dryd https://review.openstack.org/583665 | 14:51 |
*** felipemonteiro has quit IRC | 15:01 | |
*** sgrasley has joined #airshipit | 15:32 | |
cfriesen | seaneagan: maybe I hit a helm bug then...I changed a values file and ran "armada apply" and the helm release shows the change but the underlying k8s pod didn't get updated. | 15:38 |
openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests https://review.openstack.org/592679 | 15:42 |
*** vrv_ has quit IRC | 15:46 | |
openstackgerrit | Sean Eagan proposed openstack/airship-armada master: Move to semantic diffing of charts https://review.openstack.org/592754 | 15:59 |
*** hemanth_n has quit IRC | 16:26 | |
cfriesen | is anyone aware of an issue with armada (in a container) not detecting that helm has actually deployed a release? | 16:27 |
cfriesen | my armada is sitting at " armada.handlers.tiller [-] Helm install release: wait=True, timeout=1800" but "helm list" shows that the release is DEPLOYED | 16:28 |
cfriesen | if I abort the armada command and re-run it, it sees that the release is deployed and moves on to the next chart. | 16:32 |
cfriesen | I'm seeing this intermittently when running the official armada container on a CentOS host. | 16:37 |
seaneagan | cfriesen: so you deployed a release via the helm CLI and then tried to upgrade it via armada? I'm not aware of any issues with that. Are you sure the release names match up? The armada manifest specifies a release prefix that is prepended to all release names. | 16:47 |
*** roman_g has quit IRC | 16:48 | |
cfriesen | seaneagan: no, I was trying to run "armada apply" on a fresh system with no helm releases | 16:51 |
cfriesen | seaneagan: the helm release described in the manifest was deployed properly, but armada didn't detect that it was deployed and was stuck waiting for it | 16:52 |
openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests https://review.openstack.org/592679 | 16:53 |
cfriesen | seaneagan: http://paste.openstack.org/show/728323/ | 16:53 |
seaneagan | ah, you can avoid the wait by using unsequenced chart groups | 16:54 |
seaneagan | and additionally not configuring any wait in the manifests | 16:55 |
seaneagan | otherwise it will use helm's --wait on the install or upgrade, and will wait for any pod labels you specify | 16:57 |
seaneagan | https://airship-armada.readthedocs.io/en/latest/operations/guide-build-armada-yaml.html#armada-chartgroup-v1 | 17:00 |
cfriesen | seaneagan: I hope I just messed up the manifest. :) Here's the relevent section of manifest: http://paste.openstack.org/show/728324/ "helm list" shows a deployed "osh-system-ingress-controller" release. | 17:04 |
cfriesen | do I need to change the wait section to specify "release_group: osh-system-ingress-controller"? | 17:04 |
seaneagan | you'd need to remove the wait section altogether, if it sees that it will wait | 17:07 |
cfriesen | I want to wait for it to be done though | 17:08 |
cfriesen | for other charts it does wait successfully, but for this one it didn't seem to detect that it was deployed. | 17:08 |
seaneagan | ah, deployed just means that it has sent all the resources to kubernetes, not that they are fully up and running/ready | 17:10 |
seaneagan | helm's --wait waits for stuff to actually be ready | 17:11 |
cfriesen | just retried. helm shows DEPLOYED, kubectl shows two ingress pods running, armada is still waiting | 17:13 |
seaneagan | here's the actual logic used: | 17:13 |
seaneagan | https://github.com/helm/helm/blob/master/pkg/kube/wait.go#L46 | 17:13 |
seaneagan | you could try deploying the chart using `helm install/upgrade --wait` and see if you have the same issue as with armada | 17:14 |
cfriesen | Okay, I think I figured it out. I was waiting for "release_group: system-ingress-controller" but it looks like I need to wait for "release_group: osh-system-ingress-controller" | 17:15 |
cfriesen | under data.wait.labels in the manifest | 17:16 |
seaneagan | yeah, so that's the other waiting that armada does, on those labels | 17:17 |
openstackgerrit | Ahmad Mahmoudi proposed openstack/airship-shipyard master: Block site_update if there is no host in MaaS/Drydock https://review.openstack.org/583665 | 17:21 |
cfriesen | seaneagan: thanks for the help, sorry for the noise. :) | 17:21 |
*** aagate has joined #airshipit | 17:31 | |
openstackgerrit | Ahmad Mahmoudi proposed openstack/airship-shipyard master: Block site_update if there is no host in MaaS/Drydock https://review.openstack.org/583665 | 17:31 |
openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests https://review.openstack.org/592679 | 17:50 |
openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests https://review.openstack.org/592679 | 18:15 |
*** jamesgu__ has quit IRC | 19:25 | |
*** jamesgu__ has joined #airshipit | 19:26 | |
openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: caching: Add test to validate shared caching across threads https://review.openstack.org/593159 | 20:08 |
openstackgerrit | Bryan Strassner proposed openstack/airship-shipyard master: Remediate failing Bandit gate https://review.openstack.org/593166 | 20:13 |
openstackgerrit | Anthony Bellino proposed openstack/airship-armada master: Validation refactor https://review.openstack.org/591403 | 20:33 |
openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests https://review.openstack.org/592679 | 20:58 |
openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: docs: Elaborate on document layering in documentation https://review.openstack.org/578554 | 20:59 |
*** aaronsheffield has quit IRC | 21:08 | |
openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests https://review.openstack.org/592679 | 21:44 |
openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: refactor: Clean up jsonpath_replace method https://review.openstack.org/593209 | 22:26 |
openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: refactor: Clean up jsonpath_replace method https://review.openstack.org/593209 | 22:31 |
openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Initial set of Airship deployment manifests https://review.openstack.org/592062 | 22:38 |
openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests https://review.openstack.org/592679 | 22:38 |
openstackgerrit | Kaspars Skels proposed openstack/airship-treasuremap master: Add license - Apache License Version 2.0 https://review.openstack.org/592199 | 22:49 |
openstackgerrit | Felipe Monteiro proposed openstack/airship-deckhand master: refactor: Clean up jsonpath_replace method https://review.openstack.org/593209 | 23:30 |
openstackgerrit | Merged openstack/airship-treasuremap master: Add license - Apache License Version 2.0 https://review.openstack.org/592199 | 23:32 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!