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.

Sign up for GitHub

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

Jump to bottom

relax pyyaml jsonschema #11149

Closed
wants to merge 86 commits into from
Closed

relax pyyaml jsonschema #11149

wants to merge 86 commits into from

Conversation

n3o-Bhushan
Copy link

Description of your changes:

Following changes from this PR

Checklist:

alexlatchford and others added 30 commits August 17, 2020 18:05
@alexlatchford
AIP-1676: Ensure we pass through correct namespace when using the kfp…
d69f011 
… cli
@alexlatchford
AIP-1676: Add in api-namespace CLI argument
d1e0c4b
@alexlatchford
AIP-1676: Clean up manual namespace override hack
41a75fd
@alexlatchford
AIP-1676: Add in mechanism for passing in kubeflow-userid header
30f953b
@alexlatchford
AIP-1676: Hook up namespaces to experiments CLI
e4864a1
@alexlatchford
Merge branch 'master' of github.com:kubeflow/pipelines into alexla/AI…
d449076 
…P-2524
@alexlatchford
Merge branch 'master' of github.com:zillow/pipelines into feature/zg
173ecfc
@alexlatchford
AIP-653: Sym-link python SDK components to avoid poetry lacking subdi…
7e93631 
…rectory install capabilities (#1)
@Peilun-Li
Relax kubernetes sdk version range
9ddb351
@Peilun-Li
Merge pull request #6 from zillow/peilunl/relax-k8s-sdk
8a43840 
Relax kubernetes sdk version range
@alexlatchford
Merge tag '1.7.0' of github.com:kubeflow/pipelines into alexla/AIP-4600
d8e49c3 
Kubeflow Pipelines 1.7.0 release
@alexlatchford
AIP-4600: Add in publishing to ZG Artifactory
ca04bf3
@alexlatchford
Trigger CICD
7479ddf
@alexlatchford
AIP-4600: Add in credentials for publishing to Artifactory
3f68150
@alexlatchford
AIP-4600: Add in credentials for publishing to Artifactory
a8fa267
@alexlatchford
AIP-4600: Add in credentials for publishing to Artifactory
f2d7e50
@alexlatchford
AIP-4600: Add in credentials for publishing to Artifactory
876ebe7
@alexlatchford
AIP-4600: Add in credentials for publishing to Artifactory
0558130
@alexlatchford
AIP-4600: Update PYPI_REPOSITORY to to publish to ANALYTICS_PYPI_REPO…
05c3214 
…SITORY
@alexlatchford
AIP-4600: Embed package name and version into sdist
147bdbd
@alexlatchford
AIP-4600: Fix GNU sed compatibility error
9790c43
@alexlatchford
AIP-4600: Ensure + chars in KFP version are replaced when used as K8s…
248ef74 
… label
@alexlatchford
AIP-4600: Ensure + chars in KFP version are replaced when used as K8s…
95205d8 
… label
@alexlatchford
AIP-4600: Allow ZG to control semantic version of zillow-kfp
3ebfb12
@alexlatchford
fix(sdk): Get short name of complex input/output types to ensure we c…
d614fc2 
…an map to appropriate de|serializer (kubeflow#6504)

Also:
- Simplify _data_passing methods, add in type hints and docstrings.
- Remove get_deserializer_code_for_type.
@alexlatchford
fix(sdk): Update upper bound on kubernetes constraint in Python SDK (k…
6f9828d 
…ubeflow#6532)
@alexlatchford
Merge pull request #9 from zillow/alexla/AIP-4600-pt3
ecf30d2 
AIP-4600: [Part 3 of 3] Pull in upstream fixes from KFP.
@alexlatchford
AIP-4600: Update CICD pin
9a564b5
@alexlatchford
Merge pull request #8 from zillow/alexla/AIP-4600-pt2
f521386 
AIP-4600: [Part 2 of 3] Add in publishing to ZG Artifactory
@alexlatchford
Merge pull request #7 from zillow/alexla/AIP-4600
af95222 
AIP-4600: [Part 1 of 3] Pull in v1.7.0 from upstream
talebzeghmi and others added 19 commits October 18, 2022 14:49
@talebzeghmi
PR Feedback (styles)
9fbe629
@talebzeghmi
Merge pull request #20 from zillow/tz/AIP-6692-disable-retry-button
6b3fe07 
AIP-6692 KFP UI Remove Retry Button
@talebzeghmi
AIP-6621: Terminate Button UI KNOWN ISSUE btn
626be63
@talebzeghmi
typo fix
8730efc
@talebzeghmi
Merge pull request #21 from zillow/tz/AIP-6621-terminate-fix
5fcda6f 
AIP-6621: Terminate Button UI KNOWN ISSUE
@cloudw
Correctly prints run details in json format
17f85b7
@cloudw
Merge pull request #24 from zillow/yunw/cli-run-details-json
4c3cb9f 
Correctly prints run details in json format
@cloudw
Add enable / disable recurring run
3abde63
@cloudw
Fixing doc string
1fd7df1
@cloudw
Merge pull request #25 from zillow/yunw/enable-disable-recurring-run
dc9a5d8 
Add enable / disable recurring run
@Peilun-Li
relax typing-extensions
5f52ece
@Peilun-Li
Merge pull request #27 from zillow/zillow/peilunl/relax-typing-extens…
1f70f5b 
…ions

Relax typing-extensions
@tmckay
Add link to terminate instructions in docs from terminate dialog
94bcf32
@tmckay
Merge pull request #29 from zillow/taylorm/add-terminate-doc-link
5caba14 
Add link to terminate instructions in docs from terminate dialog
@n3o-Bhushan
relax google auth dependency range
af0cf59
@n3o-Bhushan
address comment and adjust range
d4744a5
@n3o-Bhushan
adjust range in setup.py
c5245ea
@n3o-Bhushan
Merge pull request #30 from n3o-Bhushan/bhushan/loosen-google-auth-dep
ab8fcb9 
Relax google auth dependency range
@n3o-Bhushan
Update setup.py
c627baf 
relax dependencies to avoid conflict with doit module
@google-oss-prow Google OSS Prow
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign james-jwu for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow Google OSS Prow
Copy link

Hi @n3o-Bhushan. Thanks for your PR.

I'm waiting for a kubeflow member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@n3o-Bhushan n3o-Bhushan deleted the bhushan/relax-pyyaml-jsonschema branch August 29, 2024 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ark-kun Ark-kun Awaiting requested review from Ark-kun

@DharmitD DharmitD Awaiting requested review from DharmitD

@HumairAK HumairAK Awaiting requested review from HumairAK

@rimolive rimolive Awaiting requested review from rimolive

@zijianjoy zijianjoy Awaiting requested review from zijianjoy

Assignees
No one assigned
Labels
needs-ok-to-test size/XXL
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@n3o-Bhushan @alexlatchford @Peilun-Li @chensun @ysk24ok @hieuhc @Bobgy @talebzeghmi @cloudw @tmckay