Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Create separate tester artifacts for each build #256

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

Merged
per1234 merged 3 commits into arduino:main from per1234:separate-artifacts
Mar 24, 2021
Merged

Create separate tester artifacts for each build #256

per1234 merged 3 commits into arduino:main from per1234:separate-artifacts
Mar 24, 2021

Conversation

Copy link
Contributor

@per1234 per1234 commented Mar 23, 2021
edited
Loading

Previously, a single workflow artifact was created by the "Arduino IDE" GitHub Actions workflow. This artifact contained
the builds for each operating system, including all three versions of the Windows build. This resulted in beta testers
needing to do a >1 GB download for every build, even though they likely needed only ~200 MB of what they downloaded.

Producing separate workflow artifacts makes it easier for beta testers to participate in the development and is less wasteful of
resources.

per1234 added 2 commits March 22, 2021 22:51
Previously, the build CI/CD workflow had many occurrences of the string "build-artifacts" used for the workflow artifact
name. This made the workflow more difficult to understand and maintain. Now a single workflow scoped environment variable
is used to define the artifact name.
Previously, a single workflow artifact was created by the "Arduino IDE" GitHub Actions workflow. This artifact contained
the builds for each operating system, including all three versions of the Windows build. This resulted in beta testers
needing to do a >1 GB download for every build, even though they likely needed only ~200 MB of what they downloaded.
Producing separate workflows makes it easier for beta testers to participate in the development and is less wasteful of
resources.
Copy link

thank you, @per1234

Copy link
Contributor

@kittaakos kittaakos left a comment

Choose a reason for hiding this comment

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

I tried the macOS artifact, and it worked. Thank you!

I would have moved the artifacts step right after the build one, but it's a personal preference.

The "Arduino IDE" GitHub Actions workflow uses a workflow artifact to transfer the build artifacts between jobs. Now that
separate tester build artifacts are produced, the monolithic job transfer artifact is superfluous once the workflow run
is finished. Deleting it avoids potential confusion for beta testers and unnecessary storage space usage.
Copy link
Contributor Author

per1234 commented Mar 24, 2021

I would have moved the artifacts step right after the build one, but it's a personal preference.

OK, it is now done: https://github.com/arduino/arduino-ide/compare/5cb544c15837e63aace2fd0163a3e187a5713293..a3287ca1de656d8d5442941b85751e6e08f5d6e5

kittaakos reacted with heart emoji

@per1234 per1234 merged commit 3fb087f into arduino:main Mar 24, 2021
@per1234 per1234 added topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement labels Oct 25, 2021
@per1234 per1234 self-assigned this Nov 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@kittaakos kittaakos kittaakos approved these changes

Reviewers whose approvals may not affect merge requirements
Labels
topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /