Skip to content

Initial implementation for storing Agama configuration provided by kernel cmdline#1866

Merged
teclator merged 6 commits intomasterfrom
initrd_agama_cmdline
Jan 8, 2025
Merged

Initial implementation for storing Agama configuration provided by kernel cmdline#1866
teclator merged 6 commits intomasterfrom
initrd_agama_cmdline

Conversation

@teclator
Copy link
Copy Markdown
Contributor

@teclator teclator commented Jan 7, 2025

Problem

As part of the linuxrc compatibilty Jira entry, we want to make sure several boot arguments are translated to environment variables that indeed affect the execution of Agama.

That includes the following patterns: LIBSTORAGE_, YAST_, Y2*, ZYPP_FULLLOG.

Some examples (useful for verifying everything actually works): LIBSTORAGE_MULTIPATH_AUTOSTART, LIBSTORAGE_MDPART, YAST_ACTIVATE_LUKS, YAST_REUSE_LVM, Y2DEBUG, ZYPP_FULLLOG

Solution

The configuration that is provided by the kernel cmdline will be stored to /etc/agama.d/cmdline.conf and the agama services will use it as an EnvironmentFile. It is an initial implementation an probably would need some more work.

Testing

  • Tested manually

@teclator teclator force-pushed the initrd_agama_cmdline branch from 4918e3d to 7f8fd62 Compare January 7, 2025 13:43
@teclator teclator force-pushed the initrd_agama_cmdline branch from 7f8fd62 to 57054c9 Compare January 8, 2025 08:26
@teclator teclator requested a review from imobachgs January 8, 2025 09:59
@teclator teclator changed the title Initial implementation for storing Agama configuration provided by kernel cmdline and exporting as env variables Initial implementation for storing Agama configuration provided by kernel cmdline Jan 8, 2025
@teclator teclator marked this pull request as ready for review January 8, 2025 10:10
Copy link
Copy Markdown
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

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

LGTM. Great job!

As already discussed in a chat, we need some documentation to explain how to test this kind of changes.

@teclator teclator force-pushed the initrd_agama_cmdline branch from 180bbf3 to 6bd88fc Compare January 8, 2025 14:19
@teclator teclator merged commit 1ec82fe into master Jan 8, 2025
@teclator teclator deleted the initrd_agama_cmdline branch January 8, 2025 14:52
@imobachgs imobachgs mentioned this pull request Jan 10, 2025
imobachgs added a commit that referenced this pull request Jan 13, 2025
Update to release version 11.

* #1495
* #1564
* #1617
* #1618
* #1625
* #1626
* #1627
* #1628
* #1630
* #1631
* #1632
* #1633
* #1634
* #1635
* #1636
* #1639
* #1640
* #1641
* #1642
* #1643
* #1644
* #1645
* #1646
* #1647
* #1648
* #1649
* #1650
* #1651
* #1652
* #1654
* #1655
* #1656
* #1657
* #1660
* #1663
* #1666
* #1667
* #1668
* #1670
* #1671
* #1673
* #1674
* #1675
* #1676
* #1677
* #1681
* #1682
* #1683
* #1684
* #1687
* #1688
* #1689
* #1690
* #1691
* #1692
* #1693
* #1694
* #1695
* #1696
* #1698
* #1699
* #1702
* #1703
* #1704
* #1705
* #1707
* #1708
* #1709
* #1710
* #1711
* #1712
* #1713
* #1714
* #1715
* #1716
* #1717
* #1718
* #1720
* #1721
* #1722
* #1723
* #1727
* #1728
* #1729
* #1731
* #1732
* #1733
* #1734
* #1735
* #1736
* #1737
* #1740
* #1741
* #1743
* #1744
* #1745
* #1746
* #1751
* #1753
* #1754
* #1755
* #1757
* #1762
* #1763
* #1764
* #1765
* #1766
* #1767
* #1769
* #1771
* #1772
* #1773
* #1774
* #1777
* #1778
* #1785
* #1786
* #1787
* #1788
* #1789
* #1790
* #1791
* #1792
* #1793
* #1794
* #1795
* #1796
* #1797
* #1798
* #1799
* #1800
* #1802
* #1803
* #1804
* #1805
* #1807
* #1808
* #1809
* #1810
* #1811
* #1812
* #1814
* #1815
* #1821
* #1822
* #1823
* #1824
* #1825
* #1826
* #1827
* #1828
* #1830
* #1831
* #1832
* #1833
* #1834
* #1835
* #1836
* #1837
* #1838
* #1839
* #1840
* #1841
* #1842
* #1843
* #1844
* #1845
* #1847
* #1848
* #1849
* #1850
* #1851
* #1854
* #1855
* #1856
* #1857
* #1860
* #1861
* #1863
* #1864
* #1865
* #1866
* #1867
* #1871
* #1872
* #1873
* #1875
* #1876
* #1877
* #1878
* #1880
* #1881
* #1882
* #1883
* #1884
* #1885
* #1886
* #1888
* #1889
* #1890
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants