Skip to content

Adding support for field dns_publish_fixed_ip in a subnet#3090

Merged
kayrus merged 4 commits intogophercloud:masterfrom
pabclsn:master
Jul 11, 2024
Merged

Adding support for field dns_publish_fixed_ip in a subnet#3090
kayrus merged 4 commits intogophercloud:masterfrom
pabclsn:master

Conversation

@pabclsn
Copy link
Copy Markdown
Contributor

@pabclsn pabclsn commented Jul 8, 2024

Prior to starting a PR, please make sure you have read our
contributor tutorial.

Prior to a PR being reviewed, there needs to be a Github issue that the PR
addresses. Replace the brackets and text below with that issue number.

Fixes #3089

Links to the line numbers/files in the OpenStack source code that support the
code in this PR:

https://github.com/openstack/neutron/blob/df02d0dfecc057e021b8e9c19270645c07f8cb79/neutron/objects/subnet.py#L233C10-L233C30

@github-actions github-actions bot added the semver:minor Backwards-compatible change label Jul 8, 2024
Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for submitting your first PR! Be sure that we will be looking at it but keep in mind
this sometimes takes a while.
Please let the maintainers know if your PR has not got enough attention after a few days.
If any doubt, please consult our PR tutorial.

@github-actions github-actions bot added semver:minor Backwards-compatible change and removed semver:minor Backwards-compatible change labels Jul 9, 2024
@kayrus
Copy link
Copy Markdown
Contributor

kayrus commented Jul 9, 2024

@pabclsn thanks for the PR. Could you fix linter errors?

@github-actions github-actions bot added semver:minor Backwards-compatible change and removed semver:minor Backwards-compatible change labels Jul 10, 2024
@github-actions github-actions bot added semver:minor Backwards-compatible change and removed semver:minor Backwards-compatible change labels Jul 10, 2024
@github-actions github-actions bot added semver:minor Backwards-compatible change and removed semver:minor Backwards-compatible change labels Jul 10, 2024
@github-actions github-actions bot added semver:minor Backwards-compatible change and removed semver:minor Backwards-compatible change labels Jul 10, 2024
@github-actions github-actions bot added semver:minor Backwards-compatible change and removed semver:minor Backwards-compatible change labels Jul 10, 2024
@kayrus
Copy link
Copy Markdown
Contributor

kayrus commented Jul 10, 2024

@pabclsn unit tests are failing

@github-actions github-actions bot added semver:minor Backwards-compatible change and removed semver:minor Backwards-compatible change labels Jul 10, 2024
@github-actions github-actions bot removed the semver:minor Backwards-compatible change label Jul 10, 2024
@github-actions github-actions bot added the semver:minor Backwards-compatible change label Jul 10, 2024
@coveralls
Copy link
Copy Markdown

coveralls commented Jul 10, 2024

Coverage Status

coverage: 78.742% (+0.005%) from 78.737%
when pulling 0337faa on pabclsn:master
into 71268a5 on gophercloud:master.

Copy link
Copy Markdown
Contributor

@kayrus kayrus left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@kayrus kayrus merged commit d842758 into gophercloud:master Jul 11, 2024
@kayrus kayrus added the backport-v1 This PR will be backported to v1 label Jul 11, 2024
@github-actions
Copy link
Copy Markdown

Failed to backport PR to v1 branch. See logs for details.

@mandre mandre added the backport-v2 This PR will be backported to v2 label Jul 16, 2024
@github-actions
Copy link
Copy Markdown

Failed to backport PR to v2 branch. See logs for details.

@mandre mandre added backport-v2 This PR will be backported to v2 and removed backport-v2 This PR will be backported to v2 labels Jul 16, 2024
@mandre mandre added backport-v1 This PR will be backported to v1 and removed backport-v1 This PR will be backported to v1 labels Jul 16, 2024
@github-actions
Copy link
Copy Markdown

Failed to backport PR to v1 branch. See logs for details.

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

Labels

backport-v1 This PR will be backported to v1 backport-v2 This PR will be backported to v2 semver:minor Backwards-compatible change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding support for dns_publish_fixed_ip

4 participants