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

sap_facts.sh - ps print only executable with path #28

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

sap_facts.sh - ps print only executable with path #28

wants to merge 3 commits into from

Commits on Jan 10, 2024

  1. Merge pull request sap-linuxlab#22 from sap-linuxlab/dev 

    collection: merge dev to main for release 0.9.1
    @sean-freeman
    sean-freeman authored Jan 10, 2024
    Configuration menu
    Copy the full SHA
    3405579 View commit details
    Browse the repository at this point in the history

  2. Merge pull request sap-linuxlab#24 from sap-linuxlab/dev 

    collection: merge dev to main for release 0.9.1 fix
    @sean-freeman
    sean-freeman authored Jan 10, 2024
    Configuration menu
    Copy the full SHA
    e247f47 View commit details
    Browse the repository at this point in the history

Commits on Aug 20, 2024

  1. sap_facts.sh - ps print only executable with path 

    The output of "ps -ef" not also includes the command but also the
process age and process arguments. Both can have digits in its string
which may interfere when grepping for an instance number like 00 or 01.

To fix this, let ps print only the executable with its path.

Example: ps with default output - 2 lines match where only 1st line should match
/usr/sap/A22/D01/exe/sapstartsrv pf=/usr/sap/A22/SYS/profile/A22_D01_sapa22u01 -D -u a22adm
/usr/sap/A22/ASCS02/exe/sapstartsrv pf=/usr/sap/A22/SYS/profile/A22_ASCS02_sapa22u01 -D -u a22adm

Example: ps with custom layout, only one line matches as expected
/usr/sap/A22/D01/exe/sapstartsrv
    @jhohwieler
    jhohwieler committed Aug 20, 2024
    Configuration menu
    Copy the full SHA
    8ee8993 View commit details
    Browse the repository at this point in the history