Skip to content

Commit 649bfd2

Browse files
authored
Merge pull request #317 from delphix/dlpx/pr/pjang-dlpx/4d2706f8-489c-41e6-b2eb-5425092ffb87
DLPX-91307 Install packages from dms-core-gate build-dependencies
2 parents 80fee86 + 659880b commit 649bfd2

File tree

2 files changed

+34
-12
lines changed

2 files changed

+34
-12
lines changed

README.md

Lines changed: 31 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ projects.
2929

3030
## System Requirements
3131

32-
This framework is intended to be run on an Ubuntu 18.04 system with some basic
32+
This framework is intended to be run on an Ubuntu 20.04 system with some basic
3333
developer packages installed, such as git, and passwordless sudo enabled. Note
3434
that it will automatically install various build-dependencies on the system, so
3535
as a safety precaution it is currently restricted to only run on an AWS instance
@@ -48,14 +48,16 @@ This quick tutorial shows how to build the packages managed by this framework.
4848
### Step 1. Create build VM
4949

5050
You need a system that meets the requirements above. For Delphix developers, you
51-
should clone the `bootstrap-18-04` group on DCoA.
51+
should clone the `dlpx-internal-buildserver-develop` group on DCoA.
5252

5353
### Step 2. Clone this repository
5454

55-
Clone this repository on the build VM.
55+
Clone this repository on the build VM. In order to clone this repository to the
56+
build VM, you may need to specify a personal access token for authentication and
57+
authorization.
5658

5759
```
58-
git clone https://github.com/delphix/linux-pkg.git
60+
git clone https://<token>@github.com/delphix/linux-pkg.git
5961
```
6062

6163
### Step 3. Build a package
@@ -639,19 +641,31 @@ hook to `config.sh`. You should use the following functions provided by
639641
* `update_upstream_from_git()` if `UPSTREAM_GIT_URL` & `UPSTREAM_GIT_BRANCH` are
640642
set.
641643

642-
#### Step 8. Add package to package-lists
644+
#### Step 8. Make the package official
643645

646+
See [Common Steps > Make the package official](#make-the-package-official).
647+
648+
#### Step 9. Submit a Pull Request for the new package
649+
650+
Once you verify that the package can be built, submit a pull request with only the
651+
`/packages/<package>` chages. Once merged, this change will create a Jenkins job
652+
`(/linux-pkg/develop/build-package/<package>/pre-push)` to
653+
build the new package as part of the automation pipeline. Without merging these changes
654+
first, we will not be able to test the automated build and integration into the appliance.
655+
656+
#### Step 10. Add package to package-lists
657+
658+
In a separate change, add the package to package-lists.
644659
See [Common Steps > Add package to package-lists](#add-package-to-package-lists).
645660

646-
#### Step 9. Test your changes
661+
#### Step 11. Test your changes
647662

648663
See section [Testing Your Changes](#testing-your-changes).
649664

650-
#### Step 10. Make the package official
665+
#### Step 12. Submit a Pull-Request to add the package to package-lists
651666

652-
See [Common Steps > Make the package official](#make-the-package-official).
653-
654-
#### Step 11. Submit a Pull-Request for your changes to linux-pkg
667+
**IMPORTANT**: This is the step which will trigger integration of this package into
668+
the appliance.
655669

656670
### In-house package
657671

@@ -753,7 +767,12 @@ TODO: complete section
753767

754768
### Testing changes to linux-pkg
755769

756-
TODO: complete section
770+
If you are testing a newly added package to linux-pkg:
771+
772+
1. Run `git-ab-pre-push -b <package>` from `linux-pkg`.
773+
774+
Note that this package must already have been added to `/packages/` in the
775+
`develop` branch, so that its specific Jenkins build job exists.
757776

758777
## Package Lists
759778

@@ -819,7 +838,7 @@ environment variable.
819838

820839
When building packages for an older version of the Delphix Appliance, the build
821840
image will need to be picked accordingly. We are currently using
822-
`bootstrap-18-04`, but this will not be the case anymore once we switch to a
841+
`dlpx-internal-buildserver-develop`, but this will not be the case anymore once we switch to a
823842
newer Ubuntu distribution.
824843

825844
## Contributing

packages/masking/config.sh

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,9 @@ DEFAULT_PACKAGE_GIT_URL="https://github.com/delphix/dms-core-gate.git"
2222
PACKAGE_DEPENDENCIES="adoptopenjdk"
2323

2424
function prepare() {
25+
logmust read_list "$WORKDIR/repo/packaging/build-dependencies"
26+
logmust install_pkgs "${_RET_LIST[@]}"
27+
2528
logmust install_pkgs "$DEPDIR"/adoptopenjdk/*.deb
2629
}
2730

0 commit comments

Comments
 (0)