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: [TKC-2882] stream service and parallel step logs #6052

Merged
merged 47 commits into from
Dec 9, 2024
Merged
Changes from 1 commit
Commits
Show all changes
47 commits
Select commit Hold shift + click to select a range
bd39964
feat: api methods for service logs
vsukhin Nov 25, 2024
2c77962
fix: client for get service logs
vsukhin Nov 25, 2024
c71f5c8
fix: change log
vsukhin Nov 25, 2024
3e70a83
fix: disable hints
vsukhin Nov 25, 2024
aa9acb3
fix: routing
vsukhin Nov 26, 2024
68fe09f
fix: show service logs
vsukhin Nov 26, 2024
c55187f
fix: check service name
vsukhin Nov 27, 2024
93b7ac6
fix: check service name
vsukhin Nov 27, 2024
cf2dd76
fix: log comment
vsukhin Nov 27, 2024
fff63c0
fix: check for testworkflow service
vsukhin Nov 27, 2024
c7bed19
fix: friendly error
vsukhin Nov 27, 2024
f34e5d1
fix: add spinner
vsukhin Nov 27, 2024
07ce308
feat: proto for service notifications
vsukhin Nov 27, 2024
9c03fe4
feat: add cloud grpc method for server notifications
vsukhin Nov 27, 2024
9d85b35
fix: change timeeout
vsukhin Nov 28, 2024
57db7a0
fix: waiting for service pod
vsukhin Nov 28, 2024
cccfcb7
fix: typo
vsukhin Nov 28, 2024
c4658c5
fix: waiting for service pod
vsukhin Nov 28, 2024
3768550
fix: add service name check
vsukhin Nov 28, 2024
c2fd448
fix: adjust help
vsukhin Nov 29, 2024
b46bb41
fix: add method to parallel step
vsukhin Nov 29, 2024
59bdce3
fix: use retry library
vsukhin Nov 29, 2024
4224930
fix: rename const
vsukhin Nov 29, 2024
bcda47f
fix: 0 attempts
vsukhin Nov 29, 2024
97b5bd3
fix: use option
vsukhin Nov 29, 2024
3e2c45f
fix: remove ctx
vsukhin Nov 29, 2024
2819e43
feat: add cli support for parallel steps
vsukhin Dec 2, 2024
81b0295
fix: rename url
vsukhin Dec 2, 2024
c71fb91
feat: api methods for parallel steps
vsukhin Dec 2, 2024
6003cde
fix: tune parallel step detection
vsukhin Dec 2, 2024
5136c31
fix: typo
vsukhin Dec 2, 2024
6b89ab0
fix: cli
vsukhin Dec 2, 2024
bcffbd5
feat: add proto for parallel step logs
vsukhin Dec 2, 2024
bfdc72f
fix: lint
vsukhin Dec 3, 2024
ce5215f
add: grpc method for parallel steps
vsukhin Dec 3, 2024
fdffbc9
fix: comments
vsukhin Dec 3, 2024
38379b4
fix: comments
vsukhin Dec 3, 2024
9d4c5fa
Merge branch 'main' into vsukhin/feature/stream-service-logs
vsukhin Dec 3, 2024
8a7f8cb
fix: typo
vsukhin Dec 3, 2024
097fb7d
fix: change proto
vsukhin Dec 3, 2024
75d7f88
fix: rename fields
vsukhin Dec 3, 2024
f37671e
Merge branch 'main' into vsukhin/feature/stream-service-logs
vsukhin Dec 3, 2024
cd1be6e
fix: use const
vsukhin Dec 4, 2024
38f959c
fix: check for empty result
vsukhin Dec 4, 2024
288be72
fix: move methods to agent package
vsukhin Dec 9, 2024
e56e548
fix: send org and env ids
vsukhin Dec 9, 2024
e899798
fix: add org and env ids
vsukhin Dec 9, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: comments
Signed-off-by: Vladislav Sukhin <[email protected]>
vsukhin committed Dec 3, 2024
commit 38379b42652f9ef03870127a2ef8ced081901d3b
2 changes: 1 addition & 1 deletion pkg/agent/testworkflows.go
Original file line number Diff line number Diff line change
@@ -397,7 +397,7 @@ func (ag *Agent) executeWorkflowParallelStepNotificationsRequest(ctx context.Con
)

if err != nil {
message := fmt.Sprintf("cannot get service pod logs: %s", err.Error())
message := fmt.Sprintf("cannot get parallel step pod logs: %s", err.Error())
ag.testWorkflowParallelStepNotificationsResponseBuffer <- &cloud.TestWorkflowParallelStepNotificationsResponse{
StreamId: req.StreamId,
SeqNo: 0,

Unchanged files with check annotations Beta

# Step 7: Example K6 Test Workflow CRD and preload Kind images
COPY ./images /images
ARG segmentio_key

Check warning on line 30 in build/kind/kind.Dockerfile

GitHub Actions / Build a Docker image for Testkube Agent

Sensitive data should not be used in the ARG or ENV commands

SecretsUsedInArgOrEnv: Do not use ARG or ENV instructions for sensitive data (ARG "segmentio_key") More info: https://docs.docker.com/go/dockerfile/rule/secrets-used-in-arg-or-env/
ENV SEGMENTIO_KEY=$segmentio_key

Check warning on line 31 in build/kind/kind.Dockerfile

GitHub Actions / Build a Docker image for Testkube Agent

Sensitive data should not be used in the ARG or ENV commands

SecretsUsedInArgOrEnv: Do not use ARG or ENV instructions for sensitive data (ENV "SEGMENTIO_KEY") More info: https://docs.docker.com/go/dockerfile/rule/secrets-used-in-arg-or-env/
ARG ga_id
ENV GA_ID=$ga_id
ARG ga_secret

Check warning on line 34 in build/kind/kind.Dockerfile

GitHub Actions / Build a Docker image for Testkube Agent

Sensitive data should not be used in the ARG or ENV commands

SecretsUsedInArgOrEnv: Do not use ARG or ENV instructions for sensitive data (ARG "ga_secret") More info: https://docs.docker.com/go/dockerfile/rule/secrets-used-in-arg-or-env/
ENV GA_SECRET=$ga_secret

Check warning on line 35 in build/kind/kind.Dockerfile

GitHub Actions / Build a Docker image for Testkube Agent

Sensitive data should not be used in the ARG or ENV commands

SecretsUsedInArgOrEnv: Do not use ARG or ENV instructions for sensitive data (ENV "GA_SECRET") More info: https://docs.docker.com/go/dockerfile/rule/secrets-used-in-arg-or-env/
ARG docker_image_version
ENV DOCKER_IMAGE_VERSION=$docker_image_version
ARG cloud_url