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

Add documentation for IPv6 in ipsubnet filter. Add unit tests #131

Closed

Conversation

sebastien-rosset
Copy link
Contributor

@sebastien-rosset sebastien-rosset commented Jan 24, 2022

SUMMARY
  1. Add example for IPv6 in ipsubnet filter documentation.
  2. Fix typo in ipsubnet documentation.
  3. Add unit tests for ipsubnet filter.
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ipsubnet filter

ADDITIONAL INFORMATION

Analysis

  1. In the netaddr module, there should be some documentation and protection to prevent a very large 2 ^ N loop. The 'IPNetwork.subnet' function can loop through 2 ^ 128 elements netaddr/netaddr#241
  2. In ansible.utils, the ipsubnet filter should not construct a list of subnets, then get subnet at index X. Instead, the subnet can be constructed using bit operations.

@sebastien-rosset sebastien-rosset changed the title Ipv6 ipsubnet Add documentation for IPv6 in ipsubnet filter. Add unit tests Jan 24, 2022
@ashwini-mhatre
Copy link
Contributor

@sebastien-rosset could you please open those bugs as separate issues?

@softwarefactory-project-zuul
Copy link
Contributor

Unable to freeze job graph: Job network-ee-unit-tests depends on network-ee-build-container-image which was not run.

@softwarefactory-project-zuul
Copy link
Contributor

Unable to freeze job graph: Job network-ee-unit-tests depends on network-ee-build-container-image which was not run.

@softwarefactory-project-zuul
Copy link
Contributor

Unable to freeze job graph: Job network-ee-unit-tests depends on network-ee-build-container-image which was not run.

@softwarefactory-project-zuul
Copy link
Contributor

Unable to freeze job graph: Job network-ee-unit-tests depends on network-ee-build-container-image which was not run.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@ashwini-mhatre
Copy link
Contributor

recheck

@ashwini-mhatre
Copy link
Contributor

@sebastien-rosset This will be trivial change instead of minor_changes. could you please update changelog fragment?

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@sebastien-rosset
Copy link
Contributor Author

@sebastien-rosset This will be trivial change instead of minor_changes. could you please update changelog fragment?

I don't see a "trivial" section in the changelogs/config.yaml file. Can you suggest a change?

https://github.com/ansible-collections/ansible.utils/blob/main/changelogs/config.yaml

@Qalthos
Copy link
Contributor

Qalthos commented Apr 25, 2022

@sebastien-rosset This will be trivial change instead of minor_changes. could you please update changelog fragment?

I don't see a "trivial" section in the changelogs/config.yaml file. Can you suggest a change?

https://github.com/ansible-collections/ansible.utils/blob/main/changelogs/config.yaml

That would be what trivial_section_name: trivial is for

Co-authored-by: Nathaniel Case <this.is@nathanielca.se>
@softwarefactory-project-zuul
Copy link
Contributor

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.

@softwarefactory-project-zuul
Copy link
Contributor

@Qalthos
Copy link
Contributor

Qalthos commented Oct 3, 2022

Replaced by #212

@Qalthos Qalthos closed this Oct 3, 2022
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

3 participants