Skip to content

Update integration script#2

Closed
EnricoMi wants to merge 3 commits intoGeorgeJahad:addingArmadaClientfrom
EnricoMi:georgejahad/addingArmadaClient
Closed

Update integration script#2
EnricoMi wants to merge 3 commits intoGeorgeJahad:addingArmadaClientfrom
EnricoMi:georgejahad/addingArmadaClient

Conversation

@EnricoMi
Copy link

@EnricoMi EnricoMi commented Feb 6, 2025

Looks like the Docker image cannot build the Scala client.

@github-actions github-actions bot added the INFRA label Feb 6, 2025
Copy link
Owner

Choose a reason for hiding this comment

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

looks like this should be:

./bin/app/armadactl submit ../spark-armada/examples/spark-pi-driver.yaml

@EnricoMi EnricoMi force-pushed the georgejahad/addingArmadaClient branch from 36ecc55 to 84ce5da Compare February 6, 2025 21:48
@GeorgeJahad
Copy link
Owner

Looks great @EnricoMi

I'll merge as soon as the tests pass

@EnricoMi EnricoMi force-pushed the georgejahad/addingArmadaClient branch from 2ae2a51 to d6fe281 Compare February 7, 2025 19:11
@GeorgeJahad
Copy link
Owner

Hey @EnricoMi

@ClifHouck merged this branch here:
G-Research#38

So maybe you should rebase this against his branch.

@EnricoMi EnricoMi closed this Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants