Skip to content

fix: bucket-hook fails with gnu wget #6520

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: bucket-hook fails with gnu wget #6520

wants to merge 1 commit into from

Conversation

emosbaugh
Copy link
Contributor

What problem are we solving?

When we change the base image to one with GNU Wget, the call to http://seaweedfs-master.seaweedfs:9333/cluster/status fails because it does not implement the HTTP HEAD method.

the issue:

root@tmp-shell:/# wget --version
GNU Wget 1.21.2 built on linux-gnu.
root@tmp-shell:/# wget --spider http://seaweedfs-master.seaweedfs:9333/cluster/status && echo "exit $?"
Spider mode enabled. Check if remote file exists.
--2025-02-07 00:26:52--  http://seaweedfs-master.seaweedfs:9333/cluster/status
Resolving seaweedfs-master.seaweedfs (seaweedfs-master.seaweedfs)... 10.244.97.67
Connecting to seaweedfs-master.seaweedfs (seaweedfs-master.seaweedfs)|10.244.97.67|:9333... connected.
HTTP request sent, awaiting response... 405 Method Not Allowed
Remote file does not exist -- broken link!!!

Apologies for our atypical use case.

How are we solving the problem?

How is the PR tested?

Manual

with the fix:

/data # wget
BusyBox v1.37.0 (2024-09-30 10:39:57 UTC) multi-call binary.
...
/data # wget -O - http://seaweedfs-master.seaweedfs:9333/cluster/status && echo "exit $?"
Connecting to seaweedfs-master.seaweedfs:9333 (10.244.97.67:9333)
writing to stdout
-                    100% |********************************************************|    96  0:00:00 ETA
written to stdout
exit 0

root@tmp-shell:/# wget --version
GNU Wget 1.21.2 built on linux-gnu.
root@tmp-shell:/# wget -O - http://seaweedfs-master.seaweedfs:9333/cluster/status && echo "exit $?"
--2025-02-07 00:26:22--  http://seaweedfs-master.seaweedfs:9333/cluster/status
Resolving seaweedfs-master.seaweedfs (seaweedfs-master.seaweedfs)... 10.244.97.67
Connecting to seaweedfs-master.seaweedfs (seaweedfs-master.seaweedfs)|10.244.97.67|:9333... connected.
HTTP request sent, awaiting response... 200 OK
Length: 96 [application/json]
Saving to: 'STDOUT'

-                           0%[                                      ]       0  --.-KB/s               {-                         100%[=====================================>]      96  --.-KB/s    in 0s

2025-02-07 00:26:22 (12.3 MB/s) - written to stdout [96/96]

exit 0

Checks

  • I have added unit tests if possible.
  • I will add related wiki document changes and link to this PR after merging.

@emosbaugh emosbaugh mentioned this pull request Feb 7, 2025
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@emosbaugh