smbd: prevent smbd from panicing with "clustering = no" but --with-cluster-support

Michael

up release to 46

up to release 45 now

disable the DNS update code, as it breaks clustering

up to release 44

added a explanatory comment on tcon check

smbd/conn: fix prototype change of conn_close_all() in proto.h.

(Cleaning "ensure we exit with non-zero status on EOF on socket"
 after rebasing to v3-3-test which has no "make proto" anymore.)

Michael

ensure we exit with non-zero status on EOF on socket, so the parent
can trigger a brlock db cleanup

fixed child exit handling and IP release handling

log unclean shutdowns

register the ctdbd reconfigure message

update configure.rpm

increase version to 43

allow nested ctdb transactions in the same manner that they are
allowed for tdb. This is needed for the registry db backend.

update to ctdb.42 release

Michael, if you do a new release, make it 43 or later please

update to transactions4 release

drop retries to 5

use CTDB_CONTROL_TRANS2_COMMIT_RETRY to prevent the counter getting
out of sync

up release to transactions3

fixed lots of places that paniced on a failed transaction_commit,
thinking it was a failure of a transaction cancel

don't leave a dangling transaction on retry error

cope with the control failing completely without returning a status

up release number

handle two special cases

1) when all nodes write the same value to the record, or when writing
   a value that is already there, we can skip the write and save
   ourselves a network transactions

2) when all remote nodes fail an update, and we then fail a replay, we
   don't need to trigger a recovery. This solves a corner case where
   we could get into a recovery loop

mark this release as a transactions test release

put a limit on the number of retries. I found a case where a recovery
could lead to it blocking forever

we need to commit, not cancel, on record destruction

removed more unused code

all persistent databases now do all stores via automatic transactions

use transactions in idmap_tdb2

fixed fetch of empty records

samba3 can't handle NOREPLY yet

cleanup debugging and fix handling of empty transaction

first cut at adding full transactions for ctdb to samba3

got rid of the redundent cache database

packaging(RHEL-CTDB): we are now based on 3.2.1

Michael

idmap tdb2: fix inconsistent mappings by checking for race and retrying to fetch mapping.

Michael

dbwrap ctdb: fix a DEBUG message.

Michael

idmap tdb2: fix broken logic in tdb2_delete_bystring().

1. use the return value that idmap_tdb2_open_perm_db() gives us
2. don't delete frep the local db if deleting from the perm db failed.
3. fix wrong interpretation of return value of the local delete

Michael

idmap tdb2: fix broken logic in tdb2_store_bystring().

1. use the return value that idmap_tdb2_open_perm_db() gives us
2. don't write to the local db if writing to the perm db failed.
3. fix wrong interpretation of return value of the local store

Michael

dbwrap ctdb: don't retry when tdb_store failed in db_ctdb_persistent_store().

Only retry when ctdbd_persisten_update() failed.

Michael

dbwrap ctdb: add a partial mapping from tdb_error to NTSTATUS and use it for store.

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.41 for the next RPM.

Michael

dbwrap ctdb: add db_ctdb_delete_persistent() and use it for persistent DBs

as delete_rec operation from fetch_locked()

Michael

dbwrap ctdb: call db_ctdb_store() in db_ctdb_delete().

to reduce code duplication.

Michael

dbwrap ctdb: add a retry loop to the persistent store operation.

This is because ctdbd can fail in performing the persistent_store
due to race conditions, and this does not mean it can't succeed
the next time.

To not loop infinitely, this makes use of a new parametric option:
"dbwrap ctdb:max store retries" (integer) which defaults to 5
and sets the upper limit for the number or repeats of the
fetch/store cycle.

Michael

dbwrap ctdb: release the lock before calling ctdbd_persistent_store()

in the persistent db_ctdb_store operation.

This is to prevent deadlocks in db_ctdb_persistent_store().

