Skip to content

Moving networkfirewall(2), networkmanager(9) resources to v1beta1 version - #401

Merged
svscheg merged 1 commit into
crossplane-contrib:mainfrom
svscheg:networkfirewall-networkmanager
Dec 29, 2022
Merged

Moving networkfirewall(2), networkmanager(9) resources to v1beta1 version#401
svscheg merged 1 commit into
crossplane-contrib:mainfrom
svscheg:networkfirewall-networkmanager

Conversation

@svscheg

@svscheg svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor

Description of your changes

Moving networkfirewall (2) resources to v1beta1 version

Moving networkmanager (9) resources to v1beta1 version

Fixes #378

I have:

  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Manually
Uptests
Moving networkfirewall (2) resources to v1beta1 version

Moving networkmanager (9) resources to v1beta1 version

@svscheg
svscheg force-pushed the networkfirewall-networkmanager branch from c302ebb to 97469d9 Compare December 28, 2022 18:04
@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkfirewall/firewallpolicy.yaml"

@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkfirewall/rulegroup.yaml"

@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkmanager/connection.yaml"

@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkmanager/customergatewayassociation.yaml"

@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkmanager/device.yaml"

@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkmanager/globalnetwork.yaml"

@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkmanager/link.yaml"

@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkmanager/linkassociation.yaml"

@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkmanager/site.yaml"

@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkmanager/transitgatewayconnectpeerassociation.yaml"

@svscheg

svscheg commented Dec 28, 2022

Copy link
Copy Markdown
Contributor Author

/test-examples="examples/networkmanager/transitgatewayregistration.yaml"

@svscheg
svscheg requested a review from sergenyalcin December 28, 2022 18:47

@sergenyalcin sergenyalcin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @svscheg I left a few comments.

Comment thread config/externalname.go Outdated
Comment thread config/externalname.go Outdated
Comment thread config/externalname.go Outdated
Comment thread config/externalname.go Outdated
@svscheg
svscheg force-pushed the networkfirewall-networkmanager branch from 97469d9 to 144b9fc Compare December 29, 2022 10:58
add resources to networkfirewall(2), networkmanager(9) groups

fix comments
@svscheg
svscheg force-pushed the networkfirewall-networkmanager branch from 144b9fc to f9d7dd7 Compare December 29, 2022 11:02

@sergenyalcin sergenyalcin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @svscheg LGTM!

@svscheg
svscheg merged commit 65a0f97 into crossplane-contrib:main Dec 29, 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.

Moving networkfirewall(2), networkmanager(9) resources to v1beta1 version

2 participants