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

GCP: GCP tarball is wrongly referenced in metadata #1078

Closed
yuqi-zhang opened this issue Jan 27, 2020 · 5 comments · Fixed by #1079
Closed

GCP: GCP tarball is wrongly referenced in metadata #1078

yuqi-zhang opened this issue Jan 27, 2020 · 5 comments · Fixed by #1079
Assignees

Comments

@yuqi-zhang
Copy link
Contributor

Bug Report

Environment

RHCOS pipeline

Expected Behavior

Generates correct tarball

Actual Behavior

Generates incorrect tarball location

Reproduction Steps

Example failure: openshift/installer#2996

Other Information

Likely due to rework in 1dc3cdd where the reference names were changed

@cgwalters cgwalters self-assigned this Jan 28, 2020
@cgwalters
Copy link
Member

Taking this since I think @darkmuggle is travelling.

It seems to me we still have the dividing lines between cosa/mantle here aren't right - in this case, it's ore that's doing the upload, and knows the URL where the content can be found, but we're reconstructing that URL (in this case incorrectly) in cosa.

cgwalters added a commit to cgwalters/mantle that referenced this issue Jan 28, 2020
coreos-assembler was trying to re-create the URL that ore computed
for the storage, and a recent refactoring broke that.

Add an option to write out the URL we used so we only need
to have its computation in one place.

Specifically, cosa started replacing `.` with `-` but ore wasn't.

xref coreos/coreos-assembler#1078
cgwalters added a commit to cgwalters/coreos-assembler that referenced this issue Jan 28, 2020
Get the URL from what mantle computed rather than trying to re-compute
it here.

Closes: coreos#1078
cgwalters added a commit to cgwalters/coreos-assembler that referenced this issue Jan 28, 2020
Get the URL from what mantle computed rather than trying to re-compute
it here.

Closes: coreos#1078
miabbott pushed a commit that referenced this issue Jan 28, 2020
Get the URL from what mantle computed rather than trying to re-compute
it here.

Closes: #1078
@miabbott
Copy link
Member

Hmm...is this actually fixed?

The last known good build had the following:

"gcp": {
        "image": "rhcos-44-81-202001241431-0",
        "url": "https://storage.googleapis.com/rhcos/rhcos/44.81.202001241431.0.tar.gz"
    }

The next build that failed had:

"gcp": {
        "image": "rhcos-44-81-202001241932-0-gcp-x86-64",
        "url": "https://storage.googleapis.com/rhcos/rhcos/rhcos-44.81.202001241932.0-gcp.x86_64.tar.gz"
    }

And after #1079, I see:

"gcp": {
        "image": "rhcos-44-81-202001291108-0-gcp-x86-64",
        "url": "https://storage.googleapis.com/rhcos/rhcos/rhcos-44-81-202001291108-0-gcp-x86-64.tar.gz"
    }

@cgwalters
Copy link
Member

What's wrong with the new one?

@yuqi-zhang
Copy link
Contributor Author

In the rework we changed the naming schema. The problem was not the new name, but rather a disconnect between the new name of the actual tarball and the new name in the meta.json file

@miabbott
Copy link
Member

I failed to notice the subtle changes of dots to dashes in the file name...sorry for the noise!

jcajka pushed a commit to jcajka/coreos-assembler that referenced this issue Mar 24, 2020
kola/spawn: fix using v2 for FCOS when using -k
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 a pull request may close this issue.

3 participants