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

openQA: Allow overriding of install repos #1474 #1545

Merged
merged 1 commit into from
Aug 20, 2024
Merged

openQA: Allow overriding of install repos #1474 #1545

merged 1 commit into from
Aug 20, 2024

Conversation

lkocman
Copy link
Contributor

@lkocman lkocman commented Aug 19, 2024 •
edited
Loading

  • Override urls by using agama.install_url=https://url1,https://url2 boot param
  • Add logger so we can print some repo related debug messages that can be captured by agama logs store

Problem

We need to be able to test installation of preproduction data. See details in #1474

Example usage

Add agama.install_url as a boot argument to the linux kernel. Multiple urls can be seprated by commas.

image

# cat /proc/cmdline 
BOOT_IMAGE=/boot/vmlinuz-6.10.5-1-default root=UUID=4fd273b6-0453-4b36-8f58-a81e70bc9b9a splash=silent mitigations=auto quiet security=apparmor agama.install_url=https://download.opensuse.org/repositories/openSUSE:/Leap:/16.0/product/repo/Leap-16.0-aarch64-ppc64le-s390x-x86_64,https://download.opensuse.org/repositories/openSUSE:/Leap:/16.0/standard/x86_64/

This will now set https://download.opensuse.org/repositories/openSUSE:/Leap:/16.0/product/repo/Leap-16.0-aarch64-ppc64le-s390x-x86_64 as an installation_url to all products. This is extremely helpful for staging testing or any testing in openQA before publishing changes to the public repository.

Note: if you specify incorrect urls, you can at least the value it in the overview screen :-)
image

@lkocman lkocman changed the title Allow overriding of install repos Issue#1474 Allow overriding of install repos #1474 Aug 19, 2024
@lkocman lkocman changed the title Allow overriding of install repos #1474 openQA: Allow overriding of install repos #1474 Aug 19, 2024
@coveralls
Copy link

coveralls commented Aug 19, 2024 •
edited
Loading

Coverage Status

coverage: 71.319% (+0.02%) from 71.298%
when pulling 42fbf98 on lkocman:openqa
into f1187eb on openSUSE:master.

@lkocman
Copy link
Contributor Author

lkocman commented Aug 20, 2024

(squashed my changes)

