Wednesday, 2018-08-15

*** sgrasley has quit IRC00:37
*** openstack has joined #airshipit02:33
*** ChanServ sets mode: +o openstack02:33
*** mardim has quit IRC03:07
*** roman_g_ has quit IRC03:13
*** roman_g_ has joined #airshipit04:24
openstackgerritFelipe Monteiro proposed openstack/airship-pegleg master: fixtures: Introduce fixture for auto-deleting Git repos  https://review.openstack.org/59049605:18
openstackgerritMerged openstack/airship-shipyard master: Update Shipyard sampleconf.rst with correct name  https://review.openstack.org/59178205:26
openstackgerritMerged openstack/airship-drydock master: Update Drydock sampleconf.rst with correct name  https://review.openstack.org/59180905:28
*** ChrisPriceAB has quit IRC09:18
*** jamesgu has quit IRC10:02
openstackgerritBryan Strassner proposed openstack/airship-shipyard master: Add redeploy_server processing  https://review.openstack.org/58379313:45
*** aaronsheffield has joined #airshipit13:55
cfriesenwhen running armada in a container and using the CLI, I assume you need to use the -v or --mount options to expose the manifest files and any local helm charts within the container?14:16
roman_g_cfriesen: docker run -d --net host -p 8000:8000 --name armada -v $(pwd)/etc/:/etc/ -v ~/.kube/:/armada/.kube/ -v $(pwd)/examples/:/examples quay.io/attcomdev/armada:latest14:18
roman_g_docker exec armada armada --help14:19
roman_g_and you got everything mounted already14:19
cfriesenroman_g_: yeah, that's what I figured.  this seems a bit cumbersome....does Airship actually use this or does it use the HTTP API primarily?14:22
roman_g_HTTP API primarily14:24
roman_g_but e.g. in airship-in-a-bottle you can see cli output14:25
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Initial set of Airship deployment manifests  https://review.openstack.org/59206214:30
cfriesenthanks.  Separate question, do you know what the criteria are for a load getting pushed to quay.io/attcomdev/armada, and will loads continue to be pushed there with airship under OpenStack?14:32
b-strquay.io/airshipit/armada   is the active quay repo to use.14:35
cfriesenb-str: http://airship-armada.readthedocs.io/en/latest/readme.html needs to be updated then. :)14:36
b-strperfect, thanks.14:38
cfriesenokay, looks like each git commit results in a new docker image being uploaded?14:39
b-stryes, that's where we're at.14:40
openstackgerritAaron Sheffield proposed openstack/airship-armada master: Updating Quay Namespace to airshipit  https://review.openstack.org/59206614:43
aaronsheffieldGood catch cfriesen, the above PS fixes the readthedocs.14:44
cfriesenb-str: aaronsheffield: looks like "attcomdev/armada" is also referenced a few other times in the armada git repo14:45
aaronsheffieldThat PS gets those also.14:45
cfriesentotally separate question...does armada do anything to prevent races?  If I run "armada apply" to update an already-applied manifest at the same time as someone runs "armada delete --manifest", what happens?14:51
openstackgerritFelipe Monteiro proposed openstack/airship-pegleg master: fixtures: Introduce fixture for auto-deleting Git repos  https://review.openstack.org/59049615:23
*** jamesgu has joined #airshipit15:38
openstackgerritMerged openstack/airship-shipyard master: Set ULID of action on DAG request  https://review.openstack.org/58931416:35
openstackgerritBryan Strassner proposed openstack/airship-shipyard master: Add redeploy_server processing  https://review.openstack.org/58379316:46
openstackgerritMerged openstack/airship-in-a-bottle master: Add docs build and publish  https://review.openstack.org/58170316:58
openstackgerritMerged openstack/airship-in-a-bottle master: (security) Airship Security Guide framework  https://review.openstack.org/58797916:58
openstackgerritMerged openstack/airship-armada master: Updating Quay Namespace to airshipit  https://review.openstack.org/59206617:03
*** aagate has quit IRC17:11
*** roman_g_ has quit IRC17:13
*** roman_g has joined #airshipit17:13
cfriesenare there docs on the armada HTTP API?17:23
openstackgerritSangeet Gupta proposed openstack/airship-promenade master: [WIP] Authentication via keystone webhook  https://review.openstack.org/59215717:28
openstackgerritSangeet Gupta proposed openstack/airship-promenade master: [WIP] Authentication via keystone webhook  https://review.openstack.org/59215717:35
b-strcfriesen: I sure can't find any - https://git.airshipit.org/cgit/airship-armada/tree/armada/api/server.py#n61 is the endpoints, but that's less than satisfying, I'm sure.  The APIs should line up with the CLI commands, but yeah, docs are missing.17:38
cfriesenb-str: thanks.  Am I correct that the HTTP API has no equivalent for "armada delete --manifest" ?17:39
b-strThis appears to be the case.17:44
*** aagate has joined #airshipit18:23
cfriesenanother question (sorry...) Is the armada HTTP server capable of handling more than one request at once, or does it run to completion then handle the next one?  (I'm wondering if this could lead to long delays for operations on one manifest if another manifest is taking a long time to apply.)18:30
openstackgerritSean Eagan proposed openstack/airship-armada master: Deterministically serialize chart objects for diffing  https://review.openstack.org/59110418:32
cfriesennever mind, found the answer in entrypoint.sh in the parameters to the uwsgi call.18:37
cfriesenI notice the armada HTTP API has no equivalent of "armada delete".  Does Airship not use this functionality?18:49
openstackgerritBryan Strassner proposed openstack/airship-in-a-bottle master: Cleanup Airship verbiage in docs  https://review.openstack.org/59217818:52
seaneagancfriesen: we do have swagger docs for the armada HTTP api19:13
seaneaganhttps://github.com/openstack/airship-armada/tree/master/swagger19:13
seaneagannot hosted anywhere yet19:14
seaneaganyou can use swagger hub to look at them though: https://app.swaggerhub.com/19:15
seaneagancorrect, the API has no delete functionality. ideally we should have parity between API and CLI.19:18
*** openstackgerrit has quit IRC19:19
*** openstackgerrit has joined #airshipit19:49
openstackgerritMerged openstack/airship-promenade master: Opening apiserver Via Ingress  https://review.openstack.org/58553119:49
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Pegleg wrapper  https://review.openstack.org/59218920:03
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Pegleg wrapper  https://review.openstack.org/59218920:04
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Initial draft for airship-seaworthy pipeline  https://review.openstack.org/59219020:05
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Initial draft for airship-seaworthy pipeline  https://review.openstack.org/59219020:09
openstackgerritMerged openstack/airship-pegleg master: fixtures: Introduce fixture for auto-deleting Git repos  https://review.openstack.org/59049620:47
seaneagancfriesen: regarding race conditions for armada, armada doesn't have any locking built in.  I am not sure whether helm itself provides any locking at the individual release level.20:50
*** aaronsheffield has quit IRC20:53
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Add license - Apache License Version 2.0  https://review.openstack.org/59219921:05
openstackgerritBryan Strassner proposed openstack/airship-in-a-bottle master: Cleanup Airship verbiage in docs  https://review.openstack.org/59217821:08
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Initial draft for airship-seaworthy pipeline  https://review.openstack.org/59219021:34
openstackgerritSangeet Gupta proposed openstack/airship-promenade master: [WIP] Authentication via keystone webhook  https://review.openstack.org/59215721:50
openstackgerritSangeet Gupta proposed openstack/airship-promenade master: [WIP] Authentication via keystone webhook  https://review.openstack.org/59215721:57
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: Initial set of Airship deployment manifests  https://review.openstack.org/59206222:19
openstackgerritAnthony Bellino proposed openstack/airship-armada master: Validation refactor  https://review.openstack.org/59140322:26
*** roman_g has quit IRC22:32
openstackgerritKaspars Skels proposed openstack/airship-treasuremap master: [wip] Initial draft for airship-seaworthy pipeline  https://review.openstack.org/59219022:58

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