Skip to content

feat: Add --instance-type-size flag for Seqera Compute#573

Merged
jonmarti merged 1 commit intomasterfrom
feat/COMP-1241-instance-type-size
Jan 21, 2026
Merged

feat: Add --instance-type-size flag for Seqera Compute#573
jonmarti merged 1 commit intomasterfrom
feat/COMP-1241-instance-type-size

Conversation

@jonmarti
Copy link
Contributor

@jonmarti jonmarti commented Jan 20, 2026

Summary

Adds a new --instance-type-size CLI option for Seqera Compute environments, allowing users to specify the size of Data Studios instances (SMALL, MEDIUM, LARGE).

Changes

  • Updated tower-java-sdk dependency from 1.97.0 to 1.101.0
  • Added --instance-type-size flag to SeqeraComputePlatform
  • Updated GraalVM reflection config for SDK 1.101.0 using runReflectionConfigGenerator
  • Fixed ListCmd API signature change (added includeTotalSize parameter)

Usage

tw compute-envs add seqera-compute \
  --name my-env \
  --workspace 12345 \
  --instance-type-size MEDIUM

Test plan

  • ./gradlew test passes
  • ./gradlew nativeCompile builds successfully
  • CLI help shows new --instance-type-size flag
  • Reflection config generated via ./gradlew runReflectionConfigGenerator

Related PRs

@jonmarti jonmarti force-pushed the feat/COMP-1241-instance-type-size branch from b1380e1 to 597ac7d Compare January 20, 2026 15:42
@jonmarti
feat(COMP-1241): Add --instance-type-size flag for Seqera Compute
36ba430 
Add support for specifying Data Studios instance size (SMALL, MEDIUM, LARGE)
when creating Seqera Compute environments.

Changes:
- Update SDK to 1.101.0
- Add --instance-type-size option to SeqeraComputePlatform
- Add SeqeraComputeCloudInstanceTypeSize to reflect-config.json for native image
- Fix ListCmd API signature change (added includeTotalSize parameter)
@jonmarti jonmarti force-pushed the feat/COMP-1241-instance-type-size branch from 597ac7d to 36ba430 Compare January 20, 2026 15:45
endre-seqera
endre-seqera approved these changes Jan 20, 2026
View reviewed changes
Copy link
Member

@endre-seqera endre-seqera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

please include some logs/screenshot of the running the cli with this new config option

@jonmarti
Copy link
Contributor Author

jonmarti commented Jan 20, 2026 •
edited
Loading

@endre-seqera here are some logs/screenshots, should I add them anywhere else?

Pro organization

With unified-compute/W1 configured as Pro organization:

./tw compute-envs add seqera-compute \                                                                                                                                                                               
    --name comp-1241-test-ce-small \
    --workspace unified-compute/W1 \
    --credentials seqera-compute-creds \
    --region eu-west-1 \
    --instance-type-size small \
    --wait AVAILABLE

New SEQERACOMPUTE-PLATFORM compute environment 'comp-1241-test-ce-small' added at [unified-compute / W1] workspace

Waiting AVAILABLE status....CREATING.......AVAILABLE  [DONE]
image

With medium:

New SEQERACOMPUTE-PLATFORM compute environment 'comp-1241-test-ce-medium' added at [unified-compute / W1] workspace

Waiting AVAILABLE status....CREATING.......AVAILABLE  [DONE]
image image

With large

New SEQERACOMPUTE-PLATFORM compute environment 'comp-1241-test-ce-large' added at [unified-compute / W1] workspace

Waiting AVAILABLE status....CREATING.......AVAILABLE  [DONE]
image

Basic organization

image

When it's a Basic organization we got same results for small but with large or medium:

ERROR: Free-tier organizations are restricted to SMALL instance type size. Requested: MEDIUM

ERROR: Free-tier organizations are restricted to SMALL instance type size. Requested: LARGE

Unsupported values

Values other than small medium or large get:

Invalid value for option '--instance-type-size': expected one of [SMALL, MEDIUM, LARGE] (case-insensitive) but was 'foo'

alberto-miranda
alberto-miranda approved these changes Jan 20, 2026
View reviewed changes
Copy link
Contributor

@alberto-miranda alberto-miranda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a very minor comment. Nothing blocking.

@alberto-miranda
Copy link
Contributor

alberto-miranda commented Jan 20, 2026

I tested this against Platform dev and it seems to do what it should. I wasn't able to retrieve the instance type for the new CEs so that part is not fully tested yet.

@jonmarti
Copy link
Contributor Author

jonmarti commented Jan 20, 2026

I tested this against Platform dev and it seems to do what it should. I wasn't able to retrieve the instance type for the new CEs so that part is not fully tested yet.

Check the logs and screenshots I posted above after doing some more tests. Also the CE views, which include the instance type.

@jonmarti jonmarti merged commit c3359a4 into master Jan 21, 2026
10 checks passed
@alberto-miranda alberto-miranda mentioned this pull request Jan 27, 2026
Merged
@alberto-miranda alberto-miranda changed the title feat(COMP-1241): Add --instance-type-size flag for Seqera Compute feat: Add --instance-type-size flag for Seqera Compute Jan 27, 2026
@alberto-miranda alberto-miranda mentioned this pull request Jan 27, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alberto-miranda alberto-miranda alberto-miranda approved these changes

@endre-seqera endre-seqera endre-seqera approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jonmarti @alberto-miranda @endre-seqera