Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-1998

Toolchain installer needs to switch from CGI to Lua mirror selection

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Critical
    • Resolution: Won't Fix
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.1.0
    • Component/s: toolchain
    • Labels:
      None

      Description

      Let's switch mirror selection from CGI to Lua to benefit from its performance, etc.

      1. BIGTOP-1998.patch
        0.7 kB
        Konstantin Boudnik
      2. BIGTOP-1998.patch
        1 kB
        Konstantin Boudnik
      3. BIGTOP-1998.patch
        1 kB
        Konstantin Boudnik
      4. BIGTOP-1998.patch
        1 kB
        Konstantin Boudnik

        Activity

        Hide
        cos Konstantin Boudnik added a comment -

        Looks like the current *.cgi works fine vie redirect or something. We'll get back to it if needed later.

        Show
        cos Konstantin Boudnik added a comment - Looks like the current *.cgi works fine vie redirect or something. We'll get back to it if needed later.
        Hide
        cos Konstantin Boudnik added a comment -

        ok, I will just close it. If we hit the snag later - we'll fix it Thanks!

        Show
        cos Konstantin Boudnik added a comment - ok, I will just close it. If we hit the snag later - we'll fix it Thanks!
        Hide
        oflebbe Olaf Flebbe added a comment -

        Is this actually needed? It works right now without this patch. And I technically do not understand why this should be necessary at al. Would propose to double check with INFRA.

        Show
        oflebbe Olaf Flebbe added a comment - Is this actually needed? It works right now without this patch. And I technically do not understand why this should be necessary at al. Would propose to double check with INFRA.
        Hide
        cos Konstantin Boudnik added a comment -

        Olaf Flebbe - any objections for the latest patch?

        Show
        cos Konstantin Boudnik added a comment - Olaf Flebbe - any objections for the latest patch?
        Hide
        cos Konstantin Boudnik added a comment -

        Looks like the original patch is no longer valid indeed. Simply replacing .cgi with .lua is sufficient, but it works as it is as well. I'd say let's close it

        Show
        cos Konstantin Boudnik added a comment - Looks like the original patch is no longer valid indeed. Simply replacing .cgi with .lua is sufficient, but it works as it is as well. I'd say let's close it
        Hide
        cos Konstantin Boudnik added a comment -

        Yeah, looks like the INFRA had wrapper the lua script into cgi. Gimme me a few minutes to validate that assumption. I will close it if everything works as it stands right now

        Show
        cos Konstantin Boudnik added a comment - Yeah, looks like the INFRA had wrapper the lua script into cgi. Gimme me a few minutes to validate that assumption. I will close it if everything works as it stands right now
        Hide
        oflebbe Olaf Flebbe added a comment - - edited

        Seems that it is not needed. Ok to close as won't fix?

        Show
        oflebbe Olaf Flebbe added a comment - - edited Seems that it is not needed. Ok to close as won't fix?
        Hide
        cos Konstantin Boudnik added a comment -

        Yeah, you're right - let's wait for a couple of days...

        Show
        cos Konstantin Boudnik added a comment - Yeah, you're right - let's wait for a couple of days...
        Hide
        oflebbe Olaf Flebbe added a comment - - edited

        I just wanted to say that enumarting patches is more user-friendly. It seems that infra is changing their lua script rapidly, since your old script didn't work for me yesterday, it is now working for me. Your new patch does NOT work for me. -1.

        curl  https://www.apache.org/dyn/closer.lua?as_json=1 
        {"backup":["http://www.eu.apache.org/dist/","http://www.us.apache.org/dist/"],"cca2":"backup","http":["http://www.eu.apache.org/dist/","http://www.us.apache.org/dist/"],"path_info":"","preferred":"http://www.eu.apache.org/dist/"}

        i.e. no "strong" in output

        Yesterday the output was HTML, not json even with the argument as_json=1

        Perhaps we should wait until the situation settles a bit?

        Show
        oflebbe Olaf Flebbe added a comment - - edited I just wanted to say that enumarting patches is more user-friendly. It seems that infra is changing their lua script rapidly, since your old script didn't work for me yesterday, it is now working for me. Your new patch does NOT work for me. -1. curl https: //www.apache.org/dyn/closer.lua?as_json=1 { "backup" :[ "http: //www.eu.apache.org/dist/" , "http://www.us.apache.org/dist/" ], "cca2" : "backup" , "http" :[ "http://www.eu.apache.org/dist/" , "http://www.us.apache.org/dist/" ], "path_info" : ""," preferred ":" http://www.eu.apache.org/dist/"} i.e. no "strong" in output Yesterday the output was HTML, not json even with the argument as_json=1 Perhaps we should wait until the situation settles a bit?
        Hide
        cos Konstantin Boudnik added a comment -

        Olaf Flebbe, JIRA marks all patch links but the latest in gray'ish color. Basically, grab the blue one. Or sort them by date-descending and look at the top one.

        Show
        cos Konstantin Boudnik added a comment - Olaf Flebbe , JIRA marks all patch links but the latest in gray'ish color. Basically, grab the blue one. Or sort them by date-descending and look at the top one.
        Hide
        oflebbe Olaf Flebbe added a comment - - edited

        Konstantin Boudnik Which patch should I review? Enumerating patches would help me. The patches seem to be identically ?

        Show
        oflebbe Olaf Flebbe added a comment - - edited Konstantin Boudnik Which patch should I review? Enumerating patches would help me. The patches seem to be identically ?
        Hide
        cos Konstantin Boudnik added a comment -

        Arghhh ... the format of the page got changed somewhat. Addressed in the latest patch

        Show
        cos Konstantin Boudnik added a comment - Arghhh ... the format of the page got changed somewhat. Addressed in the latest patch
        Hide
        cos Konstantin Boudnik added a comment -

        Ready

        Show
        cos Konstantin Boudnik added a comment - Ready

          People

          • Assignee:
            cos Konstantin Boudnik
            Reporter:
            cos Konstantin Boudnik
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development