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

Revert "mantle: use os.ReadDir for lightweight directory reading" #3137

Merged
merged 1 commit into from
Oct 28, 2022

Conversation

travier
Copy link
Member

@travier travier commented Oct 25, 2022

Reverts #3082

See #3135

I'm working on a fix for the underlying issue but revert this until it's ready.

Copy link
Contributor

@bgilbert bgilbert left a comment

Choose a reason for hiding this comment

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

LGTM

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

@dustymabe
Copy link
Member

Looks like all the "newly failing tests because we now run all the tests" are going to have to be worked through before we merge this. Otherwise we're going to break CI in various places.

One of the failures is openshift/os#1036

@dustymabe
Copy link
Member

One of the failures is openshift/os#1036

well actually that one isn't new because it's not a shared test.

The other one is: ext.config.shared.clhm.network-device-info

@travier
Copy link
Member Author

travier commented Oct 27, 2022

openshift/os#1042

@travier
Copy link
Member Author

travier commented Oct 27, 2022

/test rhcos

@HuijingHei HuijingHei merged commit 96c0295 into main Oct 28, 2022
@HuijingHei HuijingHei deleted the revert-3082-refactor/os.ReadDir branch October 28, 2022 06:44
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.

4 participants