Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Batch Insights installation script references outdated release #361

Open
zerweck opened this issue Jul 29, 2019 · 1 comment
Open

Batch Insights installation script references outdated release #361

zerweck opened this issue Jul 29, 2019 · 1 comment

Comments

@zerweck
Copy link

zerweck commented Jul 29, 2019

Is there a reason this line references a release from September 2018?

"wget https://github.com/Azure/batch-insights/releases/download/go-beta.1/batch-insights",

On the same topic, the run command seems outdated as well:

commands <- linuxWrapCommands(commands)
if (!is.null(args$applicationInsights)) {
commands <- gsub("wait",
"wait; ./batch-insights > node-stats.log &",
commands)
}

For comparison, see https://github.com/Azure/batch-insights/blob/master/README.md#linux

@brnleehng
Copy link
Collaborator

Hey @zerweck

doAzureParallel uses the go version of Batch insights due to issues of Linux installation.

Thanks,
Brian

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

No branches or pull requests

2 participants