Bug 2104792
Summary: | curl fail to reach to the satellite | ||
---|---|---|---|
Product: | Red Hat Satellite | Reporter: | Elyasaf Halle <ehalle> |
Component: | Pulp | Assignee: | satellite6-bugs <satellite6-bugs> |
Status: | POST --- | QA Contact: | Satellite QE Team <sat-qe-bz-list> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | 6.10.4 | CC: | dalley, dkliban, ggainey, rchan, rlavi, zhunting |
Target Milestone: | stream | Keywords: | Triaged |
Target Release: | Unused | ||
Hardware: | x86_64 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | pulpcore-3.40.0 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | Type: | Bug | |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Elyasaf Halle
2022-07-07 07:21:49 UTC
Is this a change or regression in behavior from an earlier release of Satellite? (In reply to Brad Buckingham from comment #1) > Is this a change or regression in behavior from an earlier release of > Satellite? I didn’t try to register to another version of satellite @Rigel The description says installing from the repo works fine, so my guess is that an index page is not being provided when you don't have a trailing slash at the end. Which is expected behavior. Without a trailing slash it doesn't look like a directory, so you don't get an index page. If this is confirmed, please close as NOTABUG. Thank you, Daniel, for looking into it. I'm deferring to ehalle as he opened the BZ. Hi Daniel when I used the CDN in the same script and the script accepted the repo the script link ( https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/16.1/html-single/framework_for_upgrades_13_to_16.1/index#validating-red-hat-openstack-platform-oldvernum-before-the-upgrade ) It's automatically from /etc/yum.repo.d/redhat.repo the URL. And that is the ansible that runs behind. --- - hosts: undercloud, overcloud vars: metadata: name: Check correctness of current repositories description: > Detect whether the repositories listed in `yum repolist` can be connected to and that there is at least one repo configured. Detect if there are any unwanted repositories (such as EPEL) enabled. groups: - pre-upgrade tasks: - name: List repositories command: 'yum repolist -v' args: warn: no changed_when: False register: repositories - name: Find repository URLs shell: 'echo "{{ repositories.stdout }}" | grep Repo-baseurl | sed "s/Repo-baseurl.*\(http[^ ]*\).*/\1/g"' register: repository_urls changed_when: False - name: Check if there is at least one repository baseurl fail: msg: No repository found in yum repolist when: repository_urls.stdout_lines|length < 1 - name: Call repository URLs uri: url: "{{ item }}" with_items: "{{ repository_urls.stdout_lines }}" - name: Find repository IDs changed_when: False shell: 'echo "{{ repositories.stdout }}" | grep Repo-id | sed "s/Repo-id.*://" | tr -d " "' register: repository_ids - name: Check if there are any unwanted repositories enabled fail: msg: Found unwanted repository {{ item.0 }} enabled when: item.0 == item.1 with_nested: - [ 'epel/x86_64' ] - "{{ repository_ids.stdout_lines }}" It's completely valid for a webserver to not redirect from a URL without a trailing slash to one with a trailing slash, just perhaps a little atypical. Is adjusting the script an option? I can file an issue to implement automatic redirection but this isn't a very common complaint for us. Hi, the script is built in the ansible triplo, it is possible to change that but I am not sure if it will help us. we can do manual commands (curl) to test. Hi, I try to do the same procedure in satellite 6.9 and it worked. Was reverted upstream The Pulp upstream bug status is at closed. Updating the external tracker on this bug. All upstream Pulp bugs are at MODIFIED+. Moving this bug to POST. Moving this out to 6.16 - we can't backport it to pulpcore 3.39, and since I don't really see any demand for it, there's no real need to work out something special. Removing triaged keyword for reevaluation. |