AA-2941 : Fix AMP installation & startup scripts to be compatible with RockyLinux Changes

Review Request #925 — Created June 18, 2025 and submitted

pmurugaiyan
AMP
amp_4_0
AA-2941
apoorva.sn, mmiriam, ngurunathan, shuinvy

Fix AMP installation & startup scripts to be compatible with RockyLinux Changes

The changes has been tested locally.

Description From Last Updated

Composer can we keep and remove later when things are working as before?

apoorva.snapoorva.sn

it is better to hold on changes related to composer for later. Since the main objective is to bring up …

apoorva.snapoorva.sn

During uninstall this step might be needed. It can be made optional to the user

apoorva.snapoorva.sn

Same comment for composer

apoorva.snapoorva.sn

Wont we need wget ?

apoorva.snapoorva.sn

Same comment for composer. Can we keep this until we things are working as before and then remove it once …

apoorva.snapoorva.sn

is this for test user or for default user ?

apoorva.snapoorva.sn

then why does the username specify as 'array' ?

apoorva.snapoorva.sn

why destination is commented ?

apoorva.snapoorva.sn
shuinvy
  1. Ship It!
  2. 
      
apoorva.sn
  1. 
      
  2. branches/amp_4_0/amp.spec (Diff revision 1)
     
     

    Composer can we keep and remove later when things are working as before?

    1. Composer not required, we can copy it manually during the development. Its an overhead on ports.

  3. branches/amp_4_0/amp.spec (Diff revision 1)
     
     

    During uninstall this step might be needed. It can be made optional to the user

  4. branches/amp_4_0/amp.spec (Diff revision 1)
     
     

    Same comment for composer

  5. branches/amp_4_0/amp.spec (Diff revision 1)
     
     

    Wont we need wget ?

    1. Its already available as part of the base image.

  6. branches/amp_4_0/amp.spec (Diff revision 1)
     
     

    Same comment for composer. Can we keep this until we things are working as before and then remove it once we have re-written the entire logic w.r.t what composer did

  7. is this for test user or for default user ?

  8. why destination is commented ?

    1. We will not be copying the site-packages as is, instead will be using the proper offline approach i.e, whl files.

  9. 
      
apoorva.sn
  1. 
      
  2. branches/amp_4_0/amp.spec (Diff revision 1)
     
     

    it is better to hold on changes related to composer for later. Since the main objective is to bring up the AMP as is

    1. No, InfluxDB will not work, telegraf also same the same. On top of it we have the services already running, so we have to sideload for our local testing during the development if required.

  3. then why does the username specify as 'array' ?

    1. its the comment issue, fixed.

  4. 
      
pmurugaiyan
apoorva.sn
  1. Ship It!
  2. 
      
pmurugaiyan
Review request changed

Status: Closed (submitted)

Loading...