Skip to content

Commit ec0615a

Browse files
authored
Merge branch 'bitwarden:main' into feat/co-location-on-readwrite-once-storage
2 parents 398f0a0 + a4b9933 commit ec0615a

File tree

17 files changed

+45
-37
lines changed

17 files changed

+45
-37
lines changed

.github/workflows/build.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ jobs:
2929
version: 'v3.13.1'
3030

3131
- name: Login to Azure - CI Subscription
32-
uses: Azure/login@4c88f01b0e3a5600e08a37889921afd060f75cf0 # v1.5.0
32+
uses: Azure/login@de95379fe4dadc2defb305917eaa7e5dde727294 # v1.5.1
3333
with:
3434
creds: ${{ secrets.AZURE_KV_CI_SERVICE_PRINCIPAL }}
3535

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ jobs:
3030
ref: main
3131

3232
- name: Login to Azure - CI Subscription
33-
uses: Azure/login@4c88f01b0e3a5600e08a37889921afd060f75cf0 # v1.5.0
33+
uses: Azure/login@de95379fe4dadc2defb305917eaa7e5dde727294 # v1.5.1
3434
with:
3535
creds: ${{ secrets.AZURE_KV_CI_SERVICE_PRINCIPAL }}
3636

.github/workflows/version-bump.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ jobs:
2121
uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
2222

2323
- name: Login to Azure - CI Subscription
24-
uses: Azure/login@4c88f01b0e3a5600e08a37889921afd060f75cf0 # v1.5.0
24+
uses: Azure/login@de95379fe4dadc2defb305917eaa7e5dde727294 # v1.5.1
2525
with:
2626
creds: ${{ secrets.AZURE_KV_CI_SERVICE_PRINCIPAL }}
2727

charts/self-host/README.md

Lines changed: 24 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -561,43 +561,51 @@ secrets:
561561
secretProviderClass: bitwarden-azure-keyvault-csi #metadata.name in example
562562
```
563563

564+
### Create Empty Azure Application Gateway Rewrite Set
565+
566+
Application Gateway ingress deployments have a few more required steps for Bitwarden to function correctly. If you are using another ingress controller, you may skip to the next section.
567+
568+
We will need to create a rewrite set on the Application Gateway. There are various ways of doing this, but we will discuss using the Azure Portal. For now we are creating an empty set for the Helm deployment to work. We will add the rewrite rule after deploying Helm.
569+
570+
1. Navigate to the Application Gateway in the Azure Portal
571+
2. Once in the Application Gateway, find the "Rewrites" blade in the left-hand navigation menu.
572+
3. Click the "+ Rewrite set" button at the top of the main page section to add a new rewrite set
573+
4. On the "Update rewrite set" page in the "Name and Association" tab set the `Name` field to the same value specified in the `appgw.ingress.kubernetes.io/rewrite-rule-set` ingress annotation
574+
5. Click Next
575+
6. Click Create
576+
564577
### Helm
565578

566579
```shell
567580
helm upgrade bitwarden bitwarden/self-host --install --devel --namespace bitwarden --values my-values.yaml
568581
```
569582

570-
### Azure Application Gateway Rewrite Set
583+
### Update Azure Application Gateway Rewrite Set
571584

572585
Application Gateway ingress deployments have one more required step for Bitwarden to function correctly. If you are using another ingress controller, you may skip to the next section.
573586

574-
We will need to create a rewrite set on the Application Gateway. There are various ways of doing this, but we will discuss using the Azure Portal.
587+
We will need to finish the rewrite set on the Application Gateway we created earlier.
575588

576-
1. Navigate to the Application Gateway in the Azure Portal
577-
2. Once in the Application Gateway, find the "Rewrites" blade in the left-hand navigation menu.
578-
3. Click the "+ Rewrite set" button at the top of the main page section to add a new rewrite set
579-
4. On the "Update rewrite set" page in the "Name and Association" tab:
580-
- Set the Name field to the same value specified in the `appgw.ingress.kubernetes.io/rewrite-rule-set` ingress annotation
581-
- Select all routing rules that start with something similar to "pr-bitwarden-bitwarden-ingress-rule-*"
582-
5. Click Next
583-
6. On the "Rewrite rule configuration" tab, click the "Add rewrite rule" button
584-
7. Enter a name for the rule. This can be anything that helps you with organization. Something simlar to "bitwarden-rewrite" will work.
585-
8. The rule sequence value does not matter for this purpose.
586-
9. Add a condition and set the following values:
589+
1. Reopen the rewrite set you created earlier.
590+
2. On the "Update rewrite set" page in the "Name and Association" tab, select all routing paths that begin with pr-bitwarden-self-host-ingress... , deselect any that do not begin with that prefix, and then select Next.
591+
3. On the "Rewrite rule configuration" tab, click the "Add rewrite rule" button.
592+
4. Enter a name for the rule. This can be anything that helps you with organization. Something similar to "bitwarden-rewrite" will work.
593+
5. The rule sequence value does not matter for this purpose.
594+
6. Add a condition and set the following values:
587595
- Type of variable to check: Server variable
588596
- Server variable: uri_path
589597
- Case-sensitive: No
590598
- Operator: equal (=)
591599
- Pattern to match: `^(\/(?!admin)[^\/]*)\/(.*)`
592600
- Click OK
593-
10. Add an action and set the following values:
601+
7. Add an action and set the following values:
594602
- Rewrite type: URL
595603
- Action type: Set
596604
- Components: URL path
597605
- URL path value: `/{var_uri_path_2}`
598606
- Re-evalueate path map: Unchecked
599607
- Click OK
600-
11. Click "Create" at the bottom of the screen
608+
8. Click "Update" at the bottom of the screen.
601609

602610
### Pointing your DNS
603611

@@ -853,7 +861,7 @@ oc create secret generic custom-secret -n bitwarden \
853861
--from-literal=globalSettings__yubico__clientId="REPLACE" \
854862
--from-literal=globalSettings__yubico__key="REPLACE" \
855863
--from-literal=SA_PASSWORD="REPLACE" # If using SQL pod
856-
# --from-literal="REPLACE" # If using your own SQL server
864+
# --from-literal=globalSettings__sqlServer__connectionString="REPLACE" # If using your own SQL server
857865
```
858866

