Skip to content

Conversation

@b-boogaard
Copy link
Contributor

Update the GitHub actions in cd.yml to use DigiCert's KeyLocker and cloud based signing solutions for increased code signing certificate security.

@b-boogaard b-boogaard requested a review from ppolewicz July 25, 2025 17:51
@b-boogaard
Copy link
Contributor Author

I'm not sure what's going on with these failures but I'm looking into it.

@b-boogaard
Copy link
Contributor Author

It looks like some tests are failing because in some cases the list of buckets that an app key is restricted to is coming back in a different order;

['clitstgetq93ee73cj9ni6m9b2sv0wng0evk5qorhksupl72-upmo6xp27g8cct',
 'clitstgetq93ee2976bia7oll2y8meyssvhhcvq54yu-z2n0r2w5b--xcozbkhn']

vs

['clitstgetq93ee2976bia7oll2y8meyssvhhcvq54yu-z2n0r2w5b--xcozbkhn',
 'clitstgetq93ee73cj9ni6m9b2sv0wng0evk5qorhksupl72-upmo6xp27g8cct']

@olzhasar-reef
Copy link
Collaborator

Hey @b-boogaard,

I'm currently working on the fix for the tests problem you've mentioned.
Will be merged soon.

@b-boogaard b-boogaard force-pushed the bboogaard/update-code-signing branch from 121a06d to 777847c Compare July 25, 2025 20:07
@b-boogaard
Copy link
Contributor Author

Thanks @olzhasar-reef!

@b-boogaard
Copy link
Contributor Author

The multi-bucket issue is resolved now 🎉 However, it's looking like python 3.8 is having a couple problems with some sync operations. I'm not sure how my changes might be causing this but I'll spend some time digging into it.

Co-authored-by: Stephen Byrd <30883208+byrdsteve@users.noreply.github.com>
Copy link
Contributor

@byrdsteve byrdsteve left a comment

Choose a reason for hiding this comment

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

LGTM

@b-boogaard b-boogaard merged commit 533b731 into master Jul 25, 2025
30 checks passed
@b-boogaard b-boogaard deleted the bboogaard/update-code-signing branch July 25, 2025 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants