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

Remove check for s3cfg #308

Merged
merged 1 commit into from
Aug 30, 2022
Merged

Remove check for s3cfg #308

merged 1 commit into from
Aug 30, 2022

Conversation

nkoenig
Copy link
Contributor

@nkoenig nkoenig commented Aug 30, 2022

Signed-off-by: Nate Koenig [email protected]

🦟 Bug fix

Summary

I moved the upload_doc.sh script from s3cfg to the aws CLI tool and forgot to remove the s3cfg check. This check is no longer needed, and prevents the nightly documentation job from completing successfully.

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Signed-off-by: Nate Koenig <[email protected]>
@github-actions github-actions bot added 🏯 fortress Ignition Fortress 🏰 citadel Ignition Citadel labels Aug 30, 2022
@nkoenig nkoenig mentioned this pull request Aug 30, 2022
8 tasks
Copy link
Contributor

@j-rivero j-rivero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to me like we copy/pasted this from other place. Seems fine to me to supress the change.

@j-rivero
Copy link
Contributor

I moved the upload_doc.sh script from s3cfg to the aws CLI tool and forgot to remove the s3cfg check. This check is no longer needed, and prevents the nightly documentation job from completing successfully.

Out of curiosity, how is the script authenticated to use the AWS cli client?

@nkoenig
Copy link
Contributor Author

nkoenig commented Aug 30, 2022

I moved the upload_doc.sh script from s3cfg to the aws CLI tool and forgot to remove the s3cfg check. This check is no longer needed, and prevents the nightly documentation job from completing successfully.

Out of curiosity, how is the script authenticated to use the AWS cli client?

The AWS cli client relies on the ~/.aws/credentials file, which stores the key information. The credentials file can be generated using:

aws configure set aws_access_key_id YOUR_ACCESS_KEY
aws configure set aws_secret_access_key YOUR_SECRET_KEY
aws configure set default.region us-east-1

@nkoenig nkoenig merged commit 106fd41 into ign-cmake2 Aug 30, 2022
@nkoenig nkoenig deleted the remove_s3cfg branch August 30, 2022 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel 🏯 fortress Ignition Fortress
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants