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

Migrate Node Network Functionality to esisdk #67

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

ajamias
Copy link
Collaborator

@ajamias ajamias commented Jul 19, 2024

This commit is to reflect the changes to move the
network attach functionality to esisdk. This is so that both python-esiclient and esi-ui can use this functionality.

Copy link
Contributor

@tzumainn tzumainn left a comment

Choose a reason for hiding this comment

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

Don't forget to update the esisdk version in requirements.txt!

@ajamias ajamias changed the title Migrate Node Network Attach Functionality to esisdk Migrate Node Network Functionality to esisdk Jul 24, 2024
This commit is to reflect the changes to move the
network attach and detach functionality to esisdk.
This is so that both python-esiclient and esi-ui
can use this functionality.
@ajamias ajamias marked this pull request as ready for review July 24, 2024 14:05
Copy link
Contributor

@tzumainn tzumainn left a comment

Choose a reason for hiding this comment

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

Looks good! Feel free to take this out of draft, and I'll merge.

@ajamias
Copy link
Collaborator Author

ajamias commented Jul 24, 2024

Should we take python<=3.7 tests out? The tests are giving an error

@tzumainn
Copy link
Contributor

Yeah, I submitted a PR for that!

@tzumainn tzumainn closed this Jul 24, 2024
@tzumainn tzumainn reopened this Jul 24, 2024
@tzumainn tzumainn merged commit a812654 into CCI-MOC:master Jul 24, 2024
5 of 11 checks passed
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