jreidinger
jreidinger reviewed Aug 20, 2024
View reviewed changes
doc/boot_arguments.md Outdated Show resolved Hide resolved
@lkocman
Allow overriding of install repos Issue#1474
42fbf98 
* Use agama.install_url=https://..,https://.. boot param
* multiple values can be separted by commas
@jreidinger jreidinger merged commit 55fabd7 into agama-project:master Aug 20, 2024
3 checks passed
@lkocman lkocman linked an issue Aug 20, 2024 that may be closed by this pull request
openQA: product-testing please provide a way to inject a custom repository #1474
Closed
imobachgs
imobachgs reviewed Aug 26, 2024
View reviewed changes
doc/boot_arguments.md
@@ -15,6 +15,22 @@ See [URL handling in the
installer](https://github.com/yast/yast-installation/blob/master/doc/url.md) to find more details
about the supported URLs.

## Custom Installation URL Configuration

You can override the default `installation_url` set in the product files [here](https://github.com/openSUSE/agama/tree/master/products.d) by passing the `agama.install_url` parameter as a boot option in the bootloader.
Copy link
Contributor

@imobachgs imobachgs Aug 26, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

Please, we should try to use the same name for the parameter, either install_url or installation_url but the same.

lkocman added a commit to lkocman/os-autoinst-distri-opensuse that referenced this pull request Aug 27, 2024
@lkocman
Utilize agama.install_url
2b966c8 
* Introduced in agama-project/agama#1545
lkocman added a commit to lkocman/os-autoinst-distri-opensuse that referenced this pull request Aug 27, 2024
@lkocman
Utilize agama.install_url
5b53dbd 
* introduced in agama-project/agama#1545
@lkocman lkocman mentioned this pull request Aug 27, 2024
Merged
lkocman added a commit to lkocman/os-autoinst-distri-opensuse that referenced this pull request Aug 27, 2024
@lkocman
Utilize agama.install_url
9e42b9a 
* introduced in agama-project/agama#1545
lkocman added a commit to lkocman/os-autoinst-distri-opensuse that referenced this pull request Aug 28, 2024
@lkocman
Utilize agama.install_url
93b16fe 
* Introduced in agama-project/agama#1545
* Pass comma separated values from AGAMA_INSTALL_URL
* Agama boot entry finally done through needle match
lkocman added a commit to lkocman/os-autoinst-distri-opensuse that referenced this pull request Aug 28, 2024
@lkocman
Utilize agama.install_url
3e853e2 
* Introduced in agama-project/agama#1545
* Pass comma separated values from AGAMA_INSTALL_URL
* Agama boot entry finally done through needle match
lkocman added a commit to lkocman/os-autoinst-distri-opensuse that referenced this pull request Aug 28, 2024
@lkocman
Utilize agama.install_url
addbf30 
* Introduced in agama-project/agama#1545
* Pass comma separated values from AGAMA_INSTALL_URL
  as agama.install_url boot param
* Agama boot entry finally done through needle match
lkocman added a commit to lkocman/os-autoinst-distri-opensuse that referenced this pull request Sep 4, 2024
@lkocman
Utilize agama.install_url
bce9973 
* Introduced in agama-project/agama#1545
* Pass comma separated values from AGAMA_INSTALL_URL
  as agama.install_url boot param
* Agama boot entry finally done through needle match
lkocman added a commit to lkocman/os-autoinst-distri-opensuse that referenced this pull request Sep 4, 2024
@lkocman
Utilize agama.install_url
42dcf03 
* Introduced in agama-project/agama#1545
* Pass REPO_0 as a param to it agama supports, comma separated list
  as agama.install_url boot param
* Agama boot entry finally done through needle match
volodymyrkatkalov pushed a commit to volodymyrkatkalov/os-autoinst-distri-opensuse that referenced this pull request Sep 11, 2024
@lkocman @volodymyrkatkalov
Utilize agama.install_url
7ce5fa6 
* Introduced in agama-project/agama#1545
* Pass REPO_0 as a param to it agama supports, comma separated list
  as agama.install_url boot param
* Agama boot entry finally done through needle match
@imobachgs imobachgs mentioned this pull request Sep 20, 2024
Merged
imobachgs added a commit that referenced this pull request Sep 20, 2024
@imobachgs
Update to release 10 (#1622)
574a1a5 
Prepare for releasing Agama 10·

* #1263
* #1330
* #1407
* #1408
* #1410
* #1411
* #1412
* #1416
* #1417
* #1419
* #1420
* #1421
* #1422
* #1423
* #1424
* #1425
* #1428
* #1429
* #1430
* #1431
* #1432
* #1433
* #1436
* #1437
* #1438
* #1439
* #1440
* #1441
* #1443
* #1444
* #1445
* #1449
* #1450
* #1451
* #1452
* #1453
* #1454
* #1455
* #1456
* #1457
* #1459
* #1460
* #1462
* #1464
* #1465
* #1466
* #1467
* #1468
* #1469
* #1470
* #1471
* #1472
* #1473
* #1475
* #1476
* #1477
* #1478
* #1479
* #1480
* #1481
* #1482
* #1483
* #1484
* #1485
* #1486
* #1487
* #1488
* #1489
* #1491
* #1492
* #1493
* #1494
* #1496
* #1497
* #1498
* #1499
* #1500
* #1501
* #1502
* #1503
* #1504
* #1505
* #1506
* #1507
* #1508
* #1510
* #1511
* #1512
* #1513
* #1514
* #1515
* #1516
* #1517
* #1518
* #1519
* #1520
* #1522
* #1523
* #1524
* #1525
* #1526
* #1527
* #1528
* #1529
* #1530
* #1531
* #1532
* #1533
* #1534
* #1535
* #1536
* #1537
* #1540
* #1541
* #1543
* #1544
* #1545
* #1546
* #1547
* #1548
* #1549
* #1550
* #1552
* #1553
* #1554
* #1555
* #1556
* #1557
* #1558
* #1559
* #1560
* #1562
* #1563
* #1565
* #1566
* #1567
* #1568
* #1569
* #1570
* #1571
* #1572
* #1573
* #1574
* #1575
* #1576
* #1577
* #1578
* #1579
* #1580
* #1581
* #1583
* #1584
* #1585
* #1586
* #1587
* #1588
* #1589
* #1590
* #1591
* #1592
* #1593
* #1596
* #1597
* #1598
* #1600
* #1602
* #1605
* #1606
* #1607
* #1608
* #1610
* #1611
* #1612
* #1613
* #1614
* #1619
* #1620
* #1621
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@imobachgs imobachgs imobachgs left review comments

@jreidinger jreidinger jreidinger 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.

openQA: product-testing please provide a way to inject a custom repository
4 participants
@lkocman @coveralls @imobachgs @jreidinger