search

mmajcica / DeploySsrs

Build-Release task for VSTS/TFS that manages Microsoft's SQL Server Reporting Services
MIT License
21 stars 21 forks source link

If multiple Shared DataSource exist with the same name at the moment of the deployment, DataSource's reference in shared Datasets can be updated to an invalid one #71

Closed GregoryOtt closed 3 years ago

GregoryOtt commented 3 years ago

I Have one staging SSRS server for multiple environments (FAT, UAT, Features, etc.) SSRS artifacts are deployed on a specifiic folder corresponding to the environment image

In each folder the structure is the same and all artifact have the same name.

While deploying, the reference to datasource in the shared datasets are updated according to the first Datasource found with the same name as the reference https://github.com/mmajcica/DeploySsrs/blob/fea9b6dcedf8b7794f73bc237ad5d63cecf5c228/task/ps_modules/ssrs.psm1#L1123

We should use reference to created Datasources when creating Datasets

mmajcica commented 3 years ago

It's been 3 years since I last worked on this. I have nowhere to test any of this. If you are actively working with it and do have a chance to spin this with various situations, feel free to fix what necessary and will be more than happy to assist.

mmajcica commented 3 years ago

@GregoryOtt can I assist you anyhow so that you can test your changes before publishing the new version in Marketplace? Should I share it with a specific organization but with different publisher compare to the one used for the public extension so that you can test it? If you are planning to spend more time on this, we can setup a call to align if necessary.

GregoryOtt commented 3 years ago

@mmajcica : I've just tested it, it works in my context. The #72 is the result of my local test on my company's Azure DevOps Server. My version of the extension is deployed on it. Because it's an Azure DevOps Server, I can package and update the extension myself, no more action from you is required.

I've no other environment/context to test it for now.

hdouglassjr commented 3 years ago

Hi @GregoryOtt , I am in the process of also setting this up with Azure DevOps, can you share your package and configuration you used to get it working on your end? I was trying to use this extension from @mmajcica , but realized in this post it has not been worked on in a while. Any help would be appreciated.

GregoryOtt commented 3 years ago

Could you please open a new issue with your problem and some details so we can investigate?