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

Reorganizing makefile and adding info to rootfs #1350

Merged
merged 2 commits into from
Apr 18, 2022

Conversation

helsaawy
Copy link
Contributor

@helsaawy helsaawy commented Apr 13, 2022

Reorganized makefile to read from top to bottom, and added additional
files to LCOW rootfs that include the time stamp of of the vhd creation,
the image build date, and its full name (pulled from a *.testdata.json
file in the yocto release).

Signed-off-by: Hamza El-Saawy hamzaelsaawy@microsoft.com

Reorganized makefile to read from top to bottom, and added additional
files to LCOW rootfs that include the time stamp of of the vhd creation,
the image build date, and its full name (pulled from a *.testdata.json
file in the LSG release, that appears to be one of the only location
of that information).

Signed-off-by: Hamza El-Saawy <hamzaelsaawy@microsoft.com>
@helsaawy helsaawy requested a review from a team as a code owner April 13, 2022 21:47
@dcantah
Copy link
Contributor

dcantah commented Apr 18, 2022

I guess something to note is this adds a dep on jq being installed (which I feel isn't too crazy) if the testdata file exists in the yocto drop.

Signed-off-by: Hamza El-Saawy <hamzaelsaawy@microsoft.com>
@helsaawy helsaawy merged commit 54a5ad8 into microsoft:master Apr 18, 2022
@helsaawy helsaawy deleted the he/make branch April 18, 2022 19:17
anmaxvl added a commit that referenced this pull request Feb 7, 2023
Sync ADO with upstream to enable including test GCS binaries as
part of dev-pipeline

Related work items: #1311, #1322, #1341, #1343, #1345, #1347, #1348, #1350, #1353, #1354, #1355, #1358, #1361, #1365, #1368, #1369, #1370
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