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

unnecessary change #2390

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

PaulTalbot-INL
Copy link
Collaborator

Do not merge

Testing regression testers.

@PaulTalbot-INL PaulTalbot-INL changed the title [WIP] unnecessary change unnecessary change Oct 24, 2024
@moosebuild
Copy link

Job Test CentOS 7 on 252737b : invalidated by @joshua-cogliati-inl

restarted civet

@joshua-cogliati-inl
Copy link
Contributor

This is a splendid unnecessary change and I highly unapprove it.

@PaulTalbot-INL
Copy link
Collaborator Author

yeah, trying to find out if the ExternalRay test fails on devel or just on Caleb's PR.

@moosebuild
Copy link

Job Test qsubs sawtooth on 252737b : invalidated by @joshua-cogliati-inl

failed rm directory not empty

@alfoa
Copy link
Collaborator

alfoa commented Oct 25, 2024

yeah, trying to find out if the ExternalRay test fails on devel or just on Caleb's PR.

also in my PR :(

@alfoa
Copy link
Collaborator

alfoa commented Oct 25, 2024

@PaulTalbot-INL @joshua-cogliati-inl I think the reason is that redis got a new release yesterday:

https://pypi.org/project/redis/#history

And in the dependencies.xml file redis for windows is not pinned to a certain version

@alfoa
Copy link
Collaborator

alfoa commented Oct 28, 2024

@PaulTalbot-INL @joshua-cogliati-inl I think the reason is that redis got a new release yesterday:

https://pypi.org/project/redis/#history

And in the dependencies.xml file redis for windows is not pinned to a certain version

downgrading the redis library to 5.1 does not help. No idea!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants