Skip to content

Commit 10cb50d

Browse files
Fixed Script Lint Issues
Signed-off-by: Utkarsh Bhatt <[email protected]>
1 parent 89fe07b commit 10cb50d

File tree

5 files changed

+22
-11
lines changed

5 files changed

+22
-11
lines changed

.github/workflows/q2q-candidate-upgrade.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,4 +63,4 @@ jobs:
6363
run: ~/actionutils.sh headexec wait_for_osds 3
6464

6565
- name: Exercise RGW again
66-
run: ~/actionutils.sh testrgw_on_lxd node-wrk0
66+
run: ~/actionutils.sh testrgw_on_headnode

.github/workflows/q2r-candidate-upgrade.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,4 +63,4 @@ jobs:
6363
run: ~/actionutils.sh headexec wait_for_osds 3
6464

6565
- name: Exercise RGW again
66-
run: ~/actionutils.sh testrgw_on_lxd node-wrk0
66+
run: ~/actionutils.sh testrgw_on_headnode

.github/workflows/tests.yml

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -303,6 +303,13 @@ jobs:
303303
- name: Enable services on wrk1
304304
run: ~/actionutils.sh headexec enable_services node-wrk1
305305

306+
- name: Enable RGW
307+
run: ~/actionutils.sh headexec enable_rgw
308+
309+
- name: Exercise RGW again
310+
run: |
311+
~/actionutils.sh testrgw_on_headnode
312+
306313
- name: Test remove node wrk3
307314
run: |
308315
set -uex
@@ -434,11 +441,7 @@ jobs:
434441
- name: Wait until 3 OSDs are up
435442
run: ~/actionutils.sh headexec wait_for_osds 3
436443

437-
- name: Install boto3 on head node.
438-
run: |
439-
~/actionutils.sh headexec install_boto3
440-
441444
- name: Exercise RGW again
442445
run: |
443-
~/actionutils.sh testrgw_on_lxd node-wrk0
446+
~/actionutils.sh testrgw_on_headnode
444447

scripts/appS3.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,10 +38,16 @@ def app_handle(args):
3838
(object_name, object_size)
3939
)
4040
# Print object IO summary:
41-
print("Object #{}: {}/{} -> Size: {}MB".format(i, bucket_name, object_name, object_size))
41+
print(
42+
"Object #{}: {}/{} -> Size: {}MB"
43+
.format(i, bucket_name, object_name, object_size)
44+
)
4245

4346
# Print Summary
44-
print("IO Summary: Object Count {}, Total Size {}MB".format(args.obj_num, sum(size for _, size in objects)))
47+
print(
48+
"IO Summary: Object Count {}, Total Size {}MB"
49+
.format(args.obj_num, sum(size for _, size in objects))
50+
)
4551

4652
# Cleanup (if asked for)
4753
if not args.no_delete:

tests/scripts/actionutils.sh

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -337,11 +337,13 @@ function testrgw() {
337337
rm keys.json
338338
}
339339

340-
function testrgw_on_lxd() {
340+
function testrgw_on_headnode() {
341341
set -eux
342342
local container="${1?missing}"
343+
local nw=$(get_lxd_network public)
344+
gw=$(echo "$nw" | cut -d/ -f1)
343345
lxc exec $container -- sh -c "microceph client s3 create testUser --json" > keys.json
344-
sudo python3 ./scripts/appS3.py http://localhost:80 keys.json --obj-num 2
346+
sudo python3 ./scripts/appS3.py http://${gw}0:80 keys.json --obj-num 2
345347

346348
# cleanup
347349
lxc exec $container -- sh -c "microceph client s3 delete testUser"

0 commit comments

Comments
 (0)