-
Notifications
You must be signed in to change notification settings - Fork 23
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.
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
WIP: ATS-812 - PoC FFmpeg skeleton - please keep (for experiments) - do not merge #364
Draft
montgolfiere
wants to merge
30
commits into
master
Choose a base branch
from
ATS-812_PoC_FFmpeg_skeleton
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
montgolfiere
commented
Mar 29, 2021
- DO NOT MERGE
- note: PoC only - limited functionality & no tests yet - see also ATS-793 epic
- before I forget, tweak docker image name (=> alfresco-transform-...)
- before I forget, tweak docker image name and also add "poc" (eg. alfresco-transform-ffmpeg-poc)
* Disable WhiteSource & Veracode scans * Disable Veracode scans (cherry picked from commit c3acdfa)
* master: (77 commits) [maven-release-plugin][skip ci] prepare for next development iteration [maven-release-plugin][skip ci] prepare release 2.3.10 ATS-876: Release ATS 1.3.2 [trigger release] (#360) ACS-1185: Bump ActiveMQ client in T-Core (#359) [maven-release-plugin][skip ci] prepare for next development iteration [maven-release-plugin][skip ci] prepare release 2.3.9 ATS-876: Release ATS 1.3.2 [trigger release] (#358) Bump alfresco/alfresco-base-java (#353) Bump alfresco/alfresco-base-java (#352) Bump alfresco/alfresco-base-java (#354) Bump alfresco/alfresco-base-java (#355) Bump alfresco/alfresco-base-java (#356) Bump alfresco/alfresco-base-java (#357) Bump alfresco/alfresco-base-java (#345) Bump alfresco/alfresco-base-java (#346) Bump alfresco/alfresco-base-java (#347) Bump alfresco/alfresco-base-java (#348) Bump alfresco/alfresco-base-java (#349) Bump alfresco/alfresco-base-java (#350) Bump spring-boot-starter-parent from 2.4.2 to 2.4.3 (#344) ...
- merge in master (~ 2.3.11-SNAPSHOT)
- add *skeleton* of unit tests based on other T-Engines (incomplete) - TODO add more tests, re: specific FFmpeg transform options (+ more src/tgt media types)
- run *skeleton* of unit tests (incomplete) - TODO add more tests, re: specific FFmpeg transform options (+ more src/tgt media types)
- temp comment-out other non-FFmpeg tests so that they don't run on Travis (DO NOT MERGE) ;-)
- trivial: fix typo in name (pom.xml)
- fix the -ve unit test files (back to same as other T-Engines) - for now, remove the empty FFmpeg t-options - TODO add some example config options (+ unit tests)
- fix typo in name (pom.xml) - temporarily downgrade to CentOS 7 base docker image (=> ~ FFmpeg 3.4.8-1) - TODO upgrade to CentOS 8 base docker image (=> ~ FFmpeg 4.2.4)
- quick PoC hack to update Dockerfile to FFmpeg 4.2.4 on CentOS 8 (from 3.4.8-1 on CentOS 7) base docker image - TODO raise story/task to do this properly (as part of epic) !
- add PoC option to convert from mp4 to either image/png or image/png - single frame based on timeOffset param (example format "00:00:00.5") - transform option names & support mimetypes subject to change - requires detailed design + (unit) regression tests - see also MM-156
montgolfiere
changed the title
Draft (WIP): ATS-812 - PoC FFmpeg skeleton - for learning experiments
Draft (WIP): ATS-812 - PoC FFmpeg skeleton - for experiments & learning
Dec 21, 2021
…PoC_FFmpeg_skeleton * commit '0c939761299358bec8084ab7675d0a4798d09b16': (97 commits) [maven-release-plugin][skip ci] prepare release 2.5.4 ACS-1998 Enhance the transform config (#484) Bump guava from 30.1.1-jre to 31.0.1-jre (#473) Bump jsoup from 1.14.2 to 1.14.3 (#475) Bump dependency.jackson.version from 2.12.5 to 2.13.0 (#476) Bump dependency.cxf.version from 3.4.4 to 3.4.5 (#477) Bump spring-boot-starter-parent from 2.5.4 to 2.5.6 (#479) ATS-970: Fix build after Travis secrets rotation (#469) Bump maven-javadoc-plugin from 3.3.0 to 3.3.1 (#466) Bump dependency.jackson.version from 2.12.4 to 2.12.5 (#464) [maven-release-plugin][skip ci] prepare for next development iteration [maven-release-plugin][skip ci] prepare release 2.5.3 ATS-956: Release Alfresco Transform Core 2.5.3 [trigger release] (#463) Bump docker-maven-plugin from 0.36.1 to 0.37.0 (#450) Bump jsoup from 1.14.1 to 1.14.2 (#451) Bump activemq-client from 5.16.2 to 5.16.3 (#452) Bump spring-boot-starter-parent from 2.5.3 to 2.5.4 (#454) Bump alfresco/alfresco-base-java (#458) Bump alfresco/alfresco-base-java (#456) Bump alfresco/alfresco-base-java (#457) ... # Conflicts: # .travis.yml
- temp comment out other T-Engine jobs (do not merge)
…PoC_FFmpeg_skeleton * commit '2fd8361a783f117592b55bbca4fc7273adf1db49': (25 commits) [maven-release-plugin][skip ci] prepare for next development iteration [maven-release-plugin][skip ci] prepare release 2.5.5-A1 [trigger release] 2.5.5-A1 (#511) [trigger release] 2.5.5-A1 Bump dependency.jackson.version from 2.13.0 to 2.13.1 Bump dependency.cxf.version from 3.4.5 to 3.5.0 Revert "Bump dependency.tika.version from 2.1.0 to 2.2.1" Bump dependency.tika.version from 2.1.0 to 2.2.1 ACS-2002 Enhance T-Router debug (#507) Bump license-maven-plugin from 2.0.0 to 2.0.1.alfresco-1 (#481) Bump bcmail-jdk15on from 1.69 to 1.70 (#492) Bump bcprov-jdk15on from 1.69 to 1.70 (#491) Bump spring-boot-starter-parent from 2.6.0 to 2.6.2 (#504) ACS-2352: Added Veracode agent-based scanning integration. Bump joda-time from 2.2 to 2.10.13 ATS-969 Tika upgrade 1.x -> 2.x (#493) ATS-968 Migrate to CentOS 7 base images (#490) ACS-1983 CentOS 7 base image for the ImageMagic transformer (#489) Update travis linux dist to Focal (#478) Bump spring-boot-starter-parent from 2.5.6 to 2.6.0 ... # Conflicts: # alfresco-transform-core-aio/alfresco-transform-core-aio-boot/pom.xml # alfresco-transform-core-aio/alfresco-transform-core-aio/pom.xml # alfresco-transform-imagemagick/alfresco-transform-imagemagick-boot/pom.xml # alfresco-transform-imagemagick/alfresco-transform-imagemagick/pom.xml # alfresco-transform-libreoffice/alfresco-transform-libreoffice-boot/pom.xml # alfresco-transform-libreoffice/alfresco-transform-libreoffice/pom.xml # alfresco-transform-misc/alfresco-transform-misc-boot/pom.xml # alfresco-transform-misc/alfresco-transform-misc/pom.xml # alfresco-transform-pdf-renderer/alfresco-transform-pdf-renderer-boot/pom.xml # alfresco-transform-pdf-renderer/alfresco-transform-pdf-renderer/pom.xml # alfresco-transform-tika/alfresco-transform-tika-boot/pom.xml # alfresco-transform-tika/alfresco-transform-tika/pom.xml # alfresco-transformer-base/pom.xml # pom.xml
… FFmpeg (do not merge)
- remove extra isTransformable check (& related isSupported src/tgt) for now - add mov to mp4 to supported src/tgt config - update lic headers to 2022
- same as other mock tests, eg. see ImageMagickControllerTest - although it seems some fields should be optional for pojo serialization (mock test)
…eOffset) to enable video/audio trim - only set default frames num (1) if target is a supported image format (note: currently configured for jpg and png)
- eg. to demo transcoding of video resolution (or even resize of frame if output to an image)
montgolfiere
changed the title
Draft (WIP): ATS-812 - PoC FFmpeg skeleton - for experiments & learning
WIP: ATS-812 - PoC FFmpeg skeleton - please keep (for experiments & learning) - do not merge
Feb 2, 2022
montgolfiere
changed the title
WIP: ATS-812 - PoC FFmpeg skeleton - please keep (for experiments & learning) - do not merge
WIP: ATS-812 - PoC FFmpeg skeleton - please keep (for experiments) - do not merge
Feb 2, 2022
- went from green to red (when building docker image - apparently change in FFmpeg install) - took the opportunity to switch to supported Java Base Image (ie. CentOS 7 LTS instead of CentOS 8) - hence, for now (for PoC) floating FFmpeg v3.x (rather than v4.x)
* master: (52 commits) [maven-release-plugin][skip ci] prepare for next development iteration [maven-release-plugin][skip ci] prepare release 2.5.7-A4 ACS-2587 Fix test failure ACS-2587 Include directAccessUrl in T-Config if supported (#539) ATS-864: fix dependabot ATS-864: Switch from fabric8 to docker mvn plugin (#436) ACS-2072 Enforce third party license compliance. [maven-release-plugin][skip ci] prepare for next development iteration [maven-release-plugin][skip ci] prepare release 2.5.7-A3 [trigger release] 2.5.7-A3 Bump activemq-client from 5.16.3 to 5.16.4 (#534) Bump maven-javadoc-plugin from 3.3.1 to 3.3.2 (#533) ATS-981 Avoid CVE-2022-23181 with spring-boot-starter-web 2.6.3 (#537) [maven-release-plugin][skip ci] prepare for next development iteration [maven-release-plugin][skip ci] prepare release 2.5.7-A2 [trigger release] 2.5.7-A2 ACS-2497 Rework httpTransformRequestUsingDirectAccessUrlTest so it can be used in AI and Aspose (#535) Bump docker-maven-plugin from 0.38.1 to 0.39.0 (#528) [maven-release-plugin][skip ci] prepare for next development iteration [maven-release-plugin][skip ci] prepare release 2.5.7-A1 ... # Conflicts: # .travis.yml # alfresco-transformer-base/src/main/java/org/alfresco/transformer/util/RequestParamMap.java Also bumped pom.xml version (for two ffmpeg folders) from 2.5.5-A2-SNAPSHOT to match others (2.5.7-A5-SNAPSHOT)
- update FFmpeg Travis job (to match others)
- fix tests: add new coreVersion (transform.core.verson) to match other T-Engines
I'm looking for this kind of transformation using ffpmeg. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.