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

SUSE URLs should use https:// #1195

Closed
viccuad opened this issue Aug 6, 2020 · 1 comment · Fixed by #1196
Closed

SUSE URLs should use https:// #1195

viccuad opened this issue Aug 6, 2020 · 1 comment · Fixed by #1196
Labels
Status: Not Implemented This issue will not be implemented due to reasons that should be listed on issue conversation thread Type: Bug Something isn't working

Comments

@viccuad
Copy link
Member

viccuad commented Aug 6, 2020

Describe the bug

From Dirk, SUSE URLs for chart dependencies (an all) should use https://, not http://

To Reproduce

git grep "http://.*suse.com"  | tac
stable/kubecf/requirements.yaml:  repository: http://opensource.suse.com/bits-service-release2/
stable/kubecf/requirements.yaml:  repository: http://opensource.suse.com/eirini-release/
stable/kubecf/requirements.lock:  repository: http://opensource.suse.com/bits-service-release/
stable/kubecf/requirements.lock:  repository: http://opensource.suse.com/eirini-release/

Expected behavior

All urls with https://

Environment

  • Others

Additional context

@viccuad viccuad mentioned this issue Aug 6, 2020
7 tasks
@jandubois
Copy link
Member

The subcharts will be removed in #1179, which will render this particular instance moot. Hopefully #1196 will catch other instances from appearing, so there is nothing left to do for this issue.

@fargozhu fargozhu added Type: Bug Something isn't working Status: Not Implemented This issue will not be implemented due to reasons that should be listed on issue conversation thread labels Aug 20, 2020
@fargozhu fargozhu modified the milestone: 2.4.0 Aug 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Status: Not Implemented This issue will not be implemented due to reasons that should be listed on issue conversation thread Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants