-
Notifications
You must be signed in to change notification settings - Fork 16.3k
Mahara PR #2 #4657
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
Merged
Merged
Mahara PR #2 #4657
Changes from 36 commits
Commits
Show all changes
100 commits
Select commit
Hold shift + click to select a range
251535c
Fresh Mahara based on https://github.com/catalyst/Azure_moodle from c…
darrinhodges a371134
Update to note that mssql is not supported by Mahara
darrinhodges a9e9f1f
Update for Mahara
darrinhodges 0e4f21a
fix shell syntax errors
darrinhodges 01a435e
Fix nginx html root for mahrar
darrinhodges 5907c38
Fix for additional serveroot hard coding in mahara
darrinhodges 099371b
Remove mssql option from other templates
darrinhodges c3b75d5
move comment down below bash init
darrinhodges 5a4feb3
move comment down below bash init
darrinhodges f8336af
Remove tika and tika references
darrinhodges c610012
Update diagram and README.md
darrinhodges 2e8c2da
Remove Redis as Azure supported Mahara versions do not support Redis
darrinhodges 597ab35
updated for testing
darrinhodges 7e7fd03
fix syntax error in network.json
darrinhodges 715a7a1
Removed GDPR,office365 and azuresearch reference and configs
darrinhodges 08cabcb
remove references to azuresearch
darrinhodges 84b99bb
Fix syntax error in azuredploy.json
darrinhodges a72c078
Fix syntax error in azuredploy.json
darrinhodges cff5f55
Fix terminology for Mahara
darrinhodges a8e3a54
Remove all code,references and configs for mssql - Mahara does not su…
darrinhodges e02f9e2
enable selectable search functionality
darrinhodges 2288ab6
add sql updates for elastic search options
darrinhodges 9689daf
fix syntax error
darrinhodges 0acb212
fix syntax error
darrinhodges 6dbae23
fix script logic
darrinhodges cc9cc76
Update terminology for Mahara
darrinhodges 82c4e88
remove references to objectfs as it requires backporting to versions …
darrinhodges 20f2b8b
Fix link to RG deletion
darrinhodges df154a7
remove remaining redis references
darrinhodges be70446
update Azure content url
darrinhodges f795df4
Remove inappropriate files and documentation
darrinhodges 486c604
Updates to various documents, removal of other files
darrinhodges 94d75bd
Remove inappropriate documentation, update README.md to fit into quic…
darrinhodges f446bf1
update stack diagram
darrinhodges 36aa24c
update azure deploy parameters template
darrinhodges 3cebb15
update url content to a different value
darrinhodges 00c9e06
reset default db
darrinhodges ebacf68
minor tidy
darrinhodges 8753f37
fix syntax error in template
darrinhodges 775feb6
add more options to gatewaysubnet var
darrinhodges ab3e27c
remove unused variable
darrinhodges a5ec5be
update metadata file
darrinhodges cb7c2cf
update api as validation command suggests
darrinhodges d467afb
Revert "update api as validation command suggests"
darrinhodges 61f8686
re-edit metadata.json
darrinhodges 40bd576
Updates to README.md
darrinhodges 3da2ce3
Revise deploy estimate in README.md
darrinhodges 5a957eb
remove env.json
darrinhodges 7377a60
added autogen for dblogin
darrinhodges 053d7d3
use gen-unique for db passwords
darrinhodges a990994
add variable
darrinhodges 1e5b035
db passwords now autogen
darrinhodges a9c4a33
update metadata.json
darrinhodges 6c0701f
remove extra comma
darrinhodges 7605b14
Fix syntax
darrinhodges 48ffa38
pick of another rogue comma
darrinhodges d6c4525
fix elasticsearch settings
darrinhodges 4c6b07b
remove autogen passwords
darrinhodges b0a2a76
update search name value for mahara
darrinhodges df284d3
Add note on Mahara to readme.md
darrinhodges e1bba0a
reset mahara db password
darrinhodges 21a67b8
reset azure db password
darrinhodges ee0df81
Add undocumented schema property to metadata.json
darrinhodges b1d2b26
Text update as per https://github.com/Azure/azure-quickstart-template…
darrinhodges 83daa04
Text update as per https://github.com/Azure/azure-quickstart-template…
darrinhodges d76a6a8
Text update as per https://github.com/Azure/azure-quickstart-template…
darrinhodges 115e943
Text update as per https://github.com/Azure/azure-quickstart-template…
darrinhodges 6c09e74
Text update as per https://github.com/Azure/azure-quickstart-template…
darrinhodges 789ee58
Text update as per https://github.com/Azure/azure-quickstart-template…
darrinhodges f18c5b0
Text update as per https://github.com/Azure/azure-quickstart-template…
darrinhodges 8769d73
Update as per https://github.com/Azure/azure-quickstart-templates/pul…
darrinhodges 8fb126a
update as per https://github.com/Azure/azure-quickstart-templates/pul…
darrinhodges 466ea6d
update as per https://github.com/Azure/azure-quickstart-templates/pul…
darrinhodges 84c6e03
Further updates to https://github.com/Azure/azure-quickstart-template…
darrinhodges 579a073
Additional updates for https://github.com/Azure/azure-quickstart-temp…
darrinhodges 66b6a02
Text update
darrinhodges 48d2e62
Update api version for nested template deployment as per Travis CI me…
darrinhodges c8ac66a
Update generated usernames with fixed length as per Travis CI
darrinhodges 64f1344
Remove default values as per https://github.com/Azure/azure-quickstar…
darrinhodges 91bf572
Remove hard coded endpoint as per https://github.com/Azure/azure-quic…
darrinhodges 1ed08bf
Altering to output lb FQDN
darrinhodges 5f1f00a
Removing static end points
darrinhodges 9cbdc6e
Remove static endpoint for lb fqdn.
darrinhodges 586cd05
Fix JSON syntax error
darrinhodges 705e8e1
replace missing variable
darrinhodges bbdb989
Fix issue with siteURL var
darrinhodges 287346d
Output ssh username from parent template
darrinhodges f576f11
Revert "Fix issue with siteURL var"
darrinhodges ad3e737
Generate FQDN and pass through to scaleset and controller templates.
darrinhodges 2654375
Add variable to deploy template
darrinhodges d74c9cc
Fix syntax error
darrinhodges 96f0c6c
Fix parameters for nested templates
darrinhodges a5ed253
Insert missing quote character
darrinhodges f1ec59a
Move siteurl config into scripts as its not relevant to the Azure tem…
darrinhodges a163444
Fix cli parameters
darrinhodges bf92296
Fix cli parameters
darrinhodges 96c616f
Fix cli parameters
darrinhodges ad5d47d
Resolve conflict
darrinhodges ab6ae1b
Text update as per https://github.com/Azure/azure-quickstart-template…
darrinhodges f20c4d0
remove deployment resource
bmoore-msft File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -23,6 +23,13 @@ | |
| }, | ||
| "type": "bool" | ||
| }, | ||
| "location": { | ||
| "type": "string", | ||
| "defaultValue": "[resourceGroup().location]", | ||
| "metadata": { | ||
| "description": "Location for all resources" | ||
| } | ||
| }, | ||
| "azureBackupSwitch": { | ||
| "defaultValue": false, | ||
| "metadata": { | ||
|
|
@@ -80,7 +87,6 @@ | |
| "type": "string" | ||
| }, | ||
| "sshUsername": { | ||
| "defaultValue": "azureadmin", | ||
| "metadata": { | ||
| "description": "ssh user name" | ||
| }, | ||
|
|
@@ -130,7 +136,6 @@ | |
| "type": "string" | ||
| }, | ||
| "dbLogin": { | ||
| "defaultValue": "dbadmin", | ||
| "metadata": { | ||
| "description": "Database admin username" | ||
| }, | ||
|
|
@@ -223,7 +228,7 @@ | |
| "nfs" | ||
| ], | ||
| "metadata": { | ||
| "description": "File server type: GlusterFS, NFS--not yet highly available" | ||
| "description": "File server type: GlusterFS, NFS--not yet highly available. Gluster uses premium managed disks therefore premium skus are required." | ||
| }, | ||
| "type": "string" | ||
| }, | ||
|
|
@@ -360,7 +365,7 @@ | |
| "resources": [ | ||
| { | ||
| "type": "Microsoft.Resources/deployments", | ||
| "apiVersion": "2017-05-10", | ||
| "apiVersion": "2016-09-01", | ||
| "name": "pid-738e3eec-68d4-4667-8377-c05c77c21f1b", | ||
|
||
| "properties": { | ||
| "mode": "Incremental", | ||
|
|
@@ -373,7 +378,7 @@ | |
| }, | ||
| { | ||
| "type": "Microsoft.Resources/deployments", | ||
| "apiVersion": "2017-05-10", | ||
| "apiVersion": "2016-09-01", | ||
| "dependsOn": [ | ||
| "Microsoft.Resources/deployments/networkTemplate" | ||
| ], | ||
|
|
@@ -399,7 +404,7 @@ | |
| { | ||
| "condition": "[parameters('azureBackupSwitch')]", | ||
| "type": "Microsoft.Resources/deployments", | ||
| "apiVersion": "2017-05-10", | ||
| "apiVersion": "2016-09-01", | ||
| "name": "recoveryTemplate", | ||
| "properties": { | ||
| "mode": "Incremental", | ||
|
|
@@ -415,7 +420,7 @@ | |
| }, | ||
| { | ||
| "type": "Microsoft.Resources/deployments", | ||
| "apiVersion": "2017-05-10", | ||
| "apiVersion": "2016-09-01", | ||
| "name": "networkTemplate", | ||
| "properties": { | ||
| "mode": "Incremental", | ||
|
|
@@ -431,7 +436,7 @@ | |
| }, | ||
| { | ||
| "type": "Microsoft.Resources/deployments", | ||
| "apiVersion": "2017-05-10", | ||
| "apiVersion": "2016-09-01", | ||
| "dependsOn": [ | ||
| "Microsoft.Resources/deployments/networkTemplate", | ||
| "Microsoft.Resources/deployments/recoveryTemplate" | ||
|
|
@@ -452,7 +457,7 @@ | |
| { | ||
| "condition": "[equals(parameters('fileServerType'),'gluster')]", | ||
| "type": "Microsoft.Resources/deployments", | ||
| "apiVersion": "2017-05-10", | ||
| "apiVersion": "2016-09-01", | ||
| "dependsOn": [ | ||
| "Microsoft.Resources/deployments/networkTemplate", | ||
| "Microsoft.Resources/deployments/recoveryTemplate" | ||
|
|
@@ -472,7 +477,7 @@ | |
| }, | ||
| { | ||
| "type": "Microsoft.Resources/deployments", | ||
| "apiVersion": "2017-05-10", | ||
| "apiVersion": "2016-09-01", | ||
| "dependsOn": [ | ||
| "Microsoft.Resources/deployments/glusterTemplate", | ||
| "Microsoft.Resources/deployments/recoveryTemplate", | ||
|
|
@@ -490,6 +495,9 @@ | |
| }, | ||
| "ctlrPubIpId": { | ||
| "value": "[reference('networkTemplate').outputs.ctlrPubIpId.value]" | ||
| }, | ||
| "siteFQDN": { | ||
| "value": "[reference('networkTemplate').outputs.siteFQDN.value]" | ||
| } | ||
| }, | ||
| "templateLink": { | ||
|
|
@@ -499,7 +507,7 @@ | |
| }, | ||
| { | ||
| "type": "Microsoft.Resources/deployments", | ||
| "apiVersion": "2017-05-10", | ||
| "apiVersion": "2016-09-01", | ||
| "dependsOn": [ | ||
| "Microsoft.Resources/deployments/controllerTemplate", | ||
| "Microsoft.Resources/deployments/networkTemplate", | ||
|
|
@@ -511,6 +519,9 @@ | |
| "parameters": { | ||
| "maharaCommon": { | ||
| "value": "[variables('maharaCommon')]" | ||
| }, | ||
| "siteFQDN": { | ||
| "value": "[reference('networkTemplate').outputs.siteFQDN.value]" | ||
| } | ||
| }, | ||
| "templateLink": { | ||
|
|
@@ -520,7 +531,7 @@ | |
| }, | ||
| { | ||
| "type": "Microsoft.Resources/deployments", | ||
| "apiVersion": "2017-05-10", | ||
| "apiVersion": "2016-09-01", | ||
| "name": "storageAccountTemplate", | ||
| "properties": { | ||
| "mode": "Incremental", | ||
|
|
@@ -538,7 +549,7 @@ | |
| "outputs": { | ||
| "siteURL": { | ||
| "type": "string", | ||
| "value": "[variables('maharaCommon').siteURL]" | ||
| "value": "[if(equals(variables('maharaCommon').siteURL,'www.example.org'),reference('networkTemplate').outputs.siteFQDN.value,'www.example.org')]" | ||
| }, | ||
| "controllerInstanceIP": { | ||
| "type": "string", | ||
|
|
@@ -572,9 +583,13 @@ | |
| "type": "string", | ||
| "value": "[variables('maharaCommon').maharaDbPass]" | ||
| }, | ||
| "sshUsername": { | ||
| "type": "string", | ||
| "value": "[variables('maharaCommon').sshUsername]" | ||
| }, | ||
| "loadBalancerDNS": { | ||
| "type": "string", | ||
| "value": "[variables('maharaCommon').lbDns]" | ||
| "value": "[reference('networkTemplate').outputs.siteFQDN.value]" | ||
| } | ||
| }, | ||
| "variables": { | ||
|
|
@@ -589,6 +604,7 @@ | |
| "documentation09": " controller - creates a controller VM and deploys code", | ||
| "documentation10": "GlusterFS Sizing guidance", | ||
| "maharaCommon": { | ||
| "location": "[parameters('location')]", | ||
| "baseTemplateUrl": "[concat(parameters('_artifactsLocation'), 'nested/')]", | ||
| "scriptLocation": "[concat(parameters('_artifactsLocation'), 'scripts/')]", | ||
| "artifactsSasToken": "[parameters('_artifactsLocationSasToken')]", | ||
|
|
@@ -643,7 +659,6 @@ | |
| "ctlrPipName": "[concat('controller-pubip-',variables('resourceprefix'))]", | ||
| "ctlrVmName": "[concat('controller-vm-',variables('resourceprefix'))]", | ||
| "ctlrVmSecrets": "[take(variables('ctlrVmSecretsArray'), if(empty(parameters('keyVaultResourceId')), 0, 1))]", | ||
| "lbDns": "[concat('lb-',variables('resourceprefix'),'.',resourceGroup().location,'.cloudapp.azure.com')]", | ||
| "lbName": "[concat('lb-',variables('resourceprefix'))]", | ||
| "lbPipName": "[concat('lb-pubip-',variables('resourceprefix'))]", | ||
| "maharaAdminPass": "[concat(toUpper('xl'), substring(uniqueString(resourceGroup().id, deployment().name), 6, 7),',1*8')]", | ||
|
|
@@ -670,7 +685,7 @@ | |
| "resourcesPrefix": "[variables('resourceprefix')]", | ||
| "searchType": "[parameters('searchType')]", | ||
| "serverName": "[concat(parameters('dbServerType'), '-',variables('resourceprefix'))]", | ||
| "siteURL": "[if(or(empty(parameters('siteURL')), equals(parameters('siteURL'), 'www.example.org')), concat('lb-',variables('resourceprefix'),'.',resourceGroup().location,'.cloudapp.azure.com'), parameters('siteURL'))]", | ||
| "siteURL": "[parameters('siteURL')]", | ||
| "sshPublicKey": "[parameters('sshPublicKey')]", | ||
| "sshUsername": "[parameters('sshUsername')]", | ||
| "sslEnforcement": "[parameters('sslEnforcement')]", | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rgardler - is this correct?