Andras/BoundlessFlowCampus2K
4
7
Fork 0
Code Issues 90 Pull requests 7 Projects Releases Packages Wiki Activity Actions

915 Windows deployment script #948

Merged
a24julot merged 3 commits from 915-Windows-deployment-script into team_1_week_7 2026-05-22 08:13:49 +00:00
Conversation 6 Commits 3 Files changed 3 +17 -4
a24timsv commented 2026-05-13 13:03:05 +00:00
Collaborator
Copy link

Added the Windows deployment script and changed the Unix script to match the coding conventions.
Also changed the names.

Relevant issue: #915

Added the Windows deployment script and changed the Unix script to match the coding conventions. Also changed the names. Relevant issue: #915
Added the Windows deployment script and changed the Unix script to match the coding conventions.
Also changed the names.
b24johka commented 2026-05-13 13:16:11 +00:00
Collaborator
Copy link

Review

  • Everything seems good

Question before merge ready:
Should we try it out on a windows machine in order to actually know that it works?

# Review - Everything seems good Question before merge ready: Should we try it out on a windows machine in order to actually know that it works?
a22erigr removed review request for b24idalu 2026-05-13 13:17:43 +00:00
a24timsv commented 2026-05-13 13:24:19 +00:00
Author
Collaborator
Copy link

Yes, try it and post result from server logs.

Yes, try it and post result from server logs.
b24johka commented 2026-05-13 13:52:20 +00:00
Collaborator
Copy link

image

Tried to execute the script on my Windows PC and it could not find the path.

![image](/attachments/8f6b2fb2-6e4c-4198-8415-7f66697c3262) Tried to execute the script on my Windows PC and it could not find the path.
image.png
14 KiB
a24julot changed target branch from Temp-Main-Week-5 to team_1_week_6 2026-05-13 13:55:20 +00:00
b24johka commented 2026-05-15 10:11:17 +00:00
Collaborator
Copy link

Apparently Windows require special permissions of some sort to SSH with the private.key, we will talk to Henrik about the key because we suspect that it is a private key, but we should probably have a public key.

Apparently Windows require special permissions of some sort to SSH with the private.key, we will talk to Henrik about the key because we suspect that it is a private key, but we should probably have a public key.
b24johka changed title from 915 Windows deployment script to WIP: 915 Windows deployment script 2026-05-15 10:11:43 +00:00
b24johka commented 2026-05-19 08:06:45 +00:00
Collaborator
Copy link

A discussion has been taking place about if it would be a simple fix to use the docker WSL (Windows Subsystem for Linux), we already have it since docker is running WSL. It could work out fine in order for Windows users to be able to deploy.
I will create a separate issue on this.

A discussion has been taking place about if it would be a simple fix to use the docker WSL (Windows Subsystem for Linux), we already have it since docker is running WSL. It could work out fine in order for Windows users to be able to deploy. I will create a separate issue on this.
b24johka changed title from WIP: 915 Windows deployment script to 915 Windows deployment script 2026-05-21 13:35:12 +00:00
b24johka changed target branch from team_1_week_6 to team_1_week_7 2026-05-21 13:40:01 +00:00
b24johka commented 2026-05-21 14:02:43 +00:00
Collaborator
Copy link

Tried this out by running a test script on the server from my Windows machine which worked.
I don't see how this would work differently than deploying the deploy script, this will be somewhat awkward to test since you need the SSH-key to verify it.

Tried this out by running a test script on the server from my Windows machine which worked. I don't see how this would work differently than deploying the deploy script, this will be somewhat awkward to test since you need the SSH-key to verify it.
a24julot merged commit 53b4ff59db into team_1_week_7 2026-05-22 08:13:49 +00:00
c24danli deleted branch 915-Windows-deployment-script 2026-05-22 12:10:43 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
b24johka
Labels
Clear labels   
_CRITICAL_

Only use this label for critical tasks that are project-scope,and must be addressed ASAP.

  
API

   
app

   
backEnd

   
Blocked, waiting for further changes

   
bug

Something is not working

  
cleanup

Issues to cleanup the codebase (restructure folders, remove deprecated or smelly code)

  
close

Issue to close

  
design

   
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
feature request

   
frontEnd

   
help wanted

Need some help

  
invalid

Something is wrong

  
low priority

   
needs input

Issue needs input from product owner or customer

  
needs review

issue is in need of review

  
project documentation

   
question

More information is needed

  
research

   
reviewed

This Issue has been reviewed

  
script

   
security

use this for security-related issues

  
SQL

  
style

   
testing

   
topLevel

Reserved for top-level, project-wide issues.

  
wontfix

This won't be fixed

No labels
_CRITICAL_
API
app
backEnd
Blocked, waiting for further changes
bug
cleanup
close
design
duplicate
enhancement
feature request
frontEnd
help wanted
invalid
low priority
needs input
needs review
project documentation
question
research
reviewed
script
security
SQL
style
testing
topLevel
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
b24johka
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!948
No description provided.