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

current linking is deprecated #88

Open
unixbird opened this issue Nov 19, 2020 · 2 comments
Open

current linking is deprecated #88

unixbird opened this issue Nov 19, 2020 · 2 comments

Comments

@unixbird
Copy link

The way the current image needs --link is deprecated according to docker as referenced https://docs.docker.com/network/links/. It says to use the same network space for it but other than docker compose how would you force osticket to connect to the db upon creation?

@bobweston
Copy link

bobweston commented Aug 2, 2022

I am not a contributor to this repo. But when starting the OSTicket container, you should use the link flag

--link osticket_mysql:mysql

So basically, start the osticket_mysql container. Then start the osticket container using the flag above

@unixbird
Copy link
Author

unixbird commented Aug 3, 2022

I am not a contributor to this repo. But when starting the OSTicket container, you should use the link flag

--link osticket_mysql:mysql

So basically, start the osticket_mysql container. Then start the osticket container using the flag above

So this question was originally asked when I was inexperienced with docker and containers in general and I read that the --link flag was becoming deprecated.

I eventually found that compose/swarm automagically does this when its already in the same network and that the --link really is semi redundant when you're telling the osticket container the creds to actually connect to the database via environmental variables.

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

No branches or pull requests

2 participants