[CLIENT-5171] CI/CD: Remove dev workflow and dev branch. Promote directly to TEST and skip DEV gate - #1186
Open
juliannguyen4 wants to merge 41 commits into
Open
Conversation
…kes promoting artifacts to TEST twice as fast compared to now.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## stage #1186 +/- ##
==========================================
+ Coverage 85.46% 85.64% +0.18%
==========================================
Files 101 102 +1
Lines 14661 14888 +227
==========================================
+ Hits 12530 12751 +221
- Misses 2131 2137 +6 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
…v-branch-and-promote-directly-to-TEST
…al dev workflow works..
…v-branch-and-promote-directly-to-TEST
Collaborator
Author
|
This is blocked because there is currently a release bundle that I'm trying to provide a customer with. I don't want to override that bundle with a "test" one |
…v-branch-and-promote-directly-to-TEST
…ng the version on a test branch.
This reverts commit 7a9237f.
…n updating the version on a test branch." This reverts commit d560260.
…om JFrog to Github actions. This will probably be used later on for nightly / weekend testing.
…v-branch-and-promote-directly-to-TEST
…ing representation, then have fromJson() convert back to JSON object in test-artifact workflow
…v-branch-and-promote-directly-to-TEST
…d when storing a server map type (#1026)
…er value, where the field is represented as an enum type in the C client (#1142) Deprecations The following API calls will emit a DeprecationWarning if an out of bounds value is passed for: - bit_resize operation's resize_flags parameter - map_create operation's map_order parameter - list_set_order and list_create operations' list_order parameter - String operation is_numeric's numeric_type parameter - String operations regex_compare and regex_replace's regex_flags parameter - list_sort operation's and ListSort expression's sort_flags and order parameters, respectively - cdt_ctx.cdt_ctx_map_key_create's order parameter - cdt_ctx.cdt_ctx_list_index_create's order parameter - All bitwise operations' bit policy parameter - All map operations and expressions' map policy parameter - All list operations and expressions' list policy parameter - All HLL operations that take in an HLL policy parameter Risks - If new members are added to C client enums, this code must be updated to account for the new member. Currently the C client does not have a sentinel value to represent the max value for each enum type Breaking change: for bit resize flags, list and map order, string ops' numeric type and regex flags, and list sort flags (except for cdt_ctx objects), a ParamError is now raised if the user passes a value < INT_MIN or > INT_MAX instead of INT64_MIN and INT64_MAX, respectively. This is not a big deal since these enum values are very well within (INT_MIN, INT_MAX), anyways.
shannonklaus
requested changes
Aug 14, 2026
shannonklaus
left a comment
shannonklaus
left a comment
There was a problem hiding this comment.
You also need to change dependabot to target stage. You should search for dev and fix all of the stale comments/etc.
Also we talked about having a rotating test plan, when is this part of the python CI/CD refactor going to be completed?
Collaborator
Author
I forgot to do this; thanks for the reminder |
Collaborator
Author
I wanted to scope this PR to just remove the |
…ld not run i.e stage tests and release bundle creation + promotion.
…v-branch-and-promote-directly-to-TEST
…the tools package image 12.1.1_2 being deleted in JFrog by the new cleanup policy
…the default tag used in the shared action). So apply this fix everywhere else that setup-aerospike-server is called.
…of the beginning jobs
…e replaced with *-docker-prod-public-local since any artifacts in JFrog dev repos will be cleaned up after 30 days
…ll previous ongoing runs. This assumes that only the latest change in stage will be release candidates.
… and dry-run in workflow run name to ensure those inputs' values are correct.
…itly target this group instead of the default one
Collaborator
Author
|
TODO - need to publish manylinux openssl image to internal stage local JFrog repo |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
This reduces the time it takes to provide QE with artifacts by upwards of 66% compared to the time it currently takes in
dev. We run valgrind and the tests from both the dev and stage workflow in parallel now instead of serially.Notes
stage-testto point to the same revision as this branchExtra changes
We are removing testing for Alma Linux 8 (binary compatible with RHEL 8) here, which is okay since we dropped support for RHEL 8 in an earlier release.
TODO
devbefore merging this PRdevafter merging