Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-2800

build: clean out old autoconf options

Details

    • 3
    • 6778

    Description

      It currently takes longer to run configure on the Lustre source than it does to build it. There are a ton of configure tests that can be removed in light of the support matrix for 2.4+

      Attachments

        Issue Links

          Activity

            [LU-2800] build: clean out old autoconf options

            All patches have been landed to master. This ticket can be closed.

            simmonsja James A Simmons added a comment - All patches have been landed to master. This ticket can be closed.
            simmonsja James A Simmons added a comment - Only three patches to go. Please review the following patches. http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/8102 http://review.whamcloud.com/#/c/5380 - depends on patch http://review.whamcloud.com/#/c/7135 from LU-3319
            simmonsja James A Simmons added a comment - We are down to the last few patches. They are: http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5383 http://review.whamcloud.com/#/c/5546 http://review.whamcloud.com/#/c/8102 http://review.whamcloud.com/#/c/5380 - depends on patch http://review.whamcloud.com/#/c/7135 for LU-3319
            simmonsja James A Simmons added a comment - The patches ready for inspection are: http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5374 http://review.whamcloud.com/#/c/5406 http://review.whamcloud.com/#/c/5371 http://review.whamcloud.com/#/c/5546
            simmonsja James A Simmons added a comment - If you have not done so please review the patches below. All of them are ready to be merged. Thank you. http://review.whamcloud.com/#/c/5339 http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5411 http://review.whamcloud.com/#/c/5367 http://review.whamcloud.com/#/c/5403 http://review.whamcloud.com/#/c/5370 http://review.whamcloud.com/#/c/5374 http://review.whamcloud.com/#/c/5406

            Next series is ready. They will not conflict with current master or each other.

            http://review.whamcloud.com/#/c/5339
            http://review.whamcloud.com/#/c/5347
            http://review.whamcloud.com/#/c/5411
            http://review.whamcloud.com/#/c/5367
            http://review.whamcloud.com/#/c/5403

            The following patches are dependent on the above and have to be applied in the specific order below.

            http://review.whamcloud.com/#/c/5370
            http://review.whamcloud.com/#/c/5374
            http://review.whamcloud.com/#/c/5406

            Besides these only a few more patches will be needed after these land. The reason for not posting those patches is that they conflict with multiple patches above.

            simmonsja James A Simmons added a comment - Next series is ready. They will not conflict with current master or each other. http://review.whamcloud.com/#/c/5339 http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5411 http://review.whamcloud.com/#/c/5367 http://review.whamcloud.com/#/c/5403 The following patches are dependent on the above and have to be applied in the specific order below. http://review.whamcloud.com/#/c/5370 http://review.whamcloud.com/#/c/5374 http://review.whamcloud.com/#/c/5406 Besides these only a few more patches will be needed after these land. The reason for not posting those patches is that they conflict with multiple patches above.
            simmonsja James A Simmons added a comment - New series ready to merge. They will not conflict with master or each other. Please review if you haven't already. http://review.whamcloud.com/#/c/5340 http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5348 http://review.whamcloud.com/#/c/5360 http://review.whamcloud.com/#/c/5411
            simmonsja James A Simmons added a comment - The follow patches are the next series to merge. They will not conflict with master or each other. Please review if you haven't already. http://review.whamcloud.com/#/c/5338 http://review.whamcloud.com/#/c/5340 http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5359 http://review.whamcloud.com/#/c/5369 http://review.whamcloud.com/#/c/5411
            simmonsja James A Simmons added a comment - The follow matches make the next series to merge. They will not conflict with master as it stands or each other. http://review.whamcloud.com/#/c/5336 http://review.whamcloud.com/#/c/5340 http://review.whamcloud.com/#/c/5343 http://review.whamcloud.com/#/c/5345 http://review.whamcloud.com/#/c/5347 http://review.whamcloud.com/#/c/5353 http://review.whamcloud.com/#/c/5384 http://review.whamcloud.com/#/c/5393 http://review.whamcloud.com/#/c/5411

            Luckily most of the smaller patches will not collide often with other non LU-2800 patches. The the big is http://review.whamcloud.com/#/c/5505 which can easily end up in the rebase treadmill.

            simmonsja James A Simmons added a comment - Luckily most of the smaller patches will not collide often with other non LU-2800 patches. The the big is http://review.whamcloud.com/#/c/5505 which can easily end up in the rebase treadmill.

            Because of the "painful resynchronizing" process, it would be great if these can get some attention from Oleg before they become stale again. I'd hate to see cleanup like this die due to lack of attention.

            prakash Prakash Surya (Inactive) added a comment - Because of the "painful resynchronizing" process, it would be great if these can get some attention from Oleg before they become stale again. I'd hate to see cleanup like this die due to lack of attention.

            People

              bogl Bob Glossman (Inactive)
              jeffm Jeff Mahoney (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: