Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix registry user in credentials output #578

Merged
merged 1 commit into from
Jul 5, 2022

Conversation

rodneyxr
Copy link
Contributor

@rodneyxr rodneyxr commented Jun 30, 2022

Description

Referenced config.ZarfRegistryPushUser instead of the hardcoded zarf-push-user which was incorrectly being displayed in the credentials output after a zarf init.

Related Issue

Fixes #577

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist before merging

  • Tests have been added/updated as necessary (add the needs-tests label)
  • Documentation has been updated as necessary (add the needs-docs label)
  • An ADR has been written as necessary (add the needs-adr label) [ 1 2 3 ]
  • (Optional) Changes have been linted locally with golangci-lint. (NOTE: We haven't turned on lint checks in the pipeline yet so linting may be hard if it shows a lot of lint errors in places that weren't touched by changes. Thus, linting is optional right now.)

@jeff-mccoy jeff-mccoy added bug 🐞 Something isn't working packager labels Jun 30, 2022
@rodneyxr rodneyxr force-pushed the 577-fix-registry-user branch 3 times, most recently from e4316a1 to 8d59eea Compare July 2, 2022 17:43
@rodneyxr
Copy link
Contributor Author

rodneyxr commented Jul 2, 2022

What is the issue with Error: needs-git-sign-off label is banned in the enforce-labels job? I thought I signed my commit correctly 😞

@jeff-mccoy
Copy link
Contributor

Sorry that's really confusing. We require a core-member sign-off of all third party contributions. One of us needs to re-sign your commits and a different member do the approval. You'll still get authorship for the commit, but that way we can track back our reviewers to touching every piece of code. I'll do the sign-off later today.

@jeff-mccoy
Copy link
Contributor

If you hover over the label we try to explain this, but clearly should have something on PR open that let's you know the process.
Screen Shot 2022-07-02 at 1 03 50 PM

@rodneyxr rodneyxr force-pushed the 577-fix-registry-user branch from 8d59eea to e9a6691 Compare July 2, 2022 18:05
@rodneyxr
Copy link
Contributor Author

rodneyxr commented Jul 2, 2022

If you hover over the label we try to explain this, but clearly should have something on PR open that let's you know the process.

Ah I read that incorrectly... I thought it said I had to sign my commits. Thanks for the info!

@jeff-mccoy
Copy link
Contributor

@YrrepNoj sign-off complete, will leave to you for final review. Thanks @rodneyxr for the PR!

@jeff-mccoy jeff-mccoy merged commit 87cb001 into zarf-dev:master Jul 5, 2022
@rodneyxr rodneyxr deleted the 577-fix-registry-user branch July 5, 2022 17:51
Noxsios pushed a commit that referenced this pull request Mar 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Registry username output after init is incorrect
3 participants