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

feat: Terraform CI #99

Merged
merged 45 commits into from
Jan 31, 2023
Merged

feat: Terraform CI #99

merged 45 commits into from
Jan 31, 2023

Conversation

foo0x29a
Copy link
Contributor

@foo0x29a foo0x29a commented Dec 22, 2022

Description

Adds new CI pipeline to:

  • Create a Kubernetes cluster
  • Deploy TCE nodes
  • Export telemetry data
  • Clean up the resources created

Screenshot 2023-01-20 at 11 48 03 AM

Screenshot 2023-01-20 at 11 48 43 AM

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@foo0x29a foo0x29a force-pushed the feat/terraform-ci branch 8 times, most recently from 3b37bf0 to 878de7b Compare December 22, 2022 17:19
@foo0x29a foo0x29a self-assigned this Dec 22, 2022
@foo0x29a foo0x29a changed the title feat: Terraform CI TOO-199: Terraform CI Dec 22, 2022
@foo0x29a foo0x29a changed the title TOO-199: Terraform CI feat: Terraform CI Dec 30, 2022
@foo0x29a foo0x29a force-pushed the feat/terraform-ci branch 2 times, most recently from 04c976b to a41ca94 Compare January 3, 2023 16:52
@foo0x29a foo0x29a force-pushed the feat/terraform-ci branch 11 times, most recently from fe81929 to 676afc5 Compare January 20, 2023 01:42
@codecov
Copy link

codecov bot commented Jan 23, 2023

Codecov Report

Base: 0.00% // Head: 0.00% // No change to project coverage 👍

Coverage data is based on head (5d81401) compared to base (6653ce8).
Patch has no changes to coverable lines.

❗ Current head 5d81401 differs from pull request most recent head 0774696. Consider uploading reports for the commit 0774696 to get more accurate results

Additional details and impacted files
@@    Coverage Diff     @@
##   main   #99   +/-   ##
==========================
==========================

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@foo0x29a foo0x29a force-pushed the feat/terraform-ci branch 3 times, most recently from 89ac441 to 3b6dc3b Compare January 24, 2023 01:52
@foo0x29a foo0x29a requested review from Freyskeyd, hadjiszs and sebastiendan and removed request for a team and sebastiendan January 24, 2023 18:53
@foo0x29a
Copy link
Contributor Author

@toposware/protocol This PR is mostly for @toposware/tools to review.
However, it would be great if you could take a look on both Cargo.toml and Dockerfile.

I made two small changes to enable gRPC over TLS.

Thanks in advance!

tools/docker-compose.yml Outdated Show resolved Hide resolved
tools/docker-compose.yml Outdated Show resolved Hide resolved
tools/liveness.sh Outdated Show resolved Hide resolved
Copy link
Member

@hadjiszs hadjiszs left a comment

Choose a reason for hiding this comment

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

@protocol This PR is mostly for @tools to review.
However, it would be great if you could take a look on both Cargo.toml and Dockerfile.

Approving for those 👍

.gitignore Show resolved Hide resolved
Copy link

@dpasirst dpasirst left a comment

Choose a reason for hiding this comment

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

see included comment regarding .gitignore.

@foo0x29a foo0x29a mentioned this pull request Jan 31, 2023
4 tasks
* chore: bump tce-all-in-one version

* fix: allow multiple jobs run in parallel

* fix: remove liveness probe

* fix: condition to run CI
@foo0x29a foo0x29a merged commit 370fc08 into main Jan 31, 2023
@foo0x29a foo0x29a deleted the feat/terraform-ci branch January 31, 2023 15:29
hadjiszs pushed a commit that referenced this pull request Mar 9, 2023
feat: add workflow to: 
- create a Kubernetes cluster
- deploy TCE nodes
- export telemetry data
- clean up the resources created

---------

Signed-off-by: Thiago Nobayashi <[email protected]>
hadjiszs pushed a commit that referenced this pull request Mar 9, 2023
feat: add workflow to: 
- create a Kubernetes cluster
- deploy TCE nodes
- export telemetry data
- clean up the resources created

---------

Signed-off-by: Thiago Nobayashi <[email protected]>
hadjiszs pushed a commit that referenced this pull request Mar 9, 2023
feat: add workflow to: 
- create a Kubernetes cluster
- deploy TCE nodes
- export telemetry data
- clean up the resources created

---------

Signed-off-by: Thiago Nobayashi <[email protected]>
hadjiszs pushed a commit that referenced this pull request Mar 9, 2023
feat: add workflow to: 
- create a Kubernetes cluster
- deploy TCE nodes
- export telemetry data
- clean up the resources created

---------

Signed-off-by: Thiago Nobayashi <[email protected]>
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