<div dir="ltr"><div>Gotcha. /me shrugs</div><div><br></div><div>However we decide to implement this, its pretty evident that this field will eventually be a core field, or at the very least treated like a core field. So I can confidently say that yes, it should be renamed.<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jan 7, 2019 at 11:06 AM Daniel Alley <<a href="mailto:dalley@redhat.com">dalley@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>The serializer just needs to remove the _artifacts field and add an
 _artifact field. Here's how I did it in docker, which is a total ripoff
 of the file plugin.</div></blockquote><div><br></div><div>I know it's fairly simple to do manually, I just meant to do so automatically (unless we also need a serializer mixin like you said).  <br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jan 7, 2019 at 10:41 AM Austin Macdonald <<a href="mailto:amacdona@redhat.com" target="_blank">amacdona@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div>The serializer just needs to remove the _artifacts field and add an _artifact field. Here's how I did it in docker, which is a total ripoff of the file plugin.</div><div><br></div><div><a href="https://github.com/pulp/pulp_docker/pull/291/" target="_blank">https://github.com/pulp/pulp_docker/pull/291/</a></div><div><br></div><div>It might be worth making a serializer mixin also? (I can almost hear jortel cringing about all these mixins)<br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jan 7, 2019 at 10:32 AM Daniel Alley <<a href="mailto:dalley@redhat.com" target="_blank">dalley@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Given that single-artifact Content is likely to be a very common pattern
 among plugins, maybe it would be best to add this as a mixin for 
pulpcore. If that's the future of this field, we should definitely make 
it _artifact.</blockquote><div><br></div><div>+1 to this, I don't much like having to redefine this in every plugin.  I'm curious about how to make it work with the serializers though. <br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jan 7, 2019 at 10:13 AM Austin Macdonald <<a href="mailto:amacdona@redhat.com" target="_blank">amacdona@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">We have single-artifact Content in Docker as well. I've gone ahead and named the field _artifact.<br><br>Given that single-artifact Content is likely to be a very common pattern among plugins, maybe it would be best to add this as a mixin for pulpcore. If that's the future of this field, we should definitely make it _artifact.<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jan 2, 2019 at 12:24 PM David Davis <<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr">In most plugins, Content only has a single artifact so we created a virtual field 'artifact' that we expose to end users. In a recent change[0], we prefixed the Content fields with underscores ('_') so we're considering renaming the field to '_artifact' to be consistent with other plugins that have '_artifacts' on Content. We could use some feedback by sprint planning (Jan 4) either here or on the issue:<div><br></div><div><a href="https://pulp.plan.io/issues/4282" target="_blank">https://pulp.plan.io/issues/4282</a><br></div><div><br></div><div>[0] <a href="https://pulp.plan.io/issues/4206" target="_blank">https://pulp.plan.io/issues/4206</a><br><div><br><div><div dir="ltr" class="gmail-m_-4717088186275247052gmail-m_5233594733378491593gmail-m_3304877146554586371gmail-m_4162004404042627214gmail-m_-6418300944509377615gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>David<br></div></div></div></div></div></div></div></div></div></div></div></div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>
</blockquote></div>
</blockquote></div>
</blockquote></div>