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

ci: Archive images on shared disk #19

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

doanac
Copy link

@doanac doanac commented Sep 13, 2024

Save files on NFS share between runs for use by other workflows

@doanac
Copy link
Author

doanac commented Sep 13, 2024

Not really for merge just yet. I'm toying with an idea on how to get data we build available to lava. This is just one piece of that process.

@doanac doanac force-pushed the archive-build branch 3 times, most recently from b323470 to 3480c06 Compare September 13, 2024 19:42
Save files on NFS share between runs for use by other workflows

Signed-off-by: Andy Doan <[email protected]>
@EmbeddedAndroid EmbeddedAndroid added the qa Quality Assurance label Sep 13, 2024
@EmbeddedAndroid EmbeddedAndroid added this to the picanha milestone Sep 13, 2024
@doanac
Copy link
Author

doanac commented Sep 13, 2024

@mwasilew - With a change like this, I can set up a file service for you to make authenticated downloads.
The URL format is: /GITHUB_RUN_ID/PLATFORM/ARTIFACT. eg:

curl -v -H "Authentication: token <SUPER SECRET VALUE>" \
 https://quic-yocto-fileserver-1029608027416.us-central1.run.app/10855251928/qcm6490-idp/core-image-minimal-qcm6490-idp.rootfs.ext4

@mwasilew
Copy link

Is the host name going to remain https://quic-yocto-fileserver-1029608027416.us-central1.run.app or there will be some more convenient CNAME?

@doanac
Copy link
Author

doanac commented Sep 16, 2024

Is the host name going to remain https://quic-yocto-fileserver-1029608027416.us-central1.run.app or there will be some more convenient CNAME?

It will be the long name. Otherwise we'll have to set up DNS, a mechanism to keep certs fresh, etc.

@mwasilew
Copy link

That's OK. Just wanted to confirm.

@doanac doanac mentioned this pull request Sep 16, 2024
@EmbeddedAndroid EmbeddedAndroid linked an issue Sep 17, 2024 that may be closed by this pull request
@@ -28,3 +28,10 @@ jobs:
mkdir build
cd build
kas build ../ci/${{ matrix.machine }}.yml
Copy link
Contributor

Choose a reason for hiding this comment

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

How about using

publish:
needs: kas

Copy link
Author

Choose a reason for hiding this comment

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

Sounds good. once we have a bootable image, I'll update this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
qa Quality Assurance
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Build packaging
4 participants