Skip to content

Conversation

@robot9706
Copy link
Contributor

  • CLI adds org.dyrectorio.service-category=_internal labels to all platform containers.
  • Added labels to agent container proto.
  • Hide containers if their service category starts with an underscore or their kubernetes namespace is kube-system.

@robot9706 robot9706 requested a review from a team as a code owner September 1, 2023 13:39
@robot9706 robot9706 self-assigned this Sep 1, 2023
@github-actions github-actions bot added pr:feat source:web The scope of the issue or pull request is web. source:cli The scope of the issue or pull request is CLI tool. lang:golang lang:typescript source:proto Protobuf related changes. labels Sep 1, 2023
@codecov-commenter
Copy link

codecov-commenter commented Sep 1, 2023

Codecov Report

Patch coverage: 20.00% and no project coverage change.

Comparison is base (81aee61) 24.13% compared to head (a1ab545) 24.13%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #807   +/-   ##
========================================
  Coverage    24.13%   24.13%           
========================================
  Files           62       62           
  Lines         5954     5958    +4     
========================================
+ Hits          1437     1438    +1     
- Misses        4411     4414    +3     
  Partials       106      106           
Files Changed Coverage Δ
golang/internal/label/label.go 0.00% <0.00%> (ø)
golang/internal/mapper/grpc.go 64.57% <33.33%> (-0.06%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@polaroi8d
Copy link
Collaborator

Please add new labels to docker compose and CLI containers.

Copy link
Contributor

@nandor-magyar nandor-magyar left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Collaborator

@polaroi8d polaroi8d left a comment

Choose a reason for hiding this comment

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

TypeError: Cannot read properties of undefined (reading 'org.dyrectorio.service-category')

Copy link
Collaborator

@polaroi8d polaroi8d left a comment

Choose a reason for hiding this comment

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

Works locally.

@polaroi8d polaroi8d merged commit da12e4c into develop Sep 5, 2023
@polaroi8d polaroi8d deleted the feat/label-service-category branch September 5, 2023 12:17
chandhuDev pushed a commit to chandhuDev/dyrectorio that referenced this pull request Nov 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang:golang lang:typescript pr:feat severity:medium source:cli The scope of the issue or pull request is CLI tool. source:proto Protobuf related changes. source:web The scope of the issue or pull request is web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants