Skip to content

Commit f64c4fc

Browse files
authored
Merge pull request #5 from lamhaison/develop
[update] - add feature to split uploaded logs
2 parents 9f3c202 + edce685 commit f64c4fc

File tree

6 files changed

+47
-9
lines changed

6 files changed

+47
-9
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,3 @@
11
aws_cli_results
2+
services/tmp.sh
23
.DS_Store

common/common.sh

Lines changed: 20 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,10 @@ aws_assume_role_disable_fast_mode() {
1818
export aws_assume_role_print_account_info=true
1919
}
2020

21+
aws_assume_role_disable_show_detail_commandline() {
22+
export aws_show_commandline=false
23+
}
24+
2125
aws_run_commandline_with_retry() {
2226
local aws_commandline=$1
2327
local silent_mode=$2
@@ -60,16 +64,26 @@ aws_run_commandline_with_logging() {
6064
aws_commandline_logging=$(echo ${aws_commandline:?'aws_commandline is unset or empty'} | tr -d '\t' | tr -d '\n')
6165
# aws_commandline_logging=$(echo ${aws_commandline})
6266
local log_file_path=${aws_cli_logs}/${ASSUME_ROLE}.log
67+
68+
if [ "$aws_show_log_uploaded" = "true" ]; then
69+
70+
local log_uploaded_file_path=${aws_cli_logs}/${ASSUME_ROLE}-uploaded.log
71+
local tee_command="tee -a ${log_file_path} ${log_uploaded_file_path}"
72+
73+
else
74+
local tee_command="tee -a ${log_file_path}"
75+
fi
76+
6377
if [ "$aws_show_commandline" = "true" ]; then
64-
local output="tee -a ${log_file_path}"
78+
local detail_commandline_tee_command="${tee_command}"
6579
else
66-
local output=">> ${log_file_path}"
80+
local detail_commandline_tee_command="${tee_command} > /dev/null"
6781
fi
6882

6983
aws_commandline_result=$(aws_run_commandline_with_retry "${aws_commandline}" "${ignored_error_when_retry}")
7084

71-
echo "-------------------------------------START--$(date '+%Y-%m-%d-%H-%M-%S')------------------------------------------------" >>${log_file_path}
72-
echo "Running commandline [ ${aws_commandline_logging} ]" | eval $output
73-
echo $aws_commandline_result | tee -a ${log_file_path}
74-
echo "-------------------------------------FINISH-$(date '+%Y-%m-%d-%H-%M-%S')------------------------------------------------" >>${log_file_path}
85+
echo "------------------------------STARTED--$(date '+%Y-%m-%d-%H-%M-%S')-----------------------------------------" | eval $tee_command >/dev/null
86+
echo "Running commandline [ ${aws_commandline_logging} ]" | eval $detail_commandline_tee_command
87+
echo $aws_commandline_result | eval $tee_command
88+
echo "------------------------------FINISHED-$(date '+%Y-%m-%d-%H-%M-%S')-----------------------------------------" | eval $tee_command >/dev/null
7589
}

common/logging.sh

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,20 @@
22

33
aws_assume_role_check_log() {
44
local log_file_path=${aws_cli_logs}/${ASSUME_ROLE}.log
5+
echo "Read the log ${log_file_path}"
56
view +$ -c 'set number' ${log_file_path}
67
}
8+
9+
aws_assume_role_check_log_uploaded() {
10+
local log_file_path=${aws_cli_logs}/${ASSUME_ROLE}-uploaded.log
11+
echo "Read the log ${log_file_path}"
12+
view +$ -c 'set number' ${log_file_path}
13+
}
14+
15+
# TODO LATER
16+
aws_assume_role_enable_log_uploaded() {
17+
export aws_show_log_uploaded=true
18+
local log_uploaded_file_path=${aws_cli_logs}/${ASSUME_ROLE}-uploaded.log
19+
touch ${log_uploaded_file_path}
20+
cat /dev/null >${log_uploaded_file_path}
21+
}

main.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@ export peco_input_expired_time=10
1717
export aws_assume_role_expired_time=60
1818
# To allow log detail of the aws cli [true|false]
1919
export aws_show_commandline=true
20+
# To allow log information to make as evident and upload to the ticket. [true|false]
21+
export aws_show_log_uploaded=false
2022
export aws_log_tail_since=120m
2123

2224
mkdir -p ${tmp_credentials}

services/ec2.sh

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,3 +141,12 @@ aws_sg_add_rule() {
141141
--cidr $(lamhaison_get_public_ip)/32
142142
"
143143
}
144+
145+
aws_region_list() {
146+
aws_run_commandline "\
147+
aws ec2 describe-regions \
148+
--all-regions \
149+
--query 'Regions[].{Name:RegionName,Endpoint:Endpoint}' \
150+
--output table
151+
"
152+
}

services/test.sh

Lines changed: 0 additions & 3 deletions
This file was deleted.

0 commit comments

Comments
 (0)