Project

Profile

Help

Issue #717

closed

Pulp-admin fails to publish rpm repo with serve-http and serve-https set false

Added by marc@chadwick.net about 9 years ago. Updated about 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
2.5
Platform Release:
2.7.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

Log output

Description of problem:

If an rpm repository has both serve-http and serve-https set to false, and if at least one rpm in its contents, publish fails. Pulp-admin displays the following error:

Task Failed

coercing to Unicode: need string or buffer, NoneType found

I am attaching a text file with the contents of ~/.pulp/server_calls.log.

The publish succeeds if there is no content in the repo.

Version-Release number of selected component (if applicable):

pulp-admin-client-2.5.3-1.el6.noarch

How reproducible:

100%

Steps to Reproduce:
1. pulp-admin rpm repo create --repo-id foo --serve-http false --serve-https false
2. Upload or copy an RPM into this repo
3. pulp-admin rpm repo publish run --repo-id foo

Actual results:
Task Failed

coercing to Unicode: need string or buffer, NoneType found

Expected results:
Publish should succeed and remove any previously published data.

Additional info:
Extra thanks to bcourt on IRC who helped track this down and asked that I open this bug

+ This bug was cloned from Bugzilla Bug #1196390 +


Files

3101cb398b6be649ae617009a245967d (2.27 KB) 3101cb398b6be649ae617009a245967d marc@chadwick.net, 03/01/2015 12:26 AM
Actions #1

Updated by bcourt about 9 years ago

This bug specifically kicks in when the incremental publish code is invoked and both serve-http and serve-https is set to false.

+ This comment was cloned from Bugzilla #1196390 comment 1 +

Actions #2

Updated by ipanova@redhat.com about 9 years ago

  • Triaged changed from No to Yes
  • Severity set to Medium
Actions #3

Updated by bmbouter about 9 years ago

  • Severity changed from Medium to 2. Medium
Actions #4

Updated by ipanova@redhat.com about 9 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com

Added by ipanova@redhat.com almost 9 years ago

Revision 23d9c093 | View on GitHub

Adding validation that http and https cannot be set both to false.

With settings both set to false no publish will occur.

closes #717 https://pulp.plan.io/issues/717

Actions #5

Updated by ipanova@redhat.com almost 9 years ago

  • Status changed from ASSIGNED to POST
Actions #6

Updated by ipanova@redhat.com almost 9 years ago

  • Platform Release set to 2.7.1
Actions #7

Updated by ipanova@redhat.com almost 9 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #8

Updated by amacdona@redhat.com over 8 years ago

  • Platform Release changed from 2.7.1 to 2.7.2
Actions #9

Updated by dkliban@redhat.com about 8 years ago

  • Status changed from MODIFIED to 5
Actions #10

Updated by semyers almost 8 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
  • Platform Release changed from 2.7.2 to 2.7.0
Actions #11

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF