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

fix "make bundleRegistry" error #245

Merged
merged 1 commit into from
Aug 28, 2019

Conversation

ffromani
Copy link
Contributor

Fixes errors preventing make bundleRegistry to work on a clean tree snapshot

@ffromani
Copy link
Contributor Author

/cc @rthallisey @djzager @tiraboschi

@ffromani
Copy link
Contributor Author

OKD-4.1.0 lane failed. The fix was already posted (not by me) on #244

@ffromani
Copy link
Contributor Author

/cc @tiraboschi @rthallisey

@ffromani
Copy link
Contributor Author

/retest

@rthallisey
Copy link

are you still seeing this on master?

@arachmani
Copy link
Contributor

ci test please

@arachmani
Copy link
Contributor

ci add to whitelist

@ffromani ffromani changed the title Bundle registry mro fix fix "make bundleRegistry" error Aug 27, 2019
@ffromani
Copy link
Contributor Author

are you still seeing this on master?

yep. One patch seems to be unneeded, so I removed from this PR. But the other is still needed.

Copy link
Contributor

@djzager djzager left a comment

Choose a reason for hiding this comment

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

LGTM. One comment.

deploy/Dockerfile Show resolved Hide resolved
Fixes this error:
```
time="2019-08-23T20:34:34Z" level=fatal msg="could not decode contents of file /registry/bundles.db into package: error converting YAML to JSON: yaml: control characters are not allowed"
```

Likely caused by the fact that the `initializer` tool mistakenly tries
to include `bundles.db` into itself.

Signed-off-by: Francesco Romani <fromani@redhat.com>
@djzager
Copy link
Contributor

djzager commented Aug 28, 2019

Merging since this isn't really related to ci.

@djzager djzager merged commit 95ee295 into kubevirt:master Aug 28, 2019
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

4 participants