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

Review service container aliases and hostname #1850

Merged
merged 15 commits into from
May 28, 2024

Conversation

mrodm
Copy link
Contributor

@mrodm mrodm commented May 21, 2024

Ensure that communication between containers can be established by adding alias when connecting the service container to the Elastic Agent network.

Alias follow this pattern svc-<service_name>, where service_name is the service name defined in the compose project (_dev/deploy/docker/docker-compose.yml) or in the test configuration file.

All compose project names now will have a random suffix (runID). For isntance: elastic-package-service-12345. This would also help us to move forward running tests in parallel.

@mrodm mrodm self-assigned this May 21, 2024
@mrodm
Copy link
Contributor Author

mrodm commented May 21, 2024

test integrations

@elasticmachine
Copy link
Collaborator

Created or updated PR in integrations repository to test this version. Check elastic/integrations#9930

@mrodm
Copy link
Contributor Author

mrodm commented May 21, 2024

test integrations

@elasticmachine
Copy link
Collaborator

Created or updated PR in integrations repository to test this version. Check elastic/integrations#9930

@mrodm
Copy link
Contributor Author

mrodm commented May 21, 2024

test integrations

@elasticmachine
Copy link
Collaborator

Created or updated PR in integrations repository to test this version. Check elastic/integrations#9930

@mrodm
Copy link
Contributor Author

mrodm commented May 22, 2024

test integrations

@elasticmachine
Copy link
Collaborator

Created or updated PR in integrations repository to test this version. Check elastic/integrations#9930

@mrodm
Copy link
Contributor Author

mrodm commented May 22, 2024

test integrations

@elasticmachine
Copy link
Collaborator

Created or updated PR in integrations repository to test this version. Check elastic/integrations#9930

@mrodm
Copy link
Contributor Author

mrodm commented May 23, 2024

test integrations

@elasticmachine
Copy link
Collaborator

Created or updated PR in integrations repository to test this version. Check elastic/integrations#9930

@mrodm
Copy link
Contributor Author

mrodm commented May 23, 2024

test integrations

@elasticmachine
Copy link
Collaborator

Created or updated PR in integrations repository to test this version. Check elastic/integrations#9930

@mrodm
Copy link
Contributor Author

mrodm commented May 23, 2024

test integrations

@elasticmachine
Copy link
Collaborator

Created or updated PR in integrations repository to test this version. Check elastic/integrations#9930

Comment on lines 14 to 18
for i in $(seq "$NUMBER_SUCCESSES"); do
echo "Iter $i"
healthcheck
sleep 1
sleep "$WAITING_TIME"
done
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Create a separate PR for these changes related to Fleet Server healthcheck #1867

@mrodm mrodm force-pushed the set-service-hostname-container-name branch from aafd7b0 to 302731c Compare May 27, 2024 09:01
@mrodm
Copy link
Contributor Author

mrodm commented May 27, 2024

test integrations

@elasticmachine
Copy link
Collaborator

Created or updated PR in integrations repository to test this version. Check elastic/integrations#9930

@mrodm mrodm marked this pull request as ready for review May 27, 2024 13:41
@mrodm mrodm requested a review from a team May 27, 2024 15:21
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @mrodm

@mrodm mrodm merged commit 3ee5da5 into elastic:main May 28, 2024
3 checks passed
@mrodm mrodm deleted the set-service-hostname-container-name branch May 28, 2024 06:56
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.

3 participants