Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-29612

Fix Firefly logout

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      Firefly's logout URL is the old oauth2_proxy URL. Override it with the new URL in the Firefly chart.

        Attachments

          Issue Links

            Activity

            Hide
            rra Russ Allbery added a comment -

            Configuration override deployed to data-dev. Gregory is working on a new release, and will close this story once that release is out and the version number bumped everywhere and confirmed working on at least one environment.

            Show
            rra Russ Allbery added a comment - Configuration override deployed to data-dev. Gregory is working on a new release, and will close this story once that release is out and the version number bumped everywhere and confirmed working on at least one environment.
            Hide
            rra Russ Allbery added a comment -

            Upgraded on data-dev, data-int, NCSA int, and NCSA teststand to the new 2.1.1-3 release.

            Show
            rra Russ Allbery added a comment - Upgraded on data-dev, data-int, NCSA int, and NCSA teststand to the new 2.1.1-3 release.
            Hide
            rra Russ Allbery added a comment -

            data-dev and data-int now updated to 2.2.0, which includes the fix for compatibility with Gafaelfawr 2.0.

            Show
            rra Russ Allbery added a comment - data-dev and data-int now updated to 2.2.0, which includes the fix for compatibility with Gafaelfawr 2.0.
            Hide
            gpdf Gregory Dubois-Felsmann added a comment -

            Russ Allbery - Let's figure out on Monday whether we can close this ticket and DM-29613.

            Show
            gpdf Gregory Dubois-Felsmann added a comment - Russ Allbery  - Let's figure out on Monday whether we can close this ticket and DM-29613 .
            Hide
            gpdf Gregory Dubois-Felsmann added a comment -

            Maybe we want to take out the config override again?

            Show
            gpdf Gregory Dubois-Felsmann added a comment - Maybe we want to take out the config override again?
            Hide
            rra Russ Allbery added a comment -

            https://github.com/lsst-sqre/phalanx/pull/725
            https://github.com/lsst-sqre/charts/pull/355

            should queue up the upgrade on all other environments and remove the override of the logout URL. This doesn't handle switching between Firefly and SUIT; we'll get that as part of a later story.

            Show
            rra Russ Allbery added a comment - https://github.com/lsst-sqre/phalanx/pull/725 https://github.com/lsst-sqre/charts/pull/355 should queue up the upgrade on all other environments and remove the override of the logout URL. This doesn't handle switching between Firefly and SUIT; we'll get that as part of a later story.
            Hide
            rra Russ Allbery added a comment -

            Merged the remaining diffs. We'll do rollout as part of deploying the fix for DM-30073

            Show
            rra Russ Allbery added a comment - Merged the remaining diffs. We'll do rollout as part of deploying the fix for DM-30073
            Hide
            rra Russ Allbery added a comment -

            Reviewed by Adam

            Show
            rra Russ Allbery added a comment - Reviewed by Adam

              People

              Assignee:
              rra Russ Allbery
              Reporter:
              rra Russ Allbery
              Reviewers:
              Gregory Dubois-Felsmann
              Watchers:
              Gregory Dubois-Felsmann, Russ Allbery
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.