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

merge main #1

Merged
merged 9 commits into from
Oct 20, 2024
Merged

merge main #1

merged 9 commits into from
Oct 20, 2024

Conversation

BenElferink
Copy link
Owner

No description provided.

blumamir and others added 9 commits October 17, 2024 17:07
Changes:

- Add `odigos describe` command to describe the general status of odigos
itself. Currently implemented Cluster Collector Information
- Add conditions to collectors group status to record if the collectors
deployment failed to succeed. For example - this can happen in the
present of validation webhook that rejects the deployment if the image
is not signed according to some requirements
- move some code from `cli` to `k8sutils` to make it available for all
modules in the code.
- some cleanups on logging, so if the deployment fails to succeed, it is
only printed once to autoscaler logs
Updates for the qryn opensource and qryn/gigapipe integrations

---------

Co-authored-by: akvlad <akvlad90@gmail.com>
Co-authored-by: Amir Blum <amirgiraffe@gmail.com>
This PR:

- record the status of autoscaler node collector info into the collector
group condition to make it available for display.
- enhance the `odigos describe` command to include info about the node
collector deployment:


![image](https://github.com/user-attachments/assets/9d45ebb0-8a3d-44c7-a3da-de3daaacb167)
- docs - add `gigapipe` destination to `mint.json` for it to show up in
docs.
- ui - fix the destination category so it shows up in the UI with the
other destinations as self hosted.
- ui - make the required qryn URL field first in the list, so it shows
up first in ui
…1593)

Co-authored-by: Tamir David <tamirdavid@Tamirs-MacBook-Pro.local>
Co-authored-by: Amir Blum <amirgiraffe@gmail.com>
…1602)

This PR moves the scheduler component to use `instrumentationconfigs`
instead of `instrumentedapplications`. There is an effort to merge them
any only use one CRD - the `instrumentationconfigs`.

The content of the object is not examined, it only checks if we have any
`instrumentationconfigs` or not. Thus it should be safe.
Update the architecture section in odigos docs:

- Document each odigos component
- Move the page to be under "Architecture" section instead of "Concepts"
- Make sure the text is up to date and reflects the current status of
odigos

---------

Co-authored-by: Daria Dovzhikova <dovzhikova@gmail.com>
Co-authored-by: Arirec <125489097+Arirec@users.noreply.github.com>
@BenElferink BenElferink merged commit c25c631 into BenElferink:main Oct 20, 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