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

basic test for service discovery #44

Merged
merged 15 commits into from
Oct 31, 2023
Merged

Conversation

przsus
Copy link
Contributor

@przsus przsus commented Oct 30, 2023

No description provided.

@jakubrak
Copy link
Contributor

Please add the tests to GH Actions

testing/tests/docker-compose.yml Outdated Show resolved Hide resolved
testing/conftest.py Show resolved Hide resolved
testing/utils.py Outdated Show resolved Hide resolved
testing/utils.py Outdated Show resolved Hide resolved
testing/utils.py Outdated Show resolved Hide resolved
testing/utils.py Outdated Show resolved Hide resolved
testing/utils.py Show resolved Hide resolved
testing/utils.py Show resolved Hide resolved
testing/utils.py Outdated Show resolved Hide resolved
testing/conftest.py Outdated Show resolved Hide resolved
@przsus przsus force-pushed the create-basic-test-for-ebpf-discovery branch from 9ac5919 to 37851e4 Compare October 31, 2023 11:51
@przsus
Copy link
Contributor Author

przsus commented Oct 31, 2023

Please add the tests to GH Actions

Added a new task for it, since it's not trivial. That is because we can't run discovery inside a github container. Discussed with @hparzych

@bartekmp
Copy link

LGTM

@przsus przsus merged commit 8fcf335 into main Oct 31, 2023
2 checks passed
@jakubrak jakubrak deleted the create-basic-test-for-ebpf-discovery branch November 10, 2023 10:22
hparzych pushed a commit that referenced this pull request Apr 29, 2024
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.

5 participants