[native-toolchain-CR] Quiet down file transfers, rename target platform variable

[Laszlo Gaal (Code Review)]

Laszlo Gaal has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9149 )

Change subject: Quiet down file transfers, rename target platform variable
..


Patch Set 3: Verified+1


--
To view, visit http://gerrit.cloudera.org:8080/9149
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4349a8ae0ab5bde2dabf5ff65db0f2db4b9ba9
Gerrit-Change-Number: 9149
Gerrit-PatchSet: 3
Gerrit-Owner: Laszlo Gaal 
Gerrit-Reviewer: Laszlo Gaal 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Tue, 13 Feb 2018 14:05:23 +
Gerrit-HasComments: No

[native-toolchain-CR] Quiet down file transfers, rename target platform variable

[Laszlo Gaal (Code Review)]

Laszlo Gaal has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/9149 )

Change subject: Quiet down file transfers, rename target platform variable
..

Quiet down file transfers, rename target platform variable

1. File download with wget and maven upload all have verbose progress
indicators. This increases noise in the build logs making it hard
to navigate to relevant details.

This patch:
- changes wget to use a less chatty progress bar
- changes maven upload to batch mode and bumps log level to warnings
  or worse
- tells "aws s3 cp" to report errors only, adding an explicit echo to
  report file copy source and estination.

2. When the toolchain is built in an automated environment (e.g. using
a Jenkins job), the build target platform identifier is communicated
in an environment variable. This patch renames this variable to
BUILD_TARGET_LABEL to untangle it from Jenkins-defined variables and
to make its role more explicit.

Tested by rebuilding the toolchain with the new script, using
a conforming Jenkins job; all artifacts were prodiced with correct
package name suffixes. Results are stored at:
s3://impala-toolchain-test/build/33-a25ad7b25c/

Change-Id: Iaa4349a8ae0ab5bde2dabf5ff65db0f2db4b9ba9
---
M functions.sh
M init.sh
2 files changed, 17 insertions(+), 10 deletions(-)

Approvals:
  Laszlo Gaal: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/9149
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaa4349a8ae0ab5bde2dabf5ff65db0f2db4b9ba9
Gerrit-Change-Number: 9149
Gerrit-PatchSet: 3
Gerrit-Owner: Laszlo Gaal 
Gerrit-Reviewer: Laszlo Gaal 
Gerrit-Reviewer: Tim Armstrong 

[native-toolchain-CR] Quiet down file transfers, rename target platform variable

[Laszlo Gaal (Code Review)]

Laszlo Gaal has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9149 )

Change subject: Quiet down file transfers, rename target platform variable
..


Patch Set 3: Code-Review+2

Rebased, unchanged, carrying Tim's +2


--
To view, visit http://gerrit.cloudera.org:8080/9149
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4349a8ae0ab5bde2dabf5ff65db0f2db4b9ba9
Gerrit-Change-Number: 9149
Gerrit-PatchSet: 3
Gerrit-Owner: Laszlo Gaal 
Gerrit-Reviewer: Laszlo Gaal 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Tue, 13 Feb 2018 13:58:02 +
Gerrit-HasComments: No

[native-toolchain-CR] Quiet down file transfers, rename target platform variable

[Tim Armstrong (Code Review)]

Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9149 )

Change subject: Quiet down file transfers, rename target platform variable
..


Patch Set 2: Code-Review+2


--
To view, visit http://gerrit.cloudera.org:8080/9149
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iaa4349a8ae0ab5bde2dabf5ff65db0f2db4b9ba9
Gerrit-Change-Number: 9149
Gerrit-PatchSet: 2
Gerrit-Owner: Laszlo Gaal 
Gerrit-Reviewer: Tim Armstrong 
Gerrit-Comment-Date: Mon, 05 Feb 2018 19:54:10 +
Gerrit-HasComments: No