openstackgerrit | Merged openstack/airship-treasuremap master: global: Add nfs-provisioner chart https://review.openstack.org/635217 | 00:04 |
---|---|---|
*** irclogbot_3 has joined #airshipit | 00:16 | |
*** cheng1 has joined #airshipit | 00:59 | |
*** mbologna has quit IRC | 01:32 | |
*** mbologna has joined #airshipit | 01:32 | |
*** Nishant_ has quit IRC | 02:11 | |
*** cheng1 has quit IRC | 02:40 | |
*** cheng1 has joined #airshipit | 03:05 | |
*** cheng1 has quit IRC | 03:23 | |
*** cheng1 has joined #airshipit | 04:02 | |
*** cheng1 has quit IRC | 04:07 | |
*** kranthikirang has joined #airshipit | 04:16 | |
*** kranthikirang has quit IRC | 04:21 | |
*** portdirect has quit IRC | 04:31 | |
*** serverascode has quit IRC | 04:32 | |
*** evgenyl has quit IRC | 04:32 | |
*** adrianreza has quit IRC | 04:32 | |
*** lamt has quit IRC | 04:32 | |
*** hogepodge has quit IRC | 04:32 | |
*** srwilkers has quit IRC | 04:32 | |
*** jbryce has quit IRC | 04:33 | |
*** mattmceuen has quit IRC | 04:33 | |
*** powerds0111 has quit IRC | 04:33 | |
*** evgenyl has joined #airshipit | 04:34 | |
*** srwilkers has joined #airshipit | 04:34 | |
*** portdirect has joined #airshipit | 04:34 | |
*** adrianreza has joined #airshipit | 04:34 | |
*** cheng1 has joined #airshipit | 04:34 | |
*** hogepodge has joined #airshipit | 04:35 | |
*** mattmceuen has joined #airshipit | 04:36 | |
*** jbryce has joined #airshipit | 04:38 | |
*** lemko has quit IRC | 04:38 | |
*** serverascode has joined #airshipit | 04:39 | |
*** kranthikirang has joined #airshipit | 06:04 | |
*** kranthikirang has quit IRC | 06:09 | |
*** lemko has joined #airshipit | 06:15 | |
*** jamesgu has quit IRC | 06:37 | |
*** dims has quit IRC | 06:39 | |
openstackgerrit | Smruti Soumitra Khuntia proposed openstack/airship-armada master: End user logging for audit traceabilty https://review.openstack.org/638124 | 07:34 |
openstackgerrit | Smruti Soumitra Khuntia proposed openstack/airship-armada master: End user logging for audit traceabilty https://review.openstack.org/638124 | 07:38 |
*** skatsaounis has joined #airshipit | 07:40 | |
*** kranthikirang has joined #airshipit | 07:52 | |
*** kranthikirang has quit IRC | 07:57 | |
openstackgerrit | Smruti Soumitra Khuntia proposed openstack/airship-armada master: End user logging for audit traceabilty https://review.openstack.org/638124 | 07:59 |
*** cheng1 has quit IRC | 09:13 | |
*** cheng1 has joined #airshipit | 09:14 | |
*** kranthikirang has joined #airshipit | 09:41 | |
*** kranthikirang has quit IRC | 09:45 | |
*** roman_g has joined #airshipit | 09:58 | |
*** cheng1 has quit IRC | 10:01 | |
*** cheng1 has joined #airshipit | 10:16 | |
openstackgerrit | Smruti Soumitra Khuntia proposed openstack/airship-drydock master: End user logging for audit traceabilty https://review.openstack.org/638115 | 10:39 |
*** wahnsinn has joined #airshipit | 10:47 | |
*** Nishant_ has joined #airshipit | 10:54 | |
*** adrianreza has quit IRC | 10:59 | |
*** portdirect has quit IRC | 11:00 | |
*** adrianreza has joined #airshipit | 11:00 | |
*** portdirect has joined #airshipit | 11:01 | |
*** cheng1 has quit IRC | 11:20 | |
*** cheng1 has joined #airshipit | 11:21 | |
*** kranthikirang has joined #airshipit | 11:29 | |
*** kranthikirang has quit IRC | 11:33 | |
*** cheng1 has quit IRC | 11:38 | |
*** alanmeadows_ has joined #airshipit | 12:07 | |
*** roman_g has quit IRC | 12:09 | |
*** alanmeadows has quit IRC | 12:14 | |
*** alanmeadows_ is now known as alanmeadows | 12:14 | |
*** cheng1 has joined #airshipit | 12:42 | |
*** altlogbot_2 has quit IRC | 13:01 | |
*** irclogbot_3 has quit IRC | 13:01 | |
*** irclogbot_2 has joined #airshipit | 13:01 | |
*** altlogbot_3 has joined #airshipit | 13:01 | |
openstackgerrit | Lakshmi Prasanna Goutham Pratapa proposed openstack/airship-promenade master: Add Liveness/Readiness probe to calico-etcd. https://review.openstack.org/645561 | 13:15 |
*** kranthikirang has joined #airshipit | 13:17 | |
*** kranthikirang has quit IRC | 13:21 | |
*** cheng1 has quit IRC | 13:22 | |
*** kranthikirang has joined #airshipit | 13:31 | |
*** juhak has quit IRC | 13:35 | |
*** juhak has joined #airshipit | 13:36 | |
*** aaronsheffield has joined #airshipit | 13:48 | |
*** juhak has quit IRC | 14:16 | |
*** juhak has joined #airshipit | 14:16 | |
*** michael-beaver has joined #airshipit | 14:19 | |
*** michaelbeaver has joined #airshipit | 14:30 | |
*** kranthikirang has quit IRC | 14:34 | |
*** michael-beaver has quit IRC | 14:34 | |
openstackgerrit | Evgeniy L proposed openstack/airship-treasuremap master: [WIP][DNM] test patch https://review.openstack.org/645012 | 14:58 |
dwalt | It looks like our docker image builds are failing due to a missing Zuul variable (`Zuul.newrev`). Does anyone know if this has been addressed by the Zuul team or if there's a more reliable path forward? I haven't come across anything yet | 15:03 |
evgenyl | Hi team, I've got some time to continue to look into our AIAB gates, and after fixing a problem with mariadb, it fails on heat-test http://paste.openstack.org/show/748246/ , and the problem is there are no logs available for heat-test pod, does it get deleted? Is there a way to prevent armada/helm/k8s/something else from deleting the pod? | 15:06 |
evgenyl | dwalt: Hmm, have not seen anything related on OpenStack dev ML, do you have a link to the failure handy? | 15:09 |
dwalt | evegnyl: here is one for a PS submitted to Armada http://logs.openstack.org/24/638124/24/check/airship-armada-docker-build-gate/589638b/ | 15:10 |
dwalt | Shipyard has one as well: http://logs.openstack.org/81/645381/1/check/airship-shipyard-image-gate/be43e0f/ | 15:10 |
*** jamesgu has joined #airshipit | 15:13 | |
dwalt | evgenyl: You can prevent Armada from deleting test pods using the `test.options.cleanup.enabled` key http://logs.openstack.org/81/645381/1/check/airship-shipyard-image-gate/be43e0f/ | 15:14 |
evgenyl | dwalt: Thanks, I will try that! | 15:16 |
*** altlogbot_3 has quit IRC | 15:21 | |
openstackgerrit | Evgeniy L proposed openstack/airship-in-a-bottle master: Downgrade ingress controller for MariaDB https://review.openstack.org/644634 | 15:21 |
dwalt | evgenyl: np. Also, I meant for that link to be https://airshipit.readthedocs.io/projects/armada/en/latest/operations/guide-build-armada-yaml.html?highlight=cleanup#test-options | 15:22 |
evgenyl | dwalt: If Armada cleans up all test pods prior to running tests, shouldn't we update all Chart manifests to cleanup = false? | 15:24 |
*** altlogbot_0 has joined #airshipit | 15:24 | |
*** irclogbot_2 has quit IRC | 15:30 | |
evgenyl | dwalt: Looking into Zuul docs, I'm wondering if we should expect newrev to always be present https://github.com/openstack-infra/zuul/blob/master/doc/source/user/jobs.rst | 15:30 |
evgenyl | https://www.irccloud.com/pastebin/lRItovpb/ | 15:31 |
*** irclogbot_1 has joined #airshipit | 15:31 | |
dwalt | evgenyl: aaronsheffield just asked in #openstack-infra | 15:31 |
aaronsheffield | So, zuul upgrade to ansible 2.7, which cares more about missing variables. We don't always have zuul.newrev, so that is causing the failure. | 15:33 |
evgenyl | Oh, interesting. | 15:33 |
dwalt | aaronsheffield: is there a way to obtain the commit sha other than the `Zuul.newrev` variable? | 15:34 |
*** dimitris_ has quit IRC | 15:35 | |
*** irclogbot_1 has quit IRC | 15:36 | |
evgenyl | dwalt: Hmm, it does not seem to be available http://paste.openstack.org/show/748250/ | 15:36 |
evgenyl | dwalt: But maybe I'm missing something. | 15:36 |
*** irclogbot_3 has joined #airshipit | 15:36 | |
evgenyl | dwalt: According to the docs newrev may also be missing. | 15:36 |
dwalt | I suppose we only really need the commit hash when the change merges. So maybe just check `zuul.newrev` exists and omit the sha from the tag if it doesn't? | 15:38 |
aaronsheffield | http://lists.openstack.org/pipermail/openstack-discuss/2019-March/004034.html, the announcement about the ansible change. | 15:40 |
*** pkaralis has quit IRC | 15:41 | |
openstackgerrit | Drew Walters proposed openstack/airship-treasuremap master: airskiff: Pull rather than build Airship images https://review.openstack.org/635231 | 15:44 |
openstackgerrit | Drew Walters proposed openstack/airship-treasuremap master: global: Uplift helm-toolkit https://review.openstack.org/645226 | 15:44 |
openstackgerrit | Drew Walters proposed openstack/airship-treasuremap master: airskiff: Reduce postgresql PVC size https://review.openstack.org/639123 | 15:44 |
openstackgerrit | Drew Walters proposed openstack/airship-treasuremap master: airskiff: Use global Airship charts https://review.openstack.org/635218 | 15:44 |
*** kranthikirang has joined #airshipit | 15:45 | |
openstackgerrit | Aaron Sheffield proposed openstack/airship-pegleg master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645631 | 15:45 |
*** kranthikirang has quit IRC | 15:49 | |
*** nick_kar has quit IRC | 15:53 | |
openstackgerrit | Aaron Sheffield proposed openstack/airship-pegleg master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645631 | 16:17 |
openstackgerrit | Merged openstack/airship-treasuremap master: Add the wait label to delete nova test pod when nova chart is upgraded https://review.openstack.org/645218 | 16:25 |
*** roman_g has joined #airshipit | 16:33 | |
*** wahnsinn has quit IRC | 16:37 | |
*** sthussey has joined #airshipit | 16:38 | |
openstackgerrit | Michael Beaver proposed openstack/airship-armada master: Fix for Pep8 failures https://review.openstack.org/645656 | 16:38 |
openstackgerrit | Aaron Sheffield proposed openstack/airship-pegleg master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645631 | 16:47 |
mattmceuen | Re: the `zuul.newrev` issue - lamt has a PS out for pegleg, with some feedback on it: https://review.openstack.org/#/c/645055/ | 16:47 |
openstackgerrit | Drew Walters proposed openstack/airship-treasuremap master: airskiff: Reduce postgresql PVC size https://review.openstack.org/639123 | 16:59 |
openstackgerrit | Drew Walters proposed openstack/airship-treasuremap master: airskiff: Use global Airship charts https://review.openstack.org/635218 | 16:59 |
openstackgerrit | Matt McEuen proposed openstack/airship-promenade master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645801 | 17:07 |
openstackgerrit | Matt McEuen proposed openstack/airship-promenade master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645801 | 17:07 |
aaronsheffield | I went with dwalt's idea (similar to lamt) and just made it conditional. If it isn't there just use an empty string. Since we aren't using newrev right now until merge, this should work. https://review.openstack.org/#/c/645631/ | 17:08 |
dwalt | aaronsheffield: taking a look now. Thanks for doing that | 17:08 |
mattmceuen | Just imitated your change for promenade as well aaronsheffield https://review.openstack.org/645801 | 17:08 |
dwalt | Thanks mattmceuen | 17:08 |
dwalt | mattmceuen: just waiting to verify that Zuul re-populates that variable when aaronsheffield’s patch merges | 17:11 |
mattmceuen | +1 | 17:12 |
seaneagan | looks like i was a little late, but added a comment to: https://review.openstack.org/#/c/645631 | 17:13 |
dwalt | seaneagan: Zuul supposedly restores the `zuul.newref` variable upon merge, so we think the commit SHA should still be included in the tag when we publish | 17:15 |
dwalt | waiting to verify that now | 17:15 |
openstackgerrit | Scott Hussey proposed openstack/airship-promenade master: (apiserver) [WIP] support key rotation https://review.openstack.org/631935 | 17:16 |
dwalt | Seems like it worked https://www.irccloud.com/pastebin/Pmh2jB0M/ | 17:22 |
openstackgerrit | Michael Beaver proposed openstack/airship-armada master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645810 | 17:22 |
michaelbeaver | Awesome, just threw one up for Armada as well | 17:23 |
*** michaelbeaver has quit IRC | 17:23 | |
*** michael-beaver has joined #airshipit | 17:23 | |
openstackgerrit | Merged openstack/airship-pegleg master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645631 | 17:23 |
*** lamt has joined #airshipit | 17:29 | |
*** roman_g has quit IRC | 17:30 | |
openstackgerrit | Aaron Sheffield proposed openstack/airship-pegleg master: Fix typo in secrets linting https://review.openstack.org/644958 | 17:38 |
*** dustinspecker has joined #airshipit | 17:45 | |
openstackgerrit | Aaron Sheffield proposed openstack/airship-drydock master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645821 | 17:53 |
openstackgerrit | Aaron Sheffield proposed openstack/airship-shipyard master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645822 | 17:54 |
openstackgerrit | Aaron Sheffield proposed openstack/airship-maas master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645824 | 18:01 |
openstackgerrit | Aaron Sheffield proposed openstack/airship-deckhand master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645825 | 18:01 |
openstackgerrit | Evgeniy L proposed openstack/airship-in-a-bottle master: Fix AIAB gate Heat test & MariaDB failures https://review.openstack.org/644634 | 18:22 |
openstackgerrit | Merged openstack/airship-pegleg master: Fix typo in secrets linting https://review.openstack.org/644958 | 18:26 |
openstackgerrit | Merged openstack/airship-promenade master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645801 | 18:27 |
openstackgerrit | Sandeep Reddy Thumma proposed openstack/airship-promenade master: Set Least Previliage for .PEM files under directory /etc/genesis. https://review.openstack.org/640775 | 18:30 |
openstackgerrit | Drew Walters proposed openstack/airship-shipyard master: images: Update default image tag to `latest` https://review.openstack.org/645381 | 18:42 |
openstackgerrit | Michael Beaver proposed openstack/airship-armada master: Fix gate failures for Docker and Pep8 https://review.openstack.org/645656 | 18:46 |
openstackgerrit | Anthony Bellino proposed openstack/airship-divingbell master: [WIP] Initial Ansible Daemonset https://review.openstack.org/640539 | 18:48 |
openstackgerrit | Merged openstack/airship-promenade master: Set Least Previliage for .PEM files under directory /etc/genesis. https://review.openstack.org/640775 | 18:51 |
openstackgerrit | Michael Beaver proposed openstack/airship-armada master: Support in Armada for locking Tiller https://review.openstack.org/632483 | 18:57 |
openstackgerrit | Michael Beaver proposed openstack/airship-armada master: Move Tiller into Armada pod https://review.openstack.org/632788 | 18:57 |
openstackgerrit | Merged openstack/airship-maas master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645824 | 19:00 |
openstackgerrit | Merged openstack/airship-shipyard master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645822 | 19:06 |
openstackgerrit | Aaron Sheffield proposed openstack/airship-deckhand master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645825 | 19:06 |
openstackgerrit | Merged openstack/airship-drydock master: Updating Docker Gate use of zuul.newrev https://review.openstack.org/645821 | 19:08 |
openstackgerrit | Merged openstack/airship-armada master: Fix gate failures for Docker and Pep8 https://review.openstack.org/645656 | 19:09 |
openstackgerrit | Dan Crank proposed openstack/airship-deckhand master: Log client-id in UCP API endpoints https://review.openstack.org/634068 | 19:27 |
openstackgerrit | PRATEEK REDDY DODDA proposed openstack/airship-armada master: [WIP] Implement Security Context for Armada https://review.openstack.org/639207 | 19:29 |
openstackgerrit | Merged openstack/airship-treasuremap master: airskiff: Pull rather than build Airship images https://review.openstack.org/635231 | 19:32 |
openstackgerrit | PRATEEK REDDY DODDA proposed openstack/airship-deckhand master: [WIP] Implement Security Context for Deckhand https://review.openstack.org/639204 | 19:45 |
*** ukk1985 has joined #airshipit | 19:46 | |
ukk1985 | Hello, I am trying to deploy Airship in bottle, but it is failing while creating "maas-import-resource" pod and it entered to CrashLoopBackOff. | 19:47 |
ukk1985 | pod log says "Import of boot resources startedImport failed!" Image import FAILED! | 19:48 |
ukk1985 | Can some one help please? | 19:48 |
evgenyl | ukk1985: Yes, AIAB is not stable lately, can you paste a complete log from maas-import-resource pod somewhere here https://paste.ubuntu.com/ ? | 19:49 |
ukk1985 | Ok | 19:50 |
*** kranthikirang has joined #airshipit | 19:50 | |
kranthikirang | evgenyl: I am trying to upgrade genesis kernel version and seeing dependencies failures; and I see this in any host that I try | 19:52 |
evgenyl | kranthikirang: `dependencies failures` - can you send the output of upgrade command? | 19:53 |
evgenyl | kranthikirang: It usually shows specific packages that cause the problem. | 19:53 |
kranthikirang | evgenyl: https://gist.github.com/kranthikirang/dae96bd6a7bf8e1ea29197617a41c3bd#file-4-15-0-34-generic | 19:54 |
evgenyl | kranthikirang: Also the only repo which is added to your host by airship is the repo that container `docker-ce`, have you configured some extra repos manually? | 19:54 |
kranthikirang | evgenyl: I have tried different ways but couldn't make it work; when I install manually linux-image-unsigned-4.15.0-34-generic it bricked the system | 19:55 |
kranthikirang | I have just installed Ubuntu-16.04.6 ISO, configured network and that's ot | 19:55 |
kranthikirang | didn't even install docker-ce as well | 19:55 |
evgenyl | kranthikirang: what does `apt-cache policy` output? | 19:56 |
kranthikirang | give me a sec | 19:56 |
evgenyl | kranthikirang: At what stage does it happen? Have you installed docker.io/started genesis.sh? | 19:56 |
kranthikirang | nope | 19:57 |
kranthikirang | evgenyl: This is freshly installed ISO; nothing at all except SSH package | 19:57 |
evgenyl | kranthikirang: This happens on a clean system? | 19:57 |
kranthikirang | I have crossed checked in different servers; somehow this is broken | 19:58 |
evgenyl | kranthikirang: I have a stupid question, have you run `apt update` prior to running install? | 19:58 |
kranthikirang | even in old Ubuntu servers when I do apt-get update and try to run apt -y install 4.15.0-34-generic it fails | 19:58 |
kranthikirang | evgenyl: Oh; I did | 19:59 |
kranthikirang | you can simply check this if you have any free host without -y | 19:59 |
kranthikirang | I have genesis under proxy hence I can't open gist.github.com from my other laptop | 20:00 |
evgenyl | kranthikirang: what about `sudo apt install linux-image-4.15.0-34-generic`? | 20:03 |
kranthikirang | evgenyl: It worked but earlier it bricked the OS; reboot didn't. work | 20:04 |
ukk1985 | @evgenyl, I have pasted the log from maas-import-resource here https://paste.ubuntu.com/p/YjTQtyrW9W/ | 20:05 |
evgenyl | kranthikirang: just a sec, you will need a bunch of other related packages. | 20:05 |
kranthikirang | ok | 20:05 |
kranthikirang | evgenyl: I will re-login | 20:07 |
evgenyl | kranthikirang: http://paste.openstack.org/show/748273/ | 20:07 |
kranthikirang | evgenyl: You want me to install all these packages? | 20:08 |
evgenyl | kranthikirang: Yes, most likely your system does not boot, because it does not have all required modules/drivers installed, this is what comes from *modules. | 20:09 |
kranthikirang | evgenyl: Let me try and get back | 20:09 |
evgenyl | ukk1985: Do you have an internet access on the environment where you run airship in a bottle? | 20:10 |
*** kranthikirang has quit IRC | 20:11 | |
*** kranthikirang has joined #airshipit | 20:12 | |
*** kranthikirang has quit IRC | 20:12 | |
*** kranthikirang has joined #airshipit | 20:13 | |
ukk1985 | @evgenyl, Yes | 20:14 |
kranthikirang | evgenyl: I have installed those packages; | 20:14 |
kranthikirang | dpkg -l | grep 4.15.0-34 | 20:14 |
kranthikirang | ii linux-headers-4.15.0-34 4.15.0-34.37~16.04.1 all Header files related to Linux kernel version 4.15.0 | 20:14 |
kranthikirang | ii linux-headers-4.15.0-34-generic 4.15.0-34.37~16.04.1 amd64 Linux kernel headers for version 4.15.0 on 64 bit x86 SMP | 20:14 |
kranthikirang | ii linux-image-4.15.0-34-generic 4.15.0-34.37~16.04.1 amd64 Signed kernel image generic | 20:14 |
kranthikirang | ii linux-modules-4.15.0-34-generic 4.15.0-34.37~16.04.1 amd64 Linux kernel extra modules for version 4.15.0 on 64 bit x86 SMP | 20:14 |
kranthikirang | ii linux-modules-extra-4.15.0-34-generic 4.15.0-34.37~16.04.1 amd64 Linux kernel extra modules for version 4.15.0 on 64 bit x86 SMP | 20:14 |
kranthikirang | evgenyl: Do I have to reboot and boot into that new kernel? | 20:14 |
evgenyl | kranthikirang: Try rebooting, you still should be able to select previous kernel version from a boot console, if it does not boot. | 20:15 |
kranthikirang | evgenyl: OK | 20:16 |
*** dustinspecker has quit IRC | 20:16 | |
kranthikirang | evgenyl: Also while linting I am seeing P009 error | 20:17 |
kranthikirang | P009 | /tmp/tmp759xptxs/airship-treasuremap/site/airship- | | 20:17 |
kranthikirang | | | seaworthy/secrets/passphrases/osh_glance_password.yaml (document | | 20:17 |
kranthikirang | | | deckhand/Passphrase/v1:osh_glance_password) is a secret, is not stored in a secrets | | 20:17 |
kranthikirang | | | path | 20:17 |
kranthikirang | evgenyl: is that ok? | 20:17 |
evgenyl | kranthikirang: Yep, I think you can ignore that. | 20:18 |
kranthikirang | evgenyl: I see P009 after every p001 | 20:18 |
kranthikirang | evgenyl: OK; That's what I thought; | 20:19 |
kranthikirang | evgenyl: thanks | 20:19 |
kranthikirang | evgenyl: Perfect; It worked now; 4.15.0-34-generic | 20:23 |
evgenyl | kranthikirang: Good! | 20:23 |
kranthikirang | evgenyl: Perhaps we should update the document with packages to be installed | 20:23 |
evgenyl | ukk1985: I'm not a huge MaaS expert, but this looks to be a known issue, what commit of AIAB are you currently on? | 20:24 |
kranthikirang | apparently googling didn't help on this issue | 20:24 |
ukk1985 | evgenyl: latest commit is 7cf604036f74ba5776dc781257d8c1167eca1822 | 20:26 |
evgenyl | kranthikirang: Have you worked with before https://review.openstack.org/ ? If yes are you willing to help improving the docs and submitting a patch? | 20:26 |
evgenyl | ukk1985: Oh, I see, the latest one :) | 20:26 |
kranthikirang | evgenyl: I did; I am a contributor to openstack-helm and ONAP; but honestly never make a fix for documentation; can you give me where to start for documentation; | 20:27 |
ukk1985 | evgenyl: Yes, I cloned AIAB repo yesterday | 20:27 |
evgenyl | kranthikirang: Awesome, here is the project https://review.openstack.org/#/admin/projects/openstack/airship-treasuremap see `doc/source/authoring_and_deployment.rst` when you clone the repo. | 20:29 |
kranthikirang | evgenyl: Perfect; Will take care | 20:29 |
kranthikirang | evgenyl: Do I have to file a bug or story to make topic for this patch? I am not sure about procedure with Airship | 20:35 |
kranthikirang | evgenyl: Do you have a doc which describes fix guidelines ? | 20:35 |
openstackgerrit | Drew Walters proposed openstack/airship-treasuremap master: airskiff: Use Minikube for Kubernetes deployment https://review.openstack.org/622610 | 21:30 |
openstackgerrit | Drew Walters proposed openstack/airship-treasuremap master: airskiff: Use Minikube for Kubernetes deployment https://review.openstack.org/622610 | 21:34 |
openstackgerrit | Dimitrios Markou proposed openstack/airship-in-a-bottle master: [WIP] Add bgp peering in virtual airship https://review.openstack.org/642171 | 21:34 |
evgenyl | kranthikirang: For the doc you can just submit a fix, no need to create a bug/story. | 21:36 |
kranthikirang | evgenyl: So I need to update commit-msg when I download git hooks? | 21:36 |
kranthikirang | evgenyl: or just a simple message for commit/fix? | 21:37 |
kranthikirang | evgenyl: When I use format like Fix install 4.15.0-34-generic and write a bit notes; Its giving me errors; I am sorry, that's how I did in openstack-helm | 21:38 |
evgenyl | kranthikirang: What kind of error do you get? | 21:39 |
kranthikirang | git commit -a | 21:39 |
kranthikirang | .git/hooks/commit-msg: 1: .git/hooks/commit-msg: Fix: not found | 21:39 |
kranthikirang | .git/hooks/commit-msg: 3: .git/hooks/commit-msg: Currently: not found | 21:39 |
kranthikirang | install: target 'broken' is not a directory | 21:39 |
kranthikirang | .git/hooks/commit-msg: 5: .git/hooks/commit-msg: dependencies.: not found | 21:39 |
kranthikirang | .git/hooks/commit-msg: 6: .git/hooks/commit-msg: packages: not found | 21:39 |
kranthikirang | evgenyl: Do I have to just use git commit -a -m "<message>"? | 21:40 |
kranthikirang | evgenyl: I am trying to follow this | 21:40 |
kranthikirang | https://gerrit-review.googlesource.com/Documentation/cmd-hook-commit-msg.html | 21:40 |
evgenyl | kranthikirang: Hmm, honestly I don't know what is the problem here, I have not done any additional configuration when started to work on airship-treasuremap repo. Did you run `git commit -a` after commiting the changes using `git commit`? `-a` is needed only when you want to change the message of your commited change. | 21:45 |
kranthikirang | evgenyl: That should be --amend; Where do you put commit message? | 21:48 |
kranthikirang | evgenyl: :) I am being confused with different projects and everything follow different approach | 21:49 |
evgenyl | ukk1985: I have seen that people have similar errors on airship mailing list, as I said I'm not really familiar with MaaS and not sure what is the problem there, I you ping me on Monday I will try to look into that and see if I possibly can help you with the issue, if it's ok. | 21:49 |
openstackgerrit | Drew Walters proposed openstack/airship-armada master: End user logging for audit traceabilty https://review.openstack.org/638124 | 21:49 |
kranthikirang | evgenyl: Please tell me how do you commit and I will follow the same; Only difference is I am behind a proxy; so, have to use https password | 21:49 |
evgenyl | kranthikirang: It's not really different from every other gerrit based workflow. | 21:49 |
evgenyl | kranthikirang: I just do 'git add file-name; git commit; git review' | 21:50 |
evgenyl | kranthikirang: When you run 'git commit' you will need to enter your commit message. | 21:50 |
kranthikirang | evgenyl: So, here you will just simple give a line of message | 21:51 |
evgenyl | kranthikirang: Yes, when you run `git commit` you just write a description for the change. | 21:52 |
kranthikirang | Password for 'https://kranthikirang@review.openstack.org': | 21:52 |
kranthikirang | remote: Processing changes: refs: 1, done | 21:52 |
kranthikirang | remote: ERROR: [827dcdf] missing Change-Id in commit message footer | 21:52 |
kranthikirang | remote: | 21:52 |
evgenyl | kranthikirang: When does it happen? On commit or on review? | 21:53 |
kranthikirang | When I first tried this commit I have used a wrong e-mail and from that time it keep saying this; | 21:53 |
kranthikirang | git review | 21:53 |
kranthikirang | evgenyl: so I have deleted the folder and clone again and seeing the same error; | 21:54 |
evgenyl | ukk1985: I've just got a successful AIAB deployment with this patch https://review.openstack.org/#/c/644634/ you can try it out, run `git fetch ssh://evgeniyl@review.openstack.org:29418/openstack/airship-in-a-bottle refs/changes/34/644634/4 && git checkout FETCH_HEAD` from your airship-in-a-bottle directory. | 21:54 |
*** lemko has quit IRC | 21:55 | |
*** ukk1985 has quit IRC | 21:56 | |
evgenyl | kranthikirang: I can only suggest installing gerrit hooks https://stackoverflow.com/questions/8845658/gerrit-error-when-change-id-in-commit-messages-are-missing and running `git commit -a` to regenerate Change-id, I'm not sure why it is not already installed. | 21:57 |
evgenyl | dwalt, kaspars__, mattmceuen, anybody else, I would appreciate if you can help to review the fix for AIAB gate https://review.openstack.org/#/c/644634/ | 21:58 |
kranthikirang | evgenyl: perhaps because of proxy | 21:58 |
evgenyl | kranthikirang: Oh, this maybe the case.. | 21:58 |
kranthikirang | https://git.openstack.org/tools/hooks/commit-msg for this I see an empty file | 21:59 |
kranthikirang | evgenyl: I have cloned using git clone https://git.openstack.org/openstack/airship-treasuremap && (cd airship-treasuremap && curl -kLo `git rev-parse --git-dir`/hooks/commit-msg https://git.openstack.org/tools/hooks/commit-msg; chmod +x `git rev-parse --git-dir`/hooks/commit-msg) | 22:00 |
dwalt | evgenyl: taking a look now! | 22:01 |
evgenyl | kranthikirang: As an option you can try generating uniq ID yourself and adding it at the end of your git message `Change-Id: 291c0b7c41804b2b2a026cbe672b9210c308dc29b` | 22:06 |
kranthikirang | evgenyl: Ok | 22:07 |
*** michael-beaver has quit IRC | 22:20 | |
openstackgerrit | kranthi kiran guttikonda proposed openstack/airship-treasuremap master: Fix install 4.15.0-34-generic https://review.openstack.org/645951 | 22:28 |
*** jamesgu has quit IRC | 22:28 | |
kranthikirang | evgenyl: Nothing worked for me; So, I have copied hooks/commit-meg from non-proxy host and copied to here :) | 22:28 |
kranthikirang | evgenyl: Then everything worked like charm :) Proxy created these many problems | 22:29 |
kranthikirang | evgenyl: Can you share your e-mail? I will add you to reviews list | 22:32 |
kranthikirang | evgenyl: So, we have to manually configure genesis node with all VLANs? | 22:51 |
*** jamesgu__ has quit IRC | 23:15 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!