Friday, 2019-03-22

openstackgerritMerged openstack/airship-treasuremap master: global: Add nfs-provisioner chart  https://review.openstack.org/63521700:04
*** irclogbot_3 has joined #airshipit00:16
*** cheng1 has joined #airshipit00:59
*** mbologna has quit IRC01:32
*** mbologna has joined #airshipit01:32
*** Nishant_ has quit IRC02:11
*** cheng1 has quit IRC02:40
*** cheng1 has joined #airshipit03:05
*** cheng1 has quit IRC03:23
*** cheng1 has joined #airshipit04:02
*** cheng1 has quit IRC04:07
*** kranthikirang has joined #airshipit04:16
*** kranthikirang has quit IRC04:21
*** portdirect has quit IRC04:31
*** serverascode has quit IRC04:32
*** evgenyl has quit IRC04:32
*** adrianreza has quit IRC04:32
*** lamt has quit IRC04:32
*** hogepodge has quit IRC04:32
*** srwilkers has quit IRC04:32
*** jbryce has quit IRC04:33
*** mattmceuen has quit IRC04:33
*** powerds0111 has quit IRC04:33
*** evgenyl has joined #airshipit04:34
*** srwilkers has joined #airshipit04:34
*** portdirect has joined #airshipit04:34
*** adrianreza has joined #airshipit04:34
*** cheng1 has joined #airshipit04:34
*** hogepodge has joined #airshipit04:35
*** mattmceuen has joined #airshipit04:36
*** jbryce has joined #airshipit04:38
*** lemko has quit IRC04:38
*** serverascode has joined #airshipit04:39
*** kranthikirang has joined #airshipit06:04
*** kranthikirang has quit IRC06:09
*** lemko has joined #airshipit06:15
*** jamesgu has quit IRC06:37
*** dims has quit IRC06:39
openstackgerritSmruti Soumitra Khuntia proposed openstack/airship-armada master: End user logging for audit traceabilty  https://review.openstack.org/63812407:34
openstackgerritSmruti Soumitra Khuntia proposed openstack/airship-armada master: End user logging for audit traceabilty  https://review.openstack.org/63812407:38
*** skatsaounis has joined #airshipit07:40
*** kranthikirang has joined #airshipit07:52
*** kranthikirang has quit IRC07:57
openstackgerritSmruti Soumitra Khuntia proposed openstack/airship-armada master: End user logging for audit traceabilty  https://review.openstack.org/63812407:59
*** cheng1 has quit IRC09:13
*** cheng1 has joined #airshipit09:14
*** kranthikirang has joined #airshipit09:41
*** kranthikirang has quit IRC09:45
*** roman_g has joined #airshipit09:58
*** cheng1 has quit IRC10:01
*** cheng1 has joined #airshipit10:16
openstackgerritSmruti Soumitra Khuntia proposed openstack/airship-drydock master: End user logging for audit traceabilty  https://review.openstack.org/63811510:39
*** wahnsinn has joined #airshipit10:47
*** Nishant_ has joined #airshipit10:54
*** adrianreza has quit IRC10:59
*** portdirect has quit IRC11:00
*** adrianreza has joined #airshipit11:00
*** portdirect has joined #airshipit11:01
*** cheng1 has quit IRC11:20
*** cheng1 has joined #airshipit11:21
*** kranthikirang has joined #airshipit11:29
*** kranthikirang has quit IRC11:33
*** cheng1 has quit IRC11:38
*** alanmeadows_ has joined #airshipit12:07
*** roman_g has quit IRC12:09
*** alanmeadows has quit IRC12:14
*** alanmeadows_ is now known as alanmeadows12:14
*** cheng1 has joined #airshipit12:42
*** altlogbot_2 has quit IRC13:01
*** irclogbot_3 has quit IRC13:01
*** irclogbot_2 has joined #airshipit13:01
*** altlogbot_3 has joined #airshipit13:01
openstackgerritLakshmi Prasanna Goutham Pratapa proposed openstack/airship-promenade master: Add Liveness/Readiness probe to calico-etcd.  https://review.openstack.org/64556113:15
*** kranthikirang has joined #airshipit13:17
*** kranthikirang has quit IRC13:21
*** cheng1 has quit IRC13:22
*** kranthikirang has joined #airshipit13:31
*** juhak has quit IRC13:35
*** juhak has joined #airshipit13:36
*** aaronsheffield has joined #airshipit13:48
*** juhak has quit IRC14:16
*** juhak has joined #airshipit14:16
*** michael-beaver has joined #airshipit14:19
*** michaelbeaver has joined #airshipit14:30
*** kranthikirang has quit IRC14:34
*** michael-beaver has quit IRC14:34
openstackgerritEvgeniy L proposed openstack/airship-treasuremap master: [WIP][DNM] test patch  https://review.openstack.org/64501214:58
dwaltIt 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 yet15:03
evgenylHi 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
evgenyldwalt: Hmm, have not seen anything related on OpenStack dev ML, do you have a link to the failure handy?15:09
dwaltevegnyl: 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
dwaltShipyard has one as well: http://logs.openstack.org/81/645381/1/check/airship-shipyard-image-gate/be43e0f/15:10
*** jamesgu has joined #airshipit15:13
dwaltevgenyl: 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
evgenyldwalt: Thanks, I will try that!15:16
*** altlogbot_3 has quit IRC15:21
openstackgerritEvgeniy L proposed openstack/airship-in-a-bottle master: Downgrade ingress controller for MariaDB  https://review.openstack.org/64463415:21
dwaltevgenyl: 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-options15:22
evgenyldwalt: 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 #airshipit15:24
*** irclogbot_2 has quit IRC15:30
evgenyldwalt: 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.rst15:30
evgenylhttps://www.irccloud.com/pastebin/lRItovpb/15:31
*** irclogbot_1 has joined #airshipit15:31
dwaltevgenyl: aaronsheffield just asked in #openstack-infra15:31
aaronsheffieldSo, 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
evgenylOh, interesting.15:33
dwaltaaronsheffield: is there a way to obtain the commit sha other than the `Zuul.newrev` variable?15:34
*** dimitris_ has quit IRC15:35
*** irclogbot_1 has quit IRC15:36
evgenyldwalt: Hmm, it does not seem to be available http://paste.openstack.org/show/748250/15:36
evgenyldwalt: But maybe I'm missing something.15:36
*** irclogbot_3 has joined #airshipit15:36
evgenyldwalt: According to the docs newrev may also be missing.15:36
dwaltI 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
aaronsheffieldhttp://lists.openstack.org/pipermail/openstack-discuss/2019-March/004034.html, the announcement about the ansible change.15:40
*** pkaralis has quit IRC15:41
openstackgerritDrew Walters proposed openstack/airship-treasuremap master: airskiff: Pull rather than build Airship images  https://review.openstack.org/63523115:44
openstackgerritDrew Walters proposed openstack/airship-treasuremap master: global: Uplift helm-toolkit  https://review.openstack.org/64522615:44
openstackgerritDrew Walters proposed openstack/airship-treasuremap master: airskiff: Reduce postgresql PVC size  https://review.openstack.org/63912315:44
openstackgerritDrew Walters proposed openstack/airship-treasuremap master: airskiff: Use global Airship charts  https://review.openstack.org/63521815:44
*** kranthikirang has joined #airshipit15:45
openstackgerritAaron Sheffield proposed openstack/airship-pegleg master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64563115:45
*** kranthikirang has quit IRC15:49
*** nick_kar has quit IRC15:53
openstackgerritAaron Sheffield proposed openstack/airship-pegleg master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64563116:17
openstackgerritMerged openstack/airship-treasuremap master: Add the wait label to delete nova test pod when nova chart is upgraded  https://review.openstack.org/64521816:25
*** roman_g has joined #airshipit16:33
*** wahnsinn has quit IRC16:37
*** sthussey has joined #airshipit16:38
openstackgerritMichael Beaver proposed openstack/airship-armada master: Fix for Pep8 failures  https://review.openstack.org/64565616:38
openstackgerritAaron Sheffield proposed openstack/airship-pegleg master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64563116:47
mattmceuenRe: the `zuul.newrev` issue - lamt has a PS out for pegleg, with some feedback on it: https://review.openstack.org/#/c/645055/16:47
openstackgerritDrew Walters proposed openstack/airship-treasuremap master: airskiff: Reduce postgresql PVC size  https://review.openstack.org/63912316:59
openstackgerritDrew Walters proposed openstack/airship-treasuremap master: airskiff: Use global Airship charts  https://review.openstack.org/63521816:59
openstackgerritMatt McEuen proposed openstack/airship-promenade master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64580117:07
openstackgerritMatt McEuen proposed openstack/airship-promenade master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64580117:07
aaronsheffieldI 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
dwaltaaronsheffield: taking a look now. Thanks for doing that17:08
mattmceuenJust imitated your change for promenade as well aaronsheffield https://review.openstack.org/64580117:08
dwaltThanks mattmceuen17:08
dwaltmattmceuen: just waiting to verify that Zuul re-populates that variable when aaronsheffield’s patch merges17:11
mattmceuen+117:12
seaneaganlooks like i was a little late, but added a comment to: https://review.openstack.org/#/c/64563117:13
dwaltseaneagan: Zuul supposedly restores the `zuul.newref` variable upon merge, so we think the commit SHA should still be included in the tag when we publish17:15
dwaltwaiting to verify that now17:15
openstackgerritScott Hussey proposed openstack/airship-promenade master: (apiserver) [WIP] support key rotation  https://review.openstack.org/63193517:16
dwaltSeems like it worked https://www.irccloud.com/pastebin/Pmh2jB0M/17:22
openstackgerritMichael Beaver proposed openstack/airship-armada master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64581017:22
michaelbeaverAwesome, just threw one up for Armada as well17:23
*** michaelbeaver has quit IRC17:23
*** michael-beaver has joined #airshipit17:23
openstackgerritMerged openstack/airship-pegleg master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64563117:23
*** lamt has joined #airshipit17:29
*** roman_g has quit IRC17:30
openstackgerritAaron Sheffield proposed openstack/airship-pegleg master: Fix typo in secrets linting  https://review.openstack.org/64495817:38
*** dustinspecker has joined #airshipit17:45
openstackgerritAaron Sheffield proposed openstack/airship-drydock master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64582117:53
openstackgerritAaron Sheffield proposed openstack/airship-shipyard master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64582217:54
openstackgerritAaron Sheffield proposed openstack/airship-maas master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64582418:01
openstackgerritAaron Sheffield proposed openstack/airship-deckhand master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64582518:01
openstackgerritEvgeniy L proposed openstack/airship-in-a-bottle master: Fix AIAB gate Heat test & MariaDB failures  https://review.openstack.org/64463418:22
openstackgerritMerged openstack/airship-pegleg master: Fix typo in secrets linting  https://review.openstack.org/64495818:26
openstackgerritMerged openstack/airship-promenade master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64580118:27
openstackgerritSandeep Reddy Thumma proposed openstack/airship-promenade master: Set Least Previliage for .PEM files under directory /etc/genesis.  https://review.openstack.org/64077518:30
openstackgerritDrew Walters proposed openstack/airship-shipyard master: images: Update default image tag to `latest`  https://review.openstack.org/64538118:42
openstackgerritMichael Beaver proposed openstack/airship-armada master: Fix gate failures for Docker and Pep8  https://review.openstack.org/64565618:46
openstackgerritAnthony Bellino proposed openstack/airship-divingbell master: [WIP] Initial Ansible Daemonset  https://review.openstack.org/64053918:48
openstackgerritMerged openstack/airship-promenade master: Set Least Previliage for .PEM files under directory /etc/genesis.  https://review.openstack.org/64077518:51
openstackgerritMichael Beaver proposed openstack/airship-armada master: Support in Armada for locking Tiller  https://review.openstack.org/63248318:57
openstackgerritMichael Beaver proposed openstack/airship-armada master: Move Tiller into Armada pod  https://review.openstack.org/63278818:57
openstackgerritMerged openstack/airship-maas master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64582419:00
openstackgerritMerged openstack/airship-shipyard master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64582219:06
openstackgerritAaron Sheffield proposed openstack/airship-deckhand master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64582519:06
openstackgerritMerged openstack/airship-drydock master: Updating Docker Gate use of zuul.newrev  https://review.openstack.org/64582119:08
openstackgerritMerged openstack/airship-armada master: Fix gate failures for Docker and Pep8  https://review.openstack.org/64565619:09
openstackgerritDan Crank proposed openstack/airship-deckhand master: Log client-id in UCP API endpoints  https://review.openstack.org/63406819:27
openstackgerritPRATEEK REDDY DODDA proposed openstack/airship-armada master: [WIP] Implement Security Context for Armada  https://review.openstack.org/63920719:29
openstackgerritMerged openstack/airship-treasuremap master: airskiff: Pull rather than build Airship images  https://review.openstack.org/63523119:32
openstackgerritPRATEEK REDDY DODDA proposed openstack/airship-deckhand master: [WIP] Implement Security Context for Deckhand  https://review.openstack.org/63920419:45
*** ukk1985 has joined #airshipit19:46
ukk1985Hello, 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
ukk1985pod log says "Import of boot resources startedImport failed!"     Image       import FAILED!19:48
ukk1985Can some one help please?19:48
evgenylukk1985: 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
ukk1985Ok19:50
*** kranthikirang has joined #airshipit19:50
kranthikirangevgenyl: I am trying to upgrade genesis kernel version and seeing dependencies failures; and I see this in any host that I try19:52
evgenylkranthikirang: `dependencies failures` - can you send the output of upgrade command?19:53
evgenylkranthikirang: It usually shows specific packages that cause the problem.19:53
kranthikirangevgenyl: https://gist.github.com/kranthikirang/dae96bd6a7bf8e1ea29197617a41c3bd#file-4-15-0-34-generic19:54
evgenylkranthikirang: 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
kranthikirangevgenyl: 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 system19:55
kranthikirangI have just installed Ubuntu-16.04.6 ISO, configured network and that's ot19:55
kranthikirangdidn't even install docker-ce as well19:55
evgenylkranthikirang: what does `apt-cache policy` output?19:56
kranthikiranggive me a sec19:56
evgenylkranthikirang: At what stage does it happen? Have you installed docker.io/started genesis.sh?19:56
kranthikirangnope19:57
kranthikirangevgenyl: This is freshly installed ISO; nothing at all except SSH package19:57
evgenylkranthikirang: This happens on a clean system?19:57
kranthikirangI have crossed checked in different servers; somehow this is broken19:58
evgenylkranthikirang: I have a stupid question, have you run `apt update` prior to running install?19:58
kranthikirangeven in old Ubuntu servers when I do apt-get update and try to run apt -y install 4.15.0-34-generic it fails19:58
kranthikirangevgenyl: Oh; I did19:59
kranthikirangyou can simply check this if you have any free host without -y19:59
kranthikirangI have genesis under proxy hence I can't open gist.github.com from my other laptop20:00
evgenylkranthikirang: what about `sudo apt install linux-image-4.15.0-34-generic`?20:03
kranthikirangevgenyl: It worked but earlier it bricked the OS; reboot didn't. work20:04
ukk1985@evgenyl, I have pasted the log from maas-import-resource here https://paste.ubuntu.com/p/YjTQtyrW9W/20:05
evgenylkranthikirang: just a sec, you will need a bunch of other related packages.20:05
kranthikirangok20:05
kranthikirangevgenyl: I will re-login20:07
evgenylkranthikirang: http://paste.openstack.org/show/748273/20:07
kranthikirangevgenyl: You want me to install all these packages?20:08
evgenylkranthikirang: 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
kranthikirangevgenyl: Let me try and get back20:09
evgenylukk1985: Do you have an internet access on the environment where you run airship in a bottle?20:10
*** kranthikirang has quit IRC20:11
*** kranthikirang has joined #airshipit20:12
*** kranthikirang has quit IRC20:12
*** kranthikirang has joined #airshipit20:13
ukk1985@evgenyl, Yes20:14
kranthikirangevgenyl: I have installed those packages;20:14
kranthikirangdpkg -l | grep 4.15.0-3420:14
kranthikirangii  linux-headers-4.15.0-34               4.15.0-34.37~16.04.1                       all          Header files related to Linux kernel version 4.15.020:14
kranthikirangii  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 SMP20:14
kranthikirangii  linux-image-4.15.0-34-generic         4.15.0-34.37~16.04.1                       amd64        Signed kernel image generic20:14
kranthikirangii  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 SMP20:14
kranthikirangii  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 SMP20:14
kranthikirangevgenyl: Do I have to reboot and boot into that new kernel?20:14
evgenylkranthikirang: Try rebooting, you still should be able to select previous kernel version from a boot console, if it does not boot.20:15
kranthikirangevgenyl: OK20:16
*** dustinspecker has quit IRC20:16
kranthikirangevgenyl: Also while linting I am seeing P009 error20:17
kranthikirangP009    |                   /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|            |                                          path20:17
kranthikirangevgenyl: is that ok?20:17
evgenylkranthikirang: Yep, I think you can ignore that.20:18
kranthikirangevgenyl: I see P009 after every p00120:18
kranthikirangevgenyl: OK; That's what I thought;20:19
kranthikirangevgenyl: thanks20:19
kranthikirangevgenyl: Perfect; It worked now; 4.15.0-34-generic20:23
evgenylkranthikirang: Good!20:23
kranthikirangevgenyl: Perhaps we should update the document with packages to be installed20:23
evgenylukk1985: 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
kranthikirangapparently googling didn't help on this issue20:24
ukk1985evgenyl: latest commit is 7cf604036f74ba5776dc781257d8c1167eca182220:26
evgenylkranthikirang: 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
evgenylukk1985: Oh, I see, the latest one :)20:26
kranthikirangevgenyl: 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
ukk1985evgenyl: Yes, I cloned AIAB repo yesterday20:27
evgenylkranthikirang: 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
kranthikirangevgenyl: Perfect; Will take care20:29
kranthikirangevgenyl: Do I have to file a bug or story to make topic for this patch? I am not sure about procedure with Airship20:35
kranthikirangevgenyl: Do you have a doc which describes fix guidelines ?20:35
openstackgerritDrew Walters proposed openstack/airship-treasuremap master: airskiff: Use Minikube for Kubernetes deployment  https://review.openstack.org/62261021:30
openstackgerritDrew Walters proposed openstack/airship-treasuremap master: airskiff: Use Minikube for Kubernetes deployment  https://review.openstack.org/62261021:34
openstackgerritDimitrios Markou proposed openstack/airship-in-a-bottle master: [WIP] Add bgp peering in virtual airship  https://review.openstack.org/64217121:34
evgenylkranthikirang: For the doc you can just submit a fix, no need to create a bug/story.21:36
kranthikirangevgenyl: So I need to update commit-msg when I download git hooks?21:36
kranthikirangevgenyl: or just a simple message for commit/fix?21:37
kranthikirangevgenyl: 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-helm21:38
evgenylkranthikirang: What kind of error do you get?21:39
kranthikiranggit commit -a21:39
kranthikirang.git/hooks/commit-msg: 1: .git/hooks/commit-msg: Fix: not found21:39
kranthikirang.git/hooks/commit-msg: 3: .git/hooks/commit-msg: Currently: not found21:39
kranthikiranginstall: target 'broken' is not a directory21:39
kranthikirang.git/hooks/commit-msg: 5: .git/hooks/commit-msg: dependencies.: not found21:39
kranthikirang.git/hooks/commit-msg: 6: .git/hooks/commit-msg: packages: not found21:39
kranthikirangevgenyl: Do I have to just use git commit -a -m "<message>"?21:40
kranthikirangevgenyl: I am trying to follow this21:40
kranthikiranghttps://gerrit-review.googlesource.com/Documentation/cmd-hook-commit-msg.html21:40
evgenylkranthikirang: 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
kranthikirangevgenyl: That should be --amend; Where do you put commit message?21:48
kranthikirangevgenyl: :) I am being confused with different projects and everything follow different approach21:49
evgenylukk1985: 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
openstackgerritDrew Walters proposed openstack/airship-armada master: End user logging for audit traceabilty  https://review.openstack.org/63812421:49
kranthikirangevgenyl: 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 password21:49
evgenylkranthikirang: It's not really different from every other gerrit based workflow.21:49
evgenylkranthikirang: I just do 'git add file-name; git commit; git review'21:50
evgenylkranthikirang: When you run 'git commit' you will need to enter your commit message.21:50
kranthikirangevgenyl: So, here you will just simple give a line of message21:51
evgenylkranthikirang: Yes, when you run `git commit` you just write a description for the change.21:52
kranthikirangPassword for 'https://kranthikirang@review.openstack.org':21:52
kranthikirangremote: Processing changes: refs: 1, done21:52
kranthikirangremote: ERROR: [827dcdf] missing Change-Id in commit message footer21:52
kranthikirangremote:21:52
evgenylkranthikirang: When does it happen? On commit or on review?21:53
kranthikirangWhen I first tried this commit I have used a wrong e-mail and from that time it keep saying this;21:53
kranthikiranggit review21:53
kranthikirangevgenyl: so I have deleted the folder and clone again and seeing the same error;21:54
evgenylukk1985: 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 IRC21:55
*** ukk1985 has quit IRC21:56
evgenylkranthikirang: 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
evgenyldwalt, 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
kranthikirangevgenyl: perhaps because of proxy21:58
evgenylkranthikirang: Oh, this maybe the case..21:58
kranthikiranghttps://git.openstack.org/tools/hooks/commit-msg for this I see an empty file21:59
kranthikirangevgenyl: 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
dwaltevgenyl: taking a look now!22:01
evgenylkranthikirang: 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
kranthikirangevgenyl: Ok22:07
*** michael-beaver has quit IRC22:20
openstackgerritkranthi kiran guttikonda proposed openstack/airship-treasuremap master: Fix install 4.15.0-34-generic  https://review.openstack.org/64595122:28
*** jamesgu has quit IRC22:28
kranthikirangevgenyl: Nothing worked for me; So, I have copied hooks/commit-meg from non-proxy host and copied to here :)22:28
kranthikirangevgenyl: Then everything worked like charm :) Proxy created these many problems22:29
kranthikirangevgenyl: Can you share your e-mail? I will add you to reviews list22:32
kranthikirangevgenyl: So, we have to manually configure genesis node with all VLANs?22:51
*** jamesgu__ has quit IRC23:15

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