Logshark icon indicating copy to clipboard operation
Logshark copied to clipboard

v.4.2.3 docker build always fails because it cannot find package Tools.TableauServerRestApi

Open til-jmac opened this issue 3 years ago • 7 comments

@zhuoyanggao Since v4.2.3 the docker build process is broken, with the error as described in https://github.com/tableau/Logshark/issues/165

The following commit appears to be the cause: https://github.com/tableau/Logshark/commit/d0c0ca9076307844e2c686436a6b77b0c680f88b#diff-6fe1e64645078ab18848d369ae5ee3d3f9ff22943ae39c5a84d61f087635daeb

Reverting this change so that the package can be found in the relevant path in the source code resolves the issue. Any reason this can't be updated in the latest version so that we can use it in docker?

Credit to @petersilv for spotting this

til-jmac avatar Feb 15 '23 11:02 til-jmac

Having the same issue

jay2610 avatar Feb 15 '23 16:02 jay2610

Hi @jay2610 and @til-jmac ,

Wonder if below work-around would help:

Revert all changes in Logshark/Logshark.csproj locally, and retry build. https://github.com/tableau/Logshark/commit/d0c0ca9076307844e2c686436a6b77b0c680f88b#diff-6fe1e64645078ab18848d369ae5ee3d3f9ff22943ae39c5a84d61f087635daeb

We might be releasing a patch for this in the coming weeks.

zhuoyanggao avatar Feb 15 '23 19:02 zhuoyanggao

@til-jmac This is due to a missing manual step in the release process. Those changes should not have been there in the 4.2.3 release.

zhuoyanggao avatar Feb 15 '23 19:02 zhuoyanggao

@til-jmac This is due to a missing manual step in the release process. Those changes should not have been there in the 4.2.3 release.

Thanks for the reply. Reverting the changes locally does work, just wanted to raise it so a fix can be incorporated.

til-jmac avatar Feb 15 '23 19:02 til-jmac

any update here ? or more detail on how to update locally?

tb582 avatar Mar 23 '23 14:03 tb582

any update here ? or more detail on how to update locally?

Open Logshark/LogShark.csproj and add this after the last </ItemGroup> (for me this was on line 60)

<ItemGroup>
<Reference Include="Tools.TableauServerRestApi">
<HintPath>..\thirdparty\Tools.TableauServerRestApi.dll</HintPath>
</Reference>
</ItemGroup> 

rickymedrano avatar Apr 24 '23 23:04 rickymedrano

how is this STILL open ??

tb582 avatar Dec 05 '23 17:12 tb582