Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Allow the export api to be called without login #4532

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

raymondz1
Copy link
Contributor

What this PR does / why we need it

Allow the export api to be called without login.

Which issue(s) this PR fixes

Fixes #

Describe testing done for PR

Release note


Additional information

Special notes for your reviewer

@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/4532/20230329191219/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@codecov
Copy link

codecov bot commented Mar 29, 2023

Codecov Report

Merging #4532 (cc02fdc) into main (fe8dd98) will decrease coverage by 0.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4532      +/-   ##
==========================================
- Coverage   49.82%   48.97%   -0.85%     
==========================================
  Files         453      483      +30     
  Lines       45395    47515    +2120     
==========================================
+ Hits        22617    23270     +653     
- Misses      20626    22048    +1422     
- Partials     2152     2197      +45     

see 35 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@raymondz1 raymondz1 added the ok-to-merge PRs should be labelled with this before merging label Apr 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants