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.

Sign up for GitHub

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

Jump to bottom

Compose add all instances in a service group #1671

Closed
razvand opened this issue May 23, 2024 · 1 comment
Closed

Compose add all instances in a service group #1671

razvand opened this issue May 23, 2024 · 1 comment
Assignees
@nderjung
Labels
kind/bug Something isn't working

Comments

@razvand
Copy link
Contributor

razvand commented May 23, 2024

Describe the bug

When using Compose, via kraft cloud compose up, a service group is created and all instances are added to that service group. That means both the service-providing instance and backing instances, such as databases or others are added to the service group.

However, when creating a service group, it load balances (round robins) between instances. Which means it will query both service-providing instances and backing instances.

Only the service-providing instance should be added to the service group.

Steps to reproduce

Use this PR and run:

kraft cloud console up
curl https://python.dal0.kraft.host
curl https://python.dal0.kraft.host
curl https://python.dal0.kraft.host
curl https://python.dal0.kraft.host

Some runs will show an error message:

<html>
<head><title>502 Bad Gateway</title></head>
<body>
<center><h1>502 Bad Gateway</h1></center>
<hr><center>openresty</center>
</body>
</html>

Expected behavior

Only the service-providing instances should be part of the service group, allowing to be queried.

Which architectures were you using or does this bug affect?

No response

Which operating system were you using or does this bug affect?

No response

Relevant log output

No response
@razvand razvand added the kind/bug Something isn't working label May 23, 2024
@craciunoiuc
Copy link
Member

Fixed in #1678

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nderjung nderjung

Labels
kind/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@razvand @nderjung @craciunoiuc