Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.12.0, Lustre 2.10.6
    • None
    • None
    • 9223372036854775807

    Description

      Looking at current codes, still many places for example README, man page etc still
      use old URL.

      Even link could redirect automatically, we'd better fix it to avoid any further confusions.

      Attachments

        Activity

          [LU-11304] URLs need be updated

          Good to know that

          wshilong Wang Shilong (Inactive) added a comment - Good to know that
          pjones Peter Jones added a comment -

          That explains the omission - Ubuntu 18.04 is not supported on b2_10 so that's why it missed the change. We are not planning to do any more landings to b2_10 - we will nee focusing on master and b2_12 going forward and neither of them have this problem

          pjones Peter Jones added a comment - That explains the omission - Ubuntu 18.04 is not supported on b2_10 so that's why it missed the change. We are not planning to do any more landings to b2_10 - we will nee focusing on master and b2_12 going forward and neither of them have this problem
          wshilong Wang Shilong (Inactive) added a comment - - edited

          Checked with latest master, it did not have this problem, that is because of following commit from DongYang:

          ommit e41bdca755954beeb47ee6653ed9e140b7338e96
          Author: Li Dongyang <dongyangli@ddn.com>
          Date:   Thu Jul 19 12:24:36 2018 -0400
          
              LU-11071 build: Add server build support for Ubuntu 18.04
              
              This enables server build for Ubuntu 18.04 LTS, the ldiskfs
              patches are based on Gael's 4.12 support,
              they apply to kernel versions 4.15.0-20.21 to 4.15.0-23.25
              
              There's also a small fix to make dpkg happy when installing
              lustre packages which requires lustre-client-utils.
              
              Test-Parameters: clientdistro=ubuntu1604 trivial
          

          Noticed the patch did not only update URL but also updated some copyright, something like following:

            Lustre is a scalable, secure, robust, highly-available cluster file system.
          - This release is maintained by Intel Corporation and available from
          - https://wiki.hpdd.intel.com/
          + This release is maintained by Whamcloud and available from
          + https://wiki.whamcloud.com/
          

          So it depends on how we are going to support for b2_10, is Ubuntu18.04 supposed to be supported on b2_10 branch? or we could port parts of this patch to fix these copyright issue.

          wshilong Wang Shilong (Inactive) added a comment - - edited Checked with latest master, it did not have this problem, that is because of following commit from DongYang: ommit e41bdca755954beeb47ee6653ed9e140b7338e96 Author: Li Dongyang <dongyangli@ddn.com> Date: Thu Jul 19 12:24:36 2018 -0400 LU-11071 build: Add server build support for Ubuntu 18.04 This enables server build for Ubuntu 18.04 LTS, the ldiskfs patches are based on Gael's 4.12 support, they apply to kernel versions 4.15.0-20.21 to 4.15.0-23.25 There's also a small fix to make dpkg happy when installing lustre packages which requires lustre-client-utils. Test-Parameters: clientdistro=ubuntu1604 trivial Noticed the patch did not only update URL but also updated some copyright, something like following: Lustre is a scalable, secure, robust, highly-available cluster file system. - This release is maintained by Intel Corporation and available from - https://wiki.hpdd.intel.com/ + This release is maintained by Whamcloud and available from + https://wiki.whamcloud.com/ So it depends on how we are going to support for b2_10, is Ubuntu18.04 supposed to be supported on b2_10 branch? or we could port parts of this patch to fix these copyright issue.
          pjones Peter Jones added a comment -

          Too bad we did not catch this before 2.10.7 went out but at least this is correct for all releases going forward

          pjones Peter Jones added a comment - Too bad we did not catch this before 2.10.7 went out but at least this is correct for all releases going forward

          There are still links in debian/ on b2_10:

          $> git checkout b2_10
          $> grep -Rn "hpdd"
          debian/control:8:Homepage: https://wiki.hpdd.intel.com/
          debian/control.main:8:Homepage: https://wiki.hpdd.intel.com/
          bougetq Quentin Bouget (Inactive) added a comment - - edited There are still links in debian/ on b2_10: $> git checkout b2_10 $> grep -Rn "hpdd" debian/control:8:Homepage: https://wiki.hpdd.intel.com/ debian/control.main:8:Homepage: https://wiki.hpdd.intel.com/

          John L. Hammond (jhammond@whamcloud.com) merged in patch https://review.whamcloud.com/33109/
          Subject: LU-11304 misc: update all url links to whamcloud
          Project: fs/lustre-release
          Branch: b2_10
          Current Patch Set:
          Commit: 653d84962cb8022891c08afcd8339863667b0df3

          gerrit Gerrit Updater added a comment - John L. Hammond (jhammond@whamcloud.com) merged in patch https://review.whamcloud.com/33109/ Subject: LU-11304 misc: update all url links to whamcloud Project: fs/lustre-release Branch: b2_10 Current Patch Set: Commit: 653d84962cb8022891c08afcd8339863667b0df3

          Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33109
          Subject: LU-11304 misc: update all url links to whamcloud
          Project: fs/lustre-release
          Branch: b2_10
          Current Patch Set: 1
          Commit: 93cfd713474a0476d349f42bad67460f1cf7fd82

          gerrit Gerrit Updater added a comment - Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33109 Subject: LU-11304 misc: update all url links to whamcloud Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 93cfd713474a0476d349f42bad67460f1cf7fd82
          pjones Peter Jones added a comment -

          Landed for 2.12

          pjones Peter Jones added a comment - Landed for 2.12

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33094/
          Subject: LU-11304 misc: update all url links to whamcloud
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 605a139a193e4ef212560673b4bb93eec763d088

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33094/ Subject: LU-11304 misc: update all url links to whamcloud Project: fs/lustre-release Branch: master Current Patch Set: Commit: 605a139a193e4ef212560673b4bb93eec763d088

          Wang Shilong (wshilong@ddn.com) uploaded a new patch: https://review.whamcloud.com/33094
          Subject: LU-11304 misc: update all url links to whamcloud
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: e5fb1ef8ff626040e8ce8478ecf080e9e42f3f18

          gerrit Gerrit Updater added a comment - Wang Shilong (wshilong@ddn.com) uploaded a new patch: https://review.whamcloud.com/33094 Subject: LU-11304 misc: update all url links to whamcloud Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: e5fb1ef8ff626040e8ce8478ecf080e9e42f3f18

          People

            wshilong Wang Shilong (Inactive)
            wshilong Wang Shilong (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: