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

Feature/lb collector #292

Merged
merged 20 commits into from
Jul 21, 2023
Merged

Conversation

heidermassu
Copy link
Contributor

@heidermassu heidermassu commented Jul 19, 2023

This PR is resolving issue #272

@github-actions
Copy link

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

@ghost ghost self-requested a review July 21, 2023 13:23
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

This looks good to me.
I tested all scripts and they're working OK. Nothing is breaking.
There are a few minor warnings here and there but that is not limiting functionality in any way.
I updated readme with additional prerequisite information.

@heidermassu heidermassu merged commit c501629 into feature/otlp-blueprint-272 Jul 21, 2023
2 of 3 checks passed
@heidermassu heidermassu deleted the feature/lb-collector branch July 21, 2023 14:24
ghost pushed a commit that referenced this pull request Jul 21, 2023
* feat: #272 starting manifest k8s building

* feat: #272 collector configuration

* feat: #272 collector configuration

* feat: #272 collector configuration

* feat: #272 collector configuration

* feat: #272 collector configuration

* feat: #272 collector config

* feat: #272 collector config

* feat: #272 collector config / changing version packge 0.40.0

* Update collector-works.yaml

* Update backend.yaml

* chore: from K8s to k8s

* Start the k8s README

* chore: cleanup

* feat: 272 - missing alteration. Jaeger structure; frontend env; prometheus target

* feat: 272 - removing RUN mv .env .env of Backend.dockerfile

* feat: #283 Public docker registry

* feat: #283 Public docker registry

* feat: #283 Public docker registry - fixing variable name and value

* readme

* feat: #283 - adding makefile k8s stufs; variable for namespace

* feat: #283 - adding makefile k8s stufs; variable for namespace

* Kind support (#284)

* chore: strt kind provisioning

* chore: testing kind stuff

* chore: makefile bootstrap

* chore: kind support

* chore: some README

* chore: some README

* Update README.md

* teste1

* Nginx

* chore: update README

* K8s deployment test (#290)

* internal route

* It works (#291)

* Merge kind makefile

* Update yml extension

---------

Co-authored-by: Heider Leandro Massu Hengstmann <[email protected]>

* chore: docs

* chore: docs

* chore: docs

* Feature/lb collector (#292)

* lb-collector

* lb-collector

* lb-collector

* lb-collector

* lb-collector

* container-registry

* updating Makefile new condictions; update readme

* update readme

* check whether the resources are ready or not

* frontend arguments as env

* frontend arguments as env

* portforward

* fixing issues related test demo

* permission chmod -R +x ./

* otel-operator and cert-manager install - fix time

* updating readme

* removing forward of make default

* update docs on prerequisites

* add execute permissions to the bash scripts

---------

Co-authored-by: Heider Leandro Massu Hengstmann <[email protected]>
Co-authored-by: Kenan Dervisevic <[email protected]>

---------

Co-authored-by: Heider Leandro Massu Hengstmann <[email protected]>
Co-authored-by: Heider Leandro Massu Hengstmann <[email protected]>
Co-authored-by: Kenan Dervisevic <[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.

2 participants