Project

General

Profile

Actions

Bug #14331

closed

using id param to add a puppet module to a content view working incorrectly

Added by John Mitsch about 8 years ago. Updated almost 4 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
API
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

If you use the id param on
POST /content_views/:content_view_id/content_view_puppet_modules

it will save that id under uuid, even though it is an active record id (and not a valid uuid)

Steps to replicate:
1) attach a puppet module to a content view with a puppet module id

you can use
hammer content-view puppet-module add --organization-id 1 --content-view-id CONTENT_VIEW_ID --id PUPPET_MODULE_ID
(replace PUPPET_MODULE_ID with the id of the puppet module, and CONTENT_VIEW_ID with the id of the content view)

2)check the api call on POST /content_views/:content_view_id/content_view_puppet_modules and the output of hammer -r content-view puppet-module list --organization-id 1 --content-view-id CONTENT_VIEW_ID

the puppet modules active record id will be in the uuid column, the uuid of the puppet module should be there

Actions #1

Updated by The Foreman Bot about 8 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to John Mitsch
  • Pull request https://github.com/Katello/katello/pull/5912 added
Actions #2

Updated by John Mitsch about 8 years ago

  • Status changed from Ready For Testing to Resolved
  • Assignee deleted (John Mitsch)
  • Pull request deleted (https://github.com/Katello/katello/pull/5912)

fixed in hammer (with a separate issue) https://github.com/Katello/hammer-cli-katello/pull/368

Actions #3

Updated by Eric Helms about 8 years ago

  • Status changed from Resolved to Rejected
  • translation missing: en.field_release set to 114
Actions #4

Updated by Jonathon Turel almost 4 years ago

  • Target version changed from Katello Backlog to Katello Recycle Bin
Actions

Also available in: Atom PDF