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 a switch to Import-BcContainerLicense to update c:\demo\settings.ps1 #2940

Closed
mjmatthiesen opened this issue Mar 16, 2023 · 2 comments
Closed

Comments

@mjmatthiesen
Copy link

Describe the issue
As bc licenses expire I use the import command so the current container still works. Eventually the container is recreated with replace-bccontainer. The settings.ps2 file still uses a url to an old license file. It never gets updated. For me, it would be beneficial to update this every time I run import-bclicense, but I can see why some partners may not want this.
Would be great if the command got an additional flag so I don't run into the issue weeks later when the container is recreated.

Scripts used to create container and cause the issue

Replace-BcServerContainer

Full output of scripts

Using license file https://old-url.com/lic.flf
Downloading c:\bcartifacts.cache\teskhczu.n3d\my\license.flf
Using WebClient
Error downloading..., retrying in 6 seconds...
Using WebClient
New-Bcimage Telemetry Correlation Id: b2f7261f-5d66-4771-b9e1-1741c739c471
Container output
Error: No such container: bcserver
Replace-BcServerContainer Telemetry Correlation Id: 4ee18306-ed80-469d-8d88-1779ebe9d9e8
Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (403) Forbidden."

Additional context

  • does it happen all the time? yes
  • did it use to work? no
@freddydk
Copy link
Contributor

I think a better fix would be to add a licensefile parameter to Replace-BcServerContainer (as that is ONLY used on Azure VMs and knows about c:\demo\settings.ps1)
In fact it already has an artifactUrl parameter which is basically the same.

freddydk added a commit to freddydk/navcontainerhelper that referenced this issue Apr 25, 2023
freddydk added a commit that referenced this issue Apr 25, 2023
@freddydk
Copy link
Contributor

A licenseFileUri parameter is added to the Replace-BcServerContainer in the next version

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

No branches or pull requests

2 participants