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

chore(ci): fix windows envs with bash shell for s3 uploads #6066

Merged

Conversation

leet4tari
Copy link
Contributor

Description
Use the bash shell to update GitHub Action Workflows envs for later use in S3 upload step.

Motivation and Context
Finally find out why env not updating for Windows workflows, seems that we need to use the bash shell to make changes.

How Has This Been Tested?
Run workflows in test branch from the repo

What process can a PR reviewer use to test or verify this change?
Check the s3 upload on Windows for any errors.

Breaking Changes

  • None
  • Requires data directory on base node to be deleted
  • Requires hard fork
  • Other - Please specify

@leet4tari leet4tari requested a review from brianp January 2, 2024 10:44
Copy link
Contributor

@brianp brianp left a comment

Choose a reason for hiding this comment

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

LGTM utAck

What's the default shell on windows?

@ghpbot-tari-project ghpbot-tari-project added the P-acks_required Process - Requires more ACKs or utACKs label Jan 2, 2024
Copy link

github-actions bot commented Jan 2, 2024

Test Results (CI)

1 264 tests   1 263 ✅  13m 28s ⏱️
   39 suites      0 💤
    1 files        1 ❌

For more details on these failures, see this check.

Results for commit 5201354.

Copy link

github-actions bot commented Jan 2, 2024

Test Results (Integration tests)

29 tests  +29   29 ✅ +29   11m 49s ⏱️ + 11m 49s
11 suites +11    0 💤 ± 0 
 2 files   + 2    0 ❌ ± 0 

Results for commit 5201354. ± Comparison against base commit 3512b74.

@leet4tari
Copy link
Contributor Author

LGTM utAck

What's the default shell on windows?
PowerShell

@leet4tari leet4tari merged commit 71c5d9d into tari-project:development Jan 8, 2024
12 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-acks_required Process - Requires more ACKs or utACKs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants