<div dir="ltr"><div dir="ltr">On Mon, May 11, 2020 at 4:46 PM Brian Bouterse <<a href="mailto:bmbouter@redhat.com">bmbouter@redhat.com</a>> wrote:</div><div class="gmail_quote"><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 class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 11, 2020 at 4:40 PM Grant Gainey <<a href="mailto:ggainey@redhat.com" target="_blank">ggainey@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">Hey folks,<div>I am making some additions to the import-export docs and went to build them locally so I could review formatting. Since the last time I did this a month ago, I'm running into a problem.  docs/Makefile now assumes that "<a href="http://pulp/.." target="_blank">http://pulp/..</a>." will resolve to something, which it certainly does not for my vagrant-box.</div></div></blockquote><div>I expected it to use localhost:24817 as the location to fetch the openapi schema from at docs-build time. For example I expect each plugin's makefile to do that with a line like this one:  <a href="https://github.com/pulp/pulp_file/blob/master/docs/Makefile#L45" target="_blank">https://github.com/pulp/pulp_file/blob/master/docs/Makefile#L45</a></div></div></div></blockquote><div><br></div><div>I'm in pulpcore, and the line is here:</div><div><br></div><div>    <a href="https://github.com/pulp/pulpcore/blob/master/docs/Makefile#L56">https://github.com/pulp/pulpcore/blob/master/docs/Makefile#L56</a></div><div><br></div><div>and was changed as part of the single-container work, it appears (eb1173a310)</div><div><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 class="gmail_quote"><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>Is this a problem with my env? A problem with the Makefile? Or a problem with the 'how to build the docs", docs, that is perhaps missing a step?</div></div></blockquote><div>Are you running `make html`? What docs are you working from?<br></div></div></div></blockquote><div><br></div><div>3.4, and yeah, I'm just running 'make html'</div><div> </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 class="gmail_quote"><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>I'm also getting two docstring errors:</div><blockquote style="margin:0px 0px 0px 40px;border:medium none;padding:0px"><div><font size="1" face="monospace">/home/vagrant/devel/pulpcore/pulpcore/app/models/exporter.py:docstring of pulpcore.app.models.PulpExport::more than one target found for cross-reference 'models.RepositoryVersion': pulpcore.app.models.RepositoryVersion, pulpcore.plugin.models.RepositoryVersion</font></div></blockquote><div>and</div><blockquote style="margin:0px 0px 0px 40px;border:medium none;padding:0px"><div><font size="1" face="monospace">/home/vagrant/devel/pulpcore/pulpcore/tasking/tasks.py:docstring of pulpcore.tasking.tasks.enqueue_with_reservation::more than one target found for cross-reference 'models.TaskGroup': pulpcore.app.models.TaskGroup, pulpcore.plugin.models.TaskGroup</font></div></blockquote><div><div>The solution is easy - explicitly specify pulpcore.app. But This didn't bite me last time I generated the docs, and I'd like to make sure it's not just a symptom of something More Wrong in my env</div></div></div></blockquote></div></div></blockquote><div> </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 class="gmail_quote"><div>I don't think this is an error with your environment, but probably a legitimate error. For anything we define in pulpcore and then import into pulpcore.plugin we probably need to have the docstring use the full path.<br></div></div></div></blockquote><div><br></div><div>Coolio, I will fix both spots - no sense in opening a new issue for TaskGroup when I'm looking right at the problem in my branch.</div><div><br></div><div>The Makefile I'm not gonna touch in my PR tho - prob wants a few more eyes</div><div><br></div><div>G</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Grant Gainey</div><div>Principal Software Engineer, Red Hat System Management Engineering</div></div></div></div></div></div>