There is a tradeoff: Usually, the record is still locked
after db->store operation. This lock is usually released
via the talloc destructor with the TALLOC_FREE to
the record. So we have two choices:

- Either re-lock the record after the call to persistent_store
  or cancel_persistent update and this way not changing any
  assumptions callers may have about the state, but possibly
  introducing new race conditions.

- Or don't lock the record again but just remove the
  talloc_destructor. This is less racy but assumes that
  the lock is always released via TALLOC_FREE of the record.

I choose the first variant for now since it seems less racy.
We can't guarantee that we succeed in getting the lock
anyways. The only real danger here is that a caller
performs multiple store operations after a fetch_locked()
which is currently not the case.

Michael

dbwrap ctdb: remove erroneously duplicated comment.

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.40 for the next RPM.

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.39 for the next RPM.

Michael

Use transaction start/cancel for persistent writes to avoid leaving the database in an inconsistent state if we crash during the operation

Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>

packaging(RHEL-CTDB): bump Release number to ctdb.37 for next rpm.

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.37 for the next RPM.

Michael

packaging(RHEL-CTDB): bump Release number to ctdb.36 for next rpm.

Michael

packaging(RHEL-CTDB): Bump release number to ctdb.35 for the next RPM.

Michael

Add a gpfs_prefetch module

This can not go upstream yet because it uses the non-GPL libgpfs. So it will
not be compiled by default and will not be included in the SOFS RPMs. But upon
Sven's request, we include it in the git tree and the source RPMs, so that it
can be built for in-house tests.

packaging(RHEL-CTDB): bump Release number to ctdb.34 for the next RPM.

Michael

packaging(RHEL-CTDB): update the build instructions in the README.

Michael

packaging(RHEL-CTDB): bump Release to ctdb.33 for the next build

Michael

packaging(RHEL-CTDB): remove the libnss_wins modules to reduce deps.

Michael

bump version number to 32

Correctly return NULL from messaging_ctdbd_connection if messaging_init failed

Exit smbstatus cleanly if messaging_init fails

Package 32-bit versions of libtalloc into the winbind rpm

Fix README: DOCS_DIR is replaced by DOCS_TARBALL

Bump up version number

packaging(RHEL-CTDB): use /usr/%{_libarch} instead of %{_libdir}

To fix creation of 32bit packges on x86_64

Michael

packaging(RHEL-CTDB): remove duplicate block from spec file.

Michael

packaging(RHEL-CTDB): fix placement and link direction of new shared libs

libwbclient, libtalloc, libtdb, and libnetapi are now installed
into /usr/lib{,64} by their SONAME. The symlink is made from
the .so files to point to the SONAME files (e.g., libtdb.so.1).

Michael

packaging(RHEL-CTDB): bump rpm release to the next version ctdb.29

Michael

packaging(RHEL-CTDB): add "service winbind condrestart" to "%post common"

Michael

packaging(RHEL-CTDB): use "/sbin/service smb" instead of "%{initdir}/smb"

Michael

packaging(RHEL-CTDB): ignore errors from init scripts in pre/post scripts.

Michael

packaging(RHEL-CTDB): in %preun, first stop the service then deactivate

Michael

packaging(RHEL-CTDB): use condrestart instead of restart in %post script

Michael

packaging(RHEL-CTDB): move service smb restart from %postun to %post

What would be the use of restarting a service that has just
been uninstalled...

Michael

packaging(RHEL-CTDB): move "chkconfig --del winbind" to "%preun common"

remains of winbind are being removed from the main package.
also stop the winbind service in the "%preun common" script.

Michael

packaging(RHEL-CTDB): remove winbind init script and its linke from main pkg

Winbind along with its libraries and init script is packaged in the
samba-common rpm.

Michael

packaging: bump the rhel-ctdb rpm releas number to ctbd.28

Michael

Do not call testparm in /etc/init.d/smb

For SOFS, we do know that we have netbios disabled, and testparm in the ctdb
shutdown event locks up due to a locked registry.tdb

