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.

Sign up for GitHub

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

Jump to bottom

Makefile: point to cargo cache when running check-clippy #2868

Merged
merged 2 commits into from
Mar 8, 2023
Merged

Makefile: point to cargo cache when running check-clippy #2868

merged 2 commits into from
Mar 8, 2023

Conversation

mjsterckx
Copy link
Contributor

@mjsterckx mjsterckx commented Mar 8, 2023 •
edited
Loading

Issue number:

N/A

Description of changes:

Mounted only relevant directories to docker container (sources, tools, and .cargo) instead of the whole working directory. Set the CARGO_HOME directory in the docker container to /tmp/.cargo so it would find the registry cache and not have to download, compile, and check each crate every time check-clippy is invoked.

Removed pinned Rust version since we use clippy from the SDK.

Testing done:

Ran cargo make check-clippy twice: first time had to download and compile several crates, second time finished immediately.

ubuntu:~/dev/bottlerocket$ cargo make check-clippy
[cargo-make] INFO - cargo make 0.36.0
[cargo-make] INFO - Build File: Makefile.toml
[cargo-make] INFO - Task: check-clippy
[cargo-make] INFO - Profile: development
[cargo-make] INFO - Running Task: check-clippy
   Compiling ring v0.16.20
   Compiling bottlerocket-variant v0.1.0 (/tmp/sources/bottlerocket-variant)
   Compiling rustls v0.20.8
    Checking testsys-config v0.1.0 (/tmp/tools/testsys-config)
    Checking webpki v0.22.0
    Checking sct v0.7.0
    Checking webpki-roots v0.22.6
    Checking tokio-rustls v0.23.4
    Checking hyper-rustls v0.23.2
    Checking aws-smithy-client v0.54.1
    Checking reqwest v0.11.13
    Checking tough v0.13.0
    Checking buildsys v0.1.0 (/tmp/tools/buildsys)
    Checking pubsys-setup v0.1.0 (/tmp/tools/pubsys-setup)
    Checking aws-types v0.54.1
    Checking aws-endpoint v0.54.1
    Checking aws-http v0.54.1
    Checking aws-sig-auth v0.54.1
    Checking aws-sdk-sso v0.24.0
    Checking aws-sdk-sts v0.24.0
    Checking aws-sdk-ec2 v0.24.0
    Checking aws-sdk-ssm v0.24.0
    Checking aws-sdk-kms v0.24.0
    Checking aws-sdk-ebs v0.24.0
    Checking aws-sdk-cloudformation v0.24.0
    Checking aws-sdk-s3 v0.24.0
    Checking aws-config v0.54.1
    Checking tough-kms v0.5.0
    Checking infrasys v0.1.0 (/tmp/tools/infrasys)
    Checking tough-ssm v0.8.0
    Checking coldsnap v0.4.3
    Checking testsys v0.1.0 (/tmp/tools/testsys)
    Checking pubsys v0.1.0 (/tmp/tools/pubsys)
    Finished dev [unoptimized + debuginfo] target(s) in 1m 01s
   Compiling ring v0.16.20
   Compiling bottlerocket-variant v0.1.0 (/tmp/sources/bottlerocket-variant)
   Compiling apiclient v0.1.0 (/tmp/sources/api/apiclient)
   Compiling imdsclient v0.1.0 (/tmp/sources/imdsclient)
   Compiling rustls v0.20.8
   Compiling apiserver v0.1.0 (/tmp/sources/api/apiserver)
   Compiling thar-be-settings v0.1.0 (/tmp/sources/api/thar-be-settings)
   Compiling shibaken v0.1.0 (/tmp/sources/api/shibaken)
   Compiling metricdog v0.1.0 (/tmp/sources/metricdog)
   Compiling corndog v0.1.0 (/tmp/sources/api/corndog)
   Compiling host-containers v0.1.0 (/tmp/sources/api/host-containers)
   Compiling driverdog v0.1.0 (/tmp/sources/driverdog)
   Compiling shimpei v0.1.0 (/tmp/sources/shimpei)
   Compiling sundog v0.1.0 (/tmp/sources/api/sundog)
   Compiling cfsignal v0.1.0 (/tmp/sources/cfsignal)
   Compiling prairiedog v0.1.0 (/tmp/sources/api/prairiedog)
   Compiling settings-committer v0.1.0 (/tmp/sources/api/settings-committer)
   Compiling certdog v0.1.0 (/tmp/sources/api/certdog)
   Compiling models v0.1.0 (/tmp/sources/models)
   Compiling netdog v0.1.0 (/tmp/sources/api/netdog)
   Compiling add-k8s-autoscaling-warm-pool-setting-metadata v0.1.0 (/tmp/sources/api/migration/migrations/v1.12.0/add-k8s-autoscaling-warm-pool-setting-metadata)
   Compiling oci-defaults-setting-metadata v0.1.0 (/tmp/sources/api/migration/migrations/v1.12.0/oci-defaults-setting-metadata)
   Compiling oci-defaults-setting v0.1.0 (/tmp/sources/api/migration/migrations/v1.12.0/oci-defaults-setting)
   Compiling add-k8s-autoscaling-warm-pool-setting v0.1.0 (/tmp/sources/api/migration/migrations/v1.12.0/add-k8s-autoscaling-warm-pool-setting)
   Compiling static-pods v0.1.0 (/tmp/sources/api/static-pods)
   Compiling early-boot-config v0.1.0 (/tmp/sources/api/early-boot-config)
   Compiling ecs-settings-applier v0.1.0 (/tmp/sources/api/ecs-settings-applier)
   Compiling pluto v0.1.0 (/tmp/sources/api/pluto)
   Compiling logdog v0.1.0 (/tmp/sources/logdog)
    Checking sct v0.7.0
    Checking webpki v0.22.0
    Checking sct v0.6.1
    Checking webpki v0.21.4
    Checking aws-sigv4 v0.48.0
    Checking ct-logs v0.8.0
    Checking rustls v0.19.1
    Checking rustls-native-certs v0.5.0
    Checking tokio-rustls v0.22.0
    Checking hyper-rustls v0.22.1
    Checking aws-smithy-client v0.48.0
    Checking hyper-proxy v0.9.1
    Checking tokio-rustls v0.23.4
    Checking hyper-rustls v0.23.2
    Checking reqwest v0.11.13
    Checking aws-types v0.48.0
    Checking aws-endpoint v0.48.0
    Checking aws-http v0.48.0
    Checking aws-sig-auth v0.48.0
   Compiling storewolf v0.1.0 (/tmp/sources/api/storewolf)
    Checking aws-sdk-sso v0.18.0
    Checking aws-sdk-sts v0.18.0
    Checking aws-sdk-cloudformation v0.18.0
    Checking aws-sdk-eks v0.18.0
    Checking tough v0.13.0
    Checking schnauzer v0.1.0 (/tmp/sources/api/schnauzer)
    Checking thar-be-updates v0.1.0 (/tmp/sources/api/thar-be-updates)
    Checking bootstrap-containers v0.1.0 (/tmp/sources/api/bootstrap-containers)
    Checking aws-config v0.48.0
    Checking migration-helpers v0.1.0 (/tmp/sources/api/migration/migration-helpers)
    Checking k8s-private-pki-path v0.1.0 (/tmp/sources/api/migration/migrations/v1.12.0/k8s-private-pki-path)
    Checking public-admin-container-v0-9-4 v0.1.0 (/tmp/sources/api/migration/migrations/v1.12.0/public-admin-container-v0-9-4)
    Checking aws-admin-container-v0-9-4 v0.1.0 (/tmp/sources/api/migration/migrations/v1.12.0/aws-admin-container-v0-9-4)
    Checking public-control-container-v0-7-0 v0.1.0 (/tmp/sources/api/migration/migrations/v1.12.0/public-control-container-v0-7-0)
    Checking k8s-registry v0.1.0 (/tmp/sources/api/migration/migrations/v1.13.0/k8s-registry)
    Checking aws-control-container-v0-7-0 v0.1.0 (/tmp/sources/api/migration/migrations/v1.12.0/aws-control-container-v0-7-0)
    Checking updog v0.1.0 (/tmp/sources/updater/updog)
    Checking migrator v0.1.0 (/tmp/sources/api/migration/migrator)
    Finished dev [unoptimized + debuginfo] target(s) in 10.44s
