Skip to content

Conversation

joaquincasares
Copy link
Contributor

No description provided.

echo "Working through Python dependencies..."
sudo dpkg -i /var/cache/apt/archives/${service}/*.deb
)
else
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is one of the bits that I kept fighting with. I was able to isolate the ordering and the fact that we need to install all the packages twice to get python3 to complete it's installation.

libdbusmenu-gtk3-4
python3
libappindicator1
)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the other bit I was fighting with. Once these virtual packages + python3 are installed, the rest of the dependencies will install as intended.

| sed 's|libgcc1|libgcc-s1|' \
| sed 's|default-dbus-session-bus|dbus-user-session|' \
| sed 's|dbus-session-bus|dbus|' \
| sed 's|^gsettings-backend$||' \
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the last bit I fought with. We need to swap out the virtual and package names, mainly for axon-dash-pdf, if I'm not mistaken. But once this is done, we'll download the correct package and upon installation, axon-dash-pdf will have the required dependencies.

@brainJamStark
Copy link
Contributor

Is this only for Cassandra air-gapped installation? Are we going to do the same for Kafka?

@joaquincasares
Copy link
Contributor Author

Thanks for catching that @brainJamStark ! I've included instructions for Kafka (but will test them tomorrow). I've also changed the wording to match support for both Cassandra and Kafka agents.

I've also made a few changes as requested by Richard over a brief live review today, mainly:

  • no longer using /var/cache/apt/archives/
  • adding an optional note for axon-dash-pdf2
  • using axon-dash-pdf2
  • adding an Upgrading section
  • adding shebang headers for easier copy-pasting
  • separating out the agent installation scripts:
    • please review so we can collectively see if we like it aesthetically. if not, we'll combine it back down into one script to avoid declaring install_dependency() twice.

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