Skip to content

Conversation

cataggar
Copy link
Member

@cataggar cataggar commented Apr 3, 2022

This makes it possible to check all the services in parallel jobs. The workflow definitions can be generated when the services are updated.

@ctaggart
Copy link
Contributor

ctaggart commented Apr 3, 2022

I was hoping it would publish 1 job at a time. Looks like it is doing 20 concurrently. :/

https://docs.github.com/en/actions/learn-github-actions/usage-limits-billing-and-administration

@cataggar
Copy link
Member Author

cataggar commented Apr 4, 2022

crates.io had to problem with the concurrent crate updates. The backoff appears that it was just for new crates. docs.rs put the azure_mgmt crates in a "priority: -1, so they get processed when capacity is available. It may take a few days. https://docs.rs/releases/queue

@cataggar cataggar changed the title publish workflows add publish workflows Apr 4, 2022
@cataggar cataggar marked this pull request as ready for review April 4, 2022 16:29
@cataggar cataggar requested a review from bmc-msft April 4, 2022 16:29
@cataggar cataggar requested a review from heaths April 4, 2022 17:20
Copy link
Contributor

@bmc-msft bmc-msft left a comment

Choose a reason for hiding this comment

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

Given these have access to CARG_REGISTRY_TOKEN, this should probably include documentation as to who has access to run these and what the process is to enable giving someone access to run these.

@cataggar
Copy link
Member Author

cataggar commented Apr 5, 2022

Given these have access to CARG_REGISTRY_TOKEN, this should probably include documentation as to who has access to run these and what the process is to enable giving someone access to run these.

The publish actions are intended for running in your own fork where only you have access to run the actions. Do not add the CARGO_REGISTRY_TOKEN to the main repository. Only add it to your repository.

How is that ^^^?

@bmc-msft
Copy link
Contributor

bmc-msft commented Apr 5, 2022

Discussion moved to the internal Teams instance for Azure SDK For Rust.

@cataggar cataggar changed the title add publish workflows test all service crates in parallel. Apr 5, 2022
@cataggar cataggar changed the title test all service crates in parallel. test all service crates in parallel Apr 5, 2022
@cataggar cataggar requested a review from bmc-msft April 5, 2022 19:30
@cataggar cataggar merged commit 9062897 into Azure:main Apr 5, 2022
@cataggar cataggar deleted the publish branch May 6, 2022 23:24
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.

3 participants