859867
### Create a service account

charts/self-host/templates/admin.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ spec:
4545
{{- end }}
4646
containers:
4747
- name: {{ template "bitwarden.admin" . }}
48-
image: "{{ .Values.component.admin.image.name }}:{{ template "bitwarden.coreVersion" }}"
48+
image: "{{ .Values.component.admin.image.name }}:{{ default ( include "bitwarden.coreVersionDefault" nil ) .Values.general.coreVersionOverride }}"
4949
envFrom:
5050
- configMapRef:
5151
name: {{ .Release.Name }}-config-map

charts/self-host/templates/api.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ spec:
4545
{{- end }}
4646
containers:
4747
- name: {{ template "bitwarden.api" . }}
48-
image: "{{ .Values.component.api.image.name }}:{{ template "bitwarden.coreVersion" }}"
48+
image: "{{ .Values.component.api.image.name }}:{{ default ( include "bitwarden.coreVersionDefault" nil ) .Values.general.coreVersionOverride }}"
4949
envFrom:
5050
- configMapRef:
5151
name: {{ .Release.Name }}-config-map

charts/self-host/templates/attachments.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ spec:
3131
{{- end }}
3232
containers:
3333
- name: {{ template "bitwarden.attachments" . }}
34-
image: "{{ .Values.component.attachments.image.name }}:{{ template "bitwarden.coreVersion" }}"
34+
image: "{{ .Values.component.attachments.image.name }}:{{ default ( include "bitwarden.coreVersionDefault" nil ) .Values.general.coreVersionOverride }}"
3535
envFrom:
3636
- configMapRef:
3737
name: {{ .Release.Name }}-config-map

charts/self-host/templates/events.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ spec:
4545
{{- end }}
4646
containers:
4747
- name: {{ template "bitwarden.events" . }}
48-
image: "{{ .Values.component.events.image.name }}:{{ template "bitwarden.coreVersion" }}"
48+
image: "{{ .Values.component.events.image.name }}:{{ default ( include "bitwarden.coreVersionDefault" nil ) .Values.general.coreVersionOverride }}"
4949
envFrom:
5050
- configMapRef:
5151
name: {{ .Release.Name }}-config-map

charts/self-host/templates/icons.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ spec:
4545
{{- end }}
4646
containers:
4747
- name: {{ template "bitwarden.icons" . }}
48-
image: "{{ .Values.component.icons.image.name }}:{{ template "bitwarden.coreVersion" }}"
48+
image: "{{ .Values.component.icons.image.name }}:{{ default ( include "bitwarden.coreVersionDefault" nil ) .Values.general.coreVersionOverride }}"
4949
envFrom:
5050
- configMapRef:
5151
name: {{ .Release.Name }}-config-map

charts/self-host/templates/identity.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ spec:
4545
{{- end }}
4646
containers:
4747
- name: {{ template "bitwarden.identity" . }}
48-
image: "{{ .Values.component.identity.image.name }}:{{ template "bitwarden.coreVersion" }}"
48+
image: "{{ .Values.component.identity.image.name }}:{{ default ( include "bitwarden.coreVersionDefault" nil ) .Values.general.coreVersionOverride }}"
4949
envFrom:
5050
- configMapRef:
5151
name: {{ .Release.Name }}-config-map

0 commit comments

Comments
 (0)