Skip to content
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

backup: TestDataDriven_backup_permissions_deprecated failed #142036

Open
cockroach-teamcity opened this issue Feb 26, 2025 · 0 comments
Open

backup: TestDataDriven_backup_permissions_deprecated failed #142036

cockroach-teamcity opened this issue Feb 26, 2025 · 0 comments
Labels
A-disaster-recovery branch-release-25.1.0-rc Used to mark GA and release blockers and technical advisories for 25.1.0-rc C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot. release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. T-disaster-recovery

Comments

@cockroach-teamcity
Copy link
Member

cockroach-teamcity commented Feb 26, 2025

backup.TestDataDriven_backup_permissions_deprecated failed on release-25.1.0-rc @ e5ec46c9865849c0b20ffa1b351372ced6c578b4:

        ----
        pq: user testuser does not have USAGE privilege on type greeting
        HINT: The existing privileges are being deprecated in favour of a fine-grained privilege model explained here https://www.cockroachlabs.com/docs/stable/backup.html#required-privileges. In a future release, to run BACKUP DATABASE, user testuser will exclusively require the BACKUP privilege on database d2.
    datadriven.go:144: 
        /var/lib/engflow/worker/work/2/exec/bazel-out/k8-fastbuild/bin/pkg/backup/backup_test_/backup_test.runfiles/com_github_cockroachdb_cockroach/pkg/backup/testdata/backup-restore/backup-permissions-deprecated:125:
        exec-sql [0 args]
        GRANT USAGE ON TYPE d2.greeting TO testuser;
        ----
    datadriven.go:144: 
        /var/lib/engflow/worker/work/2/exec/bazel-out/k8-fastbuild/bin/pkg/backup/backup_test_/backup_test.runfiles/com_github_cockroachdb_cockroach/pkg/backup/testdata/backup-restore/backup-permissions-deprecated:130:
        exec-sql [2 args]
        BACKUP DATABASE d2 INTO 'nodelocal://1/d2';
        ----
        NOTICE: The existing privileges are being deprecated in favour of a fine-grained privilege model explained here https://www.cockroachlabs.com/docs/stable/backup.html#required-privileges. In a future release, to run BACKUP DATABASE, user testuser will exclusively require the BACKUP privilege on database d2.
    datadriven.go:144: 
        /var/lib/engflow/worker/work/2/exec/bazel-out/k8-fastbuild/bin/pkg/backup/backup_test_/backup_test.runfiles/com_github_cockroachdb_cockroach/pkg/backup/testdata/backup-restore/backup-permissions-deprecated:135:
        exec-sql [2 args]
        BACKUP TABLE d2.t INTO 'nodelocal://1/d2-table';
        ----
        NOTICE: The existing privileges are being deprecated in favour of a fine-grained privilege model explained here https://www.cockroachlabs.com/docs/stable/backup.html#required-privileges. In a future release, to run BACKUP TABLE, user testuser will exclusively require the BACKUP privilege on tables: t.
    test_server_shim.go:152: automatically injected a shared process virtual cluster under test; see comment at top of test_server_shim.go for details.
    datadriven.go:144: 
        /var/lib/engflow/worker/work/2/exec/bazel-out/k8-fastbuild/bin/pkg/backup/backup_test_/backup_test.runfiles/com_github_cockroachdb_cockroach/pkg/backup/testdata/backup-restore/backup-permissions-deprecated:141:
        new-cluster [2 args]
        <no input to command>
        ----
    datadriven.go:144: 
        /var/lib/engflow/worker/work/2/exec/bazel-out/k8-fastbuild/bin/pkg/backup/backup_test_/backup_test.runfiles/com_github_cockroachdb_cockroach/pkg/backup/testdata/backup-restore/backup-permissions-deprecated:144:
         CREATE DATABASE d;
        REVOKE CONNECT ON DATABASE d FROM public;
        CREATE USER testuser;
        GRANT CONNECT ON DATABASE d TO testuser;
        expected:
        NOTICE: auto-committing transaction before processing DDL due to autocommit_before_ddl setting
        NOTICE: auto-committing transaction before processing DDL due to autocommit_before_ddl setting
        NOTICE: auto-committing transaction before processing DDL due to autocommit_before_ddl setting
        
        found:
        NOTICE: auto-committing transaction before processing DDL due to autocommit_before_ddl setting
        NOTICE: auto-committing transaction before processing DDL due to autocommit_before_ddl setting
        pq: restart transaction: TransactionRetryWithProtoRefreshError: TransactionRetryError: retry txn (RETRY_ASYNC_WRITE_FAILURE - missing intent on: /Tenant/10/Table/54/1/1050238466407137281/"legacy_payload"/1914-11-05T10:10:54.533258999Z/0): "sql txn" meta={id=e96c4804 key=/Tenant/10/Table/3/1/104/2/1 iso=Serializable pri=0.03444413 epo=0 ts=1740577744.491061881,2 min=1740577744.466740782,0 seq=13} lock=true stat=PENDING rts=1740577744.491061881,2 wto=false gul=1740577744.966740782,0
        HINT: See: https://www.cockroachlabs.com/docs/v25.1/transaction-retry-error-reference.html#retry_async_write_failure
    testutils.go:302: no Invalid Descriptors
    testutils.go:302: no Invalid Descriptors
    testutils.go:302: no Invalid Descriptors
    testutils.go:302: no Invalid Descriptors
    testutils.go:302: no Invalid Descriptors
    panic.go:626: -- test log scope end --
test logs left over in: outputs.zip/logTestDataDriven_backup_permissions_deprecated1700035422
--- FAIL: TestDataDriven_backup_permissions_deprecated (13.60s)

Parameters:

  • attempt=1
  • run=22
  • shard=16
Help

See also: How To Investigate a Go Test Failure (internal)

/cc @cockroachdb/disaster-recovery

This test on roachdash | Improve this report!

Jira issue: CRDB-48134

@cockroach-teamcity cockroach-teamcity added branch-release-25.1.0-rc Used to mark GA and release blockers and technical advisories for 25.1.0-rc C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot. release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. T-disaster-recovery labels Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-disaster-recovery branch-release-25.1.0-rc Used to mark GA and release blockers and technical advisories for 25.1.0-rc C-test-failure Broken test (automatically or manually discovered). O-robot Originated from a bot. release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. T-disaster-recovery
Projects
None yet
Development

No branches or pull requests

1 participant