Initial CLI code by stuartleeks · Pull Request #2537 · microsoft/AzureTRE

stuartleeks · 2022-09-01T15:21:26Z

Add initial code for a tre CLI for working with the AzureTRE API
Update shared service deployment script to use the CLI
Document the need to enable device code flow on root API and workspace APIs

Future work:

github-actions · 2022-09-01T15:23:05Z

Unit Test Results

515 tests 515 ✔️ 12s ⏱️
    1 suites     0 💤
    1 files     0 ❌

Results for commit 2f8d956.

♻️ This comment has been updated with latest results.

stuartleeks · 2022-10-06T09:52:18Z

/test

github-actions · 2022-10-06T09:52:38Z

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/3196210434 (with refid f02235c4)

(in response to this comment from @stuartleeks)

stuartleeks · 2022-10-06T12:11:52Z

/test

(added CLI install to CI dev container action)

github-actions · 2022-10-06T12:12:12Z

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/3197059734 (with refid f02235c4)

(in response to this comment from @stuartleeks)

stuartleeks · 2022-10-06T12:15:04Z

/test

github-actions · 2022-10-06T12:15:21Z

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/3197079475 (with refid f02235c4)

(in response to this comment from @stuartleeks)

martinpeck

LTGM! Nice 🥇

stuartleeks · 2022-10-06T13:26:00Z

/test

github-actions · 2022-10-06T13:26:21Z

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/3197542778 (with refid f02235c4)

(in response to this comment from @stuartleeks)

stuartleeks · 2022-10-06T13:58:02Z

/test

github-actions · 2022-10-06T13:58:22Z

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/3197766418 (with refid f02235c4)

(in response to this comment from @stuartleeks)

stuartleeks · 2022-10-06T14:23:44Z

/test

github-actions · 2022-10-06T14:24:07Z

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/3197955681 (with refid f02235c4)

(in response to this comment from @stuartleeks)

stuartleeks · 2022-10-06T14:40:18Z

/test

github-actions · 2022-10-06T14:40:33Z

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/3198075947 (with refid f02235c4)

(in response to this comment from @stuartleeks)

stuartleeks · 2022-10-06T14:46:34Z

/test

github-actions · 2022-10-06T14:46:54Z

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/3198121767 (with refid f02235c4)

(in response to this comment from @stuartleeks)

stuartleeks · 2022-10-06T15:03:07Z

/test

github-actions · 2022-10-06T15:03:22Z

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/3198248841 (with refid f02235c4)

(in response to this comment from @stuartleeks)

This reverts commit 8b29eac58b47974ac0d10d4b82639afd6e101c66.

stuartleeks · 2022-10-07T04:38:23Z

/test-extended

github-actions · 2022-10-07T04:38:38Z

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/3202293121 (with refid f02235c4)

(in response to this comment from @stuartleeks)

stuartleeks · 2022-10-07T05:41:57Z

/test-extended

re-run due to A hash conflict was encountered for the role Assignment ID. Please use a new Guid

github-actions · 2022-10-07T05:42:12Z

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/3202527092 (with refid f02235c4)

(in response to this comment from @stuartleeks)

stuartleeks force-pushed the sl/cli branch from 7ce88bb to 27eb18a Compare September 1, 2022 15:21

stuartleeks force-pushed the sl/cli branch from 27eb18a to 6dce40c Compare September 1, 2022 15:40

stuartleeks force-pushed the sl/cli branch 4 times, most recently from 3a72f1e to 04e568f Compare October 6, 2022 09:50

stuartleeks force-pushed the sl/cli branch from a0d8444 to 010994d Compare October 6, 2022 12:11

stuartleeks force-pushed the sl/cli branch from 010994d to 0e163af Compare October 6, 2022 12:14

stuartleeks marked this pull request as ready for review October 6, 2022 12:57

martinpeck approved these changes Oct 6, 2022

View reviewed changes

Comment thread cli/README.md Outdated

Comment thread cli/setup.py Outdated

Comment thread cli/tre/commands/shared_services/shared_service.py Outdated

stuartleeks force-pushed the sl/cli branch from 374aeda to 00599aa Compare October 6, 2022 18:14

stuartleeks added 18 commits October 6, 2022 18:30

Initial CLI code

d3e9005

Install tre CLI in dev container

324663e

Update deploy_shared_service.sh to use tre CLI

d17ab64

Install tre CLI when running dev container in CI

f0204b9

Add device code flow note to README

8e2bff7

drop old .github content

cbd8390

newlines in __init__.py for mpeck 😀

b31ec3a

Add exception stack trace to migration error logging

3b4f2da

PR feedback

b1f64ea

fixup deploy_shared_service.sh - remove old argument

6a13891

Convert shared service template to HEREDOC format to avoid escaping

8a10012

check exit code

ac3b083

fixup bash lint

ce0f575

fixup Makefile

0ed0095

update deploy_shared_services.sh quoting

c38f790

Skip SSL cert verification

8d7a485

Revert "newlines in __init__.py for mpeck 😀"

0074bb0

This reverts commit 8b29eac58b47974ac0d10d4b82639afd6e101c66.

Update CHANGELOG

2f8d956

stuartleeks force-pushed the sl/cli branch from 00599aa to 10ecafa Compare October 7, 2022 04:22

stuartleeks force-pushed the sl/cli branch from 10ecafa to 2f8d956 Compare October 7, 2022 04:45

stuartleeks merged commit 89a29dd into microsoft:main Oct 7, 2022

stuartleeks deleted the sl/cli branch October 7, 2022 08:35

Conversation

stuartleeks commented Sep 1, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Sep 1, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Unit Test Results

Uh oh!

stuartleeks commented Oct 6, 2022

Uh oh!

github-actions Bot commented Oct 6, 2022

Uh oh!

stuartleeks commented Oct 6, 2022

Uh oh!

github-actions Bot commented Oct 6, 2022

Uh oh!

stuartleeks commented Oct 6, 2022

Uh oh!

github-actions Bot commented Oct 6, 2022

Uh oh!

martinpeck left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

stuartleeks commented Oct 6, 2022

Uh oh!

github-actions Bot commented Oct 6, 2022

Uh oh!

stuartleeks commented Oct 6, 2022

Uh oh!

github-actions Bot commented Oct 6, 2022

Uh oh!

stuartleeks commented Oct 6, 2022

Uh oh!

github-actions Bot commented Oct 6, 2022

Uh oh!

stuartleeks commented Oct 6, 2022

Uh oh!

github-actions Bot commented Oct 6, 2022

Uh oh!

stuartleeks commented Oct 6, 2022

Uh oh!

github-actions Bot commented Oct 6, 2022

Uh oh!

stuartleeks commented Oct 6, 2022

Uh oh!

github-actions Bot commented Oct 6, 2022

Uh oh!

stuartleeks commented Oct 7, 2022

Uh oh!

github-actions Bot commented Oct 7, 2022

Uh oh!

stuartleeks commented Oct 7, 2022

Uh oh!

github-actions Bot commented Oct 7, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

stuartleeks commented Sep 1, 2022 •

edited

Loading

github-actions Bot commented Sep 1, 2022 •

edited

Loading