Re: On disk repos, anaconda, and Installation Source
[email protected] Wed, 19 Jun 2019 14:22:11 +0200
| Newsgroups | gmane.linux.redhat.anaconda.devel |
|---|---|
| Message-ID | <[email protected]> |
--===============6339675199885138204== Content-Type: multipart/alternative; boundary="=-gIVyFREkAHfD+MLtkW+5" --=-gIVyFREkAHfD+MLtkW+5 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Hello, So about the code and treeinfo. This repository is not loaded because it is disabled on the netboot image and the code is iterating only between enabled repositories, which is the correct behavior I would say. Yes the comment is wrong, originally it was above the ``self.set_updates_enabled(self._updates_enabled)`` line and the meaning was that all the repos there will be enabled (under some conditions) and user can adjust this by the checkbox in UI. The are of course the conditions so it's not always so or so. That comment should definitely change or the whole code should be rewritten to be more readable. You can read the conditions if you follow the `enabled` variable in this method. In short these local repositories will be enabled if there is no installation method set and it is not automatic installation -- (in other words "Closest mirror" installation). And no, this repositories shouldn't be loaded into the addon list right now. We can think about that in the further changes but now it is just implementation detail of the Closest mirror installation type. Cheers,Jirka On Fri, 2019-06-14 at 13:58 -0500, Pat Riehecky wrote: > I noticed an unexpected behavior in Fedora 30. > > > > The net install disk for Workstation ships with the > 'fedora-cisco-openh264' repo on the media. However, the repo is > not > listed under the 'Installation Source' spoke. > > > > Should this repo be loaded into the addons list? > > > > I suppose this is a bit of confusion based on this comment [1], > vs > the actual behavior where only `.treeinfo` repos that are also on > disk get enabled [2] > > > > Thoughts? > > > > Pat > > > > > > [1] > https://github.com/rhinstaller/anaconda/blob/master/pyanaconda/payload/dnfpayload.py#L1144 > > > > [2] self.addons seems to be populated via __init__.py and what it > finds in `.treeinfo` I think. > > https://github.com/rhinstaller/anaconda/blob/master/pyanaconda/payload/dnfpayload.py#L1235 > > > > -- Pat Riehecky > Fermi National Accelerator Laboratorywww.fnal.gov > www.scientificlinux.org > > > _______________________________________________Anaconda-devel-list > mailing [email protected] > https://www.redhat.com/mailman/listinfo/anaconda-devel-list --=-gIVyFREkAHfD+MLtkW+5 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: quoted-printable <html dir=3D"ltr"><head> <meta http-equiv=3D"content-type" content=3D"text/html; charset=3DUTF-8= "> </head> <body style=3D"text-align:left; direction:ltr;"><div>Hello,</div><div><br= ></div><div>So about the code and treeinfo. This repository is not loaded b= ecause it is disabled on the netboot image and the code is iterating only b= etween enabled repositories, which is the correct behavior I would say.</di= v><div><br></div><div>Yes the comment is wrong, originally it was above the= ``self.set_updates_enabled(self._updates_enabled)`` line and the meaning w= as that all the repos there will be enabled (under some conditions) and use= r can adjust this by the checkbox in UI. The are of course the conditions s= o it's not always so or so. That comment should definitely change or the wh= ole code should be rewritten to be more readable.</div><div><br></div><div>= You can read the conditions if you follow the `enabled` variable in this me= thod. In short these local repositories will be enabled if there is no inst= allation method set and it is not automatic installation -- (in other words= "Closest mirror" installation). </div><div><br></div><div>And no, this rep= ositories shouldn't be loaded into the addon list right now. We can think a= bout that in the further changes but now it is just implementation detail o= f the Closest mirror installation type.</div><div><br></div><div>Cheers,</d= iv><div>Jirka</div><div><br></div><div><br></div><div>On Fri, 2019-06-14 at= 13:58 -0500, Pat Riehecky wrote:</div><blockquote type=3D"cite" style=3D"m= argin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"> I noticed an unexpected behavior in Fedora 30.<br> <br> The net install disk for Workstation ships with the 'fedora-cisco-openh264' repo on the media. However, the repo is n= ot listed under the 'Installation Source' spoke.<br> <br> Should this repo be loaded into the addons list?<br> <br> I suppose this is a bit of confusion based on this comment [1], vs the actual behavior where only `.treeinfo` repos that are also on disk get enabled [2]<br> <br> Thoughts?<br> <br> Pat<span class=3D"pl-c1"><br> </span><br> <br> [1] <a class=3D"moz-txt-link-freetext" href=3D"https://github.com/rhinstaller/a= naconda/blob/master/pyanaconda/payload/dnfpayload.py#L1144">https://github.= com/rhinstaller/anaconda/blob/master/pyanaconda/payload/dnfpayload.py#L1144= </a><br> <br> [2] self.addons seems to be populated via __init__.py and what it finds in `.treeinfo` I think.<br> <a class=3D"moz-txt-link-freetext" href=3D"https://github.com/rhinstaller/a= naconda/blob/master/pyanaconda/payload/dnfpayload.py#L1235">https://github.= com/rhinstaller/anaconda/blob/master/pyanaconda/payload/dnfpayload.py#L1235= </a><br> <br> <pre>-- </pre><pre>Pat Riehecky</pre><pre><br></pre><pre>Fermi National= Accelerator Laboratory</pre><a class=3D"moz-txt-link-abbreviated" href=3D"= http://www.fnal.gov"><pre>www.fnal.gov</pre></a><pre><br></pre><a class=3D"= moz-txt-link-abbreviated" href=3D"http://www.scientificlinux.org"><pre>www.= scientificlinux.org</pre></a> =20 <pre>_______________________________________________</pre><pre>Anaconda-dev= el-list mailing list</pre><a href=3D"mailto:[email protected]"= ><pre>[email protected]</pre></a><pre><br></pre><a href=3D"htt= ps://www.redhat.com/mailman/listinfo/anaconda-devel-list"><pre>https://www.= redhat.com/mailman/listinfo/anaconda-devel-list</pre></a></blockquote></bod= y></html> --=-gIVyFREkAHfD+MLtkW+5-- --===============6339675199885138204== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Anaconda-devel-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/anaconda-devel-list --===============6339675199885138204==--