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

build images weekly, without cache #789

Merged
merged 2 commits into from
Jun 16, 2022
Merged

Conversation

Emilgardis
Copy link
Member

No description provided.

@Emilgardis Emilgardis requested a review from a team as a code owner June 13, 2022 21:30
@Emilgardis Emilgardis added the no changelog A valid PR without changelog (no-changelog) label Jun 13, 2022
@Emilgardis
Copy link
Member Author

Emilgardis commented Jun 13, 2022

this gives me the idea to also start making progress into buildx, it'd be cool if build-docker-images had a flag for outputting a bake file (and use a bake build for when there's multiple targets specified)

@Emilgardis Emilgardis force-pushed the no-cache-runs branch 4 times, most recently from d136a91 to 92f5a4a Compare June 14, 2022 03:28
@Emilgardis Emilgardis marked this pull request as draft June 14, 2022 11:34
@Emilgardis
Copy link
Member Author

going to split this out into two prs

@Emilgardis Emilgardis changed the title build images daily, without cache build images weekly, without cache Jun 14, 2022
@Emilgardis
Copy link
Member Author

depends on #791

@Emilgardis Emilgardis marked this pull request as ready for review June 14, 2022 17:48
@Emilgardis
Copy link
Member Author

this could also maybe implement parts of #610, checking that mounts works correctly

Copy link
Contributor

@Alexhuszagh Alexhuszagh left a comment

Choose a reason for hiding this comment

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

Minor nit.

.github/workflows/weekly.yml Outdated Show resolved Hide resolved
Comment on lines +104 to +114
.map(|m| {
format!(
"{}{}",
m.target,
if let Some(sub) = &m.sub {
format!(".{sub}")
} else {
<_>::default()
}
)
})
Copy link
Member Author

Choose a reason for hiding this comment

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

fixed this too, apparently it went away in some rebase somewhere

Copy link
Contributor

@Alexhuszagh Alexhuszagh left a comment

Choose a reason for hiding this comment

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

lgtm

@Alexhuszagh
Copy link
Contributor

bors r+

@bors
Copy link
Contributor

bors bot commented Jun 16, 2022

Build succeeded:

@bors bors bot merged commit 38ee4f2 into cross-rs:main Jun 16, 2022
@Emilgardis Emilgardis deleted the no-cache-runs branch June 16, 2022 20:07
@Emilgardis Emilgardis added this to the v0.2.2 milestone Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog A valid PR without changelog (no-changelog)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants