| openstackgerrit | Alexander Noskov proposed airship/treasuremap master: Refactor seaworthy and airsloop Jenkins jobs https://review.opendev.org/667692 | 00:05 |
|---|---|---|
| *** openstack has joined #airshipit | 13:11 | |
| *** ChanServ sets mode: +o openstack | 13:11 | |
| *** DougSchv has quit IRC | 13:18 | |
| *** howell has joined #airshipit | 13:21 | |
| *** nishantkr has joined #airshipit | 13:41 | |
| alexanderhughes | Can I get a core to look over https://review.opendev.org/#/c/666319/ please? It sets all files created by Pegleg to 640 permissions, previously they were overly permissive and since nearly everything Pegleg writes to a file is a secret there were concerns that this needed to be locked down | 13:57 |
| *** roman_g has quit IRC | 14:06 | |
| openstackgerrit | Kudaka Poorna Rajesh proposed airship/promenade master: Enable probes config for etcd https://review.opendev.org/659863 | 14:22 |
| *** DougSchv has joined #airshipit | 14:28 | |
| *** DougSchv has quit IRC | 14:36 | |
| *** amalrajgenocidex has joined #airshipit | 14:45 | |
| *** DougSchv has joined #airshipit | 14:55 | |
| *** DougSchv has quit IRC | 15:00 | |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Rename module to reflect its new location https://review.opendev.org/667469 | 15:07 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Initial airshipctl checks https://review.opendev.org/667680 | 15:17 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Initial airshipctl checks https://review.opendev.org/667680 | 15:23 |
| *** portdirect has quit IRC | 15:30 | |
| *** portdirect has joined #airshipit | 15:31 | |
| openstackgerrit | Stas Egorov proposed airship/promenade master: hyperkube image in promenade init https://review.opendev.org/666640 | 15:31 |
| openstackgerrit | Stas Egorov proposed airship/airship-in-a-bottle master: Extract hyperkube before promenade https://review.opendev.org/667170 | 15:32 |
| *** roman_g has joined #airshipit | 15:32 | |
| openstackgerrit | Alexander Hughes proposed airship/pegleg master: Set 640 file permissions at time of creation https://review.opendev.org/666319 | 15:35 |
| openstackgerrit | Roman Gorshunov proposed airship/airshipctl master: [WIP] Add basic gates and docs build https://review.opendev.org/667393 | 15:36 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Initial airshipctl checks https://review.opendev.org/667680 | 15:38 |
| *** atsgen_saket has joined #airshipit | 15:51 | |
| atsgen_saket | hi | 15:52 |
| openstackgerrit | Ian Pittwood proposed airship/spyglass-plugin-xls master: Testing for plugin https://review.opendev.org/667734 | 15:52 |
| *** DougSchv has joined #airshipit | 15:53 | |
| kskels | atsgen_saket: hello | 15:55 |
| *** DougSchv has quit IRC | 15:58 | |
| howell | does anyone have any experience with the ansible 'make' task in a zuul job? | 16:03 |
| howell | I believe I may be running make in the wrong directory: https://review.opendev.org/gitweb?p=airship/airshipctl.git;f=playbooks/airship-airshipctl-lint-test.yaml;hb=refs/changes/80/667680/12#l32 | 16:06 |
| howell | this variable: {{ zuul.project.src_dir | realpath }} should be the root of the repo, right? | 16:06 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Initial airshipctl checks https://review.opendev.org/667680 | 16:10 |
| sthussey | I was having the same issue | 16:10 |
| sthussey | using that pipeline to mount a volume into the golang docker container | 16:10 |
| sthussey | debug output shows it would be the root of the repo | 16:10 |
| howell | Right. I'm going to try to hack together something to at least get some better information on what's happening | 16:11 |
| sthussey | Here is how we use the make module in current Airship projects | 16:11 |
| sthussey | https://opendev.org/airship/drydock/src/branch/master/tools/gate/playbooks/omni_test.yaml | 16:11 |
| sthussey | We omit the 'realpath' filter | 16:11 |
| howell | ok, I'll try removing that | 16:12 |
| howell | weird if that's what's breaking it though... | 16:12 |
| *** DougSchv has joined #airshipit | 16:12 | |
| sthussey | I don't think it should. Below is the debug output from the last run I had of my zuul changes for airshipctl | 16:13 |
| sthussey | https://www.irccloud.com/pastebin/LQa8ixdK/ | 16:13 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Initial airshipctl checks https://review.opendev.org/667680 | 16:13 |
| sthussey | Which to me it looks like the pipeline expands correctly | 16:13 |
| openstackgerrit | Roman Gorshunov proposed airship/airshipctl master: [WIP] Add basic gates and docs build https://review.opendev.org/667393 | 16:17 |
| *** DougSchv has quit IRC | 16:17 | |
| *** pgaxatte has quit IRC | 16:25 | |
| openstackgerrit | Alexander Hughes proposed airship/pegleg master: Set 640 file permissions at time of creation https://review.opendev.org/666319 | 16:32 |
| howell | Removing the realpath pipeline seems to have fixed it. I have no idea why | 16:34 |
| CobHead | So I'm trying to deploy AirShip in a Bottle again after a 2 week hiatus. It still seems like the PostgreSQL container enters a crashloopbackoff. | 16:36 |
| CobHead | Logs say it still can't find the databse. | 16:36 |
| CobHead | Any updated repo I can refer to instead? Been using this: https://github.com/airshipit/treasuremap | 16:37 |
| openstackgerrit | Roman Gorshunov proposed airship/airshipctl master: [WIP] Add basic gates and docs build https://review.opendev.org/667393 | 16:38 |
| *** atsgen_saket has quit IRC | 16:45 | |
| *** gkadam has quit IRC | 16:51 | |
| kskels | CobHead: yes, we had uplift for postgress -I will check if it's already merged | 16:51 |
| kskels | that should address the postgres issue - you may re-try again and yes, treasuremap is the right reference for it now | 16:51 |
| kskels | https://review.opendev.org/#/c/657667/ | 16:52 |
| kskels | I believe we need this to make PostgresSQL more stable | 16:52 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Rename module to reflect its new location https://review.opendev.org/667469 | 16:52 |
| CobHead | I'll try again and report how it went. | 17:00 |
| CobHead | Thanks :) | 17:00 |
| howell | Could I get some reviews on this: https://review.opendev.org/#/c/667680/ | 17:14 |
| *** DougSchv has joined #airshipit | 17:15 | |
| howell | The goal for this first PS is simplicity | 17:15 |
| *** DougSchv has quit IRC | 17:19 | |
| openstackgerrit | Roman Gorshunov proposed airship/airshipctl master: [WIP] Add basic gates and docs build https://review.opendev.org/667393 | 17:25 |
| CobHead | kskels, I still get the same result @ the PostGreSQL container | 17:41 |
| kskels | yeah - the postrgress HA is not yet merged | 17:43 |
| kskels | I'll follow up see if we can merge it | 17:43 |
| kskels | we also ocasionally get it - but in our testing it happens rare | 17:43 |
| CobHead | Do you have a link for the PR? | 17:43 |
| kskels | (it's mostly due to using NFS in airship in a bottle, normally it's Ceph that is used.. so there is some issue) | 17:43 |
| kskels | https://review.opendev.org/#/c/657667/ | 17:44 |
| kskels | is the PS we need to merge and could improve the situation | 17:44 |
| openstackgerrit | Roman Gorshunov proposed airship/airshipctl master: [WIP] Add basic gates and docs build https://review.opendev.org/667393 | 17:45 |
| CobHead | So the theory is that this happens because of write permissions in NFS? | 17:45 |
| CobHead | and/or read^ | 17:46 |
| *** DougSchv has joined #airshipit | 17:59 | |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Remove travis CI config file https://review.opendev.org/667470 | 18:02 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Remove travis CI config file https://review.opendev.org/667470 | 18:02 |
| *** alexanderhughes4 has joined #airshipit | 18:04 | |
| *** DougSchv has quit IRC | 18:04 | |
| *** alexanderhughes has quit IRC | 18:06 | |
| *** amalrajgenocidex has quit IRC | 18:11 | |
| openstackgerrit | Roman Gorshunov proposed airship/airshipctl master: Add basic gates and docs build https://review.opendev.org/667393 | 18:21 |
| openstackgerrit | Alexander Noskov proposed airship/treasuremap master: WIP Uplift all components https://review.opendev.org/664469 | 18:30 |
| openstackgerrit | Alexander Noskov proposed airship/treasuremap master: Fix Keystone password for Horizon chart https://review.opendev.org/668016 | 18:30 |
| kskels | CobHead: I believe evgenyl has some more details | 18:38 |
| kskels | evgenyl: | 18:38 |
| kskels | @evgenyl | 18:38 |
| kskels | not sure how to tag properly :) | 18:38 |
| evgenyl | CobHead: This happens only with NFS backend, however the problem may be related to some race within postgresql chart, this is the reason why we want to try to pull the refactored version. | 18:39 |
| roman_g | kskels: just name works, no need to add @ | 18:44 |
| kskels | roman_g thank! | 18:46 |
| openstackgerrit | Roman Gorshunov proposed airship/docs master: [WIP] Add basic gate and docs https://review.opendev.org/668020 | 18:47 |
| openstackgerrit | Kaspars Skels proposed airship/treasuremap master: Uplift Promenade https://review.opendev.org/664467 | 18:50 |
| roman_g | sthussey: Scott, back to yesterday's issue with Git module requiring user in /etc/passwd to make temporary commit. I was looking trough code, and I'd like to ask: don't you think that _commit_ is intentional (for the cases when someone made manual changes to site docs and wants to run pegleg over updated set)? | 18:52 |
| sthussey | then why not just use the dirty repo? | 18:52 |
| roman_g | good question. | 18:54 |
| roman_g | because there could be a mess when ref is checked out | 18:54 |
| roman_g | over dirty repo | 18:54 |
| openstackgerrit | Kaspars Skels proposed airship/treasuremap master: Using tools/airship wrapper in airsloop Jenkins jobs https://review.opendev.org/667692 | 18:55 |
| sthussey | If you are checking a ref out, then why would you expect to use uncommited changes? | 18:55 |
| sthussey | I mean, this is just personal opinion. But it doesn't seem to me there is coherent design here. | 18:56 |
| roman_g | yep. we can't guess what actually user wants: ref or ref+uncommitted changes | 18:57 |
| openstackgerrit | chinasubbareddy mallavarapu proposed airship/treasuremap master: [CEPH] uplift charts and enable helm tests https://review.opendev.org/665219 | 19:03 |
| sthussey | How does committing help? | 19:04 |
| sthussey | If you commit changes, then check out a new refspec you lose the committed changes | 19:04 |
| sthussey | Unless the code is rebasing the commit on the refspec | 19:04 |
| openstackgerrit | Scott Hussey proposed airship/promenade master: Make aux etcd more conservative https://review.opendev.org/668026 | 19:08 |
| openstackgerrit | chinasubbareddy mallavarapu proposed airship/treasuremap master: [CEPH] uplift charts and enable helm tests https://review.opendev.org/665219 | 19:11 |
| roman_g | sthussey: no, it is not being rebased | 19:16 |
| roman_g | but at least state is saved in git... | 19:16 |
| roman_g | sthussey: git stash also requires user to be set (until very recent Git version, 1Q 2019) | 19:17 |
| sthussey | Seems no clean way forward | 19:27 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Initial airshipctl checks https://review.opendev.org/667680 | 19:35 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Initial airshipctl checks https://review.opendev.org/667680 | 19:50 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Switch from travis to zuul https://review.opendev.org/667680 | 19:55 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Switch from travis to zuul https://review.opendev.org/667680 | 19:56 |
| openstackgerrit | Alexander Hughes proposed airship/pegleg master: Set 640 file permissions at time of creation https://review.opendev.org/666319 | 19:57 |
| openstackgerrit | Alexander Hughes proposed airship/pegleg master: [WIP] Site and global secret decrypt. https://review.opendev.org/662847 | 19:57 |
| openstackgerrit | Roman Gorshunov proposed airship/pegleg master: Fix: don't require user to be existing in OS https://review.opendev.org/668038 | 19:57 |
| roman_g | sthussey: ^^^ - yer version with git add --all && git reset --hard | 19:58 |
| roman_g | *your | 19:58 |
| *** alexanderhughes4 has quit IRC | 20:04 | |
| roman_g | kskels: another workaround for the issue we have had https://review.opendev.org/#/c/668038/1 | 20:04 |
| *** alexanderhughes has joined #airshipit | 20:07 | |
| *** DougSchv has joined #airshipit | 20:17 | |
| openstackgerrit | Roman Gorshunov proposed airship/docs master: Add basic gate and docs https://review.opendev.org/668020 | 20:19 |
| *** DougSchv has quit IRC | 20:24 | |
| openstackgerrit | Alexander Hughes proposed airship/pegleg master: Set 640 file permissions at time of creation https://review.opendev.org/666319 | 20:35 |
| *** DougSchv has joined #airshipit | 20:38 | |
| openstackgerrit | Kaspars Skels proposed airship/treasuremap master: Minor fixes to Airsloop docs https://review.opendev.org/668042 | 20:42 |
| *** DougSchv has quit IRC | 20:43 | |
| *** alexanderhughes has quit IRC | 21:14 | |
| *** howell has quit IRC | 21:18 | |
| *** DougSchv has joined #airshipit | 21:36 | |
| *** howell has joined #airshipit | 21:41 | |
| *** DougSchv has quit IRC | 21:41 | |
| openstackgerrit | Alexander Hughes proposed airship/pegleg master: Set 640 file permissions at time of creation https://review.opendev.org/666319 | 21:42 |
| openstackgerrit | Alexander Hughes proposed airship/pegleg master: [WIP] Site and global secret decrypt. https://review.opendev.org/662847 | 21:44 |
| *** DougSchv has joined #airshipit | 21:50 | |
| *** DougSchv has quit IRC | 21:55 | |
| openstackgerrit | Alexander Hughes proposed airship/pegleg master: Implement default umask for 640 file permissions https://review.opendev.org/666319 | 22:03 |
| openstackgerrit | Alexander Hughes proposed airship/pegleg master: [WIP] Site and global secret decrypt. https://review.opendev.org/662847 | 22:05 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Switch from travis to zuul https://review.opendev.org/667680 | 22:17 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Switch from travis to zuul https://review.opendev.org/667680 | 22:31 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Switch from travis to zuul https://review.opendev.org/667680 | 22:43 |
| openstackgerrit | Ian Howell proposed airship/airshipctl master: Zuul: Switch from travis to zuul https://review.opendev.org/667680 | 22:45 |
| *** howell has quit IRC | 22:45 | |
| openstackgerrit | Alexander Noskov proposed airship/treasuremap master: Uplift Promenade https://review.opendev.org/664467 | 22:51 |
| openstackgerrit | Alexander Noskov proposed airship/treasuremap master: Align java_opts parameter for elasticsearch https://review.opendev.org/666473 | 22:51 |
| openstackgerrit | Alexander Noskov proposed airship/treasuremap master: Separate fluentbit and fluentd charts https://review.opendev.org/667236 | 22:51 |
| openstackgerrit | Alexander Noskov proposed airship/treasuremap master: Fix Keystone password for Horizon chart https://review.opendev.org/668016 | 22:52 |
| openstackgerrit | Alexander Noskov proposed airship/treasuremap master: Uplift all components https://review.opendev.org/664469 | 22:52 |
| *** aaronsheffield has quit IRC | 23:20 | |
| *** sthussey has quit IRC | 23:40 | |
| *** DougSchv has joined #airshipit | 23:51 | |
| *** DougSchv has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!