Project

General

Profile

Actions
Copy link

Bug #36844

closed

Add error handling in repo sync when trying to sync non-library repos

Added by Samir Jha about 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Samir Jha
Category:
Repositories
Target version:
Katello 4.11.0
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/Katello/katello/pull/10778
Fixed in Releases:
Katello 4.11.0
Found in Releases:
Red Hat JIRA:

Description

hammer> repository sync --id 321 --organization-id 1
Could not synchronize the repository:
undefined method `id' for nil:NilClass
hammer>

This is error sync raises from this action: `plan_action(Katello::Foreman::ContentUpdate, repo.environment, repo.content_view, repo)` when syncing non-library repos.

We shouldn't be planning sync action at all for non-library repos and the call should fail early and with proper error handling.

Actions
Copy link
 #1

Updated by Partha Aji about 1 year ago

  • Triaged changed from No to Yes
Actions
Copy link
 #2

Updated by The Foreman Bot about 1 year ago

  • Status changed from New to Ready For Testing
  • Assignee set to Samir Jha
  • Pull request https://github.com/Katello/katello/pull/10778 added
Actions
Copy link
 #3

Updated by The Foreman Bot about 1 year ago

  • Fixed in Releases Katello 4.11.0 added
Actions
Copy link
 #4

Updated by Samir Jha about 1 year ago

  • Status changed from Ready For Testing to Closed
Actions
Copy link

Also available in: Atom PDF