keep compatibility with v3-0-ctdb name for fileid:mapping option

update configure.rpm for clustered Samba usage

Add winbind:online check timeout parameter

This is a band-aid for the rather convoluted offline/online mess in winbind
right now. Winbind re-uses the offline functionality that is targeted at domain
client installations on laptops to not overload disfunctional DCs. It uses the
winbind cache timeout as the retry timeout after a DC reboot.

I am using a parametric options because when this mess is cleaned up, that
parameter needs to go away again.

I'd recommend to use something like

winbind:online check timeout = 30

in typical LAN environments. This means a reconnect is attempted every 30
seconds.

Volker

Attempt to fix bug 5684

With the ctdb checkin dde9f3f006 tdb optimized out write lock checks for
write-enabled transaction. Sadly, this also removed the possibility to ever
remove dead records left over from tdb_delete calls within a transaction.

Tridge, please check this! Did dde9f3f006 have any reason beyond performance
optimizations?

Thanks,

Volker

Attempt to fix the build on Irix CC
(cherry picked from commit 666bf8456ac44cbbbd5524af2bf4fd89e18ddf62)

Merge from Herb. print correct test status

Fix bug 5686 - libsmbclient segfaults with more than one SMBCCTX.
Here is a patch to allow many subsystems to be re-initialized. The only
functional change I made was to remove the null context tracking, as the memory
allocated here is designed to be left for the complete lifetime of the program.
Freeing this early (when all smb contexts are destroyed) could crash other
users of talloc.
Jeremy.

use variables for files used in multiple places
(cherry picked from commit cee044bc42d955c535dbb6bb372af01089d37756)

Make sure to always set errno on error path in OpenDir (and hence scan_directory).

Michael

Fix unix_convert() for "*" after changing map_nt_error_from_unix().

map_nt_error_from_unix() now assumes that it is called in
an error path and returns an error even for a given errno == 0.
The original behaviour of unix_convert() used the mapping
of errno == 0 ==> NT_STATUS_OK to return success through
an error path.

I think this must have been an oversight, and unix_convert() worked
only by coincidence (or because explicitly using the knowledge
of the conceptually wrong working of map_nt_error_from_unix().

This patch puts this straight by not interpreting errno == 0
as an error condition and proceeding in that case.

Jeremy - please check!

Michael

I found lots of places where we assume error will be set when calling
one of our virtualised functions, such as db_open(), but error is only
set when a system call fails, and it is not uncommon for us to fail a
function internally without ever making a system call. That led to us
passing back success when a function had in fact failed.

I found two places where we relied on map_nt_error_from_unix()
returning success when errno==0, but lots and lots of places where we
relied on the reverse, so I fixed those two places.

map_nt_error_from_unix() will now always return an error, returning
NT_STATUS_UNSUCCESSFUL if errno is 0
(cherry picked from commit 69d40ca4c1af925d4b0e59ddc69ef8c26e6501d1)

ensure we give an error code to any routines above that are looking
for one

cifs.upcall: negatively instantiate keys on error

When a request-key upcall exits without instantiating a key, the kernel
will negatively instantiate the key with a 60s timeout. Older kernels,
however seem to also link that key into the session keyring. This
behavior can interefere with subsequent mount attempts until the
key times out. The next request_key() call will get this negative key
even if the upcall would have worked the second time.

Fix this by having cifs.upcall negatively instantiate the key itself
with a 1s timeout and don't attach it to the session keyring.

Signed-off-by: Jeff Layton <jlayton@redhat.com>

Remove two unused variables

Do not create a new mapping if a domain with an explicit config fails

Make the docs actually build

Thanks to Karolin for the friendly build service :-)

Document idmap rewrite

Some doxygen comments for idmap

Fix prototypes

Remove "idmap alloc config : range" parameter

This was overwritten by "idmap uid/gid" anyway. These are now the range
parameters for the alloc backend.