Skip to content

remove Juniper plugin references - #348

Merged
DaanHoogland merged 5 commits into
apache:mainfrom
shapeblue:removeSrxF5
Dec 6, 2023
Merged

remove Juniper plugin references#348
DaanHoogland merged 5 commits into
apache:mainfrom
shapeblue:removeSrxF5

Conversation

@DaanHoogland

@DaanHoogland DaanHoogland commented Sep 22, 2023

Copy link
Copy Markdown
Contributor

a bit hard to define which are just plugin references and which are architectural best practices.
this implements cloudstack#7690


📚 Documentation preview 📚: https://cloudstack-documentation--348.org.readthedocs.build/en/348/

@shwstppr

Copy link
Copy Markdown
Contributor

@blueorangutan docbuild

@shwstppr shwstppr added this to the 4.19.0.0 milestone Sep 29, 2023
@blueorangutan

Copy link
Copy Markdown

@shwstppr a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/348. (QA-JID 136)

@DaanHoogland

Copy link
Copy Markdown
Contributor Author

is this lgty @shwstppr ?

@yadvr

yadvr commented Dec 1, 2023

Copy link
Copy Markdown
Member

@blueorangutan docbuild

@blueorangutan

Copy link
Copy Markdown

@rohityadavcloud a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

Comment thread source/adminguide/networking/external_firewalls_and_load_balancers.rst Outdated
@DaanHoogland DaanHoogland reopened this Dec 5, 2023

@shwstppr shwstppr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, makes sense to remove

Comment thread source/adminguide/networking/external_firewalls_and_load_balancers.rst Outdated
Comment thread source/adminguide/networking/external_firewalls_and_load_balancers.rst Outdated
@andrijapanicsb

Copy link
Copy Markdown
Contributor

LGTM, I reworded it a bit, but good to merge.

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.

5 participants