Infrastructure/ansible
5
0
Fork 0
Code Issues Pull requests 3 Projects Releases Packages Wiki Activity Actions

download: drop http only dl/secondary/archive and redirect to https #2530

Open
kevin wants to merge 1 commit from https-downloads into main
pull from: https-downloads
merge into: Infrastructure:main
Conversation 5 Commits 1 Files changed 2 +7
kevin commented 2025-03-24 17:07:33 +00:00
Contributor
Copy link

I can't off hand think of why this would break anything, but
do let me know if you think it will.

Before we merge this, we will need to adjust things in mirrormanager to
not provide http links most likely (although I suppose the redirects
will work for http clients).

Signed-off-by: Kevin Fenzi kevin@scrye.com

I can't off hand think of why this would break anything, but do let me know if you think it will. Before we merge this, we will need to adjust things in mirrormanager to not provide http links most likely (although I suppose the redirects will work for http clients). Signed-off-by: Kevin Fenzi <kevin@scrye.com>
kevin commented 2025-03-24 17:07:48 +00:00
Author
Contributor
Copy link

CC: @abompard @adrian

CC: @abompard @adrian
zuul commented 2025-03-24 17:10:37 +00:00
First-time contributor
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/913bb898f321465cb82e65e26860dc0d

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci https://fedora.softwarefactory-project.io/zuul/buildset/913bb898f321465cb82e65e26860dc0d - [fi-ansible-lint-diff ](https://fedora.softwarefactory-project.io/zuul/build/2634c304a1154058a1f421a21a1a2337) : FAILURE in 2m 46s - [fi-yamllint-diff ](https://fedora.softwarefactory-project.io/zuul/build/d4fb99cd27694d95a9c04a1a6c6e3127) : SUCCESS in 2m 05s
kevin commented 2025-03-28 17:54:38 +00:00
Author
Contributor
Copy link

Lets hold on this a bit more. There may still be some internal uses going on, so do not yet merge.

Lets hold on this a bit more. There may still be some internal uses going on, so do not yet merge.
adrian commented 2025-04-07 07:33:56 +00:00
Contributor
Copy link

Looking at MirrorManager we have two hosts with dl:

  • dl.iad2.fedoraproject.org (private mirror)
  • dl.fedoraproject.org

The entry with iad2 has only http URLs. The entry without iad2 has http and https URLs.

I can remove the http entries from dl.fedoraproject.org. Not sure what the iad2 system is used for.

Looking at MirrorManager we have two hosts with `dl`: * dl.iad2.fedoraproject.org (private mirror) * dl.fedoraproject.org The entry with iad2 has only http URLs. The entry without iad2 has http and https URLs. I can remove the http entries from `dl.fedoraproject.org`. Not sure what the `iad2` system is used for.
kevin commented 2025-04-07 22:03:06 +00:00
Author
Contributor
Copy link

The iad2 one is used to point our internal iad2 machines to the internal ip of that mirror (via dns views).

I've still got some emails out to possible internal users. Will update here once I hear back.

The iad2 one is used to point our internal iad2 machines to the internal ip of that mirror (via dns views). I've still got some emails out to possible internal users. Will update here once I hear back.
This pull request is broken due to missing fork information.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin https-downloads:https-downloads
git checkout https-downloads

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout main
git merge --no-ff https-downloads
git checkout https-downloads
git rebase main
git checkout main
git merge --ff-only https-downloads
git checkout https-downloads
git rebase main
git checkout main
git merge --no-ff https-downloads
git checkout main
git merge --squash https-downloads
git checkout main
git merge --ff-only https-downloads
git checkout main
git merge https-downloads
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
freeze-break-request

   
post-freeze
No labels
freeze-break-request
post-freeze
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Infrastructure/ansible#2530
No description provided.
Delete branch "https-downloads"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?