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

ODC-7619: Add tekton results utils to aonic library #30

Merged

Conversation

lokanandaprabhu
Copy link
Contributor

Story:

https://issues.redhat.com/browse/ODC-7619


Description:

Adding tekton results utils to aonic library and later we will use this in Openshit-console dynamic plugin.

Hooks:
useRuns - Fetch data from both k8s and tekton results
useTektonResultsRuns - Fetch data from tekton results
usePipelineRuns - Fetch PipelineRunsfrom both k8s and tekton results
useTRPipelineRuns - Fetch PipelineRuns from tekton results
useTaskRuns - Fetch TaskRuns from both k8s and tekton results
useTRTaskRuns - Fetch TaskRuns from tekton results
useTRTaskRunLog - Fetch tekton results TaskRuns logs
usePipelineRunsForPipelineOrRepository - Fetch PipelineRuns for Pipeline or Repository from both k8s and tekton results
useTaskRunsForPipelineRunOrTask - Fetch TaskRuns for PipelineRun or Task from both k8s and tekton results

Utils:
createTektonResultsUrl - Create tekton results URL


Build and lint:

Screenshot 2024-06-18 at 3 17 22 PM

Tests:

Screenshot 2024-06-18 at 3 16 25 PM Screenshot 2024-06-18 at 3 17 09 PM

Copy link

changeset-bot bot commented Jun 18, 2024

🦋 Changeset detected

Latest commit: b8adea9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@aonic-ui/pipelines Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@lokanandaprabhu
Copy link
Contributor Author

/cc @karthikjeeyar @rohitkrai03

packages/pipelines/src/hooks/useRuns.ts Outdated Show resolved Hide resolved
packages/pipelines/package.json Outdated Show resolved Hide resolved
@lokanandaprabhu lokanandaprabhu force-pushed the feature/adding-TR-utils branch from d1f7e6a to b5b5d60 Compare June 27, 2024 05:20
@lokanandaprabhu lokanandaprabhu force-pushed the feature/adding-TR-utils branch from 177c9a7 to 8e78e0a Compare July 1, 2024 07:16
packages/pipelines/src/hooks/usePipelineRuns.ts Outdated Show resolved Hide resolved
packages/pipelines/src/hooks/usePipelineRuns.ts Outdated Show resolved Hide resolved
packages/pipelines/src/utils/tekton-results-utils.ts Outdated Show resolved Hide resolved
packages/pipelines/src/utils/tekton-results-utils.ts Outdated Show resolved Hide resolved
packages/pipelines/src/types/k8s.ts Outdated Show resolved Hide resolved
packages/pipelines/src/types/k8s.ts Show resolved Hide resolved
Copy link
Collaborator

@karthikjeeyar karthikjeeyar left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates a PR is ready to be merged. label Jul 5, 2024
@karthikjeeyar
Copy link
Collaborator

@lokanandaprabhu Add yarn changeset in this PR describing the changes it brings in.

@openshift-ci openshift-ci bot removed the lgtm Indicates a PR is ready to be merged. label Jul 5, 2024
Copy link

openshift-ci bot commented Jul 5, 2024

New changes are detected. LGTM label has been removed.

Copy link
Collaborator

@rohitkrai03 rohitkrai03 left a comment

Choose a reason for hiding this comment

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

/approve

@rohitkrai03 rohitkrai03 added lgtm Indicates a PR is ready to be merged. approved Indicates a PR has been approved by an approver. labels Jul 8, 2024
@openshift-ci openshift-ci bot removed the lgtm Indicates a PR is ready to be merged. label Jul 8, 2024
Copy link

openshift-ci bot commented Jul 8, 2024

New changes are detected. LGTM label has been removed.

@lokanandaprabhu lokanandaprabhu force-pushed the feature/adding-TR-utils branch from df8f2b6 to b8adea9 Compare July 8, 2024 12:15
@rohitkrai03 rohitkrai03 added the lgtm Indicates a PR is ready to be merged. label Jul 8, 2024
@rohitkrai03 rohitkrai03 merged commit 977f231 into redhat-developer:main Jul 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver. lgtm Indicates a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants