Skip to content

Conversation

@mohamedabdallah20
Copy link
Contributor

remove $CODENAME that doesn't work properly

remove $CODENAME that doesn't work properly 

Signed-off-by: Mohammed Abdallah <[email protected]>
@mohamedabdallah20 mohamedabdallah20 requested review from a team as code owners July 27, 2025 13:19

```bash
echo "deb [signed-by=/usr/share/keyrings/fluentbit-keyring.gpg] https://packages.fluentbit.io/ubuntu/${CODENAME} ${CODENAME} main" | sudo tee /etc/apt/sources.list.d/fluent-bit.list
echo "deb [signed-by=/usr/share/keyrings/fluentbit-keyring.gpg] https://packages.fluentbit.io/ubuntu/$(grep -oP '(?<=VERSION_CODENAME=).*' /etc/os-release || lsb_release -cs) $(grep -oP '(?<=VERSION_CODENAME=).*' /etc/os-release || lsb_release -cs) main" | sudo tee /etc/apt/sources.list.d/fluent-bit.list
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we need to redirect stderr for lsb_release messages, otherwise you can get extra lines of output like No LSB modules are available.

- Redirect stderr messages, to not get extra lines of output like No LSB modules are available.
- Separate the codename variable to be more efficient and readable 

Signed-off-by: Mohammed Abdallah <[email protected]>
@patrick-stephens patrick-stephens merged commit 890e434 into fluent:master Jul 28, 2025
5 checks passed
TomlinfreeGit pushed a commit to TomlinfreeGit/fluent-bit-docs that referenced this pull request Oct 28, 2025
* Update installation on ubuntu commands 

remove $CODENAME that doesn't work properly 

Signed-off-by: Mohammed Abdallah <[email protected]>

* Update installation on ubuntu

- Redirect stderr messages, to not get extra lines of output like No LSB modules are available.
- Separate the codename variable to be more efficient and readable 

Signed-off-by: Mohammed Abdallah <[email protected]>

---------

Signed-off-by: Mohammed Abdallah <[email protected]>
Signed-off-by: Tom <[email protected]>
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.

3 participants