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.

Sign up for GitHub

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

Jump to bottom

Merge master to the new server branch #1129

Merged
merged 124 commits into from
Apr 5, 2024
Merged

Merge master to the new server branch #1129

merged 124 commits into from
Apr 5, 2024

Conversation

lslezak
Copy link
Contributor

@lslezak lslezak commented Apr 4, 2024

Problem

  • The new arch branch is behind the master

Solution

  • Merge master

Testing

  • Tested manually, compiles fine, /api/ping still works 😃

lslezak and others added 30 commits February 29, 2024 11:38
@lslezak
Web server - support SSL (HTTPS) communication
fdf6662 
- Use either the cerfificate specified via command line
  arguments or generate a self-signed certificate
- Redirect external HTTP requests to HTTPS
- Allow HTTP for internal connections (http://localhost)
- Optionally listen on a secondary address
  (to allow listening on both HTTP/80 and HTTPS/433 ports)
@lslezak
Fixes by clippy
10a6500
@lslezak
fmt fixes
05b1b09
@lslezak
Cleanup
fb7cd41
@lslezak
Documentation update
f646965
@joseivanlopez
service: Export device sid in the proposal actions
4372623
@joseivanlopez
service: Export unused slots
ca34978
@joseivanlopez
service: Export more info about block devices
02a93e9
@joseivanlopez
service: Export LVM devices
26fb99d
@joseivanlopez
service: Avoid updating D-Bus nodes of devices tree
36ff75a
@joseivanlopez
service: Export staging devices
fa1ace3
@joseivanlopez
service: Fix rubocop config
d43bb46 
- agama.gemspec was renamed, see
  #1056.
- The file name was not updated in the rubocop config.
@joseivanlopez
service: Fix typo
8128fd7
@joseivanlopez
Extend storage D-Bus API (#1071)
e35d470 
## Problem

The current result of the storage proposal is presented as a list of
actions, which could make a bit complex to understand the overall
picture of the resulting devices.

That list of actions is going to be replaced by a more visual
representation of the devices. To make this possible, more information
about the devices has to be exported on D-Bus.

* Part of https://trello.com/c/7TTVVquM.

## Solution

Add the following information to D-Bus:

* The list of actions includes the SID of the affected device.
* The partition table exports the unused slots.
* The LVM devices (volume groups, physical volumes and logical volumes)
are exported.
* The block devices includes their start block and also indicates
whether the device is encrypted.
* The staging devices are exported. 

NOTE: This PR goes to a feature branch instead of master.

## Testing

* Added unit tests
* Tested manually
@joseivanlopez
service: Add Device interface
1ba7c2b
@joseivanlopez
service: Add Partition interface
f94680d
@joseivanlopez
service: Adapt Filesystem interface
d2ce25e
@joseivanlopez
service: Update dependency
19f769a
@mchf
Turned some unwrap calls into a more reasonable handling
0653aaa
@mchf
Formatting
7b8c729
@joseivanlopez
service: Use display_name
ba93043
@joseivanlopez
service: Add LogicalVolume interface
aa1e7a3
@joseivanlopez
Extend storage D-Bus API (#1082)
64129d5 
## Problem

More information is required to properly represent the storage proposal
result:

* Device description
* Filesystem label

This is a follow-up of #1071.

## Solution

Add new D-Bus interfaces `Device` , `Partition`, `LVM.LogicalVolume` and
adapt `Filesystem` interface.

This changes require yast/yast-storage-ng#1373.

NOTE: This PR goes to a feature branch instead of master.

## Testing

* Added unit tests
* Tested manually
@yast-bot
Update web PO files
3aea9b9 
Agama-weblate commit: 6e40e55f095086ac0577a8737169f99cde56c88c
@yast-bot
Update service PO files
cc1bc84 
Agama-weblate commit: 6e40e55f095086ac0577a8737169f99cde56c88c
@yast-bot
Update translations in the product files
24d40cf 
Agama-weblate commit: 6e40e55f095086ac0577a8737169f99cde56c88c
@joseivanlopez
web: Adapt storage client to changes in D-Bus
09f3e08
@joseivanlopez
web: Load devices in the proposal page
e40435c
@joseivanlopez
Adapt storage client (#1079)
380dbb6 
## Problem

The storage D-Bus API was extended to provide all the information
required for a better representation of the proposal result, see
#1071 and
#1082. The JS storage client has
to be adapted.

## Solution

Adapt the storage client to the changes in the D-Bus API and adapt
`ProposalPage` component to read the information about the devices if
needed.

NOTE: This PR goes to a feature branch instead of master.

## Testing

* Added new unit tests
* Tested manually
@lslezak
Code review fixes
08b586f
lslezak and others added 23 commits March 27, 2024 14:26
@lslezak
Do not use Cockpit manifest for storing supported languages (#1121)
8cf36f0 
## Problem

- Continue with dropping the Cockpit dependency

## Solution

- Do not use the Cockpit manifest for storing the list of supported
languages
- Use a separate `languages.json` file instead of the `manifest.json`
- Import the JSON file directly into the JS code
- The workflow is not changed, the update script just generates a
different file
- The change is backward compatible, works with both Cockpit and the new
server

## Testing

- Updated unit tests
- Tested manually, the language selection still works fine
@imobachgs
Merge branch 'master' into infiniband_upstream
d1800bc
@imobachgs
Add infiniband to network model (#1032)
e63f418 
## Problem

No infiniband in network model

## Solution

Add infiniband to network model

## Testing

- *Added a new unit test*
- *Tested manually*
- *Tested using the migration
jcronenberg#74
@joseivanlopez @ancorgs
service: Improvements in Drive D-Bus interface
6f0f868 
Co-authored-by: Ancor Gonzalez Sosa <ancorgs@gmail.com>
@joseivanlopez
service: Fix settings conversion to Y2Storage
ee52cac
@joseivanlopez
service: Add default boot device to D-Bus
7c46e86
@joseivanlopez
service: Update Gemfile.lock
c59f80c
@joseivanlopez
web: Adapt storage client to changes in D-Bus
1e5ae20
@joseivanlopez @ancorgs
web: Add more device utils
a3067c5 
Co-authored-by: Ancor Gonzalez Sosa <ancorgs@gmail.com>
@joseivanlopez @ancorgs
web: Use FilesystemLabel util
38cee8d 
Co-authored-by: Ancor Gonzalez Sosa <ancorgs@gmail.com>
@joseivanlopez @ancorgs
web: Small fix in text
53ef918 
Co-authored-by: Ancor Gonzalez Sosa <ancorgs@gmail.com>
@joseivanlopez @dgdavid
web: Add ExpandableSelector component
f763cd4 
Co-authored-by: David Díaz González <dgonzalez@suse.de>
@joseivanlopez @dgdavid
web: Add DeviceSelectorTable and DevicesFormSelect
52ba9de 
Co-authored-by: David Díaz González <dgonzalez@suse.de>
@joseivanlopez @dgdavid
web: Add ControlledPanels
0a0fc82 
Co-authored-by: David Díaz González <dgonzalez@suse.de>
@joseivanlopez @ancorgs @dgdavid
web: Add new device selector
dea8654 
Co-authored-by: Ancor Gonzalez Sosa <ancorgs@gmail.com>
Co-authored-by: David Díaz González <dgonzalez@suse.de>
@joseivanlopez @ancorgs @dgdavid
web: Add boot device selector
1d87d66 
Co-authored-by: Ancor Gonzalez Sosa <ancorgs@gmail.com>
Co-authored-by: David Díaz González <dgonzalez@suse.de>
@joseivanlopez @ancorgs
web: Improve storage summary
5e9c002 
Co-authored-by: Ancor Gonzalez Sosa <ancorgs@gmail.com>
@joseivanlopez
New device and boot selection dialogs (#1106)
55ff0ba 
This PR implements the new UI selectors for the target installation
device and for the boot device, see
https://github.com/openSUSE/agama/blob/master/doc/storage_ui.md.

For the installation device it offers the following options:

* Select a target disk (the file systems are created over new
partitions).
* Create a new LVM volume group (the file systems are crated over
logical volumes).
* The option for reusing LVM volume groups is not offered yet.

For the boot device it offers the following options:

* To use the selected disk for installation.
* To use a different disk.
* To not configure boot partitions.

## Screenshots

<details>
<summary>Toggle</summary>

![localhost_8080_
(33)](https://github.com/openSUSE/agama/assets/1112304/63828e27-b3b1-4312-a14a-0435a86d3347)

![localhost_8080_
(32)](https://github.com/openSUSE/agama/assets/1112304/b79b84b9-d543-4558-af79-4b7cfd0d8bc7)

</details>
@joseivanlopez
service: Changelog
1b0ad17
@joseivanlopez
web: Changelog
cc317ea
@joseivanlopez
Merge branch 'master' into storage-next-device-selection
2adc9bb
@joseivanlopez
feature: Device and boot selection (#1127)
6291da5 
Merging feature branch which includes:

* #1068
* #1106
@lslezak
Merge remote-tracking branch 'origin/master' into architecture_2024_m…
fb32fdd 
…erge_master
imobachgs
imobachgs approved these changes Apr 5, 2024
View reviewed changes
Copy link
Contributor

@imobachgs imobachgs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lslezak lslezak merged commit 97518aa into architecture_2024 Apr 5, 2024
8 of 10 checks passed
@lslezak lslezak deleted the architecture_2024_merge_master branch April 5, 2024 06:26
@imobachgs imobachgs mentioned this pull request May 3, 2024
Merged
imobachgs added a commit that referenced this pull request May 6, 2024
@imobachgs
Merge architecture_2024 branch into master (#1177)
3353513 
After a few months of work, it is time to merge the `architecture_2024`
branch into `master`. It is still a work-in-progress, but all the
efforts should be go now against that branch.

## Pull requests

* #1061
* #1064
* #1073
* #1074
* #1080
* #1089
* #1091
* #1092
* #1094
* #1095
* #1099
* #1100
* #1102
* #1103
* #1112
* #1114
* #1116
* #1117
* #1119
* #1120
* #1123
* #1126
* #1129
* #1130
* #1131
* #1132
* #1133
* #1134
* #1136
* #1139
* #1140
* #1143
* #1146

## Other commits

* 8efa41f
* 9e2dec0
@imobachgs imobachgs mentioned this pull request May 17, 2024
Merged
imobachgs added a commit that referenced this pull request May 17, 2024
@imobachgs
Update to release 8 (#1232)
b13e7b7 
Prepare for releasing Agama 8. It includes the following pull requests:

* #884
* #886
* #914
* #918
* #956
* #957
* #958
* #959
* #960
* #961
* #962
* #963
* #964
* #965
* #966
* #969
* #970
* #976
* #977
* #978
* #979
* #980
* #981
* #983
* #984
* #985
* #986
* #988
* #991
* #992
* #995
* #996
* #997
* #999
* #1003
* #1004
* #1006
* #1007
* #1008
* #1009
* #1010
* #1011
* #1012
* #1014
* #1015
* #1016
* #1017
* #1020
* #1022
* #1023
* #1024
* #1025
* #1027
* #1028
* #1029
* #1030
* #1031
* #1032
* #1033
* #1034
* #1035
* #1036
* #1038
* #1039
* #1041
* #1042
* #1043
* #1045
* #1046
* #1047
* #1048
* #1052
* #1054
* #1056
* #1057
* #1060
* #1061
* #1062
* #1063
* #1064
* #1066
* #1067
* #1068
* #1069
* #1071
* #1072
* #1073
* #1074
* #1075
* #1079
* #1080
* #1081
* #1082
* #1085
* #1086
* #1087
* #1088
* #1089
* #1090
* #1091
* #1092
* #1093
* #1094
* #1095
* #1096
* #1097
* #1098
* #1099
* #1100
* #1102
* #1103
* #1104
* #1105
* #1106
* #1109
* #1110
* #1111
* #1112
* #1114
* #1116
* #1117
* #1118
* #1119
* #1120
* #1121
* #1122
* #1123
* #1125
* #1126
* #1127
* #1128
* #1129
* #1130
* #1131
* #1132
* #1133
* #1134
* #1135
* #1136
* #1138
* #1139
* #1140
* #1141
* #1142
* #1143
* #1144
* #1145
* #1146
* #1147
* #1148
* #1149
* #1151
* #1152
* #1153
* #1154
* #1155
* #1156
* #1157
* #1158
* #1160
* #1161
* #1162
* #1163
* #1164
* #1165
* #1166
* #1167
* #1168
* #1169
* #1170
* #1171
* #1172
* #1173
* #1174
* #1175
* #1177
* #1178
* #1180
* #1181
* #1182
* #1183
* #1184
* #1185
* #1187
* #1188
* #1189
* #1190
* #1191
* #1192
* #1193
* #1194
* #1195
* #1196
* #1198
* #1199
* #1200
* #1201
* #1203
* #1204
* #1205
* #1206
* #1207
* #1208
* #1209
* #1210
* #1211
* #1212
* #1213
* #1214
* #1215
* #1216
* #1217
* #1219
* #1220
* #1221
* #1222
* #1223
* #1224
* #1225
* #1226
* #1227
* #1229
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@imobachgs imobachgs imobachgs approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@lslezak @imobachgs @joseivanlopez @mchf @yast-bot @jcronenberg @ancorgs @dgdavid