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

coreos-assembler/src/cmdlib: rearrange option sequence #3211

Merged
merged 2 commits into from
Nov 17, 2022

Conversation

c4rt0
Copy link
Member

@c4rt0 c4rt0 commented Nov 17, 2022

Rearrange location of '-maxdepth' option to eliminate 'find' warning

See: #3199

Rearrange location of '-maxdepth' option to avoid 'find' warning

See: coreos#3199
jlebon
jlebon previously approved these changes Nov 17, 2022
dustymabe
dustymabe previously approved these changes Nov 17, 2022
Copy link
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

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

LGTM

We're not shipping this right now and building it broke in the
upgrade to OCP 4.11 with:

```
[2022-11-17T16:25:11.894Z] 2022-11-17 16:25:08,317 ERROR - Command returned bad exitcode
[2022-11-17T16:25:11.894Z] 2022-11-17 16:25:08,317 ERROR - COMMAND: ['buildah', '--storage-driver', 'vfs', 'from', 'scratch']
[2022-11-17T16:25:11.894Z] 2022-11-17 16:25:08,317 ERROR -  STDERR: time="2022-11-17T16:25:08Z" level=warning msg="Reading allowed ID mappings: reading subuid mappings for user \"cosa\" and subgid mappings for group \"cosa\": no subuid ranges found for user \"cosa\" in /etc/subuid"
[2022-11-17T16:25:11.894Z] time="2022-11-17T16:25:08Z" level=warning msg="Found no UID ranges set aside for user \"cosa\" in /etc/subuid."
[2022-11-17T16:25:11.894Z] time="2022-11-17T16:25:08Z" level=warning msg="Found no GID ranges set aside for user \"cosa\" in /etc/subgid."
[2022-11-17T16:25:11.894Z] Error during unshare(CLONE_NEWUSER): Function not implemented
```
@dustymabe dustymabe dismissed stale reviews from jlebon and themself via 8c49e47 November 17, 2022 17:09
@dustymabe
Copy link
Member

I pushed up a commit that should fix CI.

@dustymabe dustymabe enabled auto-merge (rebase) November 17, 2022 17:10
@dustymabe
Copy link
Member

well it didn't pick up the CI changes probably because @c4rt0 is new. I'm just going to merge.

@dustymabe dustymabe merged commit fe4ebd3 into coreos:main Nov 17, 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.

None yet

3 participants