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

Make the location of the structurizr onprem/lite configurable + document list of dependencies #8

Open
hideintheclouds opened this issue Aug 19, 2024 · 1 comment

Comments

@hideintheclouds
Copy link

At the moment, the extension does not have the possibility to configure the location of the structurizr server that will render the dsl diagrams (live-preview).

The Live preview does not seem to work after installing the extension in VS Code in an airgapped environment:

  • if structurizr lite/opnprem runs from a docker container on localhost:8000, the live preview will take the next available port: localhost:8001, but won't be able to display anything;
  • if no structurizr server runs locally, the extension will try to live preview on localhost:8000

It will be a good idea to also document the list of dependencies that the extension needs

@RVR06
Copy link
Owner

RVR06 commented Oct 19, 2024

Hi @hideintheclouds

Thanks for the feedback.

If you already have a valid structurizr/lite docker image tagged latest, Cornifer will use it.
Otherwise, download will be triggered during the first preview call by the underlying Docker engine, which may result with a very slow first run.

You also have the ability to fine tune the structurizr/lite version you want to use through the matching settings:

image

It sems to me airgapped environment should be fine with this behavior. Please refine if I missed something.

By the list of dependencies, you mean Docker and structurizr/lite image? Or something else.

Regards

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

No branches or pull requests

2 participants