Skip to content

Conversation

@jmarrero
Copy link
Member

Schema for new image as part of: https://issues.redhat.com/browse/COS-1646. Image build happening on: #2985

@jmarrero jmarrero requested a review from cgwalters August 25, 2022 21:04
cgwalters
cgwalters previously approved these changes Aug 25, 2022
@cgwalters cgwalters enabled auto-merge (rebase) August 25, 2022 21:06
@jmarrero
Copy link
Member Author

[2022-08-25T21:32:05.957Z] Aug 25 21:32:05 qemu0 kola-runext-package[1715]: error: Updating rpm-md repo 'fedora-cisco-openh264': cannot update repo 'fedora-cisco-openh264': Cannot download repomd.xml: Cannot download repodata/repomd.xml: All mirrors were tried; Last error: Downloading successful, but checksum doesn't match. Calculated: 0db88912f19858ae04ce3d89ad818679f2844afc63a4cf2ead17ecfe158be012cf9fc6b233ce61563b7215cc8583f3d0c585fd7d84769eed381eea49d9c51fd6(sha512)  Expected: 402d3374ade9cd9a8a35f738f1e215c1962d6aedc411524d2050d359519474f92c3826779ac518773fb44a88f0275981ec24976c7eb845dc775d146a18de1376(sha512)

this has been happening today all over the place... unrelated failure.

@dustymabe dustymabe disabled auto-merge August 25, 2022 22:39
@dustymabe
Copy link
Member

dustymabe commented Aug 25, 2022

Hey @jmarrero - I'm wondering if we should drop the oscontainer from the naming here. My understanding is that name means something akin to "OSTree in a container", which this isn't that IIUC. Should we consider naming it something more appropriate?

Maybe extensions-container?

@jmarrero
Copy link
Member Author

Hey @jmarrero - I'm wondering if we should drop the oscontainer from the naming here. My understanding is that name means something akin to "OSTree in a container", which this isn't that IIUC. Should we consider naming it something more appropriate?

Maybe extensions-container?

No idea what the os meant then. I assumed it meant Operating System Container and since these extensions need to be built based on the base-oscontainer then I assumed the extensions-oscontainer. Made sense. The name does not matter much to me. I am OK with extensions-container if it makes everyone happy.

@jmarrero jmarrero force-pushed the extcontainer-schema branch from b3a10d6 to bb45fe1 Compare August 26, 2022 13:21
@jmarrero jmarrero changed the title schema: Add entry for extension-oscontainer schema: Add entry for extensions-container Aug 26, 2022
@jmarrero jmarrero requested a review from dustymabe August 26, 2022 15:27
@cgwalters
Copy link
Member

Needs a rebase 🏄

@jmarrero jmarrero force-pushed the extcontainer-schema branch from bb45fe1 to 58ad17c Compare August 29, 2022 19:46
@jmarrero jmarrero requested a review from cgwalters August 29, 2022 19:46
Copy link
Member

@cgwalters cgwalters left a comment

Choose a reason for hiding this comment

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

This looks good to me as is, but I think we also want to add it to the list of artifacts.

@cgwalters cgwalters enabled auto-merge (rebase) August 29, 2022 20:13
@cgwalters cgwalters merged commit 5b01a2f into coreos:main Aug 29, 2022
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