Skip to content

Conversation

ddelong
Copy link

@ddelong ddelong commented Jan 31, 2024

Problem

deployAtEnd is desirable for our build system because it keeps the produced libraries consistent for a commit, reducing the risk of out of sync components caused by partial builds. However, due to the use of this configuration in the pom.xml files, the original Maven deploy plugin version does not actually deploy the built artifacts:

      <plugin>
        <artifactId>maven-deploy-plugin</artifactId>
        <configuration>
          <skip>true</skip>
        </configuration>
      </plugin>

Solution

Upgrade the version of maven-deploy-plugin in use from 2.8.1 to 3.1.1. This behavior was fixed in Maven Deploy Plugin 3.1.0 (MDEPLOY-226).

Testing

This build used the updated version of the deploy plugin: https://private.hubteam.com/blazar/branches/10108237/builds/6/modules/hadoop

It correctly deployed during the invocation of the last module. I also verified the artifact could be downloaded from Nexus.

Working changes

  • Change build order to avoid deploy plugin defect
  • Remove build exclusion
  • Restore original location for argument, remove weird skip config
  • Move the deployAtEnd to a different spot

@ddelong ddelong self-assigned this Jan 31, 2024
@ddelong ddelong force-pushed the ddelong-move-deploy branch from ea8c958 to d8615b2 Compare February 1, 2024 16:14
@bbeaudreault
Copy link
Member

Is this no longer needed?

@ddelong
Copy link
Author

ddelong commented Feb 5, 2024

Still needed for deployAtEnd benefits.

@ddelong ddelong force-pushed the ddelong-move-deploy branch from d8615b2 to e765c27 Compare February 5, 2024 15:17
- Change build order to avoid deploy plugin defect
- Remove build exclusion
- Restore original location for argument, remove weird skip config
- Move the deployAtEnd to a different spot
@ddelong ddelong force-pushed the ddelong-move-deploy branch from e765c27 to 6141fc2 Compare February 5, 2024 15:18
@ddelong ddelong merged commit 5c11924 into hubspot-3.3.6 Feb 5, 2024
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.

3 participants