Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Changes to introduce task server #7252

Merged
merged 8 commits into from
Nov 29, 2022
Merged

Changes to introduce task server #7252

merged 8 commits into from
Nov 29, 2022

Conversation

hanzlamateen
Copy link
Member

@hanzlamateen hanzlamateen commented Nov 23, 2022

Summary

This PR changes analytics server with task server. Following are some

  • env.local file needs to have TASKSERVER_PORT & TASKSERVER_PROCESS_INTERVAL instead of ANALYTICS_PORT & ANALYTICS_PROCESS_INTERVAL
  • In database analyticsSetting has been renamed to taskServerSetting. This needs to be reflected once PR is approved.
  • New npm package will be @xrengine/taskserver instead of @xrengine/analytics
  • The docker image in container registry needs to be created for each env's xrengine-taskserver.
  • Projects dependant on analytics server should now target task server. This PR needs to be merged with: https://github.com/TheOverlay/eepro-bridge-bot/pull/22
  • Docs updates for task server also needs to be merged: Docs changes for task server etherealengine-docs#2
  • With these changes, we will need to publish new release of helm charts.
  • Also need to make sure that [env].values.yaml file needs to have changes as shown in below image.
    image

References

closes #7056

Checklist

  • If this PR is still a WIP, convert to a draft
  • ensure all checks pass
  • When this PR is ready, mark it as "Ready for review"
  • Changes have been manually QA'd
  • Changes reviewed by at least 2 approved reviewer

QA Steps

List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.

@hanzlamateen hanzlamateen marked this pull request as ready for review November 23, 2022 03:04
@barankyle barankyle merged commit f070d82 into dev Nov 29, 2022
@barankyle barankyle deleted the task-server branch November 29, 2022 21:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Story] Rename Analytics Server to Task Server
3 participants