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

BlackCoin [BLK] Template #1484

Closed
wants to merge 39 commits into from

Conversation

tomcio2001
Copy link

Changelog

*Added BlackCoin Template for Ubuntu

@acomghbot
Copy link

No issues were found in this pull request.

@azurecla
Copy link

Hi @tomcio2001, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.

TTYL, AZPRBOT;

@acomghbot
Copy link

No issues were found in this pull request.

@tomcio2001
Copy link
Author

Ok, I will change them and remove the remains from INFX template.

@blood2
Copy link

blood2 commented Mar 28, 2016

why running the daemon as user root ?
i see this problem in Radium's template as well.
i would run it as a different user... maybe in a chrooted environment

@acomghbot
Copy link

No issues were found in this pull request.

@acomghbot
Copy link

No issues were found in this pull request.

@acomghbot
Copy link

No issues were found in this pull request.

@acomghbot
Copy link

No issues were found in this pull request.

@acomghbot
Copy link

No issues were found in this pull request.

@acomghbot
Copy link

No issues were found in this pull request.

@acomghbot
Copy link

No issues were found in this pull request.

@tomcio2001
Copy link
Author

Ok, I changed all rpc config to parameters.

@acomghbot
Copy link

No issues were found in this pull request.

@acomghbot
Copy link

No issues were found in this pull request.

@acomghbot
Copy link

The following issues were detected in this pull request:

Template: blackcoin-blockchain-ubuntu

  • The file azuredeploy.json does not contain a valid JSON object.
  • The file azuredeploy.json is not valid. Response from ARM API: BadRequest - {"error":{"code":"InvalidRequestContent","message":"The request content was invalid and could not be deserialized: 'After parsing a value an unexpected character was encountered: ". Path 'properties.template.parameters', line 77, position 3.'."}}

@acomghbot
Copy link

The following issues were detected in this pull request:

Template: blackcoin-blockchain-ubuntu

  • The file azuredeploy.json is not valid. Response from ARM API: BadRequest - {"error":{"code":"InvalidTemplateDeployment","message":"The template deployment 'dummy' is not valid according to the validation procedure. The tracking id is '2e62ada6-a5f5-4ad6-b16d-af27515231f0'. See inner errors for details. Please see http://aka.ms/arm-deploy for usage details.","details":[{"code":"InvalidDomainNameLabel","message":"The domain name label GEN-UNIQUE-11 is invalid. It must conform to the following regular expression: ^[a-z][a-z0-9-]{1,61}[a-z0-9]$.","details":[]}]}}

@acomghbot
Copy link

No issues were found in this pull request.

@allwaytogo
Copy link

Will this task be continue?

sidhujag and others added 9 commits April 6, 2016 15:42
Update azuredeploy.parameters.json
Remove the parameter, you are not using it in the sh file
forgot to remove installMethod
remove installMethod again
Update azuredeploy.parameters.json
@sidhujag
Copy link
Contributor

sidhujag commented Apr 8, 2016

your welcome

@tomcio2001
Copy link
Author

Thank you man.

@sidhujag
Copy link
Contributor

sidhujag commented Apr 14, 2016

@tomcio2001 have you contacted Mr. Gray and finished off the application to get BLK into azure? I hope I didn't waste my time helping you get this going.

sidhujag and others added 3 commits April 14, 2016 14:37
@tomcio2001
Copy link
Author

Very sorry for late reply. Where Can I contact him?

@sidhujag
Copy link
Contributor

marleyg@microsoft.com

@gatneil
Copy link
Contributor

gatneil commented Sep 13, 2016

Sorry for the late reply; is this something we still want to merge in?

@sidhujag
Copy link
Contributor

You can im not available to work on it

@gatneil
Copy link
Contributor

gatneil commented Sep 14, 2016

Thanks!

@gatneil
Copy link
Contributor

gatneil commented Sep 14, 2016

Merge no longer desired; closing

@gatneil gatneil closed this Sep 14, 2016
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

Successfully merging this pull request may close these issues.

8 participants