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

[WIP] Add minio log storage for tekton-results. #371

Closed

Conversation

AndrienkoAleksandr
Copy link
Contributor

@AndrienkoAleksandr AndrienkoAleksandr commented Nov 14, 2022

Add minio log storage for tekton-results for development instances.

Demo: https://youtu.be/MHPFu7CnKvw

Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
Signed-off-by: Oleksandr Andriienko <[email protected]>
@AndrienkoAleksandr AndrienkoAleksandr marked this pull request as draft November 14, 2022 12:31
Copy link
Contributor

@adambkaplan adambkaplan left a comment

Choose a reason for hiding this comment

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

An initial pass - good to showcase the overlays for tekton results.

developer/ckcp/config.yaml Outdated Show resolved Hide resolved
operator/gitops/argocd/argo-apps/tekton-chains.yaml Outdated Show resolved Hide resolved
@AndrienkoAleksandr AndrienkoAleksandr marked this pull request as ready for review November 28, 2022 00:00
@AndrienkoAleksandr
Copy link
Contributor Author

@adambkaplan pr is ready. But first should be merged openshift-pipelines/tektoncd-results#18 and ideally openshift-pipelines/tektoncd-results#19 too. After that I need to use newer images for tekton results deployments.

@adambkaplan
Copy link
Contributor

@AndrienkoAleksandr given our transition away from kcp, I think we should hold off on merging things new things into this repo until the infra team completes the refactor.

@Roming22 Roming22 changed the title Add minio log storage for tekton-results. [WIP] Add minio log storage for tekton-results. Dec 5, 2022
@Roming22 Roming22 marked this pull request as draft December 5, 2022 19:06
Roming22 pushed a commit that referenced this pull request Dec 19, 2022
Automatic update

Co-authored-by: auto-update <>
Co-authored-by: Michal Kovarik <[email protected]>
@AndrienkoAleksandr
Copy link
Contributor Author

Closed on favour #419

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