[cargo-make] INFO - Build Done in 76.84 seconds.
ubuntu:~/dev/bottlerocket$ cargo make check-clippy
[cargo-make] INFO - cargo make 0.36.0
[cargo-make] INFO - Build File: Makefile.toml
[cargo-make] INFO - Task: check-clippy
[cargo-make] INFO - Profile: development
[cargo-make] INFO - Running Task: check-clippy
    Finished dev [unoptimized + debuginfo] target(s) in 0.25s
    Finished dev [unoptimized + debuginfo] target(s) in 0.28s
[cargo-make] INFO - Build Done in 2.27 seconds.

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

@mjsterckx mjsterckx force-pushed the check-clippy-cache-fix branch from 8b655b4 to 58ef8de Compare March 8, 2023 15:36
@mjsterckx mjsterckx marked this pull request as ready for review March 8, 2023 15:43
@mjsterckx mjsterckx requested review from jpculp and etungsten March 8, 2023 15:43
jpmcb
jpmcb approved these changes Mar 8, 2023
View reviewed changes
Copy link
Contributor

@jpmcb jpmcb left a comment

Choose a reason for hiding this comment

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

👏🏼

stmcginnis
stmcginnis approved these changes Mar 8, 2023
View reviewed changes
Copy link
Contributor

@stmcginnis stmcginnis left a comment

Choose a reason for hiding this comment

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

Nice!

jpculp
jpculp requested changes Mar 8, 2023
View reviewed changes
.github/workflows/build.yml Outdated Show resolved Hide resolved
@mjsterckx mjsterckx force-pushed the check-clippy-cache-fix branch from 58ef8de to 9bcb97a Compare March 8, 2023 19:09
@mjsterckx mjsterckx force-pushed the check-clippy-cache-fix branch from 9bcb97a to 9af0075 Compare March 8, 2023 19:10
@mjsterckx mjsterckx requested a review from jpculp March 8, 2023 19:11
webern
webern approved these changes Mar 8, 2023
View reviewed changes
Copy link
Contributor

@webern webern left a comment

Choose a reason for hiding this comment

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

Looks good. Wait for CI to pass though.

@mjsterckx mjsterckx merged commit 7d17487 into bottlerocket-os:develop Mar 8, 2023
@mjsterckx mjsterckx deleted the check-clippy-cache-fix branch March 8, 2023 20:18
@mjsterckx mjsterckx mentioned this pull request Mar 8, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stmcginnis stmcginnis stmcginnis approved these changes

@jpculp jpculp jpculp approved these changes

@webern webern webern approved these changes

@jpmcb jpmcb jpmcb approved these changes

@etungsten etungsten etungsten approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@mjsterckx @stmcginnis @jpculp @webern @jpmcb @etungsten