• Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Minor Minor
    • core
    • Server 2008 R2 Datacenter

      The build history has the link "RSS for all" on bottom. When you click it it shows only the builds of the visible list. This is not "all". In case you scroll down in this list with the arrows buttons, it is the same list on "RSS for all" so it shows always the same.

          [JENKINS-39207] "RSS for all" shows only a few builds

          Oleg Nenashev added a comment -

          Just requires a text clarification IMO

          Oleg Nenashev added a comment - Just requires a text clarification IMO

          Ronny Borchert added a comment - - edited

          We have jobs with more than 1200 build executions. There is under the link http://srv.here.comp.org:8080/job/JobName/rssAll more than in table viewed but not all. Same for failed builds. It seems there is somewhere a break, which prefents that really all builds are shown. I looks like just the last 5 or 7 days are shown. Please find attached anonymouse generated rssAll file: rssAll.txt

          Ronny Borchert added a comment - - edited We have jobs with more than 1200 build executions. There is under the link http://srv.here.comp.org:8080/job/JobName/rssAll more than in table viewed but not all. Same for failed builds. It seems there is somewhere a break, which prefents that really all builds are shown. I looks like just the last 5 or 7 days are shown. Please find attached anonymouse generated rssAll file: rssAll.txt

          Peter Goins added a comment - - edited

          Removing my comment - my issue was with missing metrics settings for a folder.

          Peter Goins added a comment - - edited Removing my comment - my issue was with missing metrics settings for a folder.

          Steven Murphy added a comment -

          I know this is an old issue but this still appears to be the case.  The rssAll only goes back 7 days.

          Is this the intention or is it returning an incomplete list?

          Steven Murphy added a comment - I know this is an old issue but this still appears to be the case.  The rssAll only goes back 7 days. Is this the intention or is it returning an incomplete list?

          The same issue here, jenkins v2.204.2, rssAll still shows up to 7 days back, while build history is much longer.

          Daniel Jeznach added a comment - The same issue here, jenkins v2.204.2, rssAll still shows up to 7 days back, while build history is much longer.

          Alexander Samoylov added a comment - - edited

          This regression was made in https://github.com/jenkinsci/jenkins/commit/3ade716f416ad13904b212c8164c9d4f3e2f2e7d#diff-fde0630baccd1693934fc7b6bf29c31eb5bd7b5cc2138293b60f85e8bb020c4fR1569 by fcojfernandez. He replaced getBuilds() with getBuilds().newBuilds() in the doRssAll() method which was wrong, because it limited the builds with 7 days:

          -        rss(req, rsp, " all builds", getBuilds());
          +        RSS.rss(req, rsp, getDisplayName() + " all builds", getUrl(), getBuilds().newBuilds());

          Interesting is that in the same commit he changed also doRssAll() method of another class https://github.com/jenkinsci/jenkins/commit/3ade716f416ad13904b212c8164c9d4f3e2f2e7d#diff-581669f62053a1ed8fed271068c074eec26588530194da1aef0d14f722f48b30R1372 , but there he did not do this:

          -        rss(req, rsp, " all builds", getBuilds());
          +        RSS.rss(req, rsp, getDisplayName() + " all builds", getUrl(), getBuilds());

           

          Dear fcojfernandez , please return back getBuilds() how it was there till the version 2.214, because your change causes inconvenience. We cannot get RSS for all builds anymore, although we need it.

          Alexander Samoylov added a comment - - edited This regression was made in https://github.com/jenkinsci/jenkins/commit/3ade716f416ad13904b212c8164c9d4f3e2f2e7d#diff-fde0630baccd1693934fc7b6bf29c31eb5bd7b5cc2138293b60f85e8bb020c4fR1569 by fcojfernandez . He replaced getBuilds() with getBuilds().newBuilds() in the doRssAll() method which was wrong, because it limited the builds with 7 days: -        rss(req, rsp, " all builds", getBuilds()); +        RSS.rss(req, rsp, getDisplayName() + " all builds", getUrl(), getBuilds().newBuilds ()); Interesting is that in the same commit he changed also doRssAll() method of another class https://github.com/jenkinsci/jenkins/commit/3ade716f416ad13904b212c8164c9d4f3e2f2e7d#diff-581669f62053a1ed8fed271068c074eec26588530194da1aef0d14f722f48b30R1372 , but there he did not do this: -        rss(req, rsp, " all builds", getBuilds()); +        RSS.rss(req, rsp, getDisplayName() + " all builds", getUrl(), getBuilds());   Dear fcojfernandez , please return back getBuilds() how it was there till the version 2.214, because your change causes inconvenience. We cannot get RSS for all builds anymore, although we need it.

            Unassigned Unassigned
            arbeita Ronny Borchert
            Votes:
            1 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated: