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

fedora-coreos-base: add in toolbox #201

Merged
merged 1 commit into from
Oct 18, 2019
Merged

Conversation

dustymabe
Copy link
Member

@dustymabe
Copy link
Member Author

IIUC toolbox does not currently handle all of our use cases but it will in the future so let's go ahead and include it so we can fix coreos/fedora-coreos-tracker#38 and work through the issues.

@dustymabe
Copy link
Member Author

Note that there is a current issue to work out but is being worked on: containers/toolbox#267

@dustymabe
Copy link
Member Author

weird CI issue:

[  436.104942] reboot: machine restart

New image input checksum: 7a3a60563ca2722e6a8ccfcfffc311f7f2648ff6f0b0fad7daa5da4d59644da4

None

New build ID: 30.20191016.dev.0

3226 metadata, 18856 content objects imported

parse error: Unmatched '}' at line 15, column 5

parse error: Unmatched '}' at line 26, column 44

script returned exit code 4

@dustymabe
Copy link
Member Author

cc @cgwalters for input

@dustymabe
Copy link
Member Author

also has anyone seen these deprecation warnings:

compose tree:305): librepo-WARNING **: 18:48:35.107: WARNING! Deprecated LRO_MIRRORLIST used


��� Updating metadata for 'fedora-coreos-pool' 100% 

@jlebon
Copy link
Member

jlebon commented Oct 16, 2019

Yeah, they started appearing recently. Thought we maybe just needed a libdnf bump though that's not it (opened coreos/rpm-ostree#1929 anyway). Haven't gotten around to investigating deeper yet.

weird CI issue:

Hmm, yeah we've hit that in rpm-ostree CI too.

jlebon added a commit to jlebon/coreos-assembler that referenced this pull request Oct 17, 2019
There's a new update fixing the `statx` failures:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-566ad52d5e

I strongly suspect this is also the root cause behind the recent CI
failures:

coreos/fedora-coreos-config#201 (comment)
coreos/rpm-ostree#1929 (comment)

Let's just pull it in right now so we can get things unblocked.
jlebon added a commit to jlebon/coreos-assembler that referenced this pull request Oct 17, 2019
There's a new update awaiting karma that fixes the `statx` failures:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-566ad52d5e

I strongly suspect this is also the root cause behind the recent CI
failures:

coreos/fedora-coreos-config#201 (comment)
coreos/rpm-ostree#1929 (comment)

Let's just pull it in right now so we can get things unblocked.
jlebon added a commit to jlebon/coreos-assembler that referenced this pull request Oct 17, 2019
There's a new update awaiting karma that fixes the `statx` failures:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-566ad52d5e

I strongly suspect this is also the root cause behind the recent CI
failures:

coreos/fedora-coreos-config#201 (comment)
coreos/rpm-ostree#1929 (comment)

Let's just pull it in right now so we can get things unblocked.
jlebon added a commit to jlebon/coreos-assembler that referenced this pull request Oct 17, 2019
There's a new update awaiting karma that fixes the `statx` failures:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-566ad52d5e

I strongly suspect this is also the root cause behind the recent CI
failures:

coreos/fedora-coreos-config#201 (comment)
coreos/rpm-ostree#1929 (comment)

Let's just pull it in right now so we can get things unblocked.
jlebon added a commit to jlebon/coreos-assembler that referenced this pull request Oct 17, 2019
There's a new update awaiting karma that fixes the `statx` failures:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-566ad52d5e

I strongly suspect this is also the root cause behind the recent CI
failures:

coreos/fedora-coreos-config#201 (comment)
coreos/rpm-ostree#1929 (comment)

Let's just pull it in right now so we can get things unblocked.
jlebon added a commit to jlebon/coreos-assembler that referenced this pull request Oct 17, 2019
There's a new update awaiting karma that fixes the `statx` failures:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-566ad52d5e

I strongly suspect this is also the root cause behind the recent CI
failures:

coreos/fedora-coreos-config#201 (comment)
coreos/rpm-ostree#1929 (comment)

Let's just pull it in right now so we can get things unblocked.
@jlebon
Copy link
Member

jlebon commented Oct 17, 2019

Deprecation warnings addressed in rpm-software-management/libdnf#819!

jlebon added a commit to coreos/coreos-assembler that referenced this pull request Oct 17, 2019
There's a new update awaiting karma that fixes the `statx` failures:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-566ad52d5e

I strongly suspect this is also the root cause behind the recent CI
failures:

coreos/fedora-coreos-config#201 (comment)
coreos/rpm-ostree#1929 (comment)

Let's just pull it in right now so we can get things unblocked.
@dustymabe
Copy link
Member Author

are we able to retrigger CI yet?

anyone opposed to merging this?

@jlebon
Copy link
Member

jlebon commented Oct 17, 2019

The container is still rebuilding.

Do you have the final diff for this? (Or CI will give it to us once it's back up. Edit: ahh right, this needs #176)

@dustymabe
Copy link
Member Author

The final diff is:

Added:
  flatpak-session-helper-1.4.3-1.fc30.x86_64
  toolbox-0.0.15-1.fc30.noarch

@cgwalters
Copy link
Member

This makes sense, though I'd also like to land too
containers/toolbox#303
so we could consider e.g. symlinking to coretoolbox from the same RPM for our use case, since it's known to work better, until we get things unified.

@cgwalters cgwalters merged commit a4bcedf into coreos:testing-devel Oct 18, 2019
dustymabe pushed a commit to jbtrystram/fedora-coreos-config that referenced this pull request Apr 19, 2024
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.

coreos/toolbox v2
3 participants