-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[YSQL] org.yb.ysqlconnmgr.TestSessionParameters.testSessionParameters depends on TZ #22228
Open
1 task done
Labels
area/ysql
Yugabyte SQL (YSQL)
kind/enhancement
This is an enhancement of an existing feature
kind/failing-test
Tests and testing infra
priority/medium
Medium priority issue
status/awaiting-triage
Issue awaiting triage
Comments
jasonyb
added
area/ysql
Yugabyte SQL (YSQL)
status/awaiting-triage
Issue awaiting triage
labels
May 2, 2024
yugabyte-ci
added
kind/failing-test
Tests and testing infra
priority/medium
Medium priority issue
kind/enhancement
This is an enhancement of an existing feature
labels
May 2, 2024
jasonyb
pushed a commit
that referenced
this issue
May 2, 2024
…4faa000' into pg15 Summary: Merge YB master commit 01b596f titled Use ldd to determine glibc version instead of scanning binaries. and committed 2024-03-12T13:07:42-07:00 into YB pg15. This is the commit right before bitmap scan support. YB pg15 initial merge refers to 55782d5. - TestPgAlterTableColumnType.java: - YB master 833fc83 deletes the file claiming tests get moved into regress test yb_alter_table_rewrite. YB pg15 touched the file. Delete the file. - TestPgReplicationSlot.java: - consumptionOnSubsetOfColocatedTables: YB master af9178c adds this new test. Adjacent lines conflict with @ignore("YB_TODO(stiwary)") added for replicationConnectionConsumption by YB pg15. Yet, this test also needs to be ignored because of assertion failure of Assert(HistoricSnapshotActive()) in RelationGetIdentityKeyBitmap. - testInnerLSNValues: YB master cc88c41 adds this test. It fails on YB pg15. For now, ignore the test and mark with todo as is done for other replication tests. - testReplicationConnectionUpdateRestartLSN: (same as testInnerLSNValues except it is YB master 350df35) - testReplicationConnectionUpdateRestartLSNWithRestarts: (same as testReplicationConnectionUpdateRestartLSN) - explain.c: - function declarations: YB master e48f1a8 adds parameter yb_estimated_docdb_result_width to ExplainIndexScanDetails. YB pg15 changes indentation of those lines. Trivial merge. - tablecmds.c: - ATRewriteCatalogs: YB pg15 initial merge changes IsYBRelation(rel) to IsYBRelation(tab->rel) in some if conditions. YB master 833fc83 adds !yb_enable_alter_table_rewrite to those same if conditions. Trivial merge. - ATRewriteTable: YB pg15 initial merge changes heap_beginscan to table_beginscan. Adjacent lines conflict with YB master 833fc83 adding logic oldrel->rd_att = oldTupDesc; Trivial resolution. - ATPostAlterTypeParse: upstream PG 87259588d0ab0b8e742e30596afa7ae25caadb18 adds stmt->reset_default_tblspc = true; in the same location YB master 833fc83 adds an if condition. Trivial resolution. - createplan.c: - function declarations: like in explain.c, YB master e48f1a8 adds parameter yb_estimated_docdb_result_width to two functions which YB pg15 changes indentation for. Trivial resolution. - yb_decode.c: - YBDecodeInsert: YB master 7ea6665 removes RelationClose at the end of the function. Adjacent line conflict with YB pg15 merge 417e9b3 which adds extra argument to the ReorderBufferQueueChange function call. Trivial resolution. - YBDecodeDelete: like YB pg15 merge 417e9b3, pass false for the new argument of the new ReorderBufferQueueChange call. - pgoutput.c: - pgoutput_send_begin: YB master cc88c41 modifies send_replication_origin definition to have !IsYugaByteEnabled. Adjacent lines conflict with upstream PG d5a9d86d8ffcadc52ff3729cd00fbd83bc38643c. Furthermore, YB pg15 merge 417e9b3 had issues in this area before, particularly with upstream PG a8fd13cab0ba815e9925dc9676e6309f699b5f72. Upon further inspection, find that upstream PG introduced more uses of the send_repl_origin function that YB likely does not want to enter. Solve it with new code: - For all calls to send_repl_origin, make sure the local variable send_replication_origin passed to it is set to false in case !IsYugaByteEnabled. This also should cover any prior uses of the local variable. In order to do this in a way that has less diff with upstream PG, leave the initial definition as is and only overwrite it with YB logic after. This means reworking YB master cc88c41 which modifies the definition itself. - Within send_repl_origin, assert !IsYugaByteEnabled. This means reverting the extra !IsYugaByteEnabled check before calling send_repl_origin that YB pg15 merge 417e9b3 added because as long as send_replication_origin is false, calling the function should be a no-op. And this way makes diff with upstream PG smaller. - pquery.c: - function declarations: YB master 9e0f33a removes parameter isSingleRowModifyTxn from ProcessQuery. Upstream PG 8255c7a5eeba8f1a38b7a431c04909bde4f5e67d and 2f9661311b83dc481fc19f6e3bda015392010a40 change parameters and indentation of this same function. Trivial merge. - ProcessQuery: - signature: (same as function declarations minus indentation) - YbIsSingleRowModifyTxnPlanned call: YB master 9e0f33a moves some code to helper YbIsSingleRowModifyTxnPlanned in ybcModifyTable.c. YB pg15 161efd6 translates some of that code for PG 15. Transfer those edits to the new location. - PortalRunMulti: - ProcessQuery call: (same as function declarations) - ybcModifyTable.c: (see pquery.c ProcessQuery). - ruleutils.c: - function declarations: - YB master 833fc83 adds new parameter is_yb_alter_table to some functions. YB pg15 changes indentation. Trivial merge. - YB master 833fc83 also adds new function yb_decompile_pk_column_index_array where upstream PG also adds new function get_reloptions. Trivial merge. - pg_get_indexdef_worker: conflict between upstream PG 87259588d0ab0b8e742e30596afa7ae25caadb18 and YB master 833fc83 which moves the code under two if conditions. - pg_get_constraintdef_worker: - an indexId definition: YB master 833fc83 moves one indexId definition under !is_yb_alter_table condition. Upstream PG 8b069ef5dca97cd737a5fd64c420df3cd61ec1c9 and 94aa7cc5f707712f592885995a28e018c7c80488 change and move this code higher. Move the conditioning to the new location. Since both is_yb_alter_table and !is_yb_alter_table conditions need to be moved up, combine them to a single ternary operation. - tblspc definition: again, conflict between upstream PG 87259588d0ab0b8e742e30596afa7ae25caadb18 and YB master 833fc83 which moves the code under one if condition. One difference is that tblspc declaration is left outside the if scope to reduce diff with upstream PG. - guc.c: - ResetAllOptions: upstream PG 2432b1a04087edc2fd9536c7c9aa4ca03fd1b363 modifies if body and YB master d8fc67d changes if condition. Apply both. - AtEOXact_GUC: (same as ResetAllOptions) - ReportGUCOption: same upstream PG 2432b1a04087edc2fd9536c7c9aa4ca03fd1b363 moves condition (record->flags & GUC_REPORT) into new function ReportChangedGUCOptions. Same YB master d8fc67d touches that condition. Translate to the new location. - set_config_option_ext: (same as ResetAllOptions) - pathnodes.h: - IndexOptInfo: adjacent lines conflict between upstream PG 74dfe58a5927b22c744b29534e67bfdd203ac028 and YB master e48f1a8 (relation.h). - timestamp regress tests: - merge with YB master 77d71da. - costsize.c: - includes: upstream PG f09346a9c6218dd239fdf3a79a729716c0d305bd removes optimizer/var.h. Use optimizer/optimizer.h instead. This was already added by the initial merge 55782d5. - yb_get_ybctid_width: YB master e48f1a8 adds heap_open/heap_close. Change to table_open/table_close. This requires adding table.h include. - yb_get_docdb_result_width: (same as yb_get_ybctid_width) - yb_virtual_wal_client.c: - CleanupAckedTransactions: YB master 350df35 introduces code using lnext, but upstream PG 1cff1b95ab6ddae32faa3efe0d95a820dbfdc164 adds another parameter, so pass it. Similarly, list_delete_cell is also affected to remove a parameter, so don't pass it. - pg15_tests/passing_tests.tsv: - remove TestPgAlterTableColumnType tests since that file was deleted by YB master 833fc83. - TestSessionParameters.java: - SessionParameters: YB master d8fc67d expects extra_float_digits default to be 0, but upstream PG 02ddd499322ab6f2f0d58692955dc9633c2150fc changed the default to 1, so adjust accordingly. Test Plan: On Almalinux 8, check there are no regressions: #!/usr/bin/env bash set -eu ./yb_build.sh fastdebug --gcc11 pg15_tests/run_tests.sh On Almalinux 8, fastdebug, gcc11, get the following results via pg15_tests/run_tests.sh: 0 2024-05-02T10:23:26-07:00 integration-tests_cdcsdk_consistent_stream-test CDCSDKConsistentStreamTest.TestCDCSDKConsistentStreamWithForeignKeys 0 2024-05-02T10:25:12-07:00 integration-tests_cdcsdk_consumption_consistent_changes-test CDCSDKConsumptionConsistentChangesTest.TestCDCSDKConsistentStreamWithForeignKeys 1 2024-05-02T10:25:49-07:00 JAVA org.yb.pgsql.TestPgEstimatedDocdbResultWidth 1 2024-05-02T10:27:15-07:00 JAVA org.yb.pgsql.TestPgCostModelSeekNextEstimation 0 2024-05-02T10:27:38-07:00 JAVA org.yb.pgsql.TestPgRegressReplicationSlot#testPgRegressReplicationSlot 0 2024-05-02T10:28:08-07:00 integration-tests_cdcsdk_ysql-test CDCSDKYsqlTest.TestReplicationSlotDropWithActiveInvalid 0 2024-05-02T10:28:28-07:00 integration-tests_cdcsdk_consumption_consistent_changes-test CDCSDKConsumptionConsistentChangesTest.TestVWALConsumptionOnMixTables 1 2024-05-02T10:28:53-07:00 JAVA org.yb.ysqlconnmgr.TestSessionParameters 1 2024-05-02T10:33:10-07:00 JAVA org.yb.pgsql.TestPgWithoutWaitQueuesIsolationRegress 1 2024-05-02T10:33:37-07:00 JAVA org.yb.pgsql.TestPgBaseScansCostModel 0 2024-05-02T10:35:36-07:00 JAVA org.yb.pgsql.TestPgParallelReadIsolation Note org.yb.ysqlconnmgr.TestSessionParameters should pass with TZ=UTC (see issue #22228). Also: #./yb_build.sh fastdebug --gcc11 --enable-ysql-conn-mgr-test --java-test org.yb.pgsql.TestPgAuthorization ./yb_build.sh fastdebug --gcc11 --enable-ysql-conn-mgr-test --java-test 'org.yb.pgsql.TestPgSequences#testSerialTypes' ./yb_build.sh fastdebug --gcc11 --enable-ysql-conn-mgr-test --java-test 'org.yb.pgsql.TestPgDdlFaultTolerance#testDocDBMetadataFailuresForTable' ./yb_build.sh fastdebug --gcc11 --enable-ysql-conn-mgr-test --java-test 'org.yb.pgsql.TestPgSequencesWithServerCache#testSerialTypes' #./yb_build.sh fastdebug --gcc11 --enable-ysql-conn-mgr-test --java-test 'org.yb.pgsql.TestYsqlUpgrade#replacingViewKeepsCacheConsistent' ./yb_build.sh fastdebug --gcc11 --cxx-test pgwrapper_geo_transactions-test --gtest_filter GeoTransactionsTest.TestNonlocalAbort --test-args --enable_wait_queues=false --test-args --ysql_pg_conf_csv="yb_max_query_layer_retries=3" Reviewers: aagrawal, tfoucher, xCluster, hsunder Reviewed By: tfoucher Subscribers: ybase, ycdcxcluster, yql Differential Revision: https://phorge.dev.yugabyte.com/D34675
jasonyb
pushed a commit
that referenced
this issue
May 10, 2024
…0155eb9' into pg15 Summary: Merge YB master commit 12d6abe titled [#21103] DocDB: Hold Session shared pointer in the CDC State Table callback and committed 2024-03-26T20:41:12-07:00 into YB pg15. This is right before several compilation breakages on master. YB pg15 initial merge refers to 55782d5. - index.c: - reindex_relation: major conflict between YB pg15 merge 5627af5 and YB master fad94f7. A lot of context is present in the commit message of YB pg15 merge 5627af5. This time, the PG lines that were once put under (!(IsYBRelation(iRel) && iRel->rd_index->indisprimary)) were moved back out, so place them like in the order they were before (particularly, the indexNamespaceId declaration/definition goes higher up to the top of the scope). - cluster.c: - make_new_heap: YB pg15 merge 5627af5 makes trivial adjustments to code that YB master fad94f7 moves into helper YbRelationSetNewRelfileNode and calls just the helper here. - tablecmds.c: - ExecuteTruncateGuts: YB pg15 merge 5627af5 adjusts reindex_relation call. YB master fad94f7 switches is_yb_table_rewrite from false to true. Apply the same transformation in the adjusted call. - nodeYbBitmapTablescan.c: - CreateYbBitmapTableScanDesc: this function created by YB master f689455 returns what is assigned to ss_currentScanDesc, but such assignment was already done by YB pg15 merge ad2fedc in at least YbBitmapTableNext. Move that better code into YbBitmapTableNext. This also requires a signature change, and since the more future YB master 773869c is more suitable, get inspiration from that. - YbBitmapTableNext: YB pg15 merge ad2fedc adds extra lines for setting up ss_currentScanDesc, and YB master a28acc7 adds extra line node->skipped_tuples = 0 in the same location. Add the one line from master. As for the setting up of ss_currentScanDesc, see above on CreateYbBitmapTableScanDesc. - ExecReScanYbBitmapTableScan: YB master f689455 adds two things to this function: - free old node->ss.ss_currentScanDesc if exists - set new node->ss.ss_currentScanDesc The first is already handled (in a better pg15 way) by YB pg15 merge f689455. The second is also not needed because ss_currentScanDesc is already set in YbBitmapTableNext (and you can also see this part is removed by future YB master 773869c). - ExecEndYbBitmapTableScan: YB master f689455 adds "if (scanDesc)" which is likewise already done like "if (tsdesc != NULL)" by YB pg15 merge f689455. Discard the master change. - costsize.c: - includes: YB master 8fe3336 adds new include optimizer/ybcplan.h. Already, there is an effort to move such includes into the already-existing YB includes section below, such as in YB pg15 merge 5fe2012, so do the same in this case. - createplan.c: - includes: YB master 8fe3336 adds new include optimizer/ybcplan.h even though it already exists in the YB includes below. Drop this change. - function declarations: YB master 8fe3336 moves extract_pushdown_clauses (despite the lack of YB prefix, this is a YB-owned function) out to ybcplan.h. This has adjacent line conflicts, and the only changes by YB pg15 to this are indentation (which appears to be already fixed in the new location). Trivial resolution. - extract_pushdown_clauses: YB master 8fe3336 moves this function out to ybcplan.c. There have been no modifications to this function by YB pg15. Conflict is from adjacent lines. Trivial resolution. - syscache.c: - YbLoadPinnedObjectsCache: YB master 8487e59 deletes this function and moves some contents to YbInitPinnedCacheIfNeeded while YB pg15 bea1ffb deletes the whole thing. Delete the whole thing. - pg_yb_utils.c: - YBTxnDdlProcessUtility: YB master 8487e59 changes the signature of YbInitPinnedCacheIfNeeded, and YB pg15 merge 5627af5 adds #endif in the same area. Adjacent line conflict. - YbRelationSetNewRelfileNode: YB master fad94f7 moves code from make_new_heap to here, so apply the YB pg15 adjustments (which is only heap_open/heap_close to table_open/table_close). - logical.h: - LogicalDecodingContext: YB master 5c023f3 adds yb_start_decoding_at field in same area that upstream PG f95d53eded55ecbf037f6416ced6af29a2c3caca adds end_xact. Trivial merge. - syscache.h: - function declarations: (same as syscache.cc) - explain.c: - YbExplainDistinctPrefixLen: YB master e75e20d adds set_deparse_context_planstate call. Upstream PG 6ef77cf46e81f45716ec981cb08781d426181378 changes that to set_deparse_context_plan with slightly different middle argument as well. Simple resolution. - pg_yb_utils.h: - YbDdlRollbackEnabled: fix error: ‘inline’ is not at beginning of declaration [-Werror=old-style-declaration] by swapping "inline" and "bool". This was introduced by YB master 6091cc8. Not sure why it isn't an issue on master. - pg15_tests/passing_tests.tsv: - PggateTestSelect.GetTableKeyRanges: YB master cb302e0 renames PggateTestSelect.GetTableKeyRanges to PggateTestSelect.GetRangeShardedTableKeyRanges and disables the test, so remove it from this list. - PgSingleServerRestartTest.GetSafeTimeBeforeConsensusStarted: this test hits TRAP: FailedAssertion("!IsTransactionOrTransactionBlock()", File: "../../../../../../../src/postgres/src/backend/utils/activity/pgstat.c", Line: 580, PID: 1541456) but that was also the case before this merge. The issue now is that it crash loops on starting postmaster (this test kills and restarts tserver): FATAL: pre-existing shared memory block (key 33732133, ID 24018961) is still in use Bisecting commits would point to which commit introduced this issue and may help understand it, but in the interest of time, punt this to another day and just remove it from the passing list. - TestPgReplicationSlot.java: - testStartLsnValues: YB master 5c023f3 adds this test, but it fails in pg15: com.yugabyte.util.PSQLException: Database connection failed when reading from copy Ignore the test for now, just like some other tests in the same file. Test Plan: On Almalinux 8: #!/usr/bin/env bash set -eu ./yb_build.sh fastdebug --gcc11 pg15_tests/run_tests.sh Get the following results on Almalinux 8, fastdebug, gcc11: 1 2024-05-09T13:40:28-07:00 JAVA org.yb.pgsql.TestPgRegressParallel#testPgRegressParallel 0 2024-05-09T13:41:57-07:00 integration-tests_xcluster_outbound_replication_group-itest XClusterOutboundReplicationGroupTest.* 0 2024-05-09T13:42:14-07:00 integration-tests_tablet-split-itest AutomaticTabletSplitITest.SizeRatio 1 2024-05-09T13:44:49-07:00 JAVA org.yb.pgsql.TestPgCostModelSeekNextEstimation 0 2024-05-09T13:45:10-07:00 integration-tests_master_heartbeat-itest MasterHeartbeatITestWithUpgrade.ClearUniverseUuidToRecoverUniverse 0 2024-05-09T13:45:27-07:00 integration-tests_cassandra_cpp_driver-test CppCassandraDriverTest.BatchWriteDuringSoftMemoryLimit 0 2024-05-09T13:45:47-07:00 integration-tests_cdcsdk_ysql-test CDCSDKYsqlTest.TestTableIdAndPkInCDCRecordsOnNonColocatedTables 0 2024-05-09T13:46:07-07:00 integration-tests_cdcsdk_ysql-test CDCSDKYsqlTest.TestTableIdAndPkInCDCRecordsOnColocatedTables 0 2024-05-09T13:46:24-07:00 integration-tests_cdcsdk_ysql-test CDCSDKYsqlTest.TestPackedRowsWithLargeColumnValueSingleShardTransaction 0 2024-05-09T13:46:40-07:00 integration-tests_cdcsdk_ysql-test CDCSDKYsqlTest.TestPackedRowsWithLargeColumnValueSingleShardTransaction 0 2024-05-09T13:47:41-07:00 integration-tests_minicluster-snapshot-test PgCloneTest.Clone 0 2024-05-09T13:59:37-07:00 integration-tests_cdcsdk_consumption_consistent_changes-test CDCSDKConsumptionConsistentChangesTest.* 0 2024-05-09T13:59:44-07:00 consensus_raft_consensus_quorum-test RaftConsensusQuorumTest.TestRequestVote 0 2024-05-09T16:08:56-07:00 integration-tests_cdcsdk_ysql-test CDCSDKYsqlTest.TestAtomicDDLRollback 0 2024-05-09T14:08:25-07:00 integration-tests_wait_states-itest *.* 0 2024-05-09T14:08:52-07:00 integration-tests_wait_states-itest WaitStateITest/AshTestVerifyOccurrence.VerifyWaitStateEntered/kBackfillIndex_WaitForAFreeSlot 0 2024-05-09T14:10:51-07:00 JAVA org.yb.pgsql.TestPgSequences 1 2024-05-09T14:22:07-07:00 JAVA org.yb.pgsql.TestYbAsh 0 2024-05-09T14:37:03-07:00 integration-tests_cdcsdk_replica_identity-test *.* 8 2024-05-09T15:15:42-07:00 pgwrapper_pg_ddl_atomicity-test *.* 1 2024-05-09T15:16:30-07:00 JAVA org.yb.ysqlconnmgr.TestSessionParameters 1 2024-05-09T15:24:44-07:00 JAVA org.yb.ysqlconnmgr.TestDropAndRenameDb 0 2024-05-09T15:26:30-07:00 pgwrapper_pg_wait_on_conflict-test PgWaitQueuesTest.DeadlockResolvesYoungestTxn 8 2024-05-09T15:26:34-07:00 client_ql-tablet-test */QLTabletRf1TestToggleEnablePackedRow.GetTabletKeyRanges/* 1 2024-05-09T15:28:06-07:00 JAVA org.yb.pgsql.TestPgRegressMisc#testPgRegressMiscIndependent 8 2024-05-09T15:31:52-07:00 pgwrapper_pg_ddl_atomicity-test */PgLibPqTableRewrite.TestTableRewriteRollback/* 8 2024-05-09T15:32:56-07:00 pgwrapper_pg_ddl_atomicity-test */PgLibPqTableRewrite.TestTableRewriteSuccess/* 8 2024-05-09T15:36:07-07:00 tools_yb-backup-cross-feature-test */YBBackupTestWithTableRewrite.TestYSQLBackupAndRestoreAfterRewrite/* 8 2024-05-09T15:37:39-07:00 tools_yb-backup-cross-feature-test */YBBackupTestWithTableRewrite.TestYSQLBackupAndRestoreAfterFailedRewrite/* 0 2024-05-09T15:41:20-07:00 tools_yb-admin-snapshot-schedule-test */YbAdminSnapshotScheduleTestWithYsqlParam.PgsqlTestTruncate/* 8 2024-05-09T15:41:53-07:00 integration-tests_xcluster_ysql-test XClusterYsqlTest.TestTableRewriteOperations 8 2024-05-09T15:42:20-07:00 integration-tests_cdcsdk_ysql-test CDCSDKYsqlTest.TestTableRewriteOperations 0 2024-05-09T15:43:55-07:00 integration-tests_xcluster_ysql-test XClusterYsqlTest.DropTableOnConsumerThenProducer 0 2024-05-09T15:45:32-07:00 integration-tests_xcluster_ysql-test XClusterYsqlTest.ConsumerMasterRestartAfterTableDrop 0 2024-05-09T15:47:10-07:00 integration-tests_xcluster_ysql-test XClusterYsqlTest.DropTableOnProducerThenConsumer 0 2024-05-09T15:48:51-07:00 integration-tests_xcluster_ysql-test XClusterYsqlTest.ProducerMasterRestartAfterTableDrop 0 2024-05-09T15:49:32-07:00 integration-tests_xcluster_ysql-test XClusterYsqlTest.DropTableWithWorkload 0 2024-05-09T15:51:18-07:00 integration-tests_xcluster_ysql-test XClusterYsqlTest.DropTableOnProducerOnly 0 2024-05-09T15:52:10-07:00 integration-tests_xcluster_db_scoped-test XClusterDBScopedTest.DropTableOnProducerThenConsumer 0 2024-05-09T15:52:18-07:00 master_xcluster_outbound_replication_group-test XClusterOutboundReplicationGroupMockedTest.AddTableDuringCheckpoint 0 2024-05-09T15:52:26-07:00 master_xcluster_outbound_replication_group-test XClusterOutboundReplicationGroupMockedTest.DropTableDuringCheckpoint 0 2024-05-09T15:58:40-07:00 JAVA org.yb.pgsql.TestPgTransparentRestarts org.yb.ysqlconnmgr.TestSessionParameters failure is due to issue #22228. Jenkins: urgent, rebase: pg15 Reviewers: fizaa, telgersma, xCluster, hsunder Reviewed By: fizaa, telgersma Subscribers: ybase, ycdcxcluster, yql, tfoucher Differential Revision: https://phorge.dev.yugabyte.com/D34887
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area/ysql
Yugabyte SQL (YSQL)
kind/enhancement
This is an enhancement of an existing feature
kind/failing-test
Tests and testing infra
priority/medium
Medium priority issue
status/awaiting-triage
Issue awaiting triage
Jira Link: DB-11148
Description
On almalinux 8, 4899076 (master),
fails with
but this is fine:
The test assumes UTC default in
According to PG guc.c:
and PG initdb.c:
I couldn't identify where in the code, but it appears that the TZ env var at cluster creation (not initdb) determines how ysql_pg.conf gets generated to specify timezone=...
The test should not be TZ-dependent.
Issue Type
kind/failing-test
Warning: Please confirm that this issue does not contain any sensitive information
The text was updated successfully, but these errors were encountered: