Skip to content

Conversation

mironec
Copy link
Contributor

@mironec mironec commented Nov 12, 2019

Enables multiple engine versions to be tested simultaneously in CI, by reusing the GDK's unreal-engine.version file.

Engine PR to facilitate this: https://github.com/improbableio/UnrealEngine/pull/289

Reviewers: @ImprobableVlad @joshuahuburn

@mironec mironec self-assigned this Nov 12, 2019
@improbable-prow-robot
Copy link

Corresponding JIRA ticket: https://improbableio.atlassian.net/browse/GV-115

@improbable-prow-robot improbable-prow-robot added jira/GV size/L Denotes a PR that changes 150-299 lines, ignoring generated files. labels Nov 12, 2019
$deployment_url = "https://console.improbable.io/projects/${project_name}/deployments/${deployment_name}/overview"
$deployment_button = @{
type = "button"
text = ":cloud: Deployment $($i+1)"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a way we can name the deployment after the engine version? Not essential.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tried to, but hit the deployment name limit, I guess we could just change the entire format so it no longer includes the full timestamp or the gdk commit or something, but I thought it could be annoying to do.

Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps you could put it in the deployment description? I've never used it, but apparently there is a --deployment_description option when launching a deployment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Seems good, I can try that

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added this, but I can't find the description anywhere in the console :D

Copy link
Contributor

@joshuahuburn joshuahuburn left a comment

Choose a reason for hiding this comment

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

Nice job! Couple questions.


# Download the unreal-engine.version file from the GDK repo so we can run the example project builds on the same versions the GDK was run against
GDK_BRANCH_LOCAL="${GDK_BRANCH:-master}"
curl https://raw.githubusercontent.com/spatialos/UnrealGDK/$GDK_BRANCH_LOCAL/ci/unreal-engine.version -o ci/unreal-engine.version
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This seems brittle, but I tried git-ing a single file and it was also annoying.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed that we should try to change this, I will try running some experiments on how we could replace this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Didn't find a good replacement, but made this more robust in exchange

Copy link
Contributor

@ImprobableVlad ImprobableVlad 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 to me!

$deployment_url = "https://console.improbable.io/projects/${project_name}/deployments/${deployment_name}/overview"
$deployment_button = @{
type = "button"
text = ":cloud: Deployment $($i+1)"
Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps you could put it in the deployment description? I've never used it, but apparently there is a --deployment_description option when launching a deployment.

@mironec mironec merged commit 19076f0 into 0.8.0-preview-rc Nov 18, 2019
@mironec mironec deleted the feature/ci-multi-engine branch November 18, 2019 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jira/GV size/L Denotes a PR that changes 150-299 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants