Project

General

Profile

Bug #9521

Katello needs to activate pulp consumer as node

Added by Justin Sherrill about 8 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Foreman Proxy Content
Target version:
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

After chatting with jortel a pulp node needs to be activated using this method:

https://github.com/Katello/katello/blob/106bb48226f970a58b4329c6db3cd398477c49f1/app/models/katello/glue/pulp/consumer.rb#L202

when a capsule registers with the pulp_node feature.

from irc:

```
jortel | stbenjam: looking at your node sync issue, you capsule does not seem to be | activated as a node (in the pulp db)
jsherrill | jortel: when you say 'activated' what do you mean exactly?
jsherrill | which api call?
jortel | jsherrill: when you create the consumer, you need to add a special top-secret | note that indicates it's a node and what it's strategy is
jortel | jsherrill: or, in a consumer update
jsherrill | jortel: is this new?
jsherrill | (my memory is a little fuzzy)
jsherrill | and i'm trying to find where we might have ever done that
jsherrill | i dont' see anything
jsherrill | we were just registering the consumer and adding environments to sync
jsherrill | as far as i can tell
jsherrill | errr, adding repos
jsherrill | jortel: doesn't look like, it is possible pulp 2.4 didn't care?
jortel | jsherrill: yes, it's possible. iirc, we fixed a bug whereby that strategy | was not being passed to the agent.
```

my guess is that we used to call this method when pulp nodes were first implemented but that got lost along the way.

Associated revisions

Revision 24d37557 (diff)
Added by Justin Sherrill about 8 years ago

fixes #9521 - activate and deactivate pulp nodes

Revision 37cc9ee0
Added by Justin Sherrill about 8 years ago

Merge pull request #5051 from jlsherrill/9521

fixes #9521 - activate and deactivate pulp nodes

History

#1 Updated by The Foreman Bot about 8 years ago

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

#2 Updated by Justin Sherrill about 8 years ago

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

#3 Updated by Stephen Benjamin about 8 years ago

  • Bugzilla link set to 1198464

Also available in: Atom PDF