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

Fixed pattern matching error with NetworkInterface objects (SOFTWARE-5423) #3654

Merged
merged 1 commit into from
Feb 23, 2024

Conversation

sqqueak
Copy link
Contributor

@sqqueak sqqueak commented Feb 16, 2024

An error occurs when running the osg-notify script with specific parameters (see more at this ticket). This is due to a utility function trying to perform pattern matching on a NetworkInterface object, which does not have a default string representation. I modified the pattern matching logic so that it would look for patterns in the correct location and data type.

@brianhlin
Copy link
Member

@sqqueak could you rebase this on the latest master branch and force push? That'll save us a merge commit

@brianhlin
Copy link
Member

D'oh, we've had a lot of activity on this repo today. Next time could you do the force push and ping me on Slack so I can hit merge?

@sqqueak sqqueak force-pushed the fix-string-error branch 2 times, most recently from 2211cb7 to 8d552c4 Compare February 23, 2024 16:31
Copy link
Member

@brianhlin brianhlin left a comment

Choose a reason for hiding this comment

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

LGTM

@brianhlin brianhlin merged commit 99206b4 into opensciencegrid:master Feb 23, 2024
2 checks passed
@sqqueak sqqueak deleted the fix-string-error branch February 23, 2024 17:07
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.

2 participants