Skip to content

Conversation

johnprobable
Copy link
Contributor

@johnprobable johnprobable commented May 23, 2019

The deployment manager worker does not have the correct authority set, making it not update the deployment tags properly for the deployment-manager tutorial.

Also, the gamemode only gives about a second to allow players to join a session before the session is made unavailable to new players, and session time is a bit too long.

These changes will fix the authority of the deployment manager worker, change lobby time to half a minute, and session time to 10 minutes.

"value": "0"
},
"dimensions": {
"xMeters": 300,
Copy link
Contributor

Choose a reason for hiding this comment

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

We should keep the dimensions the same as previous config

Copy link
Contributor Author

@johnprobable johnprobable May 23, 2019

Choose a reason for hiding this comment

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

reverted dimension changes

},
"chunk_edge_length_meters": 50,
"streaming_query_interval": 4,
"legacy_flags": [
Copy link
Contributor

Choose a reason for hiding this comment

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

Worth checking if any of these legacy flags are required @m-samiec

Copy link

@Vatyx Vatyx May 23, 2019

Choose a reason for hiding this comment

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

The bridge_soft_handover_enabled flag needs to be there I'm pretty sure.
Also how come I don't see the enable_chunk_interest flag? It needs to be disabled since QBI is enabled

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will add those two in, though it was definitely working without them. Perhaps they default to false if not present?

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

@johnprobable johnprobable requested a review from m-samiec May 23, 2019 14:49
@m-samiec m-samiec merged commit 2698ffc into release May 24, 2019
@m-samiec m-samiec deleted the bugfix/deploymentmanager branch May 24, 2019 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants