Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Confs to envs #31

Merged
merged 10 commits into from Oct 21, 2021
Merged

Confs to envs #31

merged 10 commits into from Oct 21, 2021

Conversation

zalbiraw
Copy link
Member

  • Update Tyk products versions
  • Remove depends_on from docker-compose
  • Replace config files with environment variables

@zalbiraw
Copy link
Member Author

Still need to update the swarm docker compose files

@marksou
Copy link
Contributor

marksou commented Oct 18, 2021

@zalbiraw Let me know when I can test this as I have an Apple silicon Mac, so can test the Arm containers 😀

confs/pump.env Show resolved Hide resolved
@sedkis
Copy link
Contributor

sedkis commented Oct 21, 2021

Still need to update the swarm docker compose files

I think we can get rid of swarm completely. Just keep docker-compose.

Swarm is more for benchmarking I suppose, I doubt anyone is using them. Thoughts?

@marksou
Copy link
Contributor

marksou commented Oct 21, 2021

As we make it clear that it shouldn't be used for performance testing, we should probably remove it.

@sedkis sedkis merged commit e63029b into master Oct 21, 2021
@sedkis sedkis deleted the confs-to-envs branch October 21, 2021 19:54
TYK_DB_UI_DONTALLOWLICENSEMANAGEMENT=false
TYK_DB_UI_DONTALLOWLICENSEMANAGEMENTVIEW=false
TYK_DB_HOMEDIR=/opt/tyk-dashboard
TYK_DB_USESHARDEDANLAYTICS=false
Copy link
Contributor

@armujahid armujahid Dec 2, 2022

Choose a reason for hiding this comment

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

There's a typo in TYK_DB_USESHARDEDANALYTICS (ANLAYTICS should be ANALYTICS)

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.

None yet

4 participants