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

fix: get pl kind apiversion #1159

Merged
merged 9 commits into from
Oct 4, 2023
Merged

fix: get pl kind apiversion #1159

merged 9 commits into from
Oct 4, 2023

Conversation

jy4096
Copy link
Contributor

@jy4096 jy4096 commented Oct 4, 2023

because of kubernetes/kubernetes#80609
get pipeline doesn't return "kind" and "apiVersion"

since we know the value for these two fields, set the fields in handler manually

test result:
image

Signed-off-by: jyu6 <juanlu_yu@intuit.com>
@jy4096 jy4096 changed the base branch from main to ui-crud-1.0 October 4, 2023 19:00
Signed-off-by: jyu6 <juanlu_yu@intuit.com>
jyu6 added 5 commits October 4, 2023 12:11
Signed-off-by: jyu6 <juanlu_yu@intuit.com>
Signed-off-by: jyu6 <juanlu_yu@intuit.com>
Signed-off-by: jyu6 <juanlu_yu@intuit.com>
Signed-off-by: jyu6 <juanlu_yu@intuit.com>
Signed-off-by: jyu6 <juanlu_yu@intuit.com>
server/apis/v1/handler.go Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

Revert the change on this file, or the ci will not pass.

jyu6 added 2 commits October 4, 2023 12:22
Signed-off-by: jyu6 <juanlu_yu@intuit.com>
Signed-off-by: jyu6 <juanlu_yu@intuit.com>
@jy4096 jy4096 merged commit fa30010 into ui-crud-1.0 Oct 4, 2023
2 checks passed
@jy4096 jy4096 deleted the get-pl-kind-apiversion branch October 4, 2023 19:29
darshansimha pushed a commit to darshansimha/numaflow that referenced this pull request Oct 5, 2023
Signed-off-by: jyu6 <juanlu_yu@intuit.com>
bbehnke added a commit that referenced this pull request Oct 17, 2023
fix: sorting issue fix and arrow icon for navigating to namespace (#1157)

Signed-off-by: Darshan Simha <darshan_simha@intuit.com>
Co-authored-by: Darshan Simha <darshan_simha@intuit.com>

feat: isb update implemented and search input styling (#1158)

Signed-off-by: bbehnke <bradleybehnke@yahoo.com>

fix: get pl kind apiversion (#1159)

Signed-off-by: jyu6 <juanlu_yu@intuit.com>

feat: all spec crud (#1163)

Signed-off-by: bbehnke <bradleybehnke@yahoo.com>

Pause resume for pipelines (#1167)

Co-authored-by: Darshan Simha <darshan_simha@intuit.com>

feat: crud updates and error/loading handling (#1183)

Signed-off-by: bbehnke <bradleybehnke@yahoo.com>

fix: fixed the timing issue on resume and pause (#1185)

Co-authored-by: Darshan Simha <darshan_simha@intuit.com>

feat: error handling/display for all pages and pipeline load block (#1187)

fix: overridden changes re-merge (#1194)

Signed-off-by: Darshan Simha <darshan_simha@intuit.com>
Co-authored-by: Darshan Simha <darshan_simha@intuit.com>

fix: fixed the timing issue on resume and pause (#1197)

Signed-off-by: Darshan Simha <darshan_simha@intuit.com>
Co-authored-by: Darshan Simha <darshan_simha@intuit.com>

fix: vertex update tweaks and general text/alignment fixes (#1200)

fix: set size on sidebar slide icon (#1201)

Signed-off-by: bbehnke <bradleybehnke@yahoo.com>
bbehnke added a commit that referenced this pull request Oct 17, 2023
fix: sorting issue fix and arrow icon for navigating to namespace (#1157)

Signed-off-by: Darshan Simha <darshan_simha@intuit.com>
Co-authored-by: Darshan Simha <darshan_simha@intuit.com>

feat: isb update implemented and search input styling (#1158)

Signed-off-by: bbehnke <bradleybehnke@yahoo.com>

fix: get pl kind apiversion (#1159)

Signed-off-by: jyu6 <juanlu_yu@intuit.com>

feat: all spec crud (#1163)

Signed-off-by: bbehnke <bradleybehnke@yahoo.com>

Pause resume for pipelines (#1167)

Co-authored-by: Darshan Simha <darshan_simha@intuit.com>

feat: crud updates and error/loading handling (#1183)

Signed-off-by: bbehnke <bradleybehnke@yahoo.com>

fix: fixed the timing issue on resume and pause (#1185)

Co-authored-by: Darshan Simha <darshan_simha@intuit.com>

feat: error handling/display for all pages and pipeline load block (#1187)

fix: overridden changes re-merge (#1194)

Signed-off-by: Darshan Simha <darshan_simha@intuit.com>
Co-authored-by: Darshan Simha <darshan_simha@intuit.com>

fix: fixed the timing issue on resume and pause (#1197)

Signed-off-by: Darshan Simha <darshan_simha@intuit.com>
Co-authored-by: Darshan Simha <darshan_simha@intuit.com>

fix: vertex update tweaks and general text/alignment fixes (#1200)

fix: set size on sidebar slide icon (#1201)

Signed-off-by: bbehnke <bradleybehnke@yahoo.com>
Signed-off-by: bbehnke1 <bradley_behnke@intuit.com>
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