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
enhancement(dnstap source): Make sure ITs are independent of container #8347
enhancement(dnstap source): Make sure ITs are independent of container #8347
Conversation
…endent of container Signed-off-by: Yunkun Zhu <yzhu@bluecatnetworks.com>
✔️ Deploy Preview for vector-project ready! 🔨 Explore the source changes: 2720790 🔍 Inspect the deploy log: https://app.netlify.com/sites/vector-project/deploys/60f1c2741f348a00071b9029 😎 Browse the preview: https://deploy-preview-8347--vector-project.netlify.app |
Hi @jszwedko, this changeset is to make sure dnstap source ITs are not dependent on the container in regards to #8302 (comment). Thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aha, great, thank you @yunkunzhu !
I tested this out locally and it looks good. |
…endent of container (#8347) Signed-off-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Co-authored-by: Yunkun Zhu <yzhu@bluecatnetworks.com>
…endent of container (#8347) Signed-off-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Co-authored-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Signed-off-by: Luc Perkins <luc.perkins@datadoghq.com>
…endent of container (#8347) Signed-off-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Co-authored-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Signed-off-by: Luc Perkins <luc.perkins@datadoghq.com>
…endent of container (#8347) Signed-off-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Co-authored-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Signed-off-by: Luc Perkins <luc.perkins@datadoghq.com>
…endent of container (#8347) Signed-off-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Co-authored-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Signed-off-by: Luc Perkins <luc.perkins@datadoghq.com>
…endent of container (#8347) Signed-off-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Co-authored-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Signed-off-by: Luc Perkins <luc.perkins@datadoghq.com>
…endent of container (#8347) Signed-off-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Co-authored-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Signed-off-by: Luc Perkins <luc.perkins@datadoghq.com>
…endent of container (#8347) Signed-off-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Co-authored-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Signed-off-by: Luc Perkins <luc.perkins@datadoghq.com>
…endent of container (vectordotdev#8347) Signed-off-by: Yunkun Zhu <yzhu@bluecatnetworks.com> Co-authored-by: Yunkun Zhu <yzhu@bluecatnetworks.com>
Signed-off-by: Yunkun Zhu yzhu@bluecatnetworks.com
This enhancement is based on the comment #8302 (comment). It makes sure to do proper cleanup in the end of each IT test case and let IT test cases not dependent on the container.