Add pulp_labels to Namespaces#2172
Conversation
|
@gerrod3 hello! Do you have plans to merge this PR shortly?) |
|
@pichuzhkin Yes, sorry this got lost in the backlog. |
fixes: pulp#2139 Generated by: claude-4.5-sonnet
|
|
||
| Fields: | ||
| name (models.TextField): The name of the namespace. | ||
| pulp_labels (HStoreField): Key-value pairs for labeling and organizing namespaces. |
There was a problem hiding this comment.
I thought this was on BaseModel, but apparently not. Do you remember why?
There was a problem hiding this comment.
The original implementation before it was a HStoreField was on BaseModel. That was removed a long time ago for performance reasons. Why we switched to just adding HStoreField to each model individually I don't remember.
There was a problem hiding this comment.
Maybe the joins across master-detail were an issue
@gerrod3 Thank you so much! |
|
@pichuzhkin Next Tuesday hopefully. We have some other issues we need to fix first before we can do a release. |
3 participants
fixes: #2139
Generated by: claude-4.5-sonnet