Skip to content

Commit ba09308

Browse files
pehalavponomaryov
authored andcommitted
fix(sct): Do not require Okta for commands which do not need it
* update-conf-docs, nemesis-list, create-nemesis-pipelines * This is a hotfix and it is ugly, but for 3 commands no bigger effort is needed * Closes scylladb#10534
1 parent 452e948 commit ba09308

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

sct.py

+10-7
Original file line numberDiff line numberDiff line change
@@ -190,16 +190,19 @@ def getTestCaseNames(self, testCaseClass):
190190
type=click.Path(),
191191
expose_value=False,
192192
help="Install paths for extra python packages to install, scylla-cluster-plugins for example")
193-
def cli():
193+
@click.pass_context
194+
def cli(ctx):
194195
disable_loggers_during_startup()
195-
try_auth_with_okta()
196+
# Ugly way of filtering the few command that do not require OKTA verification
197+
if ctx.invoked_subcommand not in ("update-conf-docs", "nemesis-list", "create-nemesis-pipelines"):
198+
try_auth_with_okta()
196199

197-
key_store = KeyStore()
198-
# TODO: still leaving old keys, until we'll rebuild runner images - and reconfigure jenkins
199-
key_store.sync(keys=['scylla-qa-ec2', 'scylla-test', 'scylla_test_id_ed25519'],
200-
local_path=Path('~/.ssh/').expanduser(), permissions=0o0600)
200+
key_store = KeyStore()
201+
# TODO: still leaving old keys, until we'll rebuild runner images - and reconfigure jenkins
202+
key_store.sync(keys=['scylla-qa-ec2', 'scylla-test', 'scylla_test_id_ed25519'],
203+
local_path=Path('~/.ssh/').expanduser(), permissions=0o0600)
201204

202-
docker_hub_login(remoter=LOCALRUNNER)
205+
docker_hub_login(remoter=LOCALRUNNER)
203206

204207

205208
@cli.command('provision-resources', help="Provision resources for the test")

0 commit comments

Comments
 (0)