Skip to content

Commit

Permalink
chore: remove retry-action action on registry1 docker login (#160)
Browse files Browse the repository at this point in the history
## Description

revert using retry action on the docker login action for logging into
registry1. A separate, followup issue will address this differently in
the future. This is just reverting the change that added the reliance on
the 3rd party action that wasnt serving its purpose and had a lacking
update cadence.


## Type of change

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [x] Other (security config, docs update, etc)

## Checklist before merging

- [ ] Test, docs, adr added or updated as needed
- [ ] [Contributor Guide
Steps](https://github.com/defenseunicorns/uds-template-capability/blob/main/CONTRIBUTING.md)(https://github.com/defenseunicorns/uds-template-capability/blob/main/CONTRIBUTING.md#submitting-a-pull-request)
followed
  • Loading branch information
zachariahmiller authored Feb 8, 2024
1 parent 2c493bb commit eea0c93
Showing 1 changed file with 5 additions and 8 deletions.
13 changes: 5 additions & 8 deletions .github/actions/setup/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,12 +39,9 @@ runs:
username: dummy
password: ${{ inputs.gh_token }}

# Retries intermittent registry1 login action
- uses: Wandalen/wretry.action@v1
- name: Login to registry1
uses: docker/login-action@v3
with:
attempt_limit: 3
action: docker/login-action@v3
with: |
registry: registry1.dso.mil
username: ${{ inputs.ib_user }}
password: ${{ inputs.ib_password }}
registry: registry1.dso.mil
username: ${{ inputs.ib_user }}
password: ${{ inputs.ib_password }}

0 comments on commit eea0c93

Please sign in to comment.