Project

General

Profile

Bug #15513

mirror-on-sync option is not set properly during repository create

Added by Brad Buckingham about 3 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Repositories
Target version:
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1348070
Creating a repository with --mirror-on-sync=false results in the repository being created, but 'Mirror on Sync' is actually still set to true.

To reproduce (6.2.0 Beta2)

hammer product create --name='product1' --organization-id=1
hammer repository create \
--name='testrepo1' \
--organization-id=1 \
--product='product1' \
--content-type='yum' \
--publish-via-http=false \
--mirror-on-sync=false \
--download-policy=immediate

hammer repository info --product='product1' --name='testrepo1'

(This does not show the state of the Mirror on Sync option)

Login to the WebUI and navigate to the new repo shows Mirror on Sync = Yes

Associated revisions

Revision b912b815 (diff)
Added by Brad Buckingham about 3 years ago

fixes #15513: repository: set mirror_on_sync correctly on create (#6149)

Without this change, if the user provided mirror_on_sync=false
during repository creation, it would get ignored. As a result,
it was always true by default.

History

#1 Updated by Brad Buckingham about 3 years ago

  • Subject changed from hammer mirror-on-sync option does not work to mirror-on-sync option is not set properly during repository create
  • Assignee set to Brad Buckingham
  • Difficulty set to easy

#2 Updated by The Foreman Bot about 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/6149 added

#3 Updated by Brad Buckingham about 3 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#4 Updated by Justin Sherrill about 3 years ago

  • Legacy Backlogs Release (now unused) set to 143

Also available in: Atom PDF