pulp / pulp_rpm

RPM support for Pulp Platform
https://pulpproject.org/pulp_rpm/
GNU General Public License v2.0
48 stars 124 forks source link

Workflow: Create a distribution correction? #3000

Closed briandws2000 closed 5 months ago

briandws2000 commented 1 year ago

Hi,

I think there is a mistake in the documentation.

Location: On this page: https://docs.pulpproject.org/pulp_rpm/workflows/create_sync_publish.html#create-a-distribution-for-the-publication

Problem: Using pulp-cli commands : pulp rpm distribution create \ --name "${DIST_NAME}" \ --base-path "${BASE_PATH}" \ --publication "${PUBLICATION_HREF}"

Using httpie to talk directly to the REST API : TASK_URL=$(http POST "$BASE_ADDR"/pulp/api/v3/distributions/rpm/rpm/ \ publication="$PUBLICATION_HREF" name="$BASE_PATH" base_path="$REPO_NAME" | jq -r '.task')

Correction: I believe the httpie line should be: TASK_URL=$(http POST "$BASE_ADDR"/pulp/api/v3/distributions/rpm/rpm/ \ publication="$PUBLICATION_HREF" name="$DIST_NAME" base_path="$BASE_PATH" | jq -r '.task')

Thanks, -Sheldon

pedro-psb commented 5 months ago

That correction seems right, but we are currently focusing on improving our new documentation site (under staging_docs), and we've already dropped the httpie examples there in favor of pulp-cli examples (example of the referred section).

Since these old docs will be out when staging-docs is stable, I believe we can close that issue.

pedro-psb commented 5 months ago

The migration PR which closes this: #3443