Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • aims2 aims2
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Jira
    • Jira
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • linuxsupportlinuxsupport
  • rpmsrpms
  • aims2aims2
  • Merge requests
  • !10

Use attachments for images

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Daniel Juarez requested to merge use_attachments_for_images into master Oct 30, 2019
  • Overview 8
  • Commits 14
  • Pipelines 8
  • Changes 5

Replacing SOAP::Lite seems too much work IMHO. Not only documentation is scarce for the alternative, also examples or docs are not clear enough. It will require huge changes that will ask for a complete redesign of the code.

If we move forward with the replacement, I would suggest uploading the files in some other means as we do not know for sure how much memory reduction we can get with XML::Compile alternatives: i.e. scp?

Tweaking how we deal with attachments I could further reduce the footprint below Ben's numbers (to ~50% instead of ~60%). Clearly not enough but a workaround while we look for replacement.

CC/ @airibarr

Edited Nov 04, 2019 by Daniel Juarez
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: use_attachments_for_images