Friday, 2018-08-17

openstackgerritFelipe Monteiro proposed openstack/airship-deckhand master: Implement Barbican cache for quick secret payload/ref data  https://review.openstack.org/58511800:29
*** aagate has quit IRC00:51
openstackgerritFelipe Monteiro proposed openstack/airship-deckhand master: Implement Barbican cache for quick secret payload/ref data  https://review.openstack.org/58511801:15
openstackgerritFelipe Monteiro proposed openstack/airship-deckhand master: Clear rendered documents cache when deleting all revisions  https://review.openstack.org/59269701:55
openstackgerritFelipe Monteiro proposed openstack/airship-deckhand master: Clear rendered documents cache when deleting all revisions  https://review.openstack.org/59269702:01
openstackgerritFelipe Monteiro proposed openstack/airship-deckhand master: Clear rendered documents cache when deleting all revisions  https://review.openstack.org/59269702:04
*** roman_g has quit IRC03:06
openstackgerritAhmad Mahmoudi proposed openstack/airship-shipyard master: Block site_update if there is no host in MaaS/Dryd  https://review.openstack.org/58366506:01
*** hemanth_n has joined #airshipit06:48
*** sekharvajjula has joined #airshipit07:05
sekharvajjulaHi 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
sekharvajjulaI 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 #airshipit09:10
*** aaronsheffield has joined #airshipit12:18
openstackgerritSangeet Gupta proposed openstack/airship-promenade master: [WIP] Authentication via keystone webhook  https://review.openstack.org/59215712:21
*** hemanth_n has quit IRC12:42
*** vrv_ has joined #airshipit13:20
*** hemanth_n has joined #airshipit13:24
seaneagancfriesen: here's some background on `recreate_pods` and `force` flags:13:30
seaneaganhttps://gist.github.com/seaneagan/74b0cac6622d09206a3d94c9c07bdb4a13:30
seaneaganyes, 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 #airshipit13:51
openstackgerritFelipe Monteiro proposed openstack/airship-pegleg master: Add git and branch revision support to Pegleg CLI  https://review.openstack.org/57788614:11
*** felipemonteiro has quit IRC14:47
*** felipemonteiro has joined #airshipit14:50
openstackgerritAhmad Mahmoudi proposed openstack/airship-shipyard master: Block site_update if there is no host in MaaS/Dryd  https://review.openstack.org/58366514:51
*** felipemonteiro has quit IRC15:01
*** sgrasley has joined #airshipit15:32
cfriesenseaneagan: 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
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests  https://review.openstack.org/59267915:42
*** vrv_ has quit IRC15:46
openstackgerritSean Eagan proposed openstack/airship-armada master: Move to semantic diffing of charts  https://review.openstack.org/59275415:59
*** hemanth_n has quit IRC16:26
cfriesenis anyone aware of an issue with armada (in a container) not detecting that helm has actually deployed a release?16:27
cfriesenmy armada is sitting at " armada.handlers.tiller [-] Helm install release: wait=True, timeout=1800" but "helm list" shows that the release is DEPLOYED16:28
cfriesenif 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
cfriesenI'm seeing this intermittently when running the official armada container on a CentOS host.16:37
seaneagancfriesen: 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 IRC16:48
cfriesenseaneagan: no, I was trying to run "armada apply" on a fresh system with no helm releases16:51
cfriesenseaneagan: the helm release described in the manifest was deployed properly, but armada didn't detect that it was deployed and was stuck waiting for it16:52
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests  https://review.openstack.org/59267916:53
cfriesenseaneagan: http://paste.openstack.org/show/728323/16:53
seaneaganah, you can avoid the wait by using unsequenced chart groups16:54
seaneaganand additionally not configuring any wait in the manifests16:55
seaneaganotherwise it will use helm's --wait on the install or upgrade, and will wait for any pod labels you specify16:57
seaneaganhttps://airship-armada.readthedocs.io/en/latest/operations/guide-build-armada-yaml.html#armada-chartgroup-v117:00
cfriesenseaneagan: 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
cfriesendo I need to change the wait section to specify "release_group: osh-system-ingress-controller"?17:04
seaneaganyou'd need to remove the wait section altogether, if it sees that it will wait17:07
cfriesenI want to wait for it to be done though17:08
cfriesenfor other charts it does wait successfully, but for this one it didn't seem to detect that it was deployed.17:08
seaneaganah, deployed just means that it has sent all the resources to kubernetes, not that they are fully up and running/ready17:10
seaneaganhelm's --wait waits for stuff to actually be ready17:11
cfriesenjust retried.  helm shows DEPLOYED, kubectl shows two ingress pods running, armada is still waiting17:13
seaneaganhere's the actual logic used:17:13
seaneaganhttps://github.com/helm/helm/blob/master/pkg/kube/wait.go#L4617:13
seaneaganyou could try deploying the chart using `helm install/upgrade --wait` and see if you have the same issue as with armada17:14
cfriesenOkay, 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
cfriesenunder data.wait.labels in the manifest17:16
seaneaganyeah, so that's the other waiting that armada does, on those labels17:17
openstackgerritAhmad Mahmoudi proposed openstack/airship-shipyard master: Block site_update if there is no host in MaaS/Drydock  https://review.openstack.org/58366517:21
cfriesenseaneagan: thanks for the help, sorry for the noise. :)17:21
*** aagate has joined #airshipit17:31
openstackgerritAhmad Mahmoudi proposed openstack/airship-shipyard master: Block site_update if there is no host in MaaS/Drydock  https://review.openstack.org/58366517:31
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests  https://review.openstack.org/59267917:50
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests  https://review.openstack.org/59267918:15
*** jamesgu__ has quit IRC19:25
*** jamesgu__ has joined #airshipit19:26
openstackgerritFelipe Monteiro proposed openstack/airship-deckhand master: caching: Add test to validate shared caching across threads  https://review.openstack.org/59315920:08
openstackgerritBryan Strassner proposed openstack/airship-shipyard master: Remediate failing Bandit gate  https://review.openstack.org/59316620:13
openstackgerritAnthony Bellino proposed openstack/airship-armada master: Validation refactor  https://review.openstack.org/59140320:33
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests  https://review.openstack.org/59267920:58
openstackgerritFelipe Monteiro proposed openstack/airship-deckhand master: docs: Elaborate on document layering in documentation  https://review.openstack.org/57855420:59
*** aaronsheffield has quit IRC21:08
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests  https://review.openstack.org/59267921:44
openstackgerritFelipe Monteiro proposed openstack/airship-deckhand master: refactor: Clean up jsonpath_replace method  https://review.openstack.org/59320922:26
openstackgerritFelipe Monteiro proposed openstack/airship-deckhand master: refactor: Clean up jsonpath_replace method  https://review.openstack.org/59320922:31
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Initial set of Airship deployment manifests  https://review.openstack.org/59206222:38
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Fixes for airship-seaworthy manifests  https://review.openstack.org/59267922:38
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Add license - Apache License Version 2.0  https://review.openstack.org/59219922:49
openstackgerritFelipe Monteiro proposed openstack/airship-deckhand master: refactor: Clean up jsonpath_replace method  https://review.openstack.org/59320923:30
openstackgerritMerged openstack/airship-treasuremap master: Add license - Apache License Version 2.0  https://review.openstack.org/59219923